Skip to content

At least make pytest on a Windows laptop work#246

Merged
sverhoeven merged 2 commits intomainfrom
223-windows-test-on-local-machine
Apr 29, 2021
Merged

At least make pytest on a Windows laptop work#246
sverhoeven merged 2 commits intomainfrom
223-windows-test-on-local-machine

Conversation

@sverhoeven
Copy link
Copy Markdown
Member

@sverhoeven sverhoeven commented Apr 29, 2021

Uses list for args instead of parsing strings

Refs #223
Refs #200

PS C:\Users\stefanv\git\NLeSC\python-template> .\env\Scripts\pytest.exe
==================================================== test session starts ====================================================
platform win32 -- Python 3.9.4, pytest-4.6.11, py-1.10.0, pluggy-0.13.1
rootdir: C:\Users\stefanv\git\NLeSC\python-template, inifile: setup.cfg, testpaths: tests
plugins: cookies-0.5.1, cov-2.11.1
collected 6 items

tests\test_project.py ..                                                                                         [ 33%]
tests\test_values.py ....                                                                                        [100%]

============================================== 6 passed in 21.51 seconds ==============================================
PS C:\Users\stefanv\git\NLeSC\python-template>

@fdiblen fdiblen self-requested a review April 29, 2021 12:05
Copy link
Copy Markdown
Member

@fdiblen fdiblen left a comment

Choose a reason for hiding this comment

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

I saw it working. Thanks! 🍡

@sverhoeven sverhoeven mentioned this pull request Apr 29, 2021
@sverhoeven
Copy link
Copy Markdown
Member Author

Thanks for reviewing

@sverhoeven sverhoeven merged commit 7e9b9ac into main Apr 29, 2021
@sverhoeven sverhoeven deleted the 223-windows-test-on-local-machine branch April 29, 2021 12:22
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