Skip to content
This repository has been archived by the owner on Oct 12, 2023. It is now read-only.

Test runs no longer delete credentials. #1414

Merged
merged 1 commit into from
Oct 22, 2014
Merged

Test runs no longer delete credentials. #1414

merged 1 commit into from
Oct 22, 2014

Conversation

christav
Copy link

Also consolidated the three different but otherwise identical
credential store tests into a single file that tests all three
options (if the tests can run, that is).

Figured this would be useful when running final tests passes as without this change the test run blows away the logged in user's access tokens in the middle of the run.

Also consolidated the three different but otherwise identical
credential store tests into a single file that tests all three
options (if the tests can run, that is).
@amarzavery
Copy link
Contributor

LGTUS. Reviewed by the team.

amarzavery added a commit that referenced this pull request Oct 22, 2014
Test runs no longer delete credentials.
@amarzavery amarzavery merged commit cb3a373 into Azure:release-0.8.11 Oct 22, 2014
@christav christav deleted the fix-credential-deletion branch October 22, 2014 19:02
}

function isOsx() {
return os.platform() === 'darwin';

Choose a reason for hiding this comment

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

any risk of case sensitivity ?

Copy link
Author

Choose a reason for hiding this comment

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

Not that I've ever seen.

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.

None yet

4 participants