Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update the path of chip-all-clusters-app into scripts/tests/test_suit… #7638

Merged
merged 1 commit into from
Jun 18, 2021

Conversation

vivien-apple
Copy link
Contributor

…es.sh so it can either be runned from CI or from a build from ./gn_build.sh

Problem

./scripts/tests/test_suites.sh uses out/debug/chip-all-clusters-app for the path of the server. Building with ./gn_build.sh usually results into out/debug/standalone/chip-all-clusters-app.

Change overview

  • This PR update the path of the bash script to make it easier to run the tests suite locally.

Testing

  • This was tested locally by running ./scripts/tests/test_suites.sh.
  • The CI changed will be tested by the CI

@woody-apple
Copy link
Contributor

@woody-apple
Copy link
Contributor

/rebase

@vivien-apple vivien-apple force-pushed the Tests_UpdateTestSuitePath branch from 10193bb to 6d83bab Compare June 16, 2021 19:56
@woody-apple
Copy link
Contributor

/rebase

…es.sh so it can either be runned from CI or from a build from ./gn_build.sh
@woody-apple woody-apple force-pushed the Tests_UpdateTestSuitePath branch from 6d83bab to 673149d Compare June 17, 2021 01:46
@woody-apple
Copy link
Contributor

@mspang mspang merged commit 838c3db into project-chip:master Jun 18, 2021
nikita-s-wrk pushed a commit to nikita-s-wrk/connectedhomeip that referenced this pull request Sep 23, 2021
…es.sh so it can either be runned from CI or from a build from ./gn_build.sh (project-chip#7638)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants