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

Replace container_from_pytest_param with container_and_marks_from_pytest_param #167

Merged
merged 1 commit into from
Nov 2, 2023

Conversation

dcermak
Copy link
Owner

@dcermak dcermak commented Nov 2, 2023

  • deprecate container_from_pytest_param
  • replace it internally with container_and_marks_from_pytest_param

container_and_marks_from_pytest_param is more flexible as it encourages users to not forget about marks when creating new containers from pytest.param instances.

…est_param

- deprecate container_from_pytest_param
- replace it internally with container_and_marks_from_pytest_param

container_and_marks_from_pytest_param is more flexible as it encourages users to
not forget about marks when creating new containers from `pytest.param`
instances.
@dcermak dcermak force-pushed the param-to-cont-and-mark branch from 5c1cd18 to 1622568 Compare November 2, 2023 10:43
@dcermak dcermak enabled auto-merge November 2, 2023 10:55
Copy link

codecov bot commented Nov 2, 2023

Codecov Report

Merging #167 (1622568) into main (af62fc3) will decrease coverage by 0.17%.
The diff coverage is 87.09%.

@@            Coverage Diff             @@
##             main     #167      +/-   ##
==========================================
- Coverage   94.81%   94.64%   -0.17%     
==========================================
  Files           9        9              
  Lines        1041     1064      +23     
  Branches      214      221       +7     
==========================================
+ Hits          987     1007      +20     
- Misses         40       43       +3     
  Partials       14       14              
Files Coverage Δ
pytest_container/__init__.py 100.00% <100.00%> (ø)
pytest_container/build.py 84.21% <100.00%> (ø)
pytest_container/plugin.py 94.20% <75.00%> (ø)
pytest_container/container.py 98.34% <87.50%> (-0.66%) ⬇️

@dcermak dcermak merged commit f713074 into main Nov 2, 2023
@dcermak dcermak deleted the param-to-cont-and-mark branch November 2, 2023 11:37
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.

1 participant