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

Store tests by relative instead of absolute paths in the previous results file #28989

Merged
merged 1 commit into from
Nov 4, 2024

Conversation

loganharbour
Copy link
Member

Closes #28988

@moosebuild
Copy link
Contributor

Job Documentation, step Docs: sync website on 6d407e2 wanted to post the following:

View the site here

This comment will be updated on new commits.

@moosebuild
Copy link
Contributor

Job Coverage, step Generate coverage on 6d407e2 wanted to post the following:

Framework coverage

c80f51 #28989 6d407e
Total Total +/- New
Rate 85.08% 85.08% +0.00% -
Hits 106568 106569 +1 0
Misses 18695 18694 -1 0

Diff coverage report

Full coverage report

Modules coverage

Coverage did not change

Full coverage reports

Reports

This comment will be updated on new commits.

@loganharbour loganharbour marked this pull request as ready for review November 4, 2024 14:17
@loganharbour
Copy link
Member Author

loganharbour commented Nov 4, 2024

This is ready. It makes it easier to copy content from HPC test results into an external directory (for the bison assessment, for example).

It also removes the ability to provide multiple spec files for a single run, which was inconsistently supported and not used in practice.

@milljm
Copy link
Member

milljm commented Nov 4, 2024

I think you need to update the TestHarness hash. Unless we're ignoring this for the time being.

@loganharbour
Copy link
Member Author

I think you need to update the TestHarness hash. Unless we're ignoring this for the time being.

It's already updated, hence why https://civet.inl.gov/job/2526480/ ran

@loganharbour loganharbour merged commit 7c0efca into idaholab:next Nov 4, 2024
51 checks passed
@loganharbour loganharbour deleted the better_json branch November 4, 2024 15:38
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.

Simplify previous test results output
3 participants