Skip to content

Add getGroupRestrictionNames to BO payment preferences page object#990

Merged
Progi1984 merged 2 commits into
PrestaShop:mainfrom
PrestaEdit:fix-9858-payment-group-restrictions-shop-filter
Jun 10, 2026
Merged

Add getGroupRestrictionNames to BO payment preferences page object#990
Progi1984 merged 2 commits into
PrestaShop:mainfrom
PrestaEdit:fix-9858-payment-group-restrictions-shop-filter

Conversation

@PrestaEdit

Copy link
Copy Markdown
Contributor

Description

Adds a getGroupRestrictionNames(page) method to the BO Payment > Preferences page object (boPaymentPreferencesPage), returning the names of the customer groups listed in the "Group restrictions" table.

This is needed to assert, in a multishop context, that the group restrictions table only lists the customer groups of the currently selected shop.

Related

🤖 Generated with Claude Code

Allows tests to read the customer groups listed in the "Group restrictions"
table, needed to assert the per-shop filtering of group restrictions.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@ps-jarvis

Copy link
Copy Markdown

Hello @PrestaEdit!

This is your first pull request on ui-testing-library repository of the PrestaShop project.

Thank you, and welcome to this Open Source community!

Comment thread src/versions/develop/pages/BO/payment/preferences.ts Outdated

@Progi1984 Progi1984 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Small comment

Co-authored-by: Progi1984 <progi1984@gmail.com>
@Progi1984 Progi1984 added this to the 0.0.12 milestone Jun 10, 2026
@Progi1984 Progi1984 merged commit eb56f4d into PrestaShop:main Jun 10, 2026
2 checks passed
@github-project-automation github-project-automation Bot moved this from Ready for review to Merged in PR Dashboard Jun 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Merged

Development

Successfully merging this pull request may close these issues.

3 participants