Skip to content

Commit

Permalink
Using dws-test-driver for DWS state progression integration tests (Da…
Browse files Browse the repository at this point in the history
…taWorkflowServices#28)

* Using dws-test-driver for DWS state progression integration tests

Signed-off-by: Nathan Lee <[email protected]>

* Fixed integration test errors

Signed-off-by: Nathan Lee <[email protected]>

* code review changes

Signed-off-by: Nathan Lee <[email protected]>

* Updated dws-test-driver to main branch HEAD

Signed-off-by: Nathan Lee <[email protected]>

* Code review

Signed-off-by: Nathan Lee <[email protected]>

---------

Signed-off-by: Nathan Lee <[email protected]>
  • Loading branch information
nathandotleeathpe committed Feb 3, 2023
1 parent dab85b2 commit e14e08d
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -40,4 +40,4 @@ Feature: Integration test environment
Scenario: Kubernetes and slurm are connected
Given the kubernetes cluster kube-system UID
When the kube-system UID is queried from slurmctld
Then the UIDs match and the cluster is the same
Then the UIDs match and the cluster is the same
2 changes: 1 addition & 1 deletion testsuite/integration/src/tests/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ def pytest_bdd_apply_tag(tag, function):
return None

@given(parsers.parse('a job script:\n{script}'), target_fixture="script_path")
def write_job_script(script):
def _(script):
"""a job script: <script>"""
path = "/jobs/" + secrets.token_hex(5) + "-job.sh"
with open(path, "w") as file:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -86,4 +86,4 @@ def _(k8s, slurmctld, jobId, state):
@then(parsers.parse("the job's system comment contains the following:\n{message}"))
def _(slurmctld, jobId, message):
_,out = slurmctld.get_final_job_state(jobId)
assert message in out
assert message in out

0 comments on commit e14e08d

Please sign in to comment.