Fix linting and tests

CATcher-org · Jul 10, 2024 · bc48562 · bc48562
1 parent 2c304a4
commit bc48562
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 12 deletions.
diff --git a/src/app/shared/issue-tables/dropdownfilter.ts b/src/app/shared/issue-tables/dropdownfilter.ts
@@ -52,18 +52,18 @@ export function applyDropdownFilter(
 }
 
 function isFilteredByAssignee(filter: Filter, issue: Issue): boolean {
-
   if (issue.issueOrPr === 'Issue') {
-    return (filter.assignees.some((assignee) => issue.assignees.includes(assignee))
-          || (filter.assignees.includes("Unassigned") && issue.assignees.length === 0));
-
-  } else if (issue.issueOrPr === 'PullRequest') { 
-    return (filter.assignees.some((assignee) => issue.author === assignee)
-          || (filter.assignees.includes("Unassigned") && issue.author === null)); 
-          // note that issue.author is never == null for PRs, but is left for semantic reasons
-
-  } else { // should never occur
+    return (
+      filter.assignees.some((assignee) => issue.assignees.includes(assignee)) ||
+      (filter.assignees.includes('Unassigned') && issue.assignees.length === 0)
+    );
+  } else if (issue.issueOrPr === 'PullRequest') {
+    return (
+      filter.assignees.some((assignee) => issue.author === assignee) || (filter.assignees.includes('Unassigned') && issue.author === null)
+    );
+    // note that issue.author is never == null for PRs, but is left for semantic reasons
+  } else {
+    // should never occur
     throw new Error('Issue or PR is neither Issue nor PullRequest');
   }
-
 }
diff --git a/tests/constants/filter.constants.ts b/tests/constants/filter.constants.ts
@@ -10,7 +10,7 @@ export const DEFAULT_FILTER: Filter = {
   hiddenLabels: new Set<string>(),
   deselectedLabels: new Set<string>(),
   itemsPerPage: 20,
-  assignees: []
+  assignees: ['Unassigned']
 };
 
 export const CHANGED_FILTER: Filter = {