Skip to content

feat(tests): add GitHub Actions workflow for running tests#64

Merged
Theo-lbg merged 2 commits into
mainfrom
feat/CIInittests
Apr 15, 2026
Merged

feat(tests): add GitHub Actions workflow for running tests#64
Theo-lbg merged 2 commits into
mainfrom
feat/CIInittests

Conversation

@Theo-lbg
Copy link
Copy Markdown
Collaborator

@Theo-lbg Theo-lbg commented Apr 1, 2026

No description provided.

Comment thread .github/workflows/tests.yaml Fixed
@Theo-lbg Theo-lbg marked this pull request as ready for review April 2, 2026 09:09
Copilot AI review requested due to automatic review settings April 2, 2026 09:09
@sonarqube-xpeho
Copy link
Copy Markdown

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a dedicated GitHub Actions workflow to run the project’s Maven test suite in CI, and updates Maven test plugin configuration so unit tests and integration tests are executed in the appropriate phases.

Changes:

  • Reconfigure Maven Surefire/Failsafe to run unit tests in test and integration tests (*IntegrationTest) in integration-test/verify.
  • Add a new .github/workflows/tests.yaml workflow to run ./mvnw clean verify on pushes and pull requests.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
pom.xml Adjusts Surefire/Failsafe include/exclude patterns to separate unit vs integration tests.
.github/workflows/tests.yaml Introduces CI workflow to prepare test config/env and execute Maven verify (unit + integration).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/tests.yaml
Comment thread .github/workflows/tests.yaml
Comment thread .github/workflows/tests.yaml
@Theo-lbg Theo-lbg linked an issue Apr 2, 2026 that may be closed by this pull request
Copy link
Copy Markdown
Collaborator

@profotoce59 profotoce59 left a comment

Choose a reason for hiding this comment

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

J'allais dire d'utiliser une version plus récente pour checkout (v6 est sorti) mais au final on est aussi sur le 4 pour sonar.

@Theo-lbg Theo-lbg merged commit ff448ba into main Apr 15, 2026
13 checks passed
@Theo-lbg Theo-lbg deleted the feat/CIInittests branch April 15, 2026 08:27
@MayuriXx MayuriXx added this to the Sprint 4 milestone Apr 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Embed the CI, to launch Tests

5 participants