Merge pull request ubiquity-os-marketplace#192 from gentlementlegen/f…

…ix/priority

fix: priority default parsing

src/parser/formatting-evaluator-module.ts

@@ -175,7 +175,8 @@ export class FormattingEvaluatorModule extends BaseModule {
   }
 
   _parsePriorityLabel(labels: GitHubIssue["labels"] | undefined): number {
-    let taskPriorityEstimate = 0;
+    // Has to default to 1 in case there is no priority label
+    let taskPriorityEstimate = 1;
     if (!labels) return 1;
     for (const label of labels) {
       let priorityLabel = "";

tests/priority.test.ts

@@ -0,0 +1,31 @@
+import { ContextPlugin } from "../src/types/plugin-input";
+import { FormattingEvaluatorModule } from "../src/parser/formatting-evaluator-module";
+import { GitHubIssue } from "../src/github-types";
+import { describe, expect, it, jest } from "@jest/globals";
+
+describe("FormattingEvaluatorModule", () => {
+  const context = {
+    config: {
+      incentives: {
+        formattingEvaluator: null,
+      },
+    },
+    logger: {
+      error: jest.fn(),
+    },
+  } as unknown as ContextPlugin;
+
+  const module = new FormattingEvaluatorModule(context);
+
+  it("should default to priority 1 when no priority label is present", () => {
+    const labels: GitHubIssue["labels"] = [];
+    const priority = module["_parsePriorityLabel"](labels);
+    expect(priority).toBe(1);
+  });
+
+  it('should return priority 3 when "Priority: 3" label is present', () => {
+    const labels = ["Priority: 3"];
+    const priority = module["_parsePriorityLabel"](labels);
+    expect(priority).toBe(3);
+  });
+});