On Feb 10, 5:54 pm, "Edward K. Ream" <[email protected]> wrote:

> The fix is on the trunk at rev 2927.  All unit tests pass, but it
> looks like we are missing a test called:
> @test save a new file...

The new unit test is on the trunk at rev 2929.  It's not a perfect
test of all the c.save logic, but it comes close, and is a pretty
convincing test of the c.fileCommands.save logic.

However, this test originally did *not* catch the bug with old code.
It took more than an hour to find why the not: the call to
setCachedGlobalsElement was disabled while unit testing!

Anyway, Leo finally has a unit test for new files.  This is important:
new files are special cases in a lot of ways.  It takes significant
work to maintain the illusion that new files aren't special cases.

Edward

-- 
You received this message because you are subscribed to the Google Groups 
"leo-editor" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/leo-editor?hl=en.

Reply via email to