This looks like excellent work. I have one mis-giving:
373 - BranchCannotChangeInformationType, 374 - branch.setPrivate, 375 - True, branch.owner) 376 + CannotChangeInformationType, 377 + branch.setPrivate, True, branch.owner)
395 - BranchCannotChangeInformationType, 396 - branch.setPrivate, 397 - False, branch.owner) 398 + CannotChangeInformationType, 399 + branch.setPrivate, False, branch.owner)
Given you've added text to the branch cases that raise CannotChangeInformationType, can you flip these tests to making use of self.assertRaisesWithContent() and make certain the exception text is right.
« Back to merge proposal
This looks like excellent work. I have one mis-giving:
373 - BranchCannotCha ngeInformationT ype, ormationType,
374 - branch.setPrivate,
375 - True, branch.owner)
376 + CannotChangeInf
377 + branch.setPrivate, True, branch.owner)
395 - BranchCannotCha ngeInformationT ype, ormationType,
396 - branch.setPrivate,
397 - False, branch.owner)
398 + CannotChangeInf
399 + branch.setPrivate, False, branch.owner)
Given you've added text to the branch cases that raise CannotChangeInf ormationType, can you flip these tests to making use of self.assertRais esWithContent( ) and make certain the exception text is right.