To catch stuff like T204423: MW 1.31 install reports "InvalidArgumentException ... DatabaseDomain.php: Domain has too few or too many parts ". Dashes are special because of how DatabaseDomain handles them, so I think it's worth integrating into quibble.
Description
Description
Related Objects
Related Objects
Event Timeline
Comment Actions
Is this something a limited or partially mocked integration test could catch instead?
I don't mind whether Quibble uses it or not (either is fine I suppose), but it tends to scale better and benefit other run-times' test coverage if we "test" it directly.
Comment Actions
A unit tests for DatabaseDomain might be too low-level, a slightly higher-level of integration (whilst still not requiring an actual database) might be better.
But, it depends on whether the original issue would've been prevented. Perhaps a test case for DatabaseDomain would suffice, in which case that should suffice for this task as well.