GeoServer
  1. GeoServer
  2. GEOS-5106

Build failure in web-demo MapPreviewPageTest.testLayerNamesPrefixed caused by unwarranted order assumption

    Details

    • Type: Bug Bug
    • Status: Open Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 2.2.x
    • Fix Version/s: None
    • Component/s: Wicket UI
    • Labels:
      None
    • Environment:
    • Number of attachments :
      0

      Description

      MapPreviewPageTest.testLayerNamesPrefixed assumes that cite:streams is rendered on the second page. This is not guaranteed, and on my platform this assumption causes the build to fail.

      On my platform, commenting out these lines causes the build to pass.

      //move to next page
      tester.clickLink("table:navigatorBottom:navigator:next", true);
      

      (I deduce that cite:streams is on the first page.)

      The failure is present in both Maven and Eclipse.

        Issue Links

          Activity

          Hide
          Ben Caradoc-Davies added a comment -

          This failure was introduced by the changes in r17038 for GEOS-5099.

          Show
          Ben Caradoc-Davies added a comment - This failure was introduced by the changes in r17038 for GEOS-5099 .
          Hide
          Ben Caradoc-Davies added a comment -

          As a temporary workaround I Xed out this test in r17057.

          Show
          Ben Caradoc-Davies added a comment - As a temporary workaround I Xed out this test in r17057.
          Show
          Ben Caradoc-Davies added a comment - It was failing on our buildbot too: http://geobuilder.arrc.csiro.au/geoserver/builders/GeoT-java15/builds/6455/steps/shell_1/logs/stdio http://geobuilder.arrc.csiro.au/geoserver/builders/GeoT-java15
          Hide
          Ben Caradoc-Davies added a comment -

          Not a blocker since temporary workaround was committed.

          Show
          Ben Caradoc-Davies added a comment - Not a blocker since temporary workaround was committed.
          Hide
          Ben Caradoc-Davies added a comment -

          This test coverage is desirable. Any chance of fixing it?

          Show
          Ben Caradoc-Davies added a comment - This test coverage is desirable. Any chance of fixing it?

            People

            • Assignee:
              Alessio Fabiani
              Reporter:
              Ben Caradoc-Davies
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated: