Removed commented code.

espressif · Nov 11, 2024 · df39273 · df39273
1 parent 91f0f13
commit df39273
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 76 deletions.
diff --git a/tests/script/installCustom.test.js b/tests/script/installCustom.test.js
@@ -19,7 +19,7 @@ export function runInstallCustom(
             );
             logger.debug(`Installing IDF for targets ${targetList}`);
             logger.debug(`Recurse submodules active? : ${recursiveSubmodules}`);
-            this.timeout(5000); // Increase timeout for setup
+            this.timeout(5000);
             testRunner = new InteractiveCLITestRunner();
         });
 

diff --git a/tests/script/installWizard.test.js b/tests/script/installWizard.test.js
@@ -9,13 +9,13 @@ export function runInstallWizardTests(pathToEim) {
 
         before(async function () {
             logger.debug(`Starting installation wizard with default options`);
-            this.timeout(5000); // Increase timeout for setup
+            this.timeout(5000);
             testRunner = new InteractiveCLITestRunner();
             try {
                 await testRunner.runTerminal();
                 testRunner.sendInput(`${pathToEim}\r`);
             } catch (error) {
-                logger.debug("Error starting process:", error);
+                logger.debug(`Error starting process: ${error}`);
                 throw error;
             }
         });
@@ -102,9 +102,6 @@ export function runInstallWizardTests(pathToEim) {
             );
             expect(installationCompleted).to.be.true;
             expect(testRunner.output).to.not.include("error");
-            // expect(testRunner.output).to.include(
-            //     "Finished fetching submodules"
-            // );
             expect(testRunner.output).to.include("Downloading tools");
 
             logger.info("Installation completed");

diff --git a/tests/script/postInstall.test.js b/tests/script/postInstall.test.js
@@ -28,7 +28,7 @@ export function runPostInstallTest(
             try {
                 await testRunner.runIDFTerminal(pathToIDFScript);
             } catch (error) {
-                logger.debug("Error starting process:", error);
+                logger.debug(`Error starting process: ${error}`);
                 throw error;
             }
         });

diff --git a/tests/script/prerequisites.test.js b/tests/script/prerequisites.test.js
@@ -29,13 +29,13 @@ describe("Check if prerequisites are installed", function () {
     let testRunner;
 
     beforeEach(async function () {
-        this.timeout(5000); // Increase timeout for setup
+        this.timeout(5000);
         testRunner = new InteractiveCLITestRunner();
         try {
             await testRunner.runTerminal();
             testRunner.sendInput(`${pathToEim}\r`);
         } catch (error) {
-            logger.debug("Error starting process:", error);
+            logger.debug(`Error starting process: ${error}`);
             throw error;
         }
     });
@@ -107,73 +107,6 @@ describe("Check if prerequisites are installed", function () {
                 );
                 expect(terminalExited).to.be.true;
             });
-
-            //This should be re-enabled to run locally or when fixed on github runners powershell 7
-
-            // it("should install prerequisites and offer to install python and exit upon negative answer", async function () {
-            //     this.timeout(240000);
-            //     const promptRequisites = await testRunner.waitForOutput(
-            //         "Do you want to install prerequisites"
-            //     );
-
-            //     expect(promptRequisites).to.be.true;
-
-            //     logger.info("Question to install prerequisites passed");
-            //     testRunner.output = "";
-            //     testRunner.sendInput("y");
-
-            //     const promptPython = await testRunner.waitForOutput(
-            //         "Do you want to install Python",
-            //         240000
-            //     );
-
-            //     expect(promptPython).to.be.true;
-            //     expect(testRunner.output).to.include(
-            //         "All prerequisites are satisfied"
-            //     );
-
-            //     testRunner.sendInput("n");
-
-            //     const terminalExited = await testRunner.waitForOutput(
-            //         "Please install python3 with pip and SSL support and try again"
-            //     );
-
-            //     expect(terminalExited).to.be.true;
-            // });
-
-            // it("should install python and proceed with installation", async function () {
-            //     this.timeout(240000);
-            //     const promptRequisites = await testRunner.waitForOutput(
-            //         "Do you want to install prerequisites"
-            //     );
-
-            //     expect(promptRequisites).to.be.true;
-
-            //     logger.info("Question to install prerequisites passed");
-            //     testRunner.output = "";
-            //     testRunner.sendInput("y");
-
-            //     const promptPython = await testRunner.waitForOutput(
-            //         "Do you want to install Python",
-            //         240000
-            //     );
-
-            //     expect(promptPython).to.be.true;
-            //     expect(testRunner.output).to.include(
-            //         "All prerequisites are satisfied"
-            //     );
-
-            //     logger.info("Question to install python passed");
-            //     testRunner.output = "";
-            //     testRunner.sendInput("y");
-
-            //     const selectTargetQuestion = await testRunner.waitForOutput(
-            //         "Please select all of the target platforms",
-            //         240000
-            //     );
-
-            //     expect(selectTargetQuestion).to.be.true;
-            // });
         }
     );
 });