Skip to content

Conversation

@john-science
Copy link
Member

What is the change? Why is it being made?

Here I move more of the ARMI testing tools to the testing module:

  • armi/tests/armiRun-SHUFFLES.txt -> armi/testing/resources/armiRun-SHUFFLES.txt
  • armi/tests/armiRun-SHUFFLES.yaml -> armi/testing/resources/armiRun-SHUFFLES.yaml
  • armi/tests/COMPXS.ascii -> armi/testing/resources/COMPXS.ascii
  • armi/tests/smallHexReactor -> armi/testing/reactors/smallHexReactor

This is part of the overall work for creating a armi.testing module as outlined in #1867

SCR Information

Change Type: trivial

One-Sentence Rationale: We would like to create an armi.testing module with a small set of high-quality testing tools.

One-line Impact on Requirements: NA


Checklist

@john-science john-science added the testing Related to tests label Nov 26, 2025
@john-science
Copy link
Member Author

@opotowsky I ran that downstream test suite and found that this PR made no impact. Success!

Copy link
Member

@opotowsky opotowsky left a comment

Choose a reason for hiding this comment

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

One non-blocking comment! Approved

@john-science john-science merged commit 7cfb959 into main Dec 3, 2025
12 checks passed
@john-science john-science deleted the testing_refactor_2 branch December 3, 2025 17:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testing Related to tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants