Skip to content

Fix buildname#12

Merged
samiran-bs merged 6 commits intomainfrom
fix-buildname
Jul 2, 2021
Merged

Fix buildname#12
samiran-bs merged 6 commits intomainfrom
fix-buildname

Conversation

@roy-de-kleijn
Copy link
Copy Markdown
Contributor

No description provided.

String message = result.getThrowable().getMessage();
String reason = (message != null && message.length() > 254) ? message.substring(0, 254) : message;
markTestStatus("failed", reason, driver);
markTestStatus("failed", reason.replaceAll("[^a-zA-Z0-9]", " "), driver);
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

just thought about . (full stop) should we add that? It will make the sentences readable.

"os_version": "Big Sur",
"browser": "Chrome",
"browser_version": "latest"
"browser_version": "90.0"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

y not latest? with a hardcoded value, the code needs to be updated everytime

Comment thread build.gradle Outdated
}
systemProperty "environment", "remote"
systemProperty "test", "single"
systemProperty "testtype", "single"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

camel case?

@samiran-bs samiran-bs merged commit 219826b into main Jul 2, 2021
@samiran-bs samiran-bs deleted the fix-buildname branch July 2, 2021 15:51
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.

4 participants