Skip to content

Commit 13bdd4e

Browse files
committed
B2B-2606: Graphql Parser called at least 3 times per request
- Fixing issue where mftf edit terms & conditions tests don't cleanup data if test fails prematurely
1 parent 235bfdf commit 13bdd4e

10 files changed

+20
-7
lines changed

app/code/Magento/CheckoutAgreements/Test/Mftf/ActionGroup/AdminTermsConditionsFillTermEditFormActionGroup.xml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,9 +11,10 @@
1111
<actionGroup name="AdminTermsConditionsFillTermEditFormActionGroup">
1212
<arguments>
1313
<argument name="term"/>
14+
<argument name="termName" type="string"/>
1415
</arguments>
1516

16-
<fillField selector="{{AdminNewTermFormSection.conditionName}}" userInput="{{term.name}}" stepKey="fillFieldConditionName"/>
17+
<fillField selector="{{AdminNewTermFormSection.conditionName}}" userInput="{{termName}}" stepKey="fillFieldConditionName"/>
1718
<selectOption selector="{{AdminNewTermFormSection.isActive}}" userInput="{{term.isActive}}" stepKey="selectOptionIsActive"/>
1819
<selectOption selector="{{AdminNewTermFormSection.isHtml}}" userInput="{{term.isHtml}}" stepKey="selectOptionIsHtml"/>
1920
<selectOption selector="{{AdminNewTermFormSection.mode}}" userInput="{{term.mode}}" stepKey="selectOptionMode"/>

app/code/Magento/CheckoutAgreements/Test/Mftf/Test/AdminCreateActiveHtmlTermEntityTest.xml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,7 @@
4444
<actionGroup ref="AdminTermsConditionsOpenNewTermPageActionGroup" stepKey="openNewTerm"/>
4545
<actionGroup ref="AdminTermsConditionsFillTermEditFormActionGroup" stepKey="fillNewTerm">
4646
<argument name="term" value="activeHtmlTerm"/>
47+
<argument name="termName" value="{{activeHtmlTerm.name}}"/>
4748
</actionGroup>
4849
<actionGroup ref="AdminTermsConditionsSaveTermActionGroup" stepKey="saveNewTerm"/>
4950

app/code/Magento/CheckoutAgreements/Test/Mftf/Test/AdminCreateActiveTextTermEntityTest.xml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@
2727

2828
<actionGroup ref="AdminTermsConditionsFillTermEditFormActionGroup" stepKey="fillNewTerm">
2929
<argument name="term" value="activeTextTerm"/>
30+
<argument name="termName" value="{{activeTextTerm.name}}"/>
3031
</actionGroup>
3132
<actionGroup ref="AssertAdminTermsConditionsInGridActionGroup" stepKey="assertTermInGrid">
3233
<argument name="termName" value="{{activeTextTerm.name}}"/>

app/code/Magento/CheckoutAgreements/Test/Mftf/Test/AdminCreateDisabledTextTermEntityTest.xml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,7 @@
4343
<actionGroup ref="AdminTermsConditionsOpenNewTermPageActionGroup" stepKey="openNewTerm"/>
4444
<actionGroup ref="AdminTermsConditionsFillTermEditFormActionGroup" stepKey="fillNewTerm">
4545
<argument name="term" value="disabledTextTerm"/>
46+
<argument name="termName" value="{{disabledTextTerm.name}}"/>
4647
</actionGroup>
4748
<actionGroup ref="AdminTermsConditionsSaveTermActionGroup" stepKey="saveNewTerm"/>
4849

app/code/Magento/CheckoutAgreements/Test/Mftf/Test/AdminCreateEnabledTextTermOnMultishippingEntityTest.xml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,7 @@
5252
<actionGroup ref="AdminTermsConditionsOpenNewTermPageActionGroup" stepKey="openNewTerm"/>
5353
<actionGroup ref="AdminTermsConditionsFillTermEditFormActionGroup" stepKey="fillNewTerm">
5454
<argument name="term" value="activeTextTerm"/>
55+
<argument name="termName" value="{{activeTextTerm.name}}"/>
5556
</actionGroup>
5657
<actionGroup ref="AdminTermsConditionsSaveTermActionGroup" stepKey="saveNewTerm"/>
5758

app/code/Magento/CheckoutAgreements/Test/Mftf/Test/AdminDeleteActiveTextTermEntityTest.xml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@
2828
<actionGroup ref="AdminTermsConditionsOpenNewTermPageActionGroup" stepKey="openNewTerm"/>
2929
<actionGroup ref="AdminTermsConditionsFillTermEditFormActionGroup" stepKey="createTerm">
3030
<argument name="term" value="activeTextTerm"/>
31+
<argument name="termName" value="{{activeTextTerm.name}}"/>
3132
</actionGroup>
3233
<actionGroup ref="AdminTermsConditionsSaveTermActionGroup" stepKey="saveNewTerm"/>
3334
</before>

app/code/Magento/CheckoutAgreements/Test/Mftf/Test/AdminUpdateDisabledHtmlTermEntityTest.xml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@
3333

3434
<actionGroup ref="AdminTermsConditionsOpenGridActionGroup" stepKey="openTermsGridToDelete"/>
3535
<actionGroup ref="AdminOpenEditPageTermsConditionsByNameActionGroup" stepKey="openTermToDelete">
36-
<argument name="termName" value="{{activeTextTerm.name}}"/>
36+
<argument name="termName" value="{{disabledHtmlTerm.name}}"/>
3737
</actionGroup>
3838
<actionGroup ref="AdminTermsConditionsDeleteTermByNameActionGroup" stepKey="deleteOpenedTerm"/>
3939

@@ -43,6 +43,7 @@
4343
<actionGroup ref="AdminTermsConditionsOpenNewTermPageActionGroup" stepKey="openNewTerm"/>
4444
<actionGroup ref="AdminTermsConditionsFillTermEditFormActionGroup" stepKey="fillNewTerm">
4545
<argument name="term" value="disabledHtmlTerm"/>
46+
<argument name="termName" value="{{disabledHtmlTerm.name}}"/>
4647
</actionGroup>
4748
<actionGroup ref="AdminTermsConditionsSaveTermActionGroup" stepKey="saveNewTerm"/>
4849

@@ -52,12 +53,13 @@
5253
</actionGroup>
5354
<actionGroup ref="AdminTermsConditionsFillTermEditFormActionGroup" stepKey="fillUpdateTerm">
5455
<argument name="term" value="activeTextTerm"/>
56+
<argument name="termName" value="{{disabledHtmlTerm.name}}"/>
5557
</actionGroup>
5658
<actionGroup ref="AdminTermsConditionsSaveTermActionGroup" stepKey="saveUpdateTerm"/>
5759

5860
<actionGroup ref="AdminTermsConditionsOpenGridActionGroup" stepKey="openNewTermsGrid"/>
5961
<actionGroup ref="AssertAdminTermsConditionsInGridActionGroup" stepKey="assertTermInGrid">
60-
<argument name="termName" value="{{activeTextTerm.name}}"/>
62+
<argument name="termName" value="{{disabledHtmlTerm.name}}"/>
6163
</actionGroup>
6264

6365
<openNewTab stepKey="openStorefrontTab"/>

app/code/Magento/CheckoutAgreements/Test/Mftf/Test/AdminUpdateDisabledTextTermEntityTest.xml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,21 +22,23 @@
2222

2323
<after>
2424
<actionGroup ref="AdminOpenEditPageTermsConditionsByNameActionGroup" stepKey="openTermToDelete">
25-
<argument name="termName" value="{{activeHtmlTerm.name}}"/>
25+
<argument name="termName" value="{{disabledTextTerm.name}}"/>
2626
</actionGroup>
2727
</after>
2828

2929
<actionGroup ref="AdminTermsConditionsFillTermEditFormActionGroup" stepKey="fillNewTerm">
3030
<argument name="term" value="disabledTextTerm"/>
31+
<argument name="termName" value="{{disabledHtmlTerm.name}}"/>
3132
</actionGroup>
3233
<actionGroup ref="AdminTermsConditionsEditTermByNameActionGroup" stepKey="openUpdateTerm">
3334
<argument name="termName" value="{{disabledTextTerm.name}}"/>
3435
</actionGroup>
3536
<actionGroup ref="AdminTermsConditionsFillTermEditFormActionGroup" stepKey="fillUpdateTerm">
3637
<argument name="term" value="activeHtmlTerm"/>
38+
<argument name="termName" value="{{disabledTextTerm.name}}"/>
3739
</actionGroup>
3840
<actionGroup ref="AssertAdminTermsConditionsInGridActionGroup" stepKey="assertTermInGrid">
39-
<argument name="termName" value="{{activeHtmlTerm.name}}"/>
41+
<argument name="termName" value="{{disabledTextTerm.name}}"/>
4042
</actionGroup>
4143
<actionGroup ref="AssertStorefrontTermInCheckoutActionGroup" stepKey="assertTermInCheckout">
4244
<argument name="termCheckboxText" value="{{activeHtmlTerm.checkboxText}}"/>

app/code/Magento/CheckoutAgreements/Test/Mftf/Test/AdminUpdateEnabledTextTermEntityTest.xml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,21 +22,23 @@
2222
</annotations>
2323
<after>
2424
<actionGroup ref="AdminOpenEditPageTermsConditionsByNameActionGroup" stepKey="openTermToDelete">
25-
<argument name="termName" value="{{disabledHtmlTerm.name}}"/>
25+
<argument name="termName" value="{{activeTextTerm.name}}"/>
2626
</actionGroup>
2727
</after>
2828

2929
<actionGroup ref="AdminTermsConditionsFillTermEditFormActionGroup" stepKey="fillNewTerm">
3030
<argument name="term" value="activeTextTerm"/>
31+
<argument name="termName" value="{{activeTextTerm.name}}"/>
3132
</actionGroup>
3233
<actionGroup ref="AdminTermsConditionsEditTermByNameActionGroup" stepKey="openUpdateTerm">
3334
<argument name="termName" value="{{activeTextTerm.name}}"/>
3435
</actionGroup>
3536
<actionGroup ref="AdminTermsConditionsFillTermEditFormActionGroup" stepKey="fillUpdateTerm">
3637
<argument name="term" value="disabledHtmlTerm"/>
38+
<argument name="termName" value="{{activeTextTerm.name}}"/>
3739
</actionGroup>
3840
<actionGroup ref="AssertAdminTermsConditionsInGridActionGroup" stepKey="assertTermInGrid">
39-
<argument name="termName" value="{{disabledHtmlTerm.name}}"/>
41+
<argument name="termName" value="{{activeTextTerm.name}}"/>
4042
</actionGroup>
4143
<actionGroup ref="AssertStorefrontTermAbsentInCheckoutActionGroup" stepKey="assertTermInCheckout">
4244
<argument name="termCheckboxText" value="{{disabledHtmlTerm.checkboxText}}"/>

app/code/Magento/CheckoutAgreements/Test/Mftf/Test/StoreFrontManualTermsAndConditionsTest.xml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,7 @@
3838
<!-- Fill the Required Details-->
3939
<actionGroup ref="AdminTermsConditionsFillTermEditFormActionGroup" stepKey="fillNewTerm">
4040
<argument name="term" value="newHtmlTerm"/>
41+
<argument name="termName" value="{{newHtmlTerm.name}}"/>
4142
</actionGroup>
4243
<grabTextFrom selector="{{AdminNewTermFormSection.conditionName}}" stepKey="conditionName"/>
4344
<!-- Save Details-->

0 commit comments

Comments
 (0)