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

add smoke test and remove play json from the dependencies #74

Merged
merged 3 commits into from
Oct 5, 2023

Conversation

AurelienRichez
Copy link

@AurelienRichez AurelienRichez commented Oct 4, 2023

This removes the play json dependency because it's not needed.

As a bonus, since the previous version change crashed the app. I added a small script that :

  • call sbt stage to get a binary that is as close as possible to production and then
  • starts the application
  • check if it answers to curl calls (it does not check the content, just that it does not stop early and it answers something)

This can protect us from that problem in the future

It should not be necessary since it's pulled by the play framework
@AurelienRichez AurelienRichez force-pushed the fix/add-smoke-test branch 3 times, most recently from 8a57acf to 8878322 Compare October 4, 2023 10:47
@AurelienRichez AurelienRichez marked this pull request as ready for review October 4, 2023 10:50
@sonarcloud
Copy link

sonarcloud bot commented Oct 5, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Copy link
Member

@wjglerum wjglerum left a comment

Choose a reason for hiding this comment

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

thanks!

@wjglerum wjglerum enabled auto-merge (squash) October 5, 2023 08:55
@wjglerum wjglerum merged commit 5483e02 into main Oct 5, 2023
3 checks passed
@wjglerum wjglerum deleted the fix/add-smoke-test branch October 5, 2023 09:01
pepite pushed a commit that referenced this pull request Sep 2, 2024
* remove play json from the dependencies

It should not be necessary since it's pulled by the play framework

* Add a smoke test for the application

* add smoke test in scalatest instead of bash script
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