Skip to content

Improve error message when passing non-string ids to pytest.mark.parametrize#1861

Merged
The-Compiler merged 1 commit intopytest-dev:masterfrom
nicoddemus:ids-invalid-type-msg
Aug 24, 2016
Merged

Improve error message when passing non-string ids to pytest.mark.parametrize#1861
The-Compiler merged 1 commit intopytest-dev:masterfrom
nicoddemus:ids-invalid-type-msg

Conversation

@nicoddemus
Copy link
Member

Fix #1857

@coveralls
Copy link

Coverage Status

Coverage increased (+0.005%) to 93.035% when pulling 21a23f5 on nicoddemus:ids-invalid-type-msg into ea0feba on pytest-dev:master.

@nicoddemus nicoddemus force-pushed the ids-invalid-type-msg branch from 21a23f5 to 972a5fb Compare August 24, 2016 02:38
@coveralls
Copy link

Coverage Status

Coverage increased (+0.005%) to 93.035% when pulling 972a5fb on nicoddemus:ids-invalid-type-msg into ea0feba on pytest-dev:master.

@The-Compiler The-Compiler merged commit 3345ac9 into pytest-dev:master Aug 24, 2016
@The-Compiler
Copy link
Member

I agree fixing the error instead of making this valid again is the way to go here - thanks!

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.

3 participants