[Gzz-commits] gzz ./TODO doc/pegboard/tests_output_cleanup--v...

[Vesa Kaihlavirta]

CVSROOT:        /cvsroot/gzz
Module name:    gzz
Changes by:     Vesa Kaihlavirta <address@hidden>       03/01/09 21:42:10

Modified files:
        .              : TODO 
        doc/pegboard/tests_output_cleanup--vegai: peg.rst 

Log message:
        PEG nearly complete, ie. incomplete.

CVSWeb URLs:
http://savannah.gnu.org/cgi-bin/viewcvs/gzz/gzz/TODO.diff?tr1=1.498&tr2=1.499&r1=text&r2=text
http://savannah.gnu.org/cgi-bin/viewcvs/gzz/gzz/doc/pegboard/tests_output_cleanup--vegai/peg.rst.diff?tr1=1.1&tr2=1.2&r1=text&r2=text

Patches:
Index: gzz/TODO
diff -u gzz/TODO:1.498 gzz/TODO:1.499
--- gzz/TODO:1.498      Thu Jan  9 14:55:38 2003
+++ gzz/TODO    Thu Jan  9 21:42:10 2003
@@ -20,8 +20,8 @@
            get *THE* demo up and running
            
     vegai:
-       - PEG on how "make test" output should be cleaned;
-         do not implement before PEGging.
+       - Implement PEG on how "make test" output should be cleaned once
+          the PEG is accepted.   
        + "make committable": remove CLASSES and all object files,
          compile normal and GL and test both normal and GL;
          if either fails any test not marked failing, fail HARD.
@@ -36,7 +36,7 @@
        + genimages: allow using different formats for generated images.
          Specifically, need to be able to use gzipped S3TC files; 
          should be a LOT faster to load.
-    benaja:
+    benja:
         + DimDecorator's text is broken on GL-side. Like text seems to be
           to tight to be readable. Probably problems with coordinate
           systems? Other texts works fine.
Index: gzz/doc/pegboard/tests_output_cleanup--vegai/peg.rst
diff -u gzz/doc/pegboard/tests_output_cleanup--vegai/peg.rst:1.1 
gzz/doc/pegboard/tests_output_cleanup--vegai/peg.rst:1.2
--- gzz/doc/pegboard/tests_output_cleanup--vegai/peg.rst:1.1    Thu Jan  9 
09:00:08 2003
+++ gzz/doc/pegboard/tests_output_cleanup--vegai/peg.rst        Thu Jan  9 
21:42:10 2003
@@ -3,10 +3,147 @@
 =============================================================
 
 :Author:   Vesa Kaihlavirta
-:Last-Modified: $Date: 2003/01/09 14:00:08 $
-:Revision: $Revision: 1.1 $
+:Last-Modified: $Date: 2003/01/10 02:42:10 $
+:Revision: $Revision: 1.2 $
 :Status:   Incomplete
 :Scope:           Minor
 :Type:     ?
 
-SKELETON
\ No newline at end of file
+
+Our tests' output is seriously cluttered. This PEG will propose standards
+that, when implemented, will result in clean test output.
+
+Clean test output includes the test docstring -- which should take only one
+line -- and the result ("ok", or "fail" and traceback, or a reason why the
+test wasn't run).
+
+Problems
+========
+
+Listing individual problems from current (2003/01/09) test output.
+
+
+1. When test cost is considered, output is bizarre::
+
+    TEST COST  <function testStarts at 2220309> 10
+    TOO COSTLY TEST <function testStarts at 2220309>
+
+
+2. Garbage when initalizing test.gfx::
+
+    Init test.gfx
+    ['__doc__', '__file__', '__name__', '__path__', 'client', 'fuzzy', 'gfx',
+    'impl', 'media', 'util', 'vob']
+    ['GraphicsAPI', '__doc__', '__file__', '__name__', '__path__']
+    Jan 9, 2003 2:09:05 AM java.util.prefs.FileSystemPreferences$3 run
+    WARNING: Could not create system preferences directory. System preferences
+    are unusable.
+    GW:  address@hidden
+  
+3. gzz/media/impl/PageImageScroll.java has pa()::
+
+    Checked document of 0 pages
+
+4. Bizarre error again, not unlike 1.::
+
+    TEST FAILS IN  <function testStripBlock at 4952321> *
+    NOT RUNNING DUE TO WRONG F:  <function testStripBlock at 4952321>
+
+5. test/tools/gfx.py outputs "CBCF init".
+
+6. Some of the tests output debug information, eg::
+
+    Test space saving and re-loading ... []
+    [[[01F3F91514A6E80BCDFD6D978DACD0C40B4999DAF1]],
+    [[019FA251833B0791D550BE3D03EC67234BAACDADC1]],
+    [[01709BC0FC0687E569572437F562456C6D24FEC7FB]]]
+    [[[01F3F91514A6E80BCDFD6D978DACD0C40B4999DAF1]],
+    [[019FA251833B0791D550BE3D03EC67234BAACDADC1]],
+    [[01709BC0FC0687E569572437F562456C6D24FEC7FB]]]
+    [[[01F3F91514A6E80BCDFD6D978DACD0C40B4999DAF1]],
+    [[019FA251833B0791D550BE3D03EC67234BAACDADC1]],
+    [[01709BC0FC0687E569572437F562456C6D24FEC7FB]]]
+    [[[01F3F91514A6E80BCDFD6D978DACD0C40B4999DAF1]],
+    [[019FA251833B0791D550BE3D03EC67234BAACDADC1]],
+    [[01709BC0FC0687E569572437F562456C6D24FEC7FB]]]
+    SliceVersion([SliceVersion.Conn('urn:urn-5:7ZnIxDowCCU4ozTsf6oE45zIRTId:2',
+    'urn:urn-5:7ZnIxDowCCU4ozTsf6oE45zIRTId:1',
+    'urn:urn-5:7ZnIxDowCCU4ozTsf6oE45zIRTId:3')],
+    
{urn:urn-5:7ZnIxDowCCU4ozTsf6oE45zIRTId:1=Enf1DImpl[null,SPAN1D(address@hidden
+    0 3),null]})
+    ... (goes on and on)
+
+7. gzz/impl/mirror/MasterImpl.java outputs "Sending change:".
+
+8. Enfilades' test spurts weird things::
+
+    Test enfilades consisting of no spans and a single span. ... Jan 9, 2003
+    2:09:36 AM java.util.prefs.FileSystemPreferences checkLockFile0ErrorCode
+    WARNING: Could not lock System prefs.Unix error code 134984164. 
+    Jan 9, 2003 2:09:36 AM java.util.prefs.FileSystemPreferences syncWorld
+    Jan 9, 2003 2:09:36 AM java.util.prefs.FileSystemPreferences syncWorld
+    WARNING: Couldn't flush system prefs: 
java.util.prefs.BackingStoreException:
+    Couldn't get file lock.
+
+9. SolidBGVob test outputs debug information, and the same problem as in
+   8.::
+
+    testSingle_solidbgvob (__main__.Boxcellview) ... Jan 9, 2003 2:10:06 AM
+    java.util.prefs.Fi
+    leSystemPreferences checkLockFile0ErrorCode
+    WARNING: Could not lock System prefs.Unix error code 134984164.
+    Jan 9, 2003 2:10:06 AM java.util.prefs.FileSystemPreferences syncWorld
+    WARNING: Couldn't flush system prefs: 
java.util.prefs.BackingStoreException:
+    Couldn't get
+    file lock.
+    ok
+    testSingleNote (__main__.Coordinateplane) ... CBCF init
+    New note
+
+    PLace
+    CPV: 'urn:urn-5:nbYfgs3K6YzPEUPJJ7vfLutFDZta:1' (Uusi paperi) null
+    CPV STEP: 'urn:urn-5:nbYfgs3K6YzPEUPJJ7vfLutFDZta:2' (FOO)
+    'urn:urn-5:nbYfgs3K6YzPEUPJJ7vf
+    LutFDZta:1' (Uusi paperi)
+    CPV: 'urn:urn-5:nbYfgs3K6YzPEUPJJ7vfLutFDZta:2' (FOO)
+    'urn:urn-5:nbYfgs3K6YzPEUPJJ7vfLutFD
+    Zta:3' (100)
+    CBCF getsize ('urn:urn-5:nbYfgs3K6YzPEUPJJ7vfLutFDZta:2' (FOO),
+    gzz.view.AbstractViewConte
+    address@hidden, array([50.0, 50.0], float))
+    CPV PLACE: 100 200 50.0 50.0
+    Place:  1
+    CPV STEP: null 'urn:urn-5:nbYfgs3K6YzPEUPJJ7vfLutFDZta:1' (Uusi paperi)
+    Placed
+    ok
+
+
+Solutions
+=========
+
+1. New format for test cost output::
+
+    [test method docstring] ... skipping (cost too high: 10).
+
+2. Fix init.gfx, it presumably outputs unwanted debug information.
+
+3. Fix method, debug information.
+
+4. What the blorg is "NOT RUNNING DUE TO WRONG F:"?!
+
+5. As in 3.
+
+6. Fix test, debug information.
+
+7. As in 5.
+
+8. No idea currently. Will check into it before this PEG is active.
+
+9. As in 7. and 8.
+
+
+Alternative Solution
+====================
+
+Find a way to disable output during the tests.
+