fix empty object foreach

crazy-max · Jul 27, 2022 · 85e00ba · 85e00ba
1 parent 300306f
commit 85e00ba
Show file tree

Hide file tree

Showing 5 changed files with 28 additions and 3 deletions.
diff --git a/.res/labels.merge3.yml b/.res/labels.merge3.yml
@@ -0,0 +1 @@
+_extends: ghaction-github-labeler:.res/labels.merge1.yml
diff --git a/__tests__/labeler.test.ts b/__tests__/labeler.test.ts
@@ -173,4 +173,27 @@ describe('run', () => {
     expect(fileLabels[1]).toEqual(expect.objectContaining({name: ':bug: bug', description: 'Damn bugs'}));
     expect(() => labeler.run()).not.toThrow();
   });
+  it('extends', async () => {
+    const input = <Inputs>{
+      githubToken: process.env.GITHUB_TOKEN || 'test',
+      yamlFile: '.res/labels.merge3.yml',
+      skipDelete: true,
+      dryRun: true,
+      exclude: []
+    };
+    nock('https://api.github.com').get('/repos/crazy-max/ghaction-github-labeler/labels').reply(200, labelsFixture());
+
+    nock('https://api.github.com')
+      .get(`/repos/crazy-max/ghaction-github-labeler/contents/${encodeURIComponent(input.yamlFile as string)}`)
+      .reply(200, configFixture(input.yamlFile as string));
+
+    nock('https://api.github.com')
+      .get(`/repos/crazy-max/ghaction-github-labeler/contents/${encodeURIComponent('.res/labels.merge1.yml')}`)
+      .reply(200, configFixture('.res/labels.merge1.yml'));
+
+    const labeler = new Labeler(input);
+    const fileLabels = await labeler.fileLabels;
+    expect(fileLabels.length).toBe(15);
+    expect(() => labeler.run()).not.toThrow();
+  });
 });
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/labeler.ts b/src/labeler.ts
@@ -211,7 +211,8 @@ export class Labeler {
         const output = [] as Label[];
         configs
           .map(config => {
-            const labels = config.labels ? config.labels : config;
+            let labels = config.labels ? config.labels : config;
+            if (Object.keys(labels).length === 0) labels = undefined;
             return {labels: labels || []};
           })
           .map(config => {