Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added test for Fetcher fetchRoute + other stuff :) #103

Conversation

p1pchenk0
Copy link

  • test for fetcher
  • failing unit tests after updates

* failing unit tests after updates
@p1pchenk0 p1pchenk0 mentioned this pull request May 25, 2023
9 tasks
@codecov
Copy link

codecov bot commented May 25, 2023

Codecov Report

Patch coverage: 96.49% and project coverage change: +0.33 🎉

Comparison is base (ef81aed) 95.48% compared to head (a394845) 95.82%.

Additional details and impacted files
@@            Coverage Diff             @@
##            nuxt3     nuxt/nuxt-vitest#103      +/-   ##
==========================================
+ Coverage   95.48%   95.82%   +0.33%     
==========================================
  Files          54       55       +1     
  Lines        3898     3926      +28     
  Branches      590      604      +14     
==========================================
+ Hits         3722     3762      +40     
+ Misses        176      164      -12     
Flag Coverage Δ
vitest 95.82% <96.49%> (+0.33%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/runtime/api/fetcher/fetcher.ts 98.96% <ø> (+0.68%) ⬆️
...ntime/templates/components/core/ResourceLoader.vue 92.00% <81.81%> (-3.75%) ⬇️
src/runtime/api/auth.ts 98.10% <100.00%> (+0.03%) ⬆️
src/runtime/api/mercure.ts 100.00% <100.00%> (+1.31%) ⬆️
src/runtime/composables/process.ts 100.00% <100.00%> (ø)
src/runtime/cwa.ts 97.50% <100.00%> (+2.62%) ⬆️
src/runtime/route-middleware.ts 75.00% <100.00%> (+13.57%) ⬆️
src/runtime/storage/stores/resources/actions.ts 96.04% <100.00%> (+0.07%) ⬆️
src/runtime/storage/stores/resources/state.ts 100.00% <100.00%> (ø)
...ntime/templates/components/main/ComponentGroup.vue 100.00% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@silverbackdan
Copy link
Contributor

Fab, thanks, good to merge or do you want to keep working on this branch tomorrow?

@p1pchenk0
Copy link
Author

Fab, thanks, good to merge or do you want to keep working on this branch tomorrow?

yes, I think it would be the best option to continue working here

* moved boolean check from ComponentGroup template to computed
@p1pchenk0 p1pchenk0 changed the title Added test for Fetcher fetchRoute Added test for Fetcher fetchRoute + other stuff :) May 26, 2023
})
})

test.todo('should return correct values for client/server flags IF env is server-side', () => {
Copy link
Contributor

@silverbackdan silverbackdan May 26, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, please add a comment Disabled due to nuxt-vitest bug - track issue and progress here https://github.com/danielroe/nuxt-vitest/issues/162

@silverbackdan silverbackdan merged commit 29e07ff into components-web-app:nuxt3 May 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants