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

Get E2E tests passing #482

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Conversation

thostetler
Copy link
Member

This also adds in a bunch of docker changes and adds scripts

For now I got rid of most of the end-to-end tests.

Copy link

codecov bot commented Jun 6, 2024

Codecov Report

Attention: Patch coverage is 37.61062% with 141 lines in your changes missing coverage. Please review.

Project coverage is 23.8%. Comparing base (1c7081c) to head (50d9cc1).
Report is 20 commits behind head on master.

Current head 50d9cc1 differs from pull request most recent head 8b80ae5

Please upload reports for the commit 8b80ae5 to get more accurate results.

Additional details and impacted files
@@            Coverage Diff            @@
##           master    #482      +/-   ##
=========================================
- Coverage    47.3%   23.8%   -23.5%     
=========================================
  Files         425     565     +140     
  Lines       44389   90946   +46557     
  Branches      597     667      +70     
=========================================
+ Hits        20988   21603     +615     
- Misses      23390   69333   +45943     
+ Partials       11      10       -1     
Files Coverage Δ
src/api/config.ts 69.3% <100.0%> (ø)
src/components/AuthorAffiliations/store.tsx 83.0% <100.0%> (ø)
src/components/BibstemPicker/BibstemPicker.tsx 86.6% <100.0%> (+21.3%) ⬆️
...itationExporter/components/DescriptionCollapse.tsx 96.8% <100.0%> (ø)
...ts/CitationExporter/components/ExportContainer.tsx 95.9% <100.0%> (-0.4%) ⬇️
src/components/ClassicForm/ClassicForm.tsx 94.3% <100.0%> (+73.6%) ⬆️
src/components/DataDownloader/DataDownloader.tsx 97.7% <100.0%> (ø)
src/components/HideOnPrint/HideOnPrint.tsx 22.3% <100.0%> (ø)
src/components/Layout/AbsLayout.tsx 24.7% <100.0%> (ø)
src/components/Layout/Layout.tsx 41.4% <100.0%> (ø)
... and 27 more

... and 142 files with indirect coverage changes

@thostetler thostetler force-pushed the get-e2e-passing branch 18 times, most recently from ad55aea to 44c06a8 Compare June 6, 2024 22:30
@thostetler thostetler force-pushed the get-e2e-passing branch 26 times, most recently from 0a7886e to 50d9cc1 Compare June 12, 2024 16:22
@thostetler thostetler force-pushed the get-e2e-passing branch 2 times, most recently from 9084a3b to ec7b426 Compare June 14, 2024 13:41
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.

None yet

1 participant