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

 
 

bug #46913: Fix failure in testsuite

Submitter:  Dr. Tobias Quathamer <tt>
Submitted:  Sat 16 Jan 2016 02:34:02 PM UTC
   
 
Category:  None Severity:  3 - Normal
Item Group:  None Status:  Fixed
Privacy:  Public Assigned to:  rshann
Originator Name:  Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

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>
Group administrator
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>
Group 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>
Group administrator
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>
Group Member

 

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

Attach Files:
   
   
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
  • -email is unavailable- added by rshann (Posted a comment)
  • -email is unavailable- added by tt (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2016-01-17 tt StatusNone Fixed
        Assigned toNone rshann
        Open/ClosedOpen Closed
    2016-01-16 tt Attached File- Added fix-test.patch, #36057

    Back to the top

    Powered by Savane 3.13-caa5.
    Corresponding source code