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

SIS-23 & SIS-25: Updated CI to generate and save test reports #5

Merged
merged 11 commits into from
Mar 20, 2024
Merged

SIS-23 & SIS-25: Updated CI to generate and save test reports #5

merged 11 commits into from
Mar 20, 2024

Conversation

nhkhai
Copy link
Collaborator

@nhkhai nhkhai commented Mar 20, 2024

SIS-23: Updated CI to generate and save test reports.

nhkhai added 10 commits March 20, 2024 10:55
Added generation of the HTML test report. Fixed syntax.
Downgraded the Java JDK version to match the project.
Fixed the formatting syntax.
Added dependencies for junit-jupiter-api and  maven-surefire-plugin. This might be optional.
Updated to run all basic phases till verify then generate HTML reports.
Updated to save test artifacts and generate test report.
Fixed duplicate artifact name.
Added saving the single file html report test artifact.
Removed the redundant saved test artifact.
@nhkhai nhkhai added the enhancement New feature or request label Mar 20, 2024
@nhkhai nhkhai self-assigned this Mar 20, 2024
Copy link

gitguardian bot commented Mar 20, 2024

⚠️ GitGuardian has uncovered 1 secret following the scan of your pull request.

Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.

🔎 Detected hardcoded secret in your pull request
GitGuardian id GitGuardian status Secret Commit Filename
- Generic Password 601dc1f .github/workflows/CI.yml View secret
🛠 Guidelines to remediate hardcoded secrets
  1. Understand the implications of revoking this secret by investigating where it is used in your code.
  2. Replace and store your secret safely. Learn here the best practices.
  3. Revoke and rotate this secret.
  4. If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.

To avoid such incidents in the future consider


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

Our GitHub checks need improvements? Share your feedbacks!

@nhkhai
Copy link
Collaborator Author

nhkhai commented Mar 20, 2024

The GitGuardian Security Check failure can be ignored as it is just the default postgres password, and the authentication level is set to trust in the first place, so the password is not even applicable.

Removed extension job as it is not working properly. Base functionality is intact.
@nhkhai nhkhai merged commit 5b938ad into kiblykat:main Mar 20, 2024
1 of 2 checks passed
@nhkhai nhkhai changed the title SIS-23: Updated CI to generate and save test reports SIS-23 & SIS-25: Updated CI to generate and save test reports Mar 20, 2024
@nhkhai nhkhai requested a review from kiblykat March 20, 2024 05:20
@nhkhai
Copy link
Collaborator Author

nhkhai commented Mar 20, 2024

SIS-23 & SIS-25.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant