test: config.ts

redhat-plumbers-in-action · Oct 14, 2022 · 83556fc · 83556fc
1 parent d4b3a35
commit 83556fc
Show file tree

Hide file tree

Showing 11 changed files with 45 additions and 21 deletions.
diff --git a/dist/app.js b/dist/app.js
diff --git a/dist/app.js.map b/dist/app.js.map
diff --git a/dist/config.d.ts b/dist/config.d.ts
diff --git a/dist/config.js b/dist/config.js
diff --git a/dist/config.js.map b/dist/config.js.map
diff --git a/dist/index.js b/dist/index.js
diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/src/app.ts b/src/app.ts
@@ -40,7 +40,7 @@ const app = (probot: Probot) => {
         }
 
         await pullRequest.freeze(
-          policyItem.feedback.freezedState,
+          policyItem.feedback.frozenState,
           tag.latest,
           context
         );
@@ -56,7 +56,7 @@ const app = (probot: Probot) => {
           continue;
         }
 
-        await pullRequest.unfreeze(policyItem.feedback.unFreezedState, context);
+        await pullRequest.unfreeze(policyItem.feedback.unFreezeState, context);
         return;
       }
 

diff --git a/src/config.ts b/src/config.ts
@@ -103,11 +103,11 @@ class Feedback {
     this._unFreezeState = feedback['unfreeze-state'];
   }
 
-  get freezedState() {
+  get frozenState() {
     return this._frozenState;
   }
 
-  get unFreezedState() {
+  get unFreezeState() {
     return this._unFreezeState;
   }
 }
diff --git a/test/unit/__snapshots__/config.test.ts.snap b/test/unit/__snapshots__/config.test.ts.snap
@@ -15,6 +15,24 @@ exports[`Config Object > get policy() 1`] = `
 ]
 `;
 
+exports[`Config Object > get policy() 2`] = `
+[
+  "alpha",
+  "beta",
+]
+`;
+
+exports[`Config Object > get policy() 3`] = `
+Feedback {
+  "_frozenState": "This is No-No",
+  "_unFreezeState": "This is Yes-Yes",
+}
+`;
+
+exports[`Config Object > get policy() 4`] = `"This is No-No"`;
+
+exports[`Config Object > get policy() 5`] = `"This is Yes-Yes"`;
+
 exports[`Config Object > is invalid 1`] = `
 [
   {

diff --git a/test/unit/config.test.ts b/test/unit/config.test.ts
@@ -18,7 +18,16 @@ describe('Config Object', () => {
   });
 
   test<IConfigTestContext>('get policy()', context => {
-    context.configs.map(item => expect(item.policy).toMatchSnapshot());
+    context.configs.map(configItem => {
+      expect(configItem.policy).toMatchSnapshot();
+
+      configItem.policy.map(policyItem => {
+        expect(policyItem.tags).toMatchSnapshot();
+        expect(policyItem.feedback).toMatchSnapshot();
+        expect(policyItem.feedback.frozenState).toMatchSnapshot();
+        expect(policyItem.feedback.unFreezeState).toMatchSnapshot();
+      });
+    });
   });
 
   it<IConfigTestContext>('is valid', async context => {
@@ -45,6 +54,3 @@ describe('Config Object', () => {
     expect(Config.isConfigEmpty(null)).toEqual(true);
   });
 });
-
-describe.todo('PolicyItem Object');
-describe.todo('Feedback Object');