Add test on applytoHTTPSOptions

Author: sheldonkwok

node-client/src/config.ts

@@ -169,7 +169,7 @@ export class KubeConfig {
         this.applyAuthorizationHeader(opts);
     }
 
-    public applytoHTTPsOptions(opts: https.RequestOptions) {
+    public applytoHTTPSOptions(opts: https.RequestOptions) {
         const user = this.getCurrentUser();
 
         this.applyOptions(opts);

node-client/src/config_test.ts

@@ -1,7 +1,11 @@
-import { KubeConfig, Config } from './config';
-import { expect } from 'chai';
 import 'mocha';
 
+import { expect } from 'chai';
+import * as https from 'https';
+import * as base64 from 'base-64';
+
+import { KubeConfig, Config } from './config';
+
 const kcFileName = "testdata/kubeconfig.yaml";
 
 describe("Config", () => {
@@ -51,22 +55,22 @@ describe("KubeConfig", () => {
             let cluster1 = kc.clusters[0];
             let cluster2 = kc.clusters[1];
             expect(cluster1.name).to.equal("cluster1");
-            expect(cluster1.caData).to.equal("CADATA");
+            expect(cluster1.caData).to.equal("Q0FEQVRB");
             expect(cluster1.server).to.equal("http://example.com");
             expect(cluster2.name).to.equal("cluster2");
-            expect(cluster2.caData).to.equal("CADATA2");
+            expect(cluster2.caData).to.equal("Q0FEQVRBMg==");
             expect(cluster2.server).to.equal("http://example2.com");
 
             // check users
             expect(kc.users.length).to.equal(2);
             let user1 = kc.users[0];
             let user2 = kc.users[1];
             expect(user1.name).to.equal("user1");
-            expect(user1.certData).to.equal("USER_CADATA");
-            expect(user1.keyData).to.equal("USER_CKDATA");
+            expect(user1.certData).to.equal("VVNFUl9DQURBVEE=");
+            expect(user1.keyData).to.equal("VVNFUl9DS0RBVEE=");
             expect(user2.name).to.equal("user2");
-            expect(user2.certData).to.equal("USER2_CADATA");
-            expect(user2.keyData).to.equal("USER2_CKDATA");
+            expect(user2.certData).to.equal("VVNFUjJfQ0FEQVRB");
+            expect(user2.keyData).to.equal("VVNFUjJfQ0tEQVRB");
 
             // check contexts
             expect(kc.contexts.length).to.equal(2);
@@ -87,4 +91,20 @@ describe("KubeConfig", () => {
             // expect(kc.loadFromFile("missing.yaml")).to.throw();
         });
     });
+
+    describe("applyHTTPSOptions", () => {
+        it("should apply cert configs", () => {
+            const kc = new KubeConfig();
+            kc.loadFromFile(kcFileName);
+
+            const opts: https.RequestOptions = {};
+            kc.applytoHTTPSOptions(opts);
+
+            expect(opts).to.deep.equal({
+                ca: new Buffer('CADATA2', 'utf-8'),
+                cert: new Buffer('USER2_CADATA', 'utf-8'),
+                key: new Buffer('USER2_CKDATA', 'utf-8'),
+            });
+        });
+    });
 });

node-client/testdata/kubeconfig.yaml

@@ -1,18 +1,18 @@
 apiVersion: v1
 clusters:
 - cluster:
-    certificate-authority-data: CADATA
+    certificate-authority-data: Q0FEQVRB 
     server: http://example.com 
   name: cluster1 
 - cluster:
-    certificate-authority-data: CADATA2
+    certificate-authority-data: Q0FEQVRBMg==
     server: http://example2.com
   name: cluster2
 
 contexts:
 - context:
     cluster: cluster1 
-    user: user1 
+    user: user1
   name: context1 
 - context:
     cluster: cluster2
@@ -23,11 +23,11 @@ current-context: context2
 kind: Config
 preferences: {}
 users:
-- name: user1 
+- name: user1
   user:
-    client-certificate-data: USER_CADATA
-    client-key-data: USER_CKDATA
+    client-certificate-data: VVNFUl9DQURBVEE=
+    client-key-data: VVNFUl9DS0RBVEE=
 - name: user2
   user:
-    client-certificate-data: USER2_CADATA
-    client-key-data: USER2_CKDATA
+    client-certificate-data: VVNFUjJfQ0FEQVRB
+    client-key-data: VVNFUjJfQ0tEQVRB