feat: display all assignees instead of just primary assignee in markdown output

Co-authored-by: zkoppert <[email protected]>

Author: Copilot

markdown_writer.py

@@ -208,11 +208,12 @@ def write_to_markdown(
             else:
                 file.write(f"| {issue.title} | {issue.html_url} |")
             if "Assignee" in columns:
-                if issue.assignee:
-                    file.write(
-                        f" [{issue.assignee}](https://{endpoint}/"
-                        f"{issue.assignee}) |"
-                    )
+                if issue.assignees:
+                    assignee_links = [
+                        f"[{assignee}](https://{endpoint}/{assignee})"
+                        for assignee in issue.assignees
+                    ]
+                    file.write(f" {', '.join(assignee_links)} |")
                 else:
                     file.write(" None |")
             if "Author" in columns:

test_assignee_functionality.py

@@ -95,6 +95,89 @@ def test_assignee_column_position(self):
                 "Assignee column should appear before Author column",
             )
 
+    def test_multiple_assignees_rendering_logic(self):
+        """Test that multiple assignees are rendered correctly in assignee column."""
+        from classes import IssueWithMetrics
+        from io import StringIO
+        
+        # Test the assignee rendering logic directly
+        endpoint = "github.com"
+        columns = ["Title", "URL", "Assignee", "Author"]
+        
+        # Test case 1: Multiple assignees
+        issue_multiple = IssueWithMetrics(
+            title="Test Issue with Multiple Assignees",
+            html_url="https://github.com/test/repo/issues/1",
+            author="testuser",
+            assignee="alice",
+            assignees=["alice", "bob", "charlie"]
+        )
+        
+        # Simulate the new rendering logic 
+        if "Assignee" in columns:
+            if issue_multiple.assignees:
+                assignee_links = [
+                    f"[{assignee}](https://{endpoint}/{assignee})"
+                    for assignee in issue_multiple.assignees
+                ]
+                multiple_output = f" {', '.join(assignee_links)} |"
+            else:
+                multiple_output = " None |"
+        
+        expected_multiple = " [alice](https://github.com/alice), [bob](https://github.com/bob), [charlie](https://github.com/charlie) |"
+        self.assertEqual(multiple_output, expected_multiple, 
+                        "Multiple assignees should be rendered as comma-separated links")
+        
+        # Test case 2: Single assignee
+        issue_single = IssueWithMetrics(
+            title="Test Issue with Single Assignee",
+            html_url="https://github.com/test/repo/issues/2",
+            author="testuser",
+            assignee="alice",
+            assignees=["alice"]
+        )
+        
+        if "Assignee" in columns:
+            if issue_single.assignees:
+                assignee_links = [
+                    f"[{assignee}](https://{endpoint}/{assignee})"
+                    for assignee in issue_single.assignees
+                ]
+                single_output = f" {', '.join(assignee_links)} |"
+            else:
+                single_output = " None |"
+        
+        expected_single = " [alice](https://github.com/alice) |"
+        self.assertEqual(single_output, expected_single,
+                        "Single assignee should be rendered as a single link")
+        
+        # Test case 3: No assignees
+        issue_none = IssueWithMetrics(
+            title="Test Issue with No Assignees",
+            html_url="https://github.com/test/repo/issues/3",
+            author="testuser", 
+            assignee=None,
+            assignees=[]
+        )
+        
+        if "Assignee" in columns:
+            if issue_none.assignees:
+                assignee_links = [
+                    f"[{assignee}](https://{endpoint}/{assignee})"
+                    for assignee in issue_none.assignees
+                ]
+                none_output = f" {', '.join(assignee_links)} |"
+            else:
+                none_output = " None |"
+        
+        expected_none = " None |"
+        self.assertEqual(none_output, expected_none,
+                        "No assignees should be rendered as 'None'")
+        
+        print(f"✅ Multiple assignees test: {expected_multiple}")
+        print(f"✅ Single assignee test: {expected_single}")
+        print(f"✅ No assignees test: {expected_none}")
+
 
 if __name__ == "__main__":
     unittest.main()