Skip to content

Add verification of results via regexp#37

Closed
dvalinrh wants to merge 4 commits into
masterfrom
validate_via_reegexp
Closed

Add verification of results via regexp#37
dvalinrh wants to merge 4 commits into
masterfrom
validate_via_reegexp

Conversation

@dvalinrh

@dvalinrh dvalinrh commented Jun 23, 2025

Copy link
Copy Markdown
Contributor

Description

This adds the required files and hooks into the wrapper for verification of the test results via regexp.
Mapping results to regexp
Typical results
# connection:TPM
10:421331
20:566553
40:535533

Regexp file contents
%_header
# connection:TPM
%_multiples
[[:digit:]][[:digit:]]{0,}:[1-9][0-9.]{1,}:1$

%_header: Indicates the next line is a header that we need to match.

Field 1: %_iteration means we expect to find anumerical value
Field 2: We expect to find a non zero numerical value.

Before/After Comparison

Before change: Failures will occur on validation
After change: Passes when it should, fails when it should.

Clerical Stuff

This closes #34

Mention the JIRA ticket of the issue, this helps keep
everything together so we can find this PR easily.

Relates to JIRA: RPOPC-422

Testing

Running the wrapper (modifying the regexps to force failures)
Verified test fails when it has bad data
Verified test passes when we have good data.

@dvalinrh dvalinrh requested a review from a team June 23, 2025 09:22
@dvalinrh dvalinrh self-assigned this Jun 23, 2025
%_header
# connection:TPM
%_multiples
[[1-9]][[0-9]]{0,}:[1-9][0-9\.]{1,}$

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Invalid decimal regex

@dvalinrh

Copy link
Copy Markdown
Contributor Author

Will be using python and json instead of regexp.

@dvalinrh dvalinrh closed this Aug 19, 2025
@dvalinrh dvalinrh deleted the validate_via_reegexp branch August 19, 2025 08:44
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