-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
[Admin] Locale unique code validation feature #4610
Conversation
@@ -5,12 +5,11 @@ Feature: Locale unique code validation | |||
I want to be prevented from adding a locale with an existing code | |||
|
|||
Background: | |||
Given the store is available in the Norwegian language | |||
Given the store has locale "Norwegian" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What was wrong with the previous step? :)
7dd3c42
to
51afe24
Compare
*/ | ||
public function iShouldNotBeAbleToChoose($name) | ||
{ | ||
expect($this->createPage)->toThrow(ElementNotFoundException::class)->during('chooseName', [$name]); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We agreed to replace expect and catching exception with Assert and dedicated method on page that checks isSelectOptionAvailable
or smth similar.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
isOptionAvailable
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
When I choose Norwegian | ||
And I try to add it | ||
Then I should be notified that it is not possible | ||
Given I want to create a new locale |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
to create add a new?
bf395a9
to
24a9a03
Compare
24a9a03
to
d92cecd
Compare
👍 |
@@ -66,6 +66,7 @@ public function __construct( | |||
|
|||
/** | |||
* @Given I want to create a new locale |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be removed and the below step should be used everywhere. Task for a separate PR.
Thanks Arek! |
Based on #4594