Uploaded image for project: 'Derby'
  1. Derby
  2. DERBY-658

test harness improvements required for running on non-ASCII systems

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 10.1.1.0
    • Fix Version/s: 10.1.3.1
    • Component/s: Test
    • Labels:
      None
    • Environment:
      all but especially testing should be done on zOS

      Description

      The current functionTests test harness needs adjustment for running on non-ASCII systems like zOS.
      Currently, when using derbyTesting.jar built for instance on a windows or linux system on zOS the tests do not run, because the properties and runall files cannot be understood.
      Until now, testers on zOS had to unjar derbyTesting.jar, then run native2ascii -Cp 1047 -reverse on all appropriate files (.sql, .txt, .out, .properties, .runall, .asc, .exclude, etc).
      This is a labor intensive and error prone process. Furthermore, it causes test failures such as reported with DERBY-575, because tests may assume a certain file to be a specific length, which no longer holds true after the native2ascii conversion.

      The test harness should get modified to always read the files in the same encoding.

      Note however, that the comparison of actual .out and the master should still result in files readable on the local system, to enable a human to evaluate failures and results. At the same time, this raises the concern that someone might check-in an update to the master with an incorrect encoding.

      To resolve the main issue, I propose the following:

      • Set the default encoding in the harness.
      • for each test
        1) determine if the test encoding is set. We can probably use ij.ui.codeset - otherwise a new property is needed.
        Note that this means that .properties, .runall and .exclude files are always read in fixed/default encoding.
        2) read the master/sql files in in the default/fixed encoding unless the encoding property is set for a test
        3) Write the output out in the local encoding (the way is done currently) unless the encoding property is set (if set, write out in that encoding)
        4) Change the code that creates tmpmstr to always apply instead of only for networkserver. tmpmstr files will be created in the local encoding.
        5) Have FileCompare read tmpmstr in in the local encoding for the comparison.
        6) either document that test development/adjustment need to be at least be verified on an ascii system, or add another property that causes a copy of the actual output to be created in the default/fixed encoding.

        Attachments

        1. DERBY-658_101_20060418.diff
          11 kB
          Myrna van Lunteren
        2. DERBY-658_101_20060418.stat
          0.2 kB
          Myrna van Lunteren
        3. DERBY-658_102_20060418.diff
          14 kB
          Myrna van Lunteren
        4. DERBY-658_102_20060418.stat
          0.2 kB
          Myrna van Lunteren
        5. DERBY-658_101_20060413.diff
          16 kB
          Myrna van Lunteren
        6. DERBY-658_101_20060413.stat
          0.7 kB
          Myrna van Lunteren
        7. DERBY-658_102_20060413.diff
          16 kB
          Myrna van Lunteren
        8. DERBY-658_102_20060413.stat
          8 kB
          Myrna van Lunteren
        9. run_resource.patch.txt
          1 kB
          Samuel Andrew McIntyre
        10. DERBY-658_102_20060307_1.diff
          338 kB
          Myrna van Lunteren
        11. DERBY-658_102_20060307_1.stat
          11 kB
          Myrna van Lunteren
        12. DERBY-658_101_20060407_1.diff
          195 kB
          Myrna van Lunteren
        13. DERBY-658_101_20060407_1.stat
          10 kB
          Myrna van Lunteren

          Issue Links

            Activity

              People

              • Assignee:
                myrna Myrna van Lunteren
                Reporter:
                myrna Myrna van Lunteren
              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: