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

Add scripts for test pipeline #47

Merged
merged 7 commits into from
Feb 27, 2024
Merged

Add scripts for test pipeline #47

merged 7 commits into from
Feb 27, 2024

Conversation

navinkrishnan
Copy link
Contributor

@navinkrishnan navinkrishnan commented Feb 16, 2024

  • Copy scripts added to copy additional xmpls
  • add-all-xmpls - to setup multiple xmpls
  • additional test scripts for each xmpls

Flow to run Github action for outbound qualification:

  • git clone
  • npm run add-all-xmpls
  • npm install
  • npm test
Screenshot 2024-02-16 at 5 15 43 PM

* Copy scripts for additional xmpls
* removed test in xmpls, new tests will be added
* minor fixes in xmpls
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why is the --with-mocks options needed?

It's only needed for the remote service consumption, correct?
Is it so that other test still work once remote service consumption is added?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@DanSchlachter Yes, it was required to run along with other tests. So far, there are no conflicts.
add-all-xmpls script will copy multiple samples (Remote service, messaging, audit log, change tracking) and runs all the test at once.

@DanSchlachter DanSchlachter merged commit e01826d into main Feb 27, 2024
3 checks passed
@DanSchlachter DanSchlachter deleted the add-scripts-test branch February 27, 2024 09:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants