Skip to content

Conversation

@SergeyRuzin
Copy link
Contributor

No description provided.

CheckDropdownElements(wrapper.GroupTexts, items, $"{wrapper.Caption} groups");
}

[When("(.*?) (contain|not contain) \"(.*)\"")]
Copy link
Contributor

@Brrovko Brrovko Oct 19, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are you sure we don't have scenarios with the "when" step?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since this check in "when" step violates Gherkin syntax a bit, I think we don't have such scenarios. But I'll check

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I cannot find any usages of this step with "when". Seems ok to remove it

Brrovko
Brrovko previously approved these changes Oct 20, 2021
@Brrovko Brrovko requested a review from ILastovka October 20, 2021 07:10
ILastovka
ILastovka previously approved these changes Oct 21, 2021
@SergeyRuzin SergeyRuzin dismissed stale reviews from ILastovka and Brrovko via df88cf4 October 22, 2021 14:14
@SergeyRuzin SergeyRuzin merged commit 7228f55 into develop Oct 27, 2021
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.

5 participants