Prompt to install test framework only if test frame is not installed #6768

DonJayamanne · 2019-07-30T17:45:01Z

Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR)
Title summarizes what is changing
Has a news entry file (remember to thank yourself!)
[n/a] Appropriate comments and documentation strings in the code
[n/a] Has sufficient logging.
[n/a] Has telemetry for enhancements.
Unit tests & system/integration tests are added/updated
[n/a] Test plan is updated as appropriate
[n/a] package-lock.json has been regenerated by running npm install (if dependencies have changed)
[n/a] The wiki is updated with any design decisions/details.

src/test/testing/common/managers/baseTestManager.unit.test.ts

DonJayamanne added 2 commits July 29, 2019 13:06

Fixes

f8f5ac8

News entry

d7ef789

karrtikr reviewed Jul 30, 2019

View reviewed changes

src/test/testing/common/managers/baseTestManager.unit.test.ts Show resolved Hide resolved

karrtikr reviewed Jul 30, 2019

View reviewed changes

src/test/testing/common/managers/baseTestManager.unit.test.ts Outdated Show resolved Hide resolved

Address code review comments

5f7a51d

DonJayamanne requested a review from karrtikr July 30, 2019 18:57

karrtikr approved these changes Jul 30, 2019

View reviewed changes

DonJayamanne merged commit 7376111 into microsoft:master Jul 30, 2019

DonJayamanne deleted the pytestinstall branch July 30, 2019 19:53

lock bot locked as resolved and limited conversation to collaborators Aug 6, 2019

Provide feedback