Skip to content

Commit

Permalink
NDR enabled results in NDR on report page
Browse files Browse the repository at this point in the history
  • Loading branch information
Iqpal Mannan committed Nov 21, 2024
1 parent c2c5b24 commit e1caa04
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 12 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -191,12 +191,12 @@ public function iShouldNotBeAbleToAccessClientDetailsLink()
}

/**
* @Then /^I "(enable|disable)" the NDR for this user$/
* @Then /^I enable the NDR for this user$/
*/
public function iShouldBeAbleToToggleNdrEnabled($isEnabled)
public function iShouldBeAbleToToggleNdrEnabled()
{
$this->iVisitAdminEditUserPageForInteractingWithUser();

'enable' === $isEnabled ? $this->checkOption('admin_ndrEnabled') : $this->uncheckOption('admin_ndrEnabled');
$this->checkOption('admin_ndrEnabled');
}
}
10 changes: 1 addition & 9 deletions api/app/tests/Behat/features-v2/acl/login/login.feature
Original file line number Diff line number Diff line change
Expand Up @@ -131,17 +131,9 @@ Feature: Users logging into the service
Scenario: A user logs into the service with their primary account and toggles with the NDR checkbox attached to non-primary account
Given a super admin user accesses the admin app
When a Multi-client Lay Deputy exists and I select the non-primary user
And I "enable" the NDR for this user
And I enable the NDR for this user
And I update the details of the deputy available to me
Then a Lay Deputy tries to login with their "primary" email address
When they choose their "non-primary" Client
Then they should be on the "non-primary" Client's dashboard
And I should see the NDR report on the reports page
Given a super admin user accesses the admin app
When a Multi-client Lay Deputy exists and I select the non-primary user
When I "disable" the NDR for this user
And I update the details of the deputy available to me
Then a Lay Deputy tries to login with their "primary" email address
When they choose their "non-primary" Client
Then they should be on the "non-primary" Client's dashboard
Then I should not see the NDR report on the reports page

0 comments on commit e1caa04

Please sign in to comment.