LukasJoswiak / LukasJoswiak/draft
Improve test build process
Nobody has claimed this yet.
- Dominant language
- C++
- Stars
- 4
- Forks
- 1
- PR merge metrics
- No merged PRs in 30d
Description
The current model to compile the testing suite is to include each test file in a single file tests.cpp. This file is then compiled instead of compiling and linking each test file separately. Compiling a single file which includes all test files is not ideal because any time a test is changed the entire test suite must be rebuilt. However, because draft is a header-only library, each compiled test object file contains all of draft's definitions, resulting in duplicate definition errors on linking.
This issue is to track progress on reaching a better solution to building tests for draft.
Contributor guide
No contributing guide indexed for this repository
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
Start with tests.cpp and inspect how the testing suite is currently compiled. Compare the existing single-file include approach with the project’s available test build configuration. Done means tests can be built without recompiling the entire suite after every test change and without duplicate definition errors when test files are compiled and linked separately.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- build-system, testing
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100