Skip to content
This repository has been archived by the owner on Apr 27, 2020. It is now read-only.

Retain current context to remove dependency on minikube #38

Merged
merged 1 commit into from
Aug 6, 2018

Conversation

yuanying
Copy link
Member

@yuanying yuanying commented Aug 6, 2018

Currently test code depend on minikube environment, so test will
fail if you use extra environment.
This patch removes minikube dependency in test code.

test/suite.bats Outdated
@@ -3,14 +3,15 @@
load helper

setup() {
current_context=$(kubectl config current-context)
Copy link
Contributor

@superbrothers superbrothers Aug 6, 2018

Choose a reason for hiding this comment

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

Command substitution should be enclosed by double quotes.

current_context="$(kubectl config current-context)"

Copy link
Member Author

Choose a reason for hiding this comment

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

Done

Currently test code depend on minikube environment, so test will
fail if you use extra environment.
This patch removes minikube dependency in test code.
@yuanying yuanying force-pushed the fix-to-retain-current-context branch from 3183bc9 to 0946168 Compare August 6, 2018 05:37
@superbrothers superbrothers merged commit 1026bfd into master Aug 6, 2018
@superbrothers superbrothers deleted the fix-to-retain-current-context branch August 6, 2018 05:39
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.

2 participants