Skip to content

Adding test results validation by regexp#26

Closed
dvalinrh wants to merge 6 commits into
mainfrom
test_validation
Closed

Adding test results validation by regexp#26
dvalinrh wants to merge 6 commits into
mainfrom
test_validation

Conversation

@dvalinrh

@dvalinrh dvalinrh commented Jun 18, 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.

This adds the required files and hooks into the wrapper for verification of the test results via regexp.
Mapping results to regexp
Typical results
ht_config:sockets:threads:unit:MB/sec:cpu_affin
ht_yes_1_socket:1:2:GFlops:132:0,1
ht_yes_1_socket:1:2:GFlops:132:2,3,0,1

Regexp file contents
%_header
ht_config:sockets:threads:unit:MB/sec:cpu_affin
%_multiples
:[[:digit:]]{1,}:[[:digit:]]{1,}:GFlops:[1-9][0-9]{0,}:[ 0-9,]{1,}$

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

Field 1: Matches anything
Field 2: Expects a numerical value
Field 3: Expects a numerical value
Field 4: Expects GFLOPs
Field 5: Expects a non zero numerical value
Field 6: Expects numerical values, with commas.

Before/After Comparison

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

Clerical Stuff

This closes #22

Relates to JIRA: RPOPC-348

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 18, 2025 10:21
@dvalinrh dvalinrh self-assigned this Jun 18, 2025
@github-actions

Copy link
Copy Markdown

This relates to RPOPC-348

# Multiple lines may be present.
#
%_multiples
:[1-9][0-9]{0,}:[1-9][0-9]{0,}:GFlops:[1-9][0-9\.]{0,}:[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

Switching to use pythion and json instead of regexp.

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

Add verification test.

2 participants