Improve the error message on session creation by mykola-mokhnach · Pull Request #994 · appium/java-client

Conversation

@mykola-mokhnach

Change list

Currently users are confused by the error message, which appears on sessions creation. This change should make the error more informative and easier to fix.

Types of changes

  • No changes in production code.
  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Mykola Mokhnach added 2 commits

August 7, 2018 14:18

SrinivasanTarget

Choose a reason for hiding this comment

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

if (e.getCause() instanceof WebDriverException) {
message += " Please check the server log for more details.";
}
message += format(" Original error: %s", e.getCause().getMessage());

Choose a reason for hiding this comment

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

👍

Choose a reason for hiding this comment

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

fixed checkstyle

saikrishna321