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.

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

original fixture function should use get() #34

Closed
la10736 opened this issue May 5, 2019 · 0 comments
Closed

original fixture function should use get() #34

la10736 opened this issue May 5, 2019 · 0 comments
Milestone
rstest 0.6

Comments

@la10736
Copy link
Owner

la10736 commented May 5, 2019

Replace the given function fixture(args) by call fixture::get(args).

Take care to remove mut in args definition (it will pass just the ownership) or , simpler, shut down the warning.

When we'll implement teardown we should take get() result and discard teardown
la10736 added a commit that referenced this issue May 5, 2019
@la10736
#5 replace original fixture call by fixture body to remove warning by…
16de3ca 
… not necessary mut. We maintain both body till #34 is implemented.
@la10736 la10736 added this to the rstest 0.6 milestone Dec 7, 2019
@la10736 la10736 closed this as completed in 8a0ff08 Jan 7, 2020
la10736 added a commit that referenced this issue Jan 7, 2020
@la10736
#34 update changelog
7473e4e
la10736 added a commit that referenced this issue Jan 7, 2020
@la10736
#34 Fixed link
5d7c675
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
rstest 0.6
Development

No branches or pull requests
1 participant
@la10736