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

feat(tests): add context tests and switch to vitest #102

Merged
merged 1 commit into from
Oct 29, 2024

Conversation

virgofx
Copy link
Member

@virgofx virgofx commented Oct 29, 2024

  • Switched from Jest to Vitest for faster test execution.
  • Improved config and context providers for consistent behavior.
  • Enhanced testing with a generic test setup and utilities.
  • Added numerous tests to cover context functionality.

@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

disableBranding: false,
moduleChangeExcludePatterns: ['.gitignore', '*.md'],
moduleAssetExcludePatterns: ['tests/**', 'examples/**'],
githubToken: 'ghp_test_token_2c6912E7710c838347Ae178B4',

Check failure

Code scanning / SonarCloud

GitHub tokens should not be disclosed

<!--SONAR_ISSUE_KEY:AZLZjDllC7Zwa-T2VfA5-->Make sure this Github token gets revoked, changed, and removed from the code. <p>See more on <a href="https://sonarcloud.io/project/issues?id=terraform-module-releaser&issues=AZLZjDllC7Zwa-T2VfA5&open=AZLZjDllC7Zwa-T2VfA5&pullRequest=102">SonarCloud</a></p>
@virgofx virgofx force-pushed the feat/add-tests-for-context branch 12 times, most recently from a540a27 to 0cdd132 Compare October 29, 2024 19:16
@virgofx virgofx marked this pull request as ready for review October 29, 2024 19:23
@virgofx virgofx force-pushed the feat/add-tests-for-context branch from 0cdd132 to 8357af1 Compare October 29, 2024 19:25
- Switched from Jest to Vitest for faster test execution.
- Improved config and context providers for consistent behavior.
- Enhanced testing with a generic test setup and utilities.
- Added numerous tests to cover context functionality.
@virgofx virgofx force-pushed the feat/add-tests-for-context branch from 8357af1 to 3cf6abd Compare October 29, 2024 19:28
Copy link

Release Plan

No terraform modules updated in this pull request.

✅ Wiki Check ℹ️

Note: The following Terraform modules no longer exist in source; however, corresponding tags/releases exist. Automation tag/release deletion is disabledno subsequent action will take place.

v1.3.0, v1.2.0, v1.1.1, v1.1.0, v1.0.1, v1.0.0, v1

Powered by techpivot/terraform-module-releaser

Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
64.4% Coverage on New Code (required ≥ 80%)

See analysis details on SonarCloud

@virgofx virgofx merged commit f13b2f6 into main Oct 29, 2024
7 of 8 checks passed
@virgofx virgofx deleted the feat/add-tests-for-context branch October 29, 2024 19:30
@virgofx virgofx self-assigned this Oct 29, 2024
@release-preview release-preview bot mentioned this pull request Oct 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant