Skip to content

[stable10] Get job id only if sauce environment variables are set#35008

Merged
phil-davis merged 1 commit into
stable10from
stable10-skshetry/job-id-on-sauce
Apr 12, 2019
Merged

[stable10] Get job id only if sauce environment variables are set#35008
phil-davis merged 1 commit into
stable10from
stable10-skshetry/job-id-on-sauce

Conversation

@phil-davis

Copy link
Copy Markdown
Contributor

Backport #35007

The `getWebDriverSession` is not part of the `Mink\Driver\DriverInterface`, but is provided
by the `Selenium2Driver`. So, as Sauce uses selenium grid, this will only be used
when Sauce environment variables are set (other drivers do not have this).
@codecov

codecov Bot commented Apr 12, 2019

Copy link
Copy Markdown

Codecov Report

Merging #35008 into stable10 will decrease coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff               @@
##             stable10   #35008      +/-   ##
==============================================
- Coverage       64.29%   64.29%   -0.01%     
  Complexity      20040    20040              
==============================================
  Files            1285     1285              
  Lines           76824    76824              
  Branches         1307     1307              
==============================================
- Hits            49394    49393       -1     
- Misses          27049    27050       +1     
  Partials          381      381
Flag Coverage Δ Complexity Δ
#javascript 53.01% <ø> (ø) 0 <ø> (ø) ⬇️
#phpunit 65.49% <ø> (-0.01%) 20040 <ø> (ø)
Impacted Files Coverage Δ Complexity Δ
lib/private/Server.php 86.54% <0%> (-0.12%) 253% <0%> (ø)

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 443bf44...ed38caf. Read the comment docs.

1 similar comment
@codecov

codecov Bot commented Apr 12, 2019

Copy link
Copy Markdown

Codecov Report

Merging #35008 into stable10 will decrease coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff               @@
##             stable10   #35008      +/-   ##
==============================================
- Coverage       64.29%   64.29%   -0.01%     
  Complexity      20040    20040              
==============================================
  Files            1285     1285              
  Lines           76824    76824              
  Branches         1307     1307              
==============================================
- Hits            49394    49393       -1     
- Misses          27049    27050       +1     
  Partials          381      381
Flag Coverage Δ Complexity Δ
#javascript 53.01% <ø> (ø) 0 <ø> (ø) ⬇️
#phpunit 65.49% <ø> (-0.01%) 20040 <ø> (ø)
Impacted Files Coverage Δ Complexity Δ
lib/private/Server.php 86.54% <0%> (-0.12%) 253% <0%> (ø)

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 443bf44...ed38caf. Read the comment docs.

@phil-davis phil-davis merged commit 51c8d0b into stable10 Apr 12, 2019
@delete-merged-branch delete-merged-branch Bot deleted the stable10-skshetry/job-id-on-sauce branch April 12, 2019 07:05
@lock lock Bot locked as resolved and limited conversation to collaborators Apr 15, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants