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. » > _______________________________________________ Denemo-devel mailing list [email protected] https://lists.gnu.org/mailman/listinfo/denemo-devel
