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

unit testing standards #8

Open
jared321 opened this issue Jun 28, 2022 · 1 comment
Open

unit testing standards #8

jared321 opened this issue Jun 28, 2022 · 1 comment
Assignees

Comments

@jared321
Copy link

Klaus has pointed me toward the unit test architecture section of the User Manual, which is very useful. However, it does not specify how unit tests are supposed to communicate the SUCCESS or FAILURE status of the test. This is necessary so that test developers can do their work with the assurance that their test will communicate properly with Flash Test and therefore with CI servers.

AFAIK, this communication is via a file named unitTest_XXXX. Based on looking at a few Driver_evolveAll files in the unitTest folder, I assume that success is communicated by writing "all results conformed with expected values." If this line is not found in the file, then I suppose that implies failure. Is the value of XXXX important? Does that line have to be the final line in the file?

If this information exists elsewhere, perhaps the User Manual should point to it. If not, then perhaps someone who knows the fine details of this requirement could add it to the existing unit test section of the User Manual so that Flash-X novices are not expected to reverse engineer requirements by looking at existing (and possibly broken) tests.

Assigned to Anshu to determine if this should be done and if so to whom this Issue should be assigned.

@srichers added as he might have some input on this request.

@kweide
Copy link
Member

kweide commented Aug 4, 2022

The XXXX represents the MPI rank of the process writing the file.
I assume this is designed this way with tests in mind where several ranks each compute an error value, presumably from sone sort of distributed solution data, and the test as a whole passes IFF the error value of each rank is 0 (or below a threshold).

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

No branches or pull requests

3 participants