-
Notifications
You must be signed in to change notification settings - Fork 1
Contributing New Tests
Phil Smart edited this page Nov 20, 2024
·
8 revisions
To contribute a new test to the testbed, follow the steps below:
- Create an issue with a descriptive title of the test that includes the name of the ruleset you are testing.
- Create a new development branch based on the issue.
-
If the ruleset already exists in the
default-validator-stages.xml
file (it should), then:- Create a development branch that comprises of two distinct commits:
- Adding the tests for the new ruleset. This lets a reviewer run those and observe the fleet members who succeed on these tests vs. those who fail. A template Git commit message for this could be:
git commit -m "Add tests for checking <something>"
e.g.Add tests for checking schema validation
- The ruleset change which fixes the tests so that all fleet members succeed. A template Git commit message for this could be:
git commit -m "Fix <what> in <rule-set-name> ruleset <for?> <description?>"
e.g.Fix multi-predicate support in check_mdiop ruleset
- Adding the tests for the new ruleset. This lets a reviewer run those and observe the fleet members who succeed on these tests vs. those who fail. A template Git commit message for this could be:
- Create a development branch that comprises of two distinct commits:
-
If the ruleset does not already exist in the
default-validator-stages.xml
file, then: 2. Create a development branch that comprises of three distinct commits:- Adding the calling of the ruleset to the default pipeline in
default-validator-stages.xml
, without adding any explicit tests. If this is a separate commit, it lets a reviewer check that adding the new ruleset does not break any of the checks for other rulesets. - Adding the tests for the new ruleset. This lets a reviewer run those and observe the fleet members which succeed on these tests vs. those that fail.
- Finally, the change to the ruleset which fixes it so that all fleet members succeed.
- Adding the calling of the ruleset to the default pipeline in
- Once complete, create a PR from the branch for review.