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

feature/calculate coverage for all files #801

Merged
merged 4 commits into from
Mar 30, 2020

Conversation

adrianjost
Copy link
Contributor

Issue

As I just found out, text coverage can not be collected for vue files without a data property.
vuejs/vue-jest#32

Description

The solution is, to add (an empty) data attribute to all components

  • added empty data attribute to all components

@pullapprove pullapprove bot added waiting for review Someone should review this ASAP and removed size/L labels Mar 30, 2020
Copy link

@accesslint accesslint bot left a comment

Choose a reason for hiding this comment

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

There are accessibility issues in these changes.

src/components/atoms/ColorPicker.unit.js Outdated Show resolved Hide resolved
@codecov
Copy link

codecov bot commented Mar 30, 2020

Codecov Report

Merging #801 into develop will increase coverage by 0.44%.
The diff coverage is 89.55%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #801      +/-   ##
===========================================
+ Coverage    84.55%   84.99%   +0.44%     
===========================================
  Files           92      114      +22     
  Lines         1068     1133      +65     
  Branches       178      178              
===========================================
+ Hits           903      963      +60     
- Misses         160      165       +5     
  Partials         5        5              
Impacted Files Coverage Δ
src/components/base/BaseInput/BaseInputDefault.vue 100.00% <ø> (ø)
src/components/base/BaseProgressbar.vue 0.00% <0.00%> (ø)
src/components/base/BaseSpinner.vue 100.00% <ø> (ø)
src/components/base/BaseTitle.vue 0.00% <0.00%> (ø)
src/components/legacy/DemoBanner.vue 0.00% <0.00%> (ø)
src/components/legacy/NavigationBar.vue 50.00% <ø> (ø)
src/components/molecules/AdminTableLegend.vue 0.00% <0.00%> (ø)
src/components/molecules/FormActions.vue 0.00% <0.00%> (ø)
src/components/molecules/RenderHtmlMintEc.vue 85.71% <0.00%> (-14.29%) ⬇️
...mponents/organisms/FormDatasourceLoginWebuntis.vue 50.00% <0.00%> (-50.00%) ⬇️
... and 83 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6d75bfa...af8b4bb. Read the comment docs.

@hpi-schul-clud-bot
Copy link
Contributor

❤️ I have deployed this pull-request for you:

Docs: http://docs.801.nuxt.schul-cloud.surge.sh
Storybook: http://stories.801.nuxt.schul-cloud.surge.sh

@adrianjost adrianjost marked this pull request as ready for review March 30, 2020 18:50
@adrianjost adrianjost merged commit bb4f638 into develop Mar 30, 2020
@adrianjost adrianjost deleted the feature/coverage-for-all-files branch March 30, 2020 19:19
@adrianjost adrianjost removed the waiting for review Someone should review this ASAP label Mar 30, 2020
@pullapprove pullapprove bot added the waiting for review Someone should review this ASAP label Mar 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
waiting for review Someone should review this ASAP
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants