A bit of a naming collision here -- it just seems a bit dodgy to overwrite the definition of TestValue derived from iris.tests.unit.analysis.maths with a new, local one.  E.G. if you added another TestSomething(...TestValue) in this module it would probably fail.
Can maybe call it just "TestAdd", or whilst it's the only one just "Test" ?