bugDenemo - Bugs: bug #46913, Fix failure in testsuite

 
 

bug #46913: Fix failure in testsuite

Submitted by:  Dr. Tobias Quathamer <tt>
Submitted on:  Sat 16 Jan 2016 02:34:02 PM UTC  
 
Category: NoneSeverity: 3 - Normal
Item Group: NoneStatus: Fixed
Privacy: PublicAssigned to: Richard Shann <rshann>
Open/Closed: Closed

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Sun 17 Jan 2016 05:28:52 PM UTC, comment #3:

I should say that I have manually imported all the musicXML files to check that none of them cause Denemo to crash or hang, which is as much as the test could be doing AFAIK.
The scm test failure is more interesting ... it loads differently depending on whether the pref spillover is set true or not

Richard Shann <rshann>
Project AdministratorIn charge of this item.
Sun 17 Jan 2016 11:01:43 AM UTC, comment #2:

Thanks, now it should be possible to work out why the two other test groups (MusicXML and scm) fail.

Dr. Tobias Quathamer <tt>
Project Member
Sat 16 Jan 2016 05:50:23 PM UTC, comment #1:

http://denemo.org/maintainer-page/
gives the target
make -C tests check
for running the tests.
Running that the tests are failing as you describe, I've no idea why :( so I've applied your patch to work around it.

Richard Shann <rshann>
Project AdministratorIn charge of this item.
Sat 16 Jan 2016 02:34:02 PM UTC, original submission:

Hi,

during building denemo, I ran "make check" which failed the integration tests due to the same test case path used multiple times.

I've attached a small patch which ensures a unique test case path for the various tests. This way, the first two test groups pass. It's generated against the current git master HEAD.

The last two lines in the patch temporarily disable the testing of MusicXML and "scm" files, as those fail for other reasons I'm not able to debug.

However, I thought that you might like to include the patch to enable testing again.

Regards,
Tobias

Dr. Tobias Quathamer <tt>
Project Member

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

Attached Files
file #36057:  fix-test.patch added by tt (2KiB - text/x-patch - Fix failure in testsuite)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by rshann (Posted a comment)
  • -unavailable- added by tt (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 4 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Sun 17 Jan 2016 02:49:29 PM UTCttStatusNone=>Fixed
      Assigned toNone=>rshann
      Open/ClosedOpen=>Closed
    Sat 16 Jan 2016 02:34:02 PM UTCttAttached File-=>Added fix-test.patch, #36057

    Back to the top


    Powered by Savane 3.1-cleanup1