-
Notifications
You must be signed in to change notification settings - Fork 0
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
[BI-1692] - Change role names #142
Conversation
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.
Looks good.
@@ -15,7 +15,7 @@ Feature: Logging with Member | |||
And user can see "Program Administration" in navigation | |||
|
|||
@BI-823 | |||
Scenario: Logging in as a breeder of one program | |||
Scenario: Logging in as a Read Only of one program |
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.
Was this an issue in the description of the test?
@@ -28,14 +28,14 @@ Feature: Logging with Member | |||
And user can see "Program Administration" in navigation | |||
|
|||
@BI-845 | |||
Scenario: Logging in as a breeder of one program | |||
Scenario: Logging in as a Read Only of one program |
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.
Seems like this was just a labeling issue of the scenario like the last one
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.
Yes, I figured I might as well fix the labels since I was changing them anyway.
Description
Story: BI-1692 - Change role names
TAF changes in response to changing role domains as follows:
Areas updated to use new role domains:
Dependencies
bi-web: feature/BI-1692
bi-api: feature/BI-1692
Testing
Please include a link to a successful run of TAF for this change
Checklist: