[Uptime] Add mapping for REST API test data #35249
Merged
+1,420
−0
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Mappings are currently absent for the data directory we use for Uptime's REST API. This caused some problems when backporting other test-related changes to
7.x
. Specifically, certain values were being counted that would have been ignored if the proper mappings were present, which broke some API tests.Rather than modifying that backport PR post-review to make it work, I decided it was better to open a PR to add the necessary mapping file to
master
first, and then backport it with an updated version number to7.x
. At that point I can remove the added mapping from the above-linked PR and retest only those changes.Testing
If this is passing CI then it should be ok to merge.
Checklist
Use
strikethroughsto remove checklist items you don't feel are applicable to this PR.For maintainers