Remove no longer needed test-utility.js.

Since the previous commit (which changed the way the test_iframe is hidden from the user), we no longer need to manually call the utility methods `showTestIframe` and `hideTestIframe`. Since these methods were the only ones contained in test-utility.js, that file is now obsolete.
geoext · Nov 27, 2014 · e74b595 · e74b595
1 parent c87074c
commit e74b595
Show file tree

Hide file tree

Showing 6 changed files with 0 additions and 137 deletions.
diff --git a/tests/container/VectorLegend.html b/tests/container/VectorLegend.html
@@ -4,8 +4,6 @@
 
 <script src="http://openlayers.org/api/2.13.1/OpenLayers.js"></script>
 
-<script src="../test-utility.js"></script>
-
 <script type="text/javascript">
 
     Ext.Loader.setConfig({
@@ -134,7 +132,6 @@
     }
 
     function test_saferemove(t) {
-        showTestIframe();
         t.plan(3);
 
         var map = new OpenLayers.Map();
@@ -170,7 +167,6 @@
 
                 legend.destroy();
                 map.destroy();
-                hideTestIframe();
             });
         });
     }

diff --git a/tests/panel/Map.html b/tests/panel/Map.html
@@ -6,8 +6,6 @@
 
     <script src="http://openlayers.org/api/2.13.1/OpenLayers.js"></script>
 
-    <script src="../test-utility.js"></script>
-
     <script type="text/javascript">
         Ext.Loader.setConfig({
             disableCaching: false,
@@ -32,11 +30,6 @@
         function test_mappanel(t) {
             t.plan(4)
 
-            // make the hidden span element visible, because setSize does not
-            // work in iframes that are inside a hidden container in FF and IE.
-            // (hidden again at the end of the test)
-            showTestIframe();
-
             var moveToCnt;
 
             var map = createMap();
@@ -75,18 +68,11 @@
             map.destroy();
 
             mapPanel.destroy();
-
-            hideTestIframe();
         }
 
         function test_allOverlaysAndFallThrough(t) {
             t.plan(12);
 
-            // make the hidden span element visible, because setSize does not
-            // work in iframes that are inside a hidden container in FF and IE.
-            // (hidden again at the end of the test)
-            showTestIframe();
-
             var map, panel;
             // a flag being reset after each test to see if a fallThrough
             // warning has been issued upon Ext.create('GeoExt.panel.Map', {})
@@ -175,18 +161,11 @@
 
             // Restore the original warning method.
             GeoExt.panel.Map.prototype.warnMapFallThrough = origWarnMapFallThrough;
-
-            hideTestIframe();
         }
 
         function test_zoom(t) {
             t.plan(1);
 
-            // make the hidden span element visible, because setSize does not
-            // work in iframes that are inside a hidden container in FF and IE.
-            // (hidden again at the end of the test)
-            showTestIframe();
-
             var panel = Ext.create('GeoExt.panel.Map', {
                 title: "GeoExt MapPanel",
                 renderTo: "mappanel",
@@ -199,18 +178,11 @@
             t.eq(panel.map.zoom, 4, "zoom correctly set");
 
             panel.destroy();
-
-            hideTestIframe();
         }
 
         function test_extent(t) {
             t.plan(4);
 
-            // make the hidden span element visible, because setSize does not
-            // work in iframes that are inside a hidden container in FF and IE.
-            // (hidden again at the end of the test)
-            showTestIframe();
-
             var map, panel, log = {};
 
             map = createMap();
@@ -273,18 +245,11 @@
             panel.map.addLayer(new OpenLayers.Layer());
             t.ok(panel.map.getExtent(), "map has an extent after layer is added");
             panel.destroy();
-
-            hideTestIframe();
         }
 
         function test_center(t) {
             t.plan(3);
 
-            // make the hidden span element visible, because setSize does not
-            // work in iframes that are inside a hidden container in FF and IE.
-            // (hidden again at the end of the test)
-            showTestIframe();
-
             var map, panel, log = {};
 
             map = createMap();
@@ -337,8 +302,6 @@
             // since we created the map, we destroy it
             map.destroy();
             panel.destroy();
-
-            hideTestIframe();
         }
 
         function test_destroy(t) {
@@ -352,11 +315,6 @@
              */
 
             t.plan(3);
-
-            // make the hidden span element visible, because setSize does not
-            // work in iframes that are inside a hidden container in FF and IE.
-            // (hidden again at the end of the test)
-            showTestIframe();
 
             var panel = Ext.create('GeoExt.panel.Map', {
                 renderTo: "mappanel",
@@ -380,20 +338,13 @@
                 t.fail("panel.destroy causes problems: " + err);
             }
             t.ok(!panel.map, "panel has no reference to a map");
-
-            hideTestIframe();
         }
 
         function test_applyState_called(t) {
             t.plan(1);
 
             // set up
 
-            // make the hidden span element visible, because setSize does not
-            // work in iframes that are inside a hidden container in FF and IE.
-            // (hidden again at the end of the test)
-            showTestIframe();
-
             var provider, applyState, mapPanel, log;
 
             provider = new Ext.state.Provider();
@@ -425,19 +376,13 @@
 
             GeoExt.panel.Map.prototype.applyState = applyState;
             mapPanel.destroy();
-            hideTestIframe();
         }
 
         function test_applyState(t) {
             t.plan(21);
 
             // set up
 
-            // make the hidden span element visible, because setSize does not
-            // work in iframes that are inside a hidden container in FF and IE.
-            // (hidden again at the end of the test)
-            showTestIframe();
-
             var state;
 
             var layers = [
@@ -549,19 +494,13 @@
             // tear down
 
             mapPanel.destroy();
-            hideTestIframe();
         }
 
         function test_getState_called(t) {
             t.plan(6);
 
             // set up
 
-            // make the hidden span element visible, because setSize does not
-            // work in iframes that are inside a hidden container in FF and IE.
-            // (hidden again at the end of the test)
-            showTestIframe();
-
             var getState, mapPanel, log;
 
             getState = GeoExt.panel.Map.prototype.getState;
@@ -641,20 +580,13 @@
                 GeoExt.panel.Map.prototype.getState = getState;
                 mapPanel.destroy();
             });
-
-            hideTestIframe();
         }
 
         function test_getState(t) {
             t.plan(14);
 
             // set up
 
-            // make the hidden span element visible, because setSize does not
-            // work in iframes that are inside a hidden container in FF and IE.
-            // (hidden again at the end of the test)
-            showTestIframe();
-
             var state;
 
             // test
@@ -732,11 +664,9 @@
                  "state.opacity_barid correctly set");
 
             mapPanel.destroy();
-            hideTestIframe();
         }
 
         function test_add(t) {
-            showTestIframe();
             t.plan(1);
             var layers = [
                 new OpenLayers.Layer("foo", {visibility: true}),
@@ -758,13 +688,11 @@
             t.ok(mapPanel.items.get(0).getEl().dom.parentNode === mapPanel.body.dom,
                  "Map Panel item has the panels's body div as parent.");
             mapPanel.destroy();
-            hideTestIframe();
         }
 
         // This test is originally coming from issue #61:
         // https://github.com/geoext/geoext2/pull/61
         function test_destroyInTabbedPanelTest(t) {
-            showTestIframe();
             t.plan(3);
 
              var layers = [
@@ -829,8 +757,6 @@
              mappanel.destroy();
 
              t.eq(panel.items.length, 0, "Panel destroy was successful");
-
-             hideTestIframe();
          }
 
          function test_events(t) {

diff --git a/tests/selection/FeatureModel.html b/tests/selection/FeatureModel.html
@@ -6,8 +6,6 @@
 
     <script src="http://openlayers.org/api/2.13.1/OpenLayers.js"></script>
 
-    <script src="../test-utility.js"></script>
-
     <script type="text/javascript">
         Ext.Loader.setConfig({
             disableCaching: false,
@@ -397,7 +395,6 @@
         }
 
         function test_autopan_map_on_selection(t) {
-            showTestIframe();
             t.plan(4);
 
             /*
@@ -490,7 +487,6 @@
                  * Tear down
                  */
                 grid.destroy();
-                hideTestIframe();
             });
 
         }

diff --git a/tests/slider/Zoom.html b/tests/slider/Zoom.html
@@ -5,8 +5,6 @@
 
     <script src="http://openlayers.org/api/2.13.1/OpenLayers.js"></script>
 
-    <script src="../test-utility.js"></script>
-
     <script type="text/javascript">
         Ext.Loader.setConfig({
             disableCaching: false,
@@ -22,7 +20,6 @@
         ]);
 
         function test_zoomslider(t) {
-            showTestIframe();
             t.plan(7);
 
             var map = new OpenLayers.Map({
@@ -72,15 +69,13 @@
                     }
 
                     map.destroy();
-                    hideTestIframe();
                 });
             });
 
 
         }
 
         function test_zoomslider_aggressive(t) {
-            showTestIframe();
             t.plan(2);
             var slider1 = new GeoExt.ZoomSlider({
                 renderTo: document.body
@@ -105,7 +100,6 @@
 
             slider1.destroy();
             slider2.destroy();
-            hideTestIframe();
         }
 
     </script>

diff --git a/tests/test-utility.js b/tests/test-utility.js