I saw you added some .xml test files. For the moment, only .mxml files are loaded, I will edit the test so .xml are loaded too.
2014-04-04 18:34 GMT+02:00 Richard Shann <[email protected]>: > On Fri, 2014-04-04 at 16:12 +0200, Éloi Rivard wrote: > > Test suite now tries to open, save and re-open every .denemo file in > > "examples" and every .mxml files in "tests/integration-data". > > That's a good start, but I think it really needs to check the > saved .denemo file has not regressed. We would need to check in a set of > expected .denemo files, which could be done quite neatly by make check > doing a diff against the expected .denemo file if it exists else copying > the .denemo file it has created into the directory of expected .denemo > files. Once a "git add" has been done on the "expected" directory then > make check will be doing a regression test. > This is particularly needed while we have no test that any scheme script > ended without throwing an exception from Guile. > There are no scripts that deliberately cause errors now - I updated the > one that did that - but many scripts trigger an error if the user > responds strangely or if the user is using the guile command line in > Denemo. Such a guile throw wouldn't happen for scripts we use as tests > (such as mxml import) so we could put something in the current guile > throw handler for script errors that make check could detect, without > having to exit Denemo abnormally. (I'm not sure what that would be in > the environment that travis uses - perhaps a file that is written to if > the standard_handler() in view.c is executed). > > But even if with that I think we need to check for changes to > generated .denemo files, (the most common problem would be that some > innocent-looking change would result in the cursor not being in the same > position and then the score generated would end up completely > different). > > Does that sound ok to you? > > Richard > > > > > > If you want to do some more mxml checks, you just needs to add some > > files in tests/integration-data > > > > > > > > 2014-04-03 0:29 GMT+02:00 Éloi Rivard <[email protected]>: > > Richard, > > > > Does the "refactor" branch seem ok to you ? > > > > > > > > 2014-03-31 18:05 GMT+02:00 Richard Shann > > <[email protected]>: > > > > Jeremiah - please don't build from the latest commit > > > http://git.savannah.gnu.org/gitweb/?p=denemo.git;a=commit;h=af6ae4aaa753e77e5dada20f5328959cbe76971f > > > > as it breaks Denemo in ways I have yet to fathom. > > Attached is a file > > which just crashed trying to open it. > > This file could be corrupt (because of the crash) ... > > > > Richard > > > > > > > > > > -- > > Éloi Rivard - [email protected] > > > > « On perd plus à être indécis qu'à se tromper. » > > > > > > > > > > -- > > Éloi Rivard - [email protected] > > > > « On perd plus à être indécis qu'à se tromper. » > > > > > -- Éloi Rivard - [email protected] « On perd plus à être indécis qu'à se tromper. »
_______________________________________________ Denemo-devel mailing list [email protected] https://lists.gnu.org/mailman/listinfo/denemo-devel
