Skip to content

Synchronize driver variables and correct Cucumber resource locations#4

Open
Mahaklambda1811 wants to merge 1 commit intoLambdaTest:masterfrom
Mahaklambda1811:master
Open

Synchronize driver variables and correct Cucumber resource locations#4
Mahaklambda1811 wants to merge 1 commit intoLambdaTest:masterfrom
Mahaklambda1811:master

Conversation

@Mahaklambda1811
Copy link
Copy Markdown

Corrected Feature File Path:
Updated the @CucumberOptions in TestRunner.java from src/test/resources/features to src/main/java/resources/features

Cleaned Up Hooks:
Removed executeScript commands from Hook.java to prevent unnecessary overhead during the simplified launch test.

Fixed Compilation Errors:
Resolved "cannot find symbol" errors by synchronizing variable names;
Also updated Hook.java to use the driver variable defined in TestRunner.java.

options.setPlatformName("Android");
options.setDeviceName("Galaxy S21");
options.setPlatformVersion("13");
options.setApp("lt://APP10160171061769791489195489");
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggested change
options.setApp("lt://APP10160171061769791489195489");
options.setApp("<your_app_id>");

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

what is the use of this?

import org.testng.Assert;
import io.cucumber.testng.AbstractTestNGCucumberTests;
import io.cucumber.testng.CucumberOptions;
import io.cucumber.testng.TestNGCucumberRunner;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

removed the whole code?

Map<String, Object> ltOptions = new HashMap<>();
// Ensure these environment variables are set on your machine
ltOptions.put("user", System.getenv("LT_USERNAME"));
ltOptions.put("accessKey", System.getenv("LT_ACCESS_KEY"));
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Update the Readme.md as well

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Why have we removed this one?

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