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

Removed h5test.* from the C++ test #4629

Closed
wants to merge 10 commits into from

Conversation

bmribler
Copy link
Contributor

@bmribler bmribler commented Jul 8, 2024

Revised the C++ tests to eliminate the use of private header files. The solution is to use some functions in the tools library and duplicate some functions from h5test.*. Some of these duplicated functions can be added to the tools library at a later time, however.

Also, there are many small changes to improve the overall outputs of the tests.

Note: The changes to c++/test/Makefile.am will be committed soon.

(Issue GH-4517)

Revised the C++ tests to eliminate the use of private header files. The solution is to use some functions in the tools library and duplicate some functions from h5test.*.  Some of these duplicated functions can be added to the tools library at a later time, however.

Also, there are many small changes to improve the overall outputs of the tests.

Note: The changes to c++/test/Makefile.in will be committed soon.

(Issue HDFGroupGH-4517)
@bmribler bmribler marked this pull request as draft July 8, 2024 08:41
@bmribler bmribler marked this pull request as ready for review July 9, 2024 07:23
@byrnHDF
Copy link
Contributor

byrnHDF commented Jul 9, 2024

Need to change the CMakeLists.txt file needs the same changes as Makefile.am.

@bmribler bmribler marked this pull request as draft July 9, 2024 14:03
@jhendersonHDF
Copy link
Collaborator

This works, but from a maintenance perspective I don't know that duplicating a bunch of the library's testing framework is really the way that you'd want to go with this.

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.

3 participants