Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

Closes #3977: Add basic navigational UI tests #3978

Merged
merged 1 commit into from
Jul 16, 2019

Conversation

AaronMT
Copy link
Contributor

@AaronMT AaronMT commented Jul 10, 2019

Pull Request checklist

  • Quality: This PR builds and passes detekt/ktlint checks (A pre-push hook is recommended)

  • Tests: This PR includes thorough tests or an explanation of why it does not

  • Changelog: This PR includes a changelog entry or does not need one

  • Accessibility: The code in this PR follows accessibility best practices or does not include any user facing features

  • Adds UI tests (and assets) for basic navigation

@AaronMT AaronMT requested a review from rpappalax July 10, 2019 15:09
@AaronMT AaronMT requested a review from a team as a code owner July 10, 2019 15:09
Copy link
Contributor

@jhugman jhugman left a comment

Choose a reason for hiding this comment

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

I love the robot pattern!

This is a good start. There are a few changes I've highlighted.

You should probably also run a linter over the code: whitespace in kotlin, javascript and html is all bit uneven.

@AaronMT AaronMT force-pushed the NavigationToolbar-UI-Tests branch 3 times, most recently from ced66ba to e8ffe86 Compare July 15, 2019 19:23
@AaronMT AaronMT force-pushed the NavigationToolbar-UI-Tests branch from e8ffe86 to 0ed5b37 Compare July 16, 2019 15:00
- Adds UI tests (and assets) for basic navigation

fix: review changes

fix: linter cleanup

fix: detekt cleanup

fix: adjust wait approaches
@AaronMT AaronMT force-pushed the NavigationToolbar-UI-Tests branch from 0ed5b37 to 432fa67 Compare July 16, 2019 15:21
@AaronMT AaronMT requested a review from rpappalax July 16, 2019 16:27
Copy link
Contributor

@rpappalax rpappalax left a comment

Choose a reason for hiding this comment

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

looks great 👍

@AaronMT AaronMT dismissed jhugman’s stale review July 16, 2019 20:01

Fixed in squashed commit

@rpappalax rpappalax merged commit 6aa6cf8 into mozilla-mobile:master Jul 16, 2019
@AaronMT AaronMT deleted the NavigationToolbar-UI-Tests branch July 19, 2019 16:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants