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

Fix tests #472

Merged
merged 7 commits into from
Apr 23, 2023
Merged

Fix tests #472

merged 7 commits into from
Apr 23, 2023

Conversation

henrixapp
Copy link
Collaborator

@henrixapp henrixapp commented Apr 21, 2023

  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

Fix all checks. Tests are executed sequentially to utilize cache.

  • Please check if the PR fulfills these requirements

    • E2E Tests for the changes have been added via Cypress
    • Meaningful rspec tests have been added
    • Docs have been added / updated
  • What is the current behavior? (You can also link to an open issue here)

  • What is the new behavior (if this is a feature change)?

  • Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

  • Other information:

Splines and others added 2 commits April 21, 2023 20:10
- Update checkout action to version v3
- Use new docker layer caching version
- Simplify commands to handle docker compose shell commands
- Update codecov action
@henrixapp henrixapp changed the base branch from main to mampf-next April 21, 2023 19:03
@codecov
Copy link

codecov bot commented Apr 21, 2023

Codecov Report

Merging #472 (36d856d) into mampf-next (7db4b45) will increase coverage by 1.21%.
The diff coverage is 72.30%.

@@              Coverage Diff               @@
##           mampf-next     #472      +/-   ##
==============================================
+ Coverage       65.44%   66.65%   +1.21%     
==============================================
  Files             302      311       +9     
  Lines            8965     9360     +395     
==============================================
+ Hits             5867     6239     +372     
- Misses           3098     3121      +23     
Impacted Files Coverage Δ
app/controllers/main_controller.rb 40.62% <ø> (-1.80%) ⬇️
app/helpers/lectures_helper.rb 35.18% <ø> (+0.18%) ⬆️
app/helpers/media_helper.rb 32.87% <0.00%> (+0.93%) ⬆️
app/models/manuscript.rb 19.81% <0.00%> (-0.28%) ⬇️
app/models/term.rb 68.57% <0.00%> (-7.15%) ⬇️
spec/controllers/media_controller_spec.rb 100.00% <ø> (ø)
app/models/item.rb 52.29% <20.00%> (-0.27%) ⬇️
app/models/user_cleaner.rb 22.38% <22.38%> (ø)
app/controllers/profile_controller.rb 23.47% <25.00%> (+0.05%) ⬆️
app/helpers/users_helper.rb 42.85% <25.00%> (-23.81%) ⬇️
... and 25 more

... and 4 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@henrixapp henrixapp marked this pull request as ready for review April 22, 2023 13:18
@henrixapp henrixapp requested a review from Splines April 22, 2023 13:20
Copy link
Member

@Splines Splines left a comment

Choose a reason for hiding this comment

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

Wow, thanks for making this work, much appreciated. After this, #441 is not needed anymore, right?


- name: Pull docker images
run: docker compose pull --ignore-buildable
working-directory: docker/run_cypress_tests
Copy link
Member

Choose a reason for hiding this comment

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

Maybe rename the folder to docker/run_tests as it is not specifically for Cypress tests anymore.

@henrixapp henrixapp merged commit a0557b8 into mampf-next Apr 23, 2023
@henrixapp henrixapp deleted the fix_tests branch April 23, 2023 12:18
@henrixapp
Copy link
Collaborator Author

Yes #441 can be closed, I cherry-picked one of your commits.

@Splines Splines added the tests Unit tests, integration tests etc. label Apr 23, 2023
@Splines Splines mentioned this pull request May 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tests Unit tests, integration tests etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants