Comment out some tests

muffinista · Feb 2, 2024 · dbe451e · dbe451e
1 parent a907bc3
commit dbe451e
Show file tree

Hide file tree

Showing 8 changed files with 50 additions and 44 deletions.
diff --git a/test/lib/package.js b/test/lib/package.js
@@ -119,24 +119,24 @@ describe("Package", function() {
     });
   });
 
-  describe("downloadFile", function() {
-    var testUrl = "https://test.file/savers.zip";
-    beforeEach(function() {
-      nock("https://test.file").
-        get("/savers.zip").
-        reply(200, () => {
-          return fs.createReadStream(zipPath);
-        });
-      rimrafSync(workingDir);
-      fs.mkdirSync(workingDir);
-    });
-
-    xit("works", async function() {
-      let p = new Package(attrs);
-      const dest = await p.downloadFile(testUrl);
-      assert(fs.existsSync(dest));
-    });
-  });
+  // describe("downloadFile", function() {
+  //   var testUrl = "https://test.file/savers.zip";
+  //   beforeEach(function() {
+  //     nock("https://test.file").
+  //       get("/savers.zip").
+  //       reply(200, () => {
+  //         return fs.createReadStream(zipPath);
+  //       });
+  //     rimrafSync(workingDir);
+  //     fs.mkdirSync(workingDir);
+  //   });
+
+  //   xit("works", async function() {
+  //     let p = new Package(attrs);
+  //     const dest = await p.downloadFile(testUrl);
+  //     assert(fs.existsSync(dest));
+  //   });
+  // });
 
   describe("zipToSavers", function() {
     var p;

diff --git a/test/lib/prefs.js b/test/lib/prefs.js
@@ -179,7 +179,7 @@ describe("SaverPrefs", function() {
   });
 
   // systemSource
-  describe("sources", function() {
+  describe("systemSource", function() {
     beforeEach(function() {
       prefs = new SaverPrefs(tmpdir);
     });

diff --git a/test/main/power.js b/test/main/power.js
@@ -1,3 +1,4 @@
+/* eslint-disable mocha/no-setup-in-describe */
 "use strict";
 
 

diff --git a/test/ui/about.js b/test/ui/about.js
@@ -1,3 +1,4 @@
+/* eslint-disable mocha/no-setup-in-describe */
 "use strict";
 
 import assert from 'assert';

diff --git a/test/ui/bootstrap.js b/test/ui/bootstrap.js
@@ -1,3 +1,4 @@
+/* eslint-disable mocha/no-setup-in-describe */
 "use strict";
 
 import assert from 'assert';

diff --git a/test/ui/prefs.js b/test/ui/prefs.js
@@ -1,3 +1,4 @@
+/* eslint-disable mocha/no-setup-in-describe */
 "use strict";
 
 import assert from 'assert';

diff --git a/test/ui/settings.js b/test/ui/settings.js
@@ -1,3 +1,4 @@
+/* eslint-disable mocha/no-setup-in-describe */
 "use strict";
 
 import assert from 'assert';
@@ -71,20 +72,20 @@ describe("Settings", function() {
     await helpers.waitFor(app, "prefs");
   });
 
-  it.skip("allows setting path via dialog", async function() {
-    const [fileChooser] = await Promise.all([
-      window.waitForEvent("filechooser"),
-      window.click("button.pick")
-    ]);
-    await fileChooser.setFiles("/not/a/real/path");
+  // it.skip("allows setting path via dialog", async function() {
+  //   const [fileChooser] = await Promise.all([
+  //     window.waitForEvent("filechooser"),
+  //     window.click("button.pick")
+  //   ]);
+  //   await fileChooser.setFiles("/not/a/real/path");
 
-    await window.click("button.save");
-    await helpers.sleep(closeWindowDelay);
+  //   await window.click("button.save");
+  //   await helpers.sleep(closeWindowDelay);
 
-    assert.strictEqual("/not/a/real/path", currentPrefs().localSource);
+  //   assert.strictEqual("/not/a/real/path", currentPrefs().localSource);
 
-    await helpers.waitFor(app, "prefs");
-  });
+  //   await helpers.waitFor(app, "prefs");
+  // });
 
   it("clears localSource", async function() {
     let ls = currentPrefs().localSource;
@@ -102,22 +103,22 @@ describe("Settings", function() {
   });
 
 
-  // dialogs don't work yet
-  // @see https://github.com/microsoft/playwright/issues/8278
-  it.skip("resets defaults", async function() {
-    window = await helpers.waitFor(app, "settings");
+  // // dialogs don't work yet
+  // // @see https://github.com/microsoft/playwright/issues/8278
+  // it.skip("resets defaults", async function() {
+  //   window = await helpers.waitFor(app, "settings");
 
-    window.on("dialog", async dialog => {
-      console.log(dialog.message());
-      await dialog.accept();
-    });
+  //   window.on("dialog", async dialog => {
+  //     console.log(dialog.message());
+  //     await dialog.accept();
+  //   });
 
-    await window.click("button.reset-to-defaults");
-    await helpers.waitForText(window, "body", "Settings reset", true);
-    await helpers.sleep(closeWindowDelay);
+  //   await window.click("button.reset-to-defaults");
+  //   await helpers.waitForText(window, "body", "Settings reset", true);
+  //   await helpers.sleep(closeWindowDelay);
 
-    assert.strictEqual("", currentPrefs().localSource);
+  //   assert.strictEqual("", currentPrefs().localSource);
 
-    await helpers.waitFor(app, "prefs");
-  });
+  //   await helpers.waitFor(app, "prefs");
+  // });
 });
diff --git a/test/ui/tray.js b/test/ui/tray.js
@@ -1,3 +1,4 @@
+/* eslint-disable mocha/no-setup-in-describe */
 "use strict";
 
 import assert from 'assert';
Original file line number	Diff line number	Diff line change
		@@ -1,3 +1,4 @@
		/* eslint-disable mocha/no-setup-in-describe */
		"use strict";


Expand Down