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

fixtures: sample_workflow_workspace with path #35

Merged

Conversation

dinosk
Copy link
Member

@dinosk dinosk commented Nov 29, 2018

  • Modifies the sample_workflow_workspace fixture to have a
    parametrized path.

Connects reanahub/reana-client#193

Signed-off-by: Dinos Kousidis dinos.kousidis@cern.ch

@dinosk dinosk force-pushed the parametrized-sample-workspace-path branch 2 times, most recently from aa2e96e to c2e4d5f Compare November 30, 2018 15:35
* Modifies the sample_workflow_workspace fixture to have a
  parametrized path, and also check if the path exists before
  creating the workspace. As it uses the tmp_shared_volume_path
  which has module scope, the sample workspaces created for tests
  are removed in the end of each test file.

Signed-off-by: Dinos Kousidis <dinos.kousidis@cern.ch>
@diegodelemos diegodelemos force-pushed the parametrized-sample-workspace-path branch from c2e4d5f to 530a0c3 Compare December 3, 2018 15:21
@ghost ghost assigned diegodelemos Dec 3, 2018
@diegodelemos diegodelemos merged commit eb169b4 into reanahub:master Dec 4, 2018
@ghost ghost removed the Status: in review label Dec 4, 2018
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