Merge pull request #431 from brendandburns/oidc

Add a test for exec token caching and fix the implementation.

Author: k8s-ci-robot

src/config.ts

@@ -531,6 +531,7 @@ export function findObject<T extends Named>(list: T[], name: string, key: string
     for (const obj of list) {
         if (obj.name === name) {
             if (obj[key]) {
+                obj[key].name = name;
                 return obj[key];
             }
             return obj;

src/config_test.ts

@@ -924,6 +924,32 @@ describe('KubeConfig', () => {
                 expect(opts.headers.Authorization).to.equal(`Bearer ${token}`);
             }
         });
+        it('should cache exec with name', async () => {
+            const config = new KubeConfig();
+            const token = 'token';
+            const responseStr = `{
+                "apiVersion": "client.authentication.k8s.io/v1beta1",
+                "kind": "ExecCredential",
+                "status": {
+                  "token": "${token}"
+                }
+              }`;
+            config.loadFromClusterAndUser(
+                { skipTLSVerify: false } as Cluster,
+                {
+                    name: 'exec',
+                    exec: {
+                        command: 'echo',
+                        args: [`${responseStr}`],
+                    },
+                } as User,
+            );
+            // TODO: inject the exec command here?
+            const opts = {} as requestlib.Options;
+            await config.applyToRequest(opts);
+            expect((KubeConfig as any).authenticators[1].tokenCache['exec']).to.deep.equal(JSON.parse(responseStr));
+        });
+
         it('should throw with no command.', () => {
             const config = new KubeConfig();
             config.loadFromClusterAndUser(