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

test: create cucumber test directory if necessary #3452

Merged
merged 1 commit into from
Oct 13, 2021

Conversation

CjS77
Copy link
Collaborator

@CjS77 CjS77 commented Oct 13, 2021

The run-tests.sh script fails on a fresh install because the cucumber
output folder doesn't exist. This PR simple checks and creates the
folder if necessary.

The `run-tests.sh` script fails on a fresh install because the cucumber
output folder doesn't exist. This PR simple checks and creates the
folder if necessary.
Copy link
Contributor

@delta1 delta1 left a comment

Choose a reason for hiding this comment

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

Good catch

@stringhandler stringhandler changed the title Create cucumber test directory if necessary test: create cucumber test directory if necessary Oct 13, 2021
@aviator-app aviator-app bot removed the mq-failed label Oct 13, 2021
@aviator-app aviator-app bot merged commit 90eabe2 into development Oct 13, 2021
@stringhandler stringhandler deleted the cucumber_test_mkdir branch October 13, 2021 20:17
sdbondi added a commit to sdbondi/tari that referenced this pull request Oct 15, 2021
* development:
  fix: add display_currency_decimal method (tari-project#3445)
  fix: add sanity checks to prepare_new_block (tari-project#3448)
  test: profile wallet sqlite database operations (tari-project#3451)
  test: create cucumber test directory if necessary (tari-project#3452)
  chore: improve cucumber tags and run time speed (tari-project#3439)
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.

3 participants