IBX-10333 [Behat] Added scenario for hide later coverage #1731
Conversation
pawlakadrian
force-pushed
the
IBX-10333-hide-later-content
branch
from
7a50371 to
f0aaf3e
Compare
|
|
Regression: |
ezrobot
requested review from
KamilSznajdrowicz,
barbaragr,
juskora,
katarzynazawada,
micszo and
tomaszszopinski
and removed request for
a team
tomaszszopinski
left a comment
tomaszszopinski
left a comment
There was a problem hiding this comment.
Just a few nitpicks :)
| | title | short_title | intro | | ||
| | TestArticleToHideLater | TestArticleToHideLater | TestArticleIntro | | ||
| And I'm on Content view Page for "ContentManagement/TestArticleToHideLater" | ||
| When I perform the "Hide" action |
There was a problem hiding this comment.
Lets only use When/Then once per scenario. Focus of this scenario should a be a scheduler run.
| | title | short_title | intro | | ||
| | TestArticleToCancelHideLater | TestArticleToCancelHideLater | TestArticleIntro | | ||
| And I'm on Content view Page for "ContentManagement/TestArticleToCancelHideLater" | ||
| When I perform the "Hide" action |
| use Ibexa\Behat\Browser\Element\Criterion\ElementTextCriterion; | ||
| use Ibexa\Behat\Browser\Locator\VisibleCSSLocator; | ||
|
|
||
| class CancelContentDialog extends Dialog |
There was a problem hiding this comment.
It seems that Dialog component should be enough here.
This locator https://github.com/ibexa/admin-ui/blob/main/src/lib/Behat/Component/Dialog.php#L35C47-L35C177 covers Cancel scheduled hiding button
There was a problem hiding this comment.
Right, working as expected.
pawlakadrian
force-pushed
the
IBX-10333-hide-later-content
branch
2 times, most recently
from
4c7194b to
5d86a63
Compare
pawlakadrian
force-pushed
the
IBX-10333-hide-later-content
branch
from
da2886f to
12685f2
Compare
pawlakadrian
changed the title
pawlakadrian
force-pushed
the
IBX-10333-hide-later-content
branch
from
202c3ed to
e5a5753
Compare
|
6 participants
Description:
New scenario for hide later content coverage
For QA:
Documentation: