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

Parametrize test file for test runner #599

Merged
merged 1 commit into from
Jan 14, 2025
Merged

Conversation

lerealno
Copy link
Contributor

Description

Tests file for tests runner is now parametrized

Related Issue

No related issues

Changes Made

  • The test files to use by test runner action was hardcoded to "tests.yaml" with this change any file can be used for testing using test runner action. The parameter is optional and a default value of "tests.yaml" has been set to avoid any compatibility conflicts with any previously defined process/client.
  • [ x ] The code follows the project's coding standards.
  • [ x ] No Intel Internal IP is present within the changes.
  • [ x ] The documentation has been updated to reflect any changes in functionality.

Validation

  • [ x ] I have tested any changes in container groups locally with test_runner.py with all existing tests passing, and I have added new tests where applicable.

@lerealno lerealno requested a review from jitendra42 as a code owner January 14, 2025 14:06
Copy link

github-actions bot commented Jan 14, 2025

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails

Scanned Files

Copy link

coveralls-official bot commented Jan 14, 2025

Pull Request Test Coverage Report for Build 12772387856

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 88.485%

Totals Coverage Status
Change from base Build 12757850257: 0.0%
Covered Lines: 146
Relevant Lines: 165

💛 - Coveralls

@lerealno lerealno force-pushed the luis/test-runner-change branch from 063a75e to ef1d130 Compare January 14, 2025 16:46
@lerealno lerealno merged commit 92a56c2 into main Jan 14, 2025
31 checks passed
@lerealno lerealno deleted the luis/test-runner-change branch January 14, 2025 16:56
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