Improve define/declare mixin tests
Nobody has claimed this yet.
- Dominant language
- C++
- Stars
- 693
- Forks
- 49
- PR merge metrics
- No merged PRs in 30d
Description
Currently most of the tests of object manipulation from mixins created using DYNAMIX_DECLARE_MIXIN and DYNAMIX_DEFINE_MIXIN are in t-define_mixin.cpp (and some in t-msg.cpp). It's too dense and likely there are use cases which are missed. Split them in more files using a test library with mixins, rules, and the like.
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
Read t-define_mixin.cpp and the related tests in t-msg.cpp to map the existing DYNAMIX_DECLARE_MIXIN and DYNAMIX_DEFINE_MIXIN coverage. Organize the cases into focused test files with a shared library for mixins, rules, and related objects; done means the tests remain equivalent while making missed use cases easier to add.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- testing
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100