Bug 1469054 - Skip some test cases in test_adb.js on Windows. r?jdescottes draft
authorHiroyuki Ikezoe <hikezoe@mozilla.com>
Thu, 09 Aug 2018 15:43:44 +0900
changeset 827793 938fe3b313597bc0eae6ebcadd56a84460738a7c
parent 827792 f10bec1379b96d7487aee9572d1df1bcdcfa67d1
push id118584
push userhikezoe@mozilla.com
push dateThu, 09 Aug 2018 06:45:13 +0000
reviewersjdescottes
bugs1469054
milestone63.0a1
Bug 1469054 - Skip some test cases in test_adb.js on Windows. r?jdescottes On Windows testNoTargetBinaries fails since we don't remove extracted binaries in the profile directory when the extension is removed. The original adbhelper addon doesn't remove them either, so this problem has been underlying originally. We will fix it in a later bug. Also on Windows, testStartAndStop and testTrackDevices fail. That's because, I guess, the adb mock (adb.py) isn't launched properly. I will handle it once after I take a windows laptop back. MozReview-Commit-ID: 15wo1S0pX37
devtools/shared/adb/test/test_adb.js
--- a/devtools/shared/adb/test/test_adb.js
+++ b/devtools/shared/adb/test/test_adb.js
@@ -95,17 +95,19 @@ add_task(async function testNoAdbJSON() 
   await extension.startup();
 
   const adbBinary = await getFileForBinary();
   equal(adbBinary, null);
 
   await extension.unload();
 });
 
-add_task(async function testNoTargetBinaries() {
+add_task({
+  skip_if: () => mozinfo.os == "win" // bug 1482008
+}, async function testNoTargetBinaries() {
   const extension = ExtensionTestUtils.loadExtension({
     manifest: {
       version: "1.0",
       applications: {
         gecko: { id: "adb@mozilla.org" }
       }
     },
     files: {
@@ -143,17 +145,19 @@ add_task(async function testExtract() {
   await extension.startup();
 
   const adbBinary = await getFileForBinary();
   ok(await adbBinary.exists);
 
   await extension.unload();
 });
 
-add_task(async function testStartAndStop() {
+add_task({
+  skip_if: () => mozinfo.os == "win" // bug 1482008
+}, async function testStartAndStop() {
   const extension = ExtensionTestUtils.loadExtension({
     manifest: {
       version: "1.0",
       applications: {
         gecko: { id: "adb@mozilla.org" }
       }
     },
     files: {
@@ -175,17 +179,19 @@ add_task(async function testStartAndStop
   ok(await check(), "adb is now running");
 
   await ADB.stop(true /* sync */);
   ok(!ADB.ready);
 
   await extension.unload();
 });
 
-add_task(async function testTrackDevices() {
+add_task({
+  skip_if: () => mozinfo.os == "win" // bug 1482008
+}, async function testTrackDevices() {
   const extension = ExtensionTestUtils.loadExtension({
     manifest: {
       version: "1.0",
       applications: {
         gecko: { id: "adb@mozilla.org" }
       }
     },
     files: {