Reverse and merge tests

microsoft · Jan 19, 2021 · e591480 · e591480
1 parent 5a49b97
commit e591480
Showing 1 changed file with 8 additions and 12 deletions.
diff --git a/src/arduino/arduino.ts b/src/arduino/arduino.ts
@@ -550,10 +550,10 @@ export class ArduinoApp {
                 return false;
             }
 
-            if (!this.useArduinoCli()) {
-                args.push("--upload");
-            } else {
+            if (this.useArduinoCli()) {
                 args.push("compile", "--upload");
+            } else {
+                args.push("--upload");
             }
 
             if (dc.port) {
@@ -586,17 +586,13 @@ export class ArduinoApp {
                 return false;
             }
 
-            if (!this.useArduinoCli()) {
-                args.push("--upload");
-            } else {
-                // TODO: add the --clean argument to the cli args when v 0.14 is released (this will clean up the build folder after uploading)
-                args.push("upload");
-            }
-
             if (this.useArduinoCli()) {
-                args.push("--programmer", programmer)
+                args.push("upload",
+                    "--programmer", programmer);
             } else {
-                args.push("--useprogrammer", "--pref", `programmer=arduino:${programmer}`);
+                args.push("--upload",
+                    "--useprogrammer",
+                    "--pref", `programmer=arduino:${programmer}`);
             }
 
             args.push("--port", dc.port);