Skip to content
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

Selected input scenarios for basic joins and advance scenarios for joiner #6

Closed
wants to merge 1 commit into from

Conversation

rahuldash171
Copy link
Collaborator

Tests Covered :
Advance Inner Join
Advance Outer Join
Basic Inner Join with selected inputs
Basic outer join with selected inputs and required input

return SeleniumDriver.getDriver().findElement(By.xpath(xpath));
}

public static WebElement requiredInputCheckboxFirstFile(String pluginName){
Copy link
Collaborator

Choose a reason for hiding this comment

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

Update: pluginName to input.
Remove the second locator and use the index as a parameter.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Done.

@@ -36,6 +36,16 @@ public void uncheckPluginFieldAliasCheckBox(String plugin, String field) {
JoinerActions.uncheckPluginFieldAliasCheckBox(plugin, field);
}

@Then("Click on the required input checkbox for first schema {string}")
Copy link
Collaborator

Choose a reason for hiding this comment

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

Add index and remove the second steps

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Done , removed second and used index

@rahuldash171 rahuldash171 force-pushed the Joiner_new_Features branch 2 times, most recently from edb228e to c190827 Compare August 29, 2023 14:53
Selected input scenarios for basic joins.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants