Skip to content

Commit

Permalink
Merge pull request #19 from CrowdJustice/fixing-commands
Browse files Browse the repository at this point in the history
adding start of path to pytest command plus running cypress and karma outside the container
  • Loading branch information
phil-bell authored Mar 2, 2022
2 parents cb79dc6 + 1674add commit 93f251a
Show file tree
Hide file tree
Showing 2 changed files with 22 additions and 3 deletions.
19 changes: 19 additions & 0 deletions .github/pull_request_template.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,19 @@
### What does this PR do?

### How do you test it?

### What should the release note say?

<!--
Release notes must be between the tags to be
automatically added to the latest changes.
Remove the tags for no release notes.
-->

<release-notes><release-notes>

<!-- Review reminders!
Tag `@Design` to review non-trivial visual changes
Tag `@Frontend` to review non-trivial frontend changes
Warn Product about any significant database changes
-->
6 changes: 3 additions & 3 deletions legl_dev/main.py
Original file line number Diff line number Diff line change
Expand Up @@ -114,7 +114,7 @@ def pytest(
f"docker compose "
"exec backend pytest "
"--html=unit_test_results.html "
f"{extra_args} {path}"
f"{extra_args} /code/{path}"
)
),
]
Expand Down Expand Up @@ -154,7 +154,7 @@ def cypress():
steps = Steps(
steps=[
Command(
command=f"docker compose exec frontend yarn run cypress open",
command=f"yarn run cypress open",
)
]
)
Expand Down Expand Up @@ -246,7 +246,7 @@ def jstest():

steps = Steps(
steps=[
Command(command=(f"docker compose exec frontend yarn run test")),
Command(command=(f"yarn run test")),
Command(
command="open js-test-results/index.html",
),
Expand Down

0 comments on commit 93f251a

Please sign in to comment.