Skip to content
This repository has been archived by the owner on Dec 8, 2022. It is now read-only.

Disabling standalone install for webdriver manager #349

Merged
merged 2 commits into from
Jan 17, 2018

Conversation

Blackbaud-SteveBrush
Copy link
Member

@Blackbaud-SteveBrush Blackbaud-SteveBrush commented Dec 18, 2017

Addresses: blackbaud/skyux2#1370

SPA now passing:
https://blackbaud.visualstudio.com/Products/_build/index?buildId=456209&_a=summary

Works on the Browserstack SPA as well:
https://blackbaud.visualstudio.com/Products/_build/index?buildId=456217&_a=summary

@Blackbaud-BobbyEarl: So this inadvertently started working again. Originally, the issue only occurred when trying to download the "standalone" JAR from the remote (setting --standalone false fixed the issue at the time). After a few hours, the remote server was working again. At any rate, would this change still be appropriate, since we aren't specifically running standalone, but executing e2e directly against the Chrome driver?

@codecov-io
Copy link

codecov-io commented Dec 18, 2017

Codecov Report

Merging #349 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #349   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          65     65           
  Lines        1631   1631           
  Branches      254    254           
=====================================
  Hits         1631   1631
Flag Coverage Δ
#builder 100% <100%> (ø) ⬆️
#runtime 100% <ø> (ø) ⬆️
#srcapp 100% <ø> (ø) ⬆️
Impacted Files Coverage Δ
cli/e2e.js 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 00324a2...2e1a139. Read the comment docs.

Copy link
Contributor

@blackbaud-brandonhare blackbaud-brandonhare left a comment

Choose a reason for hiding this comment

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

Looks good to me. This has the added bonus of speeding up e2e tests!

@Blackbaud-SteveBrush Blackbaud-SteveBrush merged commit 75c0c44 into master Jan 17, 2018
@Blackbaud-SteveBrush Blackbaud-SteveBrush deleted the fix-webdriver-manager branch January 17, 2018 18:00
Blackbaud-MikitaYankouski pushed a commit to Blackbaud-MikitaYankouski/skyux-builder that referenced this pull request May 3, 2019
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.

4 participants