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

252/feature/desktop search e2 etest #314

Merged

Conversation

giacomo-cgn
Copy link
Collaborator

Adds e2e desktop search tests with testcafe and mocked request response for search, solves issue #252

@cdrini cdrini self-assigned this May 20, 2020
Copy link
Contributor

@cdrini cdrini left a comment

Choose a reason for hiding this comment

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

Nice! This is looking great :) Mostly just some comments on readability/consistency.

tests/e2e/helpers/desktopSearch.js Outdated Show resolved Hide resolved
tests/e2e/helpers/desktopSearch.js Outdated Show resolved Hide resolved
tests/e2e/helpers/desktopSearch.js Outdated Show resolved Hide resolved
tests/e2e/helpers/desktopSearch.js Outdated Show resolved Hide resolved
tests/e2e/helpers/mockSearch.js Outdated Show resolved Hide resolved
tests/e2e/helpers/mockSearch.js Outdated Show resolved Hide resolved
tests/e2e/helpers/mockSearch.js Outdated Show resolved Hide resolved
tests/e2e/models/Navigation.js Show resolved Hide resolved
tests/e2e/models/Navigation.js Outdated Show resolved Hide resolved
tests/e2e/helpers/desktopSearch.js Show resolved Hide resolved
Separated succesful and unsuccesful search tests
Integrated Navigation.js with searchBox and querySign for better code consistency
Removed br.BRcontainer and br.shell existance check
Modified searchPin
@cdrini cdrini force-pushed the 252/feature/desktopSearchE2Etest branch from 0208013 to d57139e Compare May 22, 2020 14:48
Copy link
Contributor

@cdrini cdrini left a comment

Choose a reason for hiding this comment

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

Lgtm! I rebased on master, and will merge once travis is done 👍

@cdrini cdrini merged commit a52ff1f into internetarchive:master May 22, 2020
@giacomo-cgn giacomo-cgn deleted the 252/feature/desktopSearchE2Etest branch May 23, 2020 09:58
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.

2 participants