Remove TestFakeMonths test case. The method it was testing no longer exist so presumably this isnt needed anymore
Moved test data creation for TestTallyMembership into the the testcase itself. I think Django is doing some in memory QuerySet for testcases - so the Transactions being created didnt have valid FKs to the test Member and User when they were created outside the testcase. Putting them all together fixes it. Not sure if we will want to do this long term but reckon it is good enough to get all our tests passing again :+1:
tannercollin
commented
2 years ago
Tests were failing so fixed those up:
TestFakeMonthstest case. The method it was testing no longer exist so presumably this isnt needed anymoreTestTallyMembershipinto the the testcase itself. I think Django is doing some in memoryQuerySetfor testcases - so theTransactions being created didnt have valid FKs to the testMemberandUserwhen they were created outside the testcase. Putting them all together fixes it. Not sure if we will want to do this long term but reckon it is good enough to get all our tests passing again :+1: