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

test_dsync: add test_expectfail.sh, test_existence.sh, and stubs for two other tests #596

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

ofaaland
Copy link
Collaborator

@ofaaland ofaaland commented Nov 6, 2024

Add the following tests:

  • test_expectfail.sh: add tests for missing source or destination parent
  • test_existence.sh: verify set of files synced is correct

Add stubs for the following tests:

  • test_metadata.sh: verify metadata of synced files is correct
  • test_data.sh: verify data in synced files is correct

Add a small set of bash functions for performing set operations, for use with dsync tests.

Add a small test driver and two data files for testing the bash functions.

Add a small set of bash functions for performing set operations,
for use with dsync tests.

Add a small test driver and two data files for testing the bash
functions.

Signed-off-by: Olaf Faaland <faaland1@llnl.gov>
Add the following tests:
	test_expectfail.sh: add tests for missing source or destination parent
	test_existence.sh: verify set of files synced is correct

Add stubs for the following tests:
	test_metadata.sh: verify metadata of synced files is correct
	test_data.sh: verify data in synced files is correct

Signed-off-by: Olaf Faaland <faaland1@llnl.gov>
@ofaaland
Copy link
Collaborator Author

ofaaland commented Nov 6, 2024

@gonsie @carbonneau1 I intend this to be a template for other tests, so this is a good time to discuss or object to anything about how I implemented these tests - writing tests as shell scripts, the organization of the tests, the fact that they're single-node tests, etc. Thanks!

Use 1/8 of total cores up to a max of 16 cores on the node.

Signed-off-by: Olaf Faaland <faaland1@llnl.gov>
@ofaaland
Copy link
Collaborator Author

ofaaland commented Nov 6, 2024

Hi @carbonneau1 please see the two new commits, running dsync under "mpirun", and copying a directory tree from /usr/include for use as test data.

We want a deep tree with many files and directories.  dfilemaker1 creates only
a flat tree with all files in one directory.

Instead, copy from /usr/include/c++ which should be reasonably large and
in terms of file count, and have some depth to the tree, on
any installation.

Signed-off-by: Olaf Faaland <faaland1@llnl.gov>
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