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

More than one instance of extension suits #234

Open
Tiberivs opened this issue Mar 21, 2021 · 1 comment
Open

More than one instance of extension suits #234

Tiberivs opened this issue Mar 21, 2021 · 1 comment

Comments

@Tiberivs
Copy link
Contributor

At now multiple instances of extension suits include exist inside test suits tree (see diagram).
EmbedDiagram

This may to lead to unwanted behavior. For example, method checkout.SetupSuite() will be called twice - one for each embedded struct.

@denis-tingaikin
Copy link
Member

denis-tingaikin commented Mar 22, 2021

@Tiberivs I'm not quite following how this can be implemented. The main issue that Suits can be reusable and this case we again facing the same issue that one extension will be called twice.

For ur example,

We have suits A,B,C.

Suite B has not dependencies.
Suite A is using in setup for suite C.

So I understand your proposal means do some B' and A' that will include additional root logic.

And now imagine that we need to add a new suite D that should use setup from suites B, C

So we got the same issue: A'->B'->D where extensions calling twice.

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

No branches or pull requests

2 participants