Skip to content

Skip all the createWriteStream() functional tests on OSX. - #10449

Merged
Eric Snow (ericsnowcurrently) merged 1 commit into
microsoft:masterfrom
ericsnowcurrently:fix-9953-skip-createWriteStream-functional-tests-on-OSX
Mar 6, 2020
Merged

Eric Snow (ericsnowcurrently) merged 1 commit into
microsoft:masterfrom
ericsnowcurrently:fix-9953-skip-createWriteStream-functional-tests-on-OSX

Conversation

@ericsnowcurrently

Copy link
Copy Markdown

(for #9953)

A while back we skipped one of the "raw" createWriteStream() functional tests (but only on OSX). However, other tests in the same suite are having the same problem. So we are skipping all the tests in the suite for now (to be resolved in #10031).

@ericsnowcurrently Eric Snow (ericsnowcurrently) added the no-changelog No news entry required label Mar 5, 2020
@sonarqubecloud

sonarqubecloud Bot commented Mar 5, 2020

Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@codecov-io

Codecov (codecov-io) commented Mar 5, 2020 •

Copy link
Copy Markdown

Codecov Report

Merging #10449 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #10449      +/-   ##
==========================================
- Coverage   60.95%   60.94%   -0.02%     
==========================================
  Files         571      578       +7     
  Lines       30809    31190     +381     
  Branches     4387     4429      +42     
==========================================
+ Hits        18779    19008     +229     
- Misses      11077    11216     +139     
- Partials      953      966      +13
Impacted Files Coverage Δ
src/datascience-ui/react-common/arePathsSame.ts 75% <0%> (-12.5%) ⬇️
src/client/common/utils/platform.ts 64.7% <0%> (-11.77%) ⬇️
src/client/linters/pydocstyle.ts 86.66% <0%> (-2.23%) ⬇️
src/client/activation/node/manager.ts 28.3% <0%> (-1.7%) ⬇️
src/client/datascience/debugLocationTracker.ts 76.56% <0%> (-1.57%) ⬇️
...tascience/interactive-ipynb/nativeEditorStorage.ts 49.36% <0%> (-1.29%) ⬇️
src/client/datascience/jupyter/serverSelector.ts 90.76% <0%> (-1.07%) ⬇️
src/client/common/process/proc.ts 14.49% <0%> (-0.73%) ⬇️
src/client/datascience/jupyter/jupyterSession.ts 65.35% <0%> (-0.43%) ⬇️
.../datascience/interactive-common/interactiveBase.ts 5.71% <0%> (-0.01%) ⬇️
... and 29 more

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 63befc6...54af0e7. Read the comment docs.

@ericsnowcurrently
Eric Snow (ericsnowcurrently) merged commit 4ecda64 into microsoft:master Mar 6, 2020
@ericsnowcurrently
Eric Snow (ericsnowcurrently) deleted the fix-9953-skip-createWriteStream-functional-tests-on-OSX branch March 6, 2020 15:03
@lock lock Bot locked as resolved and limited conversation to collaborators Mar 17, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

no-changelog No news entry required

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants