-
Notifications
You must be signed in to change notification settings - Fork 3
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
Test cases updates for NCCR, VIBE, OA Intake #376
base: master
Are you sure you want to change the base?
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.
please review the comments
@@ -91,15 +91,14 @@ Feature: Create OA Intake form | |||
And User logs out of OA Intake application | |||
|
|||
@selenium @Alena @Regression @OAIntake-332 @OAIntake-670 @OAIntake-669 | |||
|
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.
extra blank space on line 94
@Given("User can verify page header is {string}") | ||
public void user_can_verify_page_header_is(String header) { | ||
softAssert.assertEquals(oaIntakePage.leadershipQueuePageHeader.getText(), header); | ||
oaIntakeStepsImplementation.verifyHeadrDisplayed(header); |
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.
missing javadoc statement for this method
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 the existing Javadoc has been removed:
/**
* verify Requestor (Requested For) Queue tab is displayed
*/
@@ -8,6 +8,8 @@ | |||
import com.nci.automation.web.CommonUtils; | |||
import com.nci.automation.web.WebDriverUtils; | |||
import org.testng.Assert; | |||
|
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.
extra space on line 11
@@ -67,6 +67,11 @@ public class AwardNominationPage { | |||
@FindBy(xpath = "//span[.=' Tickets ']") | |||
public WebElement sectionTickets; | |||
|
|||
|
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.
extra space on line 70
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.
Hi,
I agree with Luis's comments and added one more for your review. Thanks!
@Given("User can verify page header is {string}") | ||
public void user_can_verify_page_header_is(String header) { | ||
softAssert.assertEquals(oaIntakePage.leadershipQueuePageHeader.getText(), header); | ||
oaIntakeStepsImplementation.verifyHeadrDisplayed(header); |
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 the existing Javadoc has been removed:
/**
* verify Requestor (Requested For) Queue tab is displayed
*/
And User can verify Optional Responses option "In My Own Words - A text box will appear below when checked." is displayed | ||
|
||
|
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.
Please remove the empty line
No description provided.