OpenLP

Code review comment for lp://qastaging/~gabriel-loo/openlp/songselectimportfix

  1. songselectimportfix
  2. Merge into trunk
Revision history for this message
Gabriel Loo (gabriel-loo) wrote :

Hi Raoul,

I've added unit tests as requested.

Here is the Jenkins output:

lp:~gabriel-loo/openlp/songselectimportfix (revision 2642)
[SUCCESS] https://ci.openlp.io/job/Branch-01-Pull/1456/
[SUCCESS] https://ci.openlp.io/job/Branch-02-Functional-Tests/1373/
[SUCCESS] https://ci.openlp.io/job/Branch-03-Interface-Tests/1311/
[SUCCESS] https://ci.openlp.io/job/Branch-04a-Windows_Functional_Tests/1115/
[SUCCESS] https://ci.openlp.io/job/Branch-04b-Windows_Interface_Tests/706/
[SUCCESS] https://ci.openlp.io/job/Branch-05a-Code_Analysis/773/
[SUCCESS] https://ci.openlp.io/job/Branch-05b-Test_Coverage/641/

review: Needs Resubmitting

« Back to merge proposal