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

Fix TestPerformanceDataIsNotDuplicated test #184

Merged
merged 1 commit into from
Jan 26, 2023
Merged

Conversation

atc0005
Copy link
Owner

@atc0005 atc0005 commented Jan 26, 2023

Test began failing with work on PR branch for PR-175 which expanded validation support for PerformanceData field values.

The previous test metric values used words in place of expected field values:

  • literal "U"
  • character or values in the regex character class "[-0-9.]"

This commit moves the self-documenting Value field strings to slice element comments and replaces them with placeholder numeric values which pass Value field validation.

refs GH-175

Test began failing with work on PR branch for PR-175 which
expanded validation support for PerformanceData field values.

The previous test metric values used words in place of expected
field values:

- literal "U"
- character or values in the regex character class "[-0-9.]"

This commit moves the self-documenting Value field strings to
slice element comments and replaces them with placeholder
numeric values which pass Value field validation.

refs GH-175
@atc0005 atc0005 added bug Something isn't working tests labels Jan 26, 2023
@atc0005 atc0005 added this to the v0.14.0 milestone Jan 26, 2023
@atc0005 atc0005 self-assigned this Jan 26, 2023
@atc0005 atc0005 merged commit ab6f026 into master Jan 26, 2023
@atc0005 atc0005 deleted the fix-test-metric-values branch January 26, 2023 14:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant