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

Upgrades to intl-tel-input:14.0.0 and fixes protractor tests #127

Closed
wants to merge 4 commits into from
Closed

Upgrades to intl-tel-input:14.0.0 and fixes protractor tests #127

wants to merge 4 commits into from

Conversation

pitpit
Copy link

@pitpit pitpit commented Oct 5, 2018

fixes #126

End-to-end test with protractor and firefox were broken :

  • Travis do not support firefox 36.0.4 anymore, it installs 56.0.2 when running the tests
  • [email protected] do not support firefox 56.0.2

I try updating protractor to each major latest version (4.0.14, 3.3.0, 2.5.1) but no one supports firefox 56.0.2.
I had to update protractor to latest release: 5.4.1 using gecko webdriver 0.19.0.

Unfortunately [email protected] is only supported on node>=4.9.x, so I replaced test coverage from Node from 0.12 to 5.x by Node from 6.x to 10.x in .travis.yml

@pitpit pitpit changed the title Upgrade to intl-tel-input:14.0.0 [WIP] Upgrade to intl-tel-input:14.0.0 Oct 5, 2018
@pitpit
Copy link
Author

pitpit commented Oct 8, 2018

protractor tests failed because rate limit of github is reached...

(node:4224) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): Error: Failed to make Github request, rate limit reached.

I will give a try tomorrow

@pitpit pitpit changed the title [WIP] Upgrade to intl-tel-input:14.0.0 Upgrades to intl-tel-input:14.0.0 and fixes selenium tests Oct 8, 2018
@pitpit pitpit changed the title Upgrades to intl-tel-input:14.0.0 and fixes selenium tests Upgrades to intl-tel-input:14.0.0 and fixes protractor tests Oct 8, 2018
install:
- "npm install"
- "./node_modules/bower/bin/bower install"
before_script:
- "export DISPLAY=:99.0"
- "sh -e /etc/init.d/xvfb start"
Copy link
Author

@pitpit pitpit Oct 9, 2018

Choose a reason for hiding this comment

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

xvfb is not needed anymore, because we're using firefox headless mode

@pitpit
Copy link
Author

pitpit commented Oct 9, 2018

hi @bighappyface, could you add a github access token to travis to avoid rate limit in travis ? I fixed all tests but it continues to fail because that rate limit reached...

@pitpit
Copy link
Author

pitpit commented Nov 21, 2018

up

@pitpit pitpit closed this by deleting the head repository Jun 17, 2024
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.

it does not work with last release of intl-tel-input
1 participant