-
Notifications
You must be signed in to change notification settings - Fork 3
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
Updated system_test_mapping.json with additional tests #220
Conversation
PR Description updated to latest commit (9f248dd) |
PR Analysis(review updated until commit 9f248dd)
PR Feedback💡 General suggestions: The PR is well-structured and the changes are clear. The addition of the JSON validation script is a good practice to ensure the integrity of the JSON file. However, it would be beneficial to add more detailed descriptions for the tests in the 'system_test_mapping.json' file to provide more context about what each test is doing. 🤖 Code feedback:
✨ Usage tips:
|
Persistent review updated to latest commit 9f248dd |
system_test_mapping.json
Outdated
"users-notification-service", | ||
"config-service" | ||
], | ||
"description": "testing teams/slack alert channels with compliance and vulnerabilities notifications", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should be slack
system_test_mapping.json
Outdated
"users-notification-service", | ||
"config-service" | ||
], | ||
"description": "testing teams/slack alert channels with compliance and vulnerabilities notifications", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should be teams
Signed-off-by: yonatanamz <[email protected]>
Updated system_test_mapping.json with additional tests
Type
Enhancement, Tests
Description
This PR includes several enhancements related to system tests:
PR changes walkthrough
1 files
json_validator.py
.github/workflows/scripts/json_validator.py
A new Python script was added to validate the JSON format of
the 'system_test_mapping.json' file. The script reads the
JSON file and checks if it is correctly formatted. If there
is a JSONDecodeError, it prints the error and exits with a
status code of 1.
1 files
validate_json.yaml
.github/workflows/validate_json.yaml
The inline Python code for JSON validation was replaced with
a call to the newly added 'json_validator.py' script. This
change makes the YAML file cleaner and the validation
process more modular.
1 files
system_test_mapping.json
system_test_mapping.json
- Descriptions were added to some tests.
- Two new
**Several updates were made to the system tests mapping:
Additional target repositories were added to various
tests.
tests, 'teams_alerts' and 'slack_alerts', were added,
including their target, target repositories, description,
and skip_on_environment fields.**