test_IO clean-up
Open
Nobody has claimed this yet.
- Dominant language
- C++
- Stars
- 160
- Forks
- 113
- Avg merge
- 12d 15h
- Merged PRs (30d)
- 1
Description
I think test_OutputfileFormat.cxx is now obsolete (superseded by files in test/IO). It can therefore be removed (also from test/CMakeLists.txt). @rijobro, can you confirm?
I think also best to add a CMakeLists.txt in test/IO and include the subdir, as opposed to adding them via test/CMakeLists.txt.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Inspect test_OutputfileFormat.cxx and test/CMakeLists.txt, then compare them with the files already under test/IO. Confirm whether the older test is superseded, remove it and its CMake entry if so, and add test/IO/CMakeLists.txt with the parent-directory inclusion. Done means the test layout and CMake configuration no longer reference the obsolete file.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cmake, cpp
- Domain
- build-system, testing
- Issue type
- Refactor
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100