Skip to content

Commit

Permalink
Minor adjustments on ci workflow
Browse files Browse the repository at this point in the history
  • Loading branch information
Fabricio-ESP committed Nov 22, 2024
1 parent 3b83822 commit bf5b24f
Show file tree
Hide file tree
Showing 6 changed files with 10 additions and 6 deletions.
4 changes: 4 additions & 0 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -149,8 +149,12 @@ jobs:
name: test-results-${{ matrix.package_name }}-${{matrix.run_on}}.zip
path: |
./tests/results-pre-test.xml
./tests/results-pre-install-test.xml
./tests/results-default-test.xml
./tests/results-variation1-test.xml
./tests/results-non-interactive-test.xml
./tests/results-CNMirror-test.xml
./tests/results-CNMirror2-test.xml
./tests/test.log
publish-test-results:
Expand Down
2 changes: 1 addition & 1 deletion tests/run_pre_test.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -23,4 +23,4 @@ Expand-Archive node_modules.zip

# Run tests using npm run AllTest
npm run pre-test
npm run pre-install
# npm run pre-install
2 changes: 1 addition & 1 deletion tests/runs/CNmirrors2.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ if (process.env.EIM_FILE_PATH) {
pathToEim = path.join(os.homedir(), "eim-cli/eim");
}

logger.debug(`Starting installation using alternative download mirrors`);
logger.debug(`Starting installation using mirror jihulab and dl.espressif.cn`);

const targetList = ["esp32"]; // targets used for IDF installation
const idfVersionList = ["v5.0.7"]; // IDF versions to be installed
Expand Down
2 changes: 1 addition & 1 deletion tests/script/installNonInteractive.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ export function runInstallNonInteractive(pathToEim, args = []) {

it("Should install IDF using specified parameters", async function () {
logger.info("Sent command line for IDF installation");
testRunner.sendInput(`${pathToEim} ${args.join(" ")} -n true \r`);
testRunner.sendInput(`${pathToEim} ${args.join(" ")} -n true\r`);

const installationSuccessful = await testRunner.waitForOutput(
"Successfully installed IDF",
Expand Down
2 changes: 1 addition & 1 deletion tests/script/postInstall.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ export function runPostInstallTest(
logger.info("Error to start IDF terminal");
logger.info(testRunner.output);
throw new Error(
"One test in teh chain failed, aborting the entire suite."
"One test in the chain failed, aborting the entire suite."
);
}
});
Expand Down
4 changes: 2 additions & 2 deletions tests/script/prerequisitesInstallRunner.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ describe("Check Pre-requisites installation on Windows", function () {
});

afterEach(async function () {
this.timeout(10000);
this.timeout(20000);
if (this.currentTest.state === "failed") {
logger.info(
`Terminal output on failure: >>\r ${testRunner.output}`
Expand All @@ -49,7 +49,7 @@ describe("Check Pre-requisites installation on Windows", function () {
try {
await testRunner.stop(6000);
} catch {
logger.debug("Error to clean up terminal after test");
logger.info("Error to clean up terminal after test");
}
testRunner = null;
});
Expand Down

0 comments on commit bf5b24f

Please sign in to comment.