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

Update protractor to fix travis e2e tests #3747

Merged
merged 10 commits into from
Aug 13, 2019

Conversation

richard-cox
Copy link
Contributor

  • Travis uses chrome stable (now v76)
  • Protractor uses webdriver npm package to download chromedriver
  • Chromedriver only supported chrome v71-75
  • Fix is to update protractor, which updates webdriver, which fetches updated chromedriver

- Travis uses chrome stable (now v76)
- Protractor uses webdriver npm package to download chromedriver
- Chromedriver only supported chrome v71-75
- Fix is to update protractor, which updates webdriver, which fetches updated chromedriver
@cfdreddbot
Copy link

✅ Hey richard-cox! The commit authors and yourself have already signed the CLA.

@richard-cox richard-cox removed the request for review from KlapTrap August 2, 2019 09:45
@codecov
Copy link

codecov bot commented Aug 2, 2019

Codecov Report

Merging #3747 into v2-master will decrease coverage by <.01%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##           v2-master    #3747      +/-   ##
=============================================
- Coverage      51.87%   51.87%   -0.01%     
=============================================
  Files            765      765              
  Lines          22425    22425              
  Branches        4029     4029              
=============================================
- Hits           11634    11633       -1     
- Misses         10791    10792       +1

Copy link
Contributor

@nwmac nwmac left a comment

Choose a reason for hiding this comment

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

A few comments on use of control flow.

@nwmac nwmac added the needs attention This PR needs attention label Aug 12, 2019
@richard-cox richard-cox added ready for review and removed in progress needs attention This PR needs attention labels Aug 12, 2019
Copy link
Contributor

@nwmac nwmac left a comment

Choose a reason for hiding this comment

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

LGTM

@nwmac nwmac merged commit 8bb4236 into v2-master Aug 13, 2019
@nwmac nwmac deleted the fix-travis-webdrive-incompatability branch August 13, 2019 07:43
KlapTrap pushed a commit that referenced this pull request Aug 16, 2019
* v2-master:
  Fix e2e test
  Fix cf init script given changes in broker service (#3746)
  Update protractor to fix travis e2e tests (#3747)
  Same Tweaks at space level - Show edit quota on space level quota details page - Ensure we return from edit stepper from org's space quota list, org level quota page and space level quota page - Apply permissions to edit quota
  Tweaks - Show edit quota on org level quota details page - Ensure we return from edit stepper from cf quota list, cf level quota page and org level quota page - Apply permissions to edit quota - Change org --> space in space level quota page - Fix validation for sub 0 values
  Small tweaks
  created unlimited-input comp / refactored quota forms
  org/space quotas: listing and management
KlapTrap pushed a commit that referenced this pull request Aug 20, 2019
* v3-master: (26 commits)
  Added fixme
  Fix e2e test
  Fix cf init script given changes in broker service (#3746)
  Update protractor to fix travis e2e tests (#3747)
  Moved more cf components from core to cf package
  Two tweaks - Move entityCatalogue clear into testEntityCatalogue, optionally return test if in test env - Move EffectsFeatureTestModule & TEST_CATALOGUE_ENTITIES into own test file
  Moved some cf components from core to cf package
  Removed some comments
  Fix cf-autoscaler unit tests
  Entity validation fixes
  Same Tweaks at space level - Show edit quota on space level quota details page - Ensure we return from edit stepper from org's space quota list, org level quota page and space level quota page - Apply permissions to edit quota
  Tweaks - Show edit quota on org level quota details page - Ensure we return from edit stepper from cf quota list, cf level quota page and org level quota page - Apply permissions to edit quota - Change org --> space in space level quota page - Fix validation for sub 0 values
  Small tweaks
  WIP
  Fix entity relations from parents test
  Build
  Fix imports after merge
  WIP
  Core - fix compile & current user permissions
  First pass autoscaler unit tests
  ...
KlapTrap pushed a commit that referenced this pull request Aug 20, 2019
* v3-master:
  Fix e2e test
  Fix cf init script given changes in broker service (#3746)
  Update protractor to fix travis e2e tests (#3747)
  Same Tweaks at space level - Show edit quota on space level quota details page - Ensure we return from edit stepper from org's space quota list, org level quota page and space level quota page - Apply permissions to edit quota
  Tweaks - Show edit quota on org level quota details page - Ensure we return from edit stepper from cf quota list, cf level quota page and org level quota page - Apply permissions to edit quota - Change org --> space in space level quota page - Fix validation for sub 0 values
  Small tweaks
  created unlimited-input comp / refactored quota forms
  org/space quotas: listing and management
KlapTrap pushed a commit that referenced this pull request Aug 21, 2019
* fix-route-create: (26 commits)
  Fix domain dropdown on route creation
  Added fav reducers back
  Added fixme
  Fix e2e test
  Fix cf init script given changes in broker service (#3746)
  Update protractor to fix travis e2e tests (#3747)
  Two tweaks - Move entityCatalogue clear into testEntityCatalogue, optionally return test if in test env - Move EffectsFeatureTestModule & TEST_CATALOGUE_ENTITIES into own test file
  Removed some comments
  Fix cf-autoscaler unit tests
  Entity validation fixes
  Same Tweaks at space level - Show edit quota on space level quota details page - Ensure we return from edit stepper from org's space quota list, org level quota page and space level quota page - Apply permissions to edit quota
  Tweaks - Show edit quota on org level quota details page - Ensure we return from edit stepper from cf quota list, cf level quota page and org level quota page - Apply permissions to edit quota - Change org --> space in space level quota page - Fix validation for sub 0 values
  Small tweaks
  WIP
  Fix entity relations from parents test
  Build
  Fix imports after merge
  WIP
  Core - fix compile & current user permissions
  First pass autoscaler unit tests
  ...
KlapTrap pushed a commit that referenced this pull request Aug 21, 2019
* entity-pipeline: (30 commits)
  Tidy up get request action method.
  Small fixes
  Fixed user apce roles data source
  Merge changes
  Fix a few low hanging fruit, added some TODOs for me
  Added backward compatility to old action types
  Added fixme
  Fix e2e test
  Fix cf init script given changes in broker service (#3746)
  Update protractor to fix travis e2e tests (#3747)
  Two tweaks - Move entityCatalogue clear into testEntityCatalogue, optionally return test if in test env - Move EffectsFeatureTestModule & TEST_CATALOGUE_ENTITIES into own test file
  Removed some comments
  Fix cf-autoscaler unit tests
  Entity validation fixes
  Same Tweaks at space level - Show edit quota on space level quota details page - Ensure we return from edit stepper from org's space quota list, org level quota page and space level quota page - Apply permissions to edit quota
  Tweaks - Show edit quota on org level quota details page - Ensure we return from edit stepper from cf quota list, cf level quota page and org level quota page - Apply permissions to edit quota - Change org --> space in space level quota page - Fix validation for sub 0 values
  Small tweaks
  WIP
  Fix entity relations from parents test
  Build
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants