Skip to content

Move vcr_test_base to testsdk#3224

Merged
troydai merged 2 commits intoAzure:masterfrom
troydai:move.vcr
May 8, 2017
Merged

Move vcr_test_base to testsdk#3224
troydai merged 2 commits intoAzure:masterfrom
troydai:move.vcr

Conversation

@troydai
Copy link
Copy Markdown
Contributor

@troydai troydai commented May 5, 2017

To enable run automation on a built package without setting up a development environment.

@troydai troydai requested a review from derekbekoe May 5, 2017 23:35
@codecov-io
Copy link
Copy Markdown

codecov-io commented May 5, 2017

Codecov Report

Merging #3224 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3224   +/-   ##
=======================================
  Coverage   70.35%   70.35%           
=======================================
  Files         381      381           
  Lines       24777    24777           
  Branches     3785     3785           
=======================================
  Hits        17431    17431           
  Misses       6237     6237           
  Partials     1109     1109
Impacted Files Coverage Δ
...ure-cli-testsdk/azure/cli/testsdk/vcr_test_base.py 69.18% <ø> (ø)

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 80569f7...1a3728c. Read the comment docs.

@troydai troydai requested review from tjprescott and yugangw-msft May 5, 2017 23:49
Copy link
Copy Markdown
Member

@tjprescott tjprescott left a comment

Choose a reason for hiding this comment

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

LGTM.

@troydai troydai merged commit 76c2d18 into Azure:master May 8, 2017
@troydai troydai deleted the move.vcr branch May 8, 2017 16:08
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.

5 participants