add test coverage for weird case of plugin with multiple impls of same spec#446
add test coverage for weird case of plugin with multiple impls of same spec#446wahuneke wants to merge 1 commit into
Conversation
dc9fb8e to
52d8e31
Compare
RonnyPfannschmidt
left a comment
There was a problem hiding this comment.
well done, those tests verify that my assessment from just/only looking at the code was wrong and now we have a demonstration of the correctness for those cases
87cc255 to
479123e
Compare
479123e to
dedb892
Compare
|
Rebased onto current main and updated for merge:
|
These tests are attempting to find @RonnyPfannschmidt 's theoretical issue described in issue #431 . Conversation in 431 is ongoing. This PR is here for reference. It is a DRAFT.
NOTE: these tests currently pass. They do not detect any new issues.