Skip to content

Remove unittest dependency for tests #885

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.

Sign up for GitHub

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

Jump to bottom
Closed
psafont opened this issue Aug 28, 2018 · 1 comment
Closed

Remove unittest dependency for tests #885

psafont opened this issue Aug 28, 2018 · 1 comment
Labels
enhancement help wanted

Comments

@psafont
Copy link
Contributor

psafont commented Aug 28, 2018

Right now the tests depend on unittest because they are coded in unittest style and on pytestm because they are run on pytest.

This is unnecessary since pytest can take care of everything.

Let's remove the unittest dependency from the test suite and track the progress here.
@psafont psafont modified the milestone: unplanned Aug 28, 2018
psafont added a commit to psafont/cwltool that referenced this issue Aug 28, 2018
@psafont
Enhancement: revamp testing for extension
a3d13a5 
Change image for workflow needing python to make it compatible with
Windows.
Changed style to pytest, as part of common-workflow-language#885
Uncomment all code, but still skip those tests
psafont added a commit to psafont/cwltool that referenced this issue Aug 28, 2018
@psafont
Enhancement: revamp testing for extension
9f6238e 
Change image for workflows needing python to make it compatible with
Windows.
Changed style to pytest, as part of common-workflow-language#885
Uncomment all code, but still skip those tests
psafont added a commit to psafont/cwltool that referenced this issue Aug 29, 2018
@psafont
Enhancement: revamp testing for extension
4bdf862 
Change image for workflows needing python to make it compatible with
Windows.
Changed style to pytest, as part of common-workflow-language#885
Uncomment all code, but still skip those tests
psafont added a commit to psafont/cwltool that referenced this issue Aug 31, 2018
@psafont
Enhancement: revamp testing for extension
566e5bf 
Change image for workflows needing python to make it compatible with
Windows.
Changed style to pytest, as part of common-workflow-language#885
Uncomment all code, but still skip those tests
@psafont psafont mentioned this issue Sep 6, 2018
Merged
@psafont
Copy link
Contributor Author

psafont commented Sep 10, 2018

Implemented by #895

@psafont psafont closed this as completed Sep 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement help wanted
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@psafont