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

Implementing flick action in ajio mobile #241

Merged
merged 4 commits into from
Jul 7, 2023

Conversation

raghavgarg1996
Copy link
Contributor

Implemented flick action gesture in ajio mobile.

driver.waitTillElementIsPresent(By.id("com.ril.ajio:id/layout_category_container")).click();
public ProductScreen selectProduct() {
LOGGER.info("Selection of Product in the result page");
if (!(driver.isElementPresent(byProductLayoutId))) {
Copy link
Contributor

Choose a reason for hiding this comment

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

do we need extra braces in if condition?
if(!driver.isElementPresent(byProductLayoutId))

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes - when we land on the AJIO app on the first time - few sections got highlighted - and web elements are present below these elements

@@ -15,6 +16,8 @@ public class SearchScreenAndroid
private static final Logger LOGGER = Logger.getLogger(SCREEN_NAME);
private static final By byResultsId = By.id("com.ril.ajio:id/tv_count_plp_header_is");
private static final By byProductId = By.id("com.ril.ajio:id/plp_row_product_iv");
private static final By byProductListTitleId = By.id("com.ril.ajio:id/toolbar_title_tv");
public static final By byProductLayoutId = By.id("com.ril.ajio:id/layout_category_container");
Copy link
Contributor

Choose a reason for hiding this comment

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

public variable?
are you using this VAR in another class?

@anandbagmar anandbagmar merged commit 5534cf6 into znsio:main Jul 7, 2023
2 checks passed
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.

3 participants