Open artsiomkaltovich opened 5 years ago
Hey! Thanks for pr!
Как там Казань?
Все норм, только чутка холодновато сейчас :)
Hello @TvoroG.
Have you had a chance to review the code? Do you have any comments? Will the PR be merged?
Regards, Artyom.
Hey!
Have you had a chance to review the code?
Forget to send comments :)
Hi @TvoroG and @ArtyomKaltovich,
Is this PR in the works? If not, I'm willing to take over, I really need this kind of functionality.
Hello @elilutsky.
Looking up every collection drammatically slow down the performance of the tests, I've tried to do it optional, but failed :)
You can try it.
What if you could somehow explicitly mark a dictionary, list, ... that it needs to be searched for nested things?
E.g.:
LazyFixtureList(['a', 'b', lazy_fixture('c'), 'd'])
And maybe there a way of adding some optional syntactic sugar, like this
from pytest_lazyfixture import Lazy
Lazy+['a', 'b', lazy_fixture('c'), 'd']
Lazy+{'x': 1, 'y': lazy_fixture('z')}
Hey, @ArtyomKaltovich 👋🏼, r u gonna finish this issue, or mb you wanna reassign to somebody else?
Hey, @ArtyomKaltovich 👋🏼, r u gonna finish this issue, or mb you wanna reassign to somebody else?
Hello @ruslan-korneev. You can take it if you want.
Hello the PR has following motivation:
1) Fix for https://github.com/TvoroG/pytest-lazy-fixture/issues/24.
Usage:
More samples can be found in tests.
2) Extends current syntax for creating list of fixtures:
with more possibilities:
Do not hesitate to express your opinion about code and syntax.
Regards, Artyom.
P.S. Как там Казань? Почему творог, а не эчпочмак? :)