openlp-core team mailing list archive

[Raoul Snyman <raoul@xxxxxxxxxxx>]

Review: Needs Fixing

Line 17, 26: need a space after the comma

  range(0,len(VerseType.names)) => range(0, len(VerseType.names))


Line 76: Your comment doesn't match what you're actually testing:

  # THEN: The result should be VerseType.Other
  self.assertIsNone(...
-- 
https://code.launchpad.net/~strada/openlp/bug-1312629/+merge/217295
Your team OpenLP Core is subscribed to branch lp:openlp.