Merge pull request #154 from brendandburns/ut

Add a test for empty users.

Author: k8s-ci-robot

src/config_test.ts

@@ -11,6 +11,7 @@ import { bufferFromFileOrString, findHomeDir, findObject, KubeConfig, Named } fr
 import { Cluster, newClusters, newContexts, newUsers, User } from './config_types';
 
 const kcFileName = 'testdata/kubeconfig.yaml';
+const kcNoUserFileName = 'testdata/empty-user-kubeconfig.yaml';
 
 /* tslint:disable: no-empty */
 describe('Config', () => { });
@@ -174,6 +175,13 @@ describe('KubeConfig', () => {
         });
     });
 
+    describe('loadEmptyUser', () => {
+        it('should load a kubeconfig with an empty user', () => {
+            const kc = new KubeConfig();
+            kc.loadFromFile(kcNoUserFileName);
+        });
+    });
+
     describe('applyHTTPSOptions', () => {
         it('should apply cert configs', () => {
             const kc = new KubeConfig();

testdata/empty-user-kubeconfig.yaml

@@ -0,0 +1,29 @@
+apiVersion: v1
+clusters:
+- cluster:
+    certificate-authority-data: Q0FEQVRB 
+    server: http://example.com 
+  name: cluster1 
+- cluster:
+    certificate-authority-data: Q0FEQVRBMg==
+    server: http://example2.com
+    insecure-skip-tls-verify: true
+  name: cluster2
+
+contexts:
+- context:
+    cluster: cluster1 
+    user: user1
+  name: context1 
+- context:
+    cluster: cluster2
+    user: user2
+  name: context2
+
+current-context: context2 
+kind: Config
+preferences: {}
+users:
+- name: user1
+  user:
+- name: user2