-
Type: Improvement
-
Resolution: Unresolved
-
Priority: Major - P3
-
None
-
Affects Version/s: None
-
Component/s: Test Coverage
-
Labels:None
CrudTest is expecting a failed insertMany to just return an error result but still include all the various counts (insertedCount, etc). TransactionTest is expecting an exception to be thrown. JsonPoweredCrudTestHelper has to deal with both situations, and it results in spaghetti code where JsonPoweredCrudTestHelper has to check whether there's an active transaction and either throw or not based on that.
We should clean up this technical debt.