Gruntfile.js

@@ -161,12 +161,9 @@ module.exports = function(grunt) {
             normal: {
                 options: {
                     urls: [ "http://localhost:8000/test/test.html" ],
-                    timeout: 10000,
+                    timeout: 10000
-                    puppeteer: {
-                        headless: 'new'
                 }
             },
-            },
             coverage: {
                 options: {
                     urls: [ "http://localhost:8000/test/coverage.html" ],

changelog.txt

@@ -6,7 +6,6 @@ OPENSEADRAGON CHANGELOG
 * BREAKING CHANGE: Dropped support for IE11 (#2300, #2361 @AndrewADev)
 * DEPRECATION: The OpenSeadragon.createCallback function is no longer recommended (#2367 @akansjain)
 * Fixed: Sometimes if the viewport was flipped and the user zoomed in far enough, it would flip back (#2364 @SebDelile)
-* Test improvements (#2382 @AndrewADev)
 
 4.1.0:
 

package.json

@@ -35,7 +35,7 @@
         "grunt-contrib-compress": "^2.0.0",
         "grunt-contrib-concat": "^2.1.0",
         "grunt-contrib-connect": "^3.0.0",
-        "grunt-contrib-qunit": "^7.0.1",
+        "grunt-contrib-qunit": "^6.2.0",
         "grunt-contrib-uglify": "^5.0.1",
         "grunt-contrib-watch": "^1.1.0",
         "grunt-eslint": "^24.0.1",

test/modules/basic.js

@@ -231,36 +231,25 @@
             return;
         }
 
-        viewer.addHandler('open', function () {
+        viewer.addHandler("open", function () {
             assert.ok(!OpenSeadragon.isFullScreen(), 'Started out not fullscreen');
 
-            const checkEnteringPreFullScreen = (event) => {
+            const checkEnteringPreFullScreen = function(event) {
                 viewer.removeHandler('pre-full-screen', checkEnteringPreFullScreen);
                 assert.ok(event.fullScreen, 'Switching to fullscreen');
                 assert.ok(!OpenSeadragon.isFullScreen(), 'Not yet fullscreen');
             };
 
-            const checkExitingFullScreen = (event) => {
+            // The fullscreen mode is always denied during tests so we are
+            // exiting directly.
+            const checkExitingFullScreen = function(event) {
                 viewer.removeHandler('full-screen', checkExitingFullScreen);
-                assert.ok(!event.fullScreen, 'Disabling fullscreen');
+                assert.ok(!event.fullScreen, 'Exiting fullscreen');
-                assert.ok(!OpenSeadragon.isFullScreen(), 'Fullscreen disabled');
+                assert.ok(!OpenSeadragon.isFullScreen(), 'Disabled fullscreen');
                 done();
-            }
-
-            // The 'new' headless mode allows us to enter fullscreen, so verify
-            // that we see the correct values returned. We will then close out
-            // of fullscreen to check the same values when exiting.
-            const checkAcquiredFullScreen = (event) => {
-                viewer.removeHandler('full-screen', checkAcquiredFullScreen);
-                viewer.addHandler('full-screen', checkExitingFullScreen);
-                assert.ok(event.fullScreen, 'Acquired fullscreen');
-                assert.ok(OpenSeadragon.isFullScreen(), 'Fullscreen enabled');
-                viewer.setFullScreen(false);
             };
-
+            viewer.addHandler("pre-full-screen", checkEnteringPreFullScreen);
-
+            viewer.addHandler("full-screen", checkExitingFullScreen);
-            viewer.addHandler('pre-full-screen', checkEnteringPreFullScreen);
-            viewer.addHandler('full-screen', checkAcquiredFullScreen);
             viewer.setFullScreen(true);
         });