Fix add users

Author: RSTZZZ

src/python/add_update_member.py

@@ -55,12 +55,13 @@ def process_role_data(parsed: Dict, prefix: str = "") -> Optional[Dict]:
 
     research_directions = parsed.get(f"{prefix}research-directions")
     if research_directions and research_directions != "_No response_":
-        print(research_directions)
-        role["research_directions"] = [
-        direction.strip() 
-        for direction in research_directions.split('-') 
-        if direction.strip()
-    ]
+        # Handle both string and list inputs for research directions
+        if isinstance(research_directions, str):
+            # Split by commas if it's a comma-separated string
+            directions = [d.strip() for d in research_directions.split(',')]
+        else:
+            directions = research_directions
+        role["research_directions"] = directions
 
     return role
 

tests/data/add_member/in.md

@@ -36,7 +36,7 @@ Mila, McGill
 
 ### Current_role-research-directions
 
-online-toxicity, temporal-graph-learning
+online-toxicity, poli-sci
 
 
 ### Current_role-advisor

tests/data/add_member/out.json

@@ -8,7 +8,7 @@
         "affiliation": "Mila, McGill",
         "research_directions": [
             "online-toxicity",
-            "temporal-graph-learning"
+            "poli-sci"
         ],
         "advisor": "Reihaneh Rabbany",
         "start_date": "Fall 2019"

tests/test_add_update_member.py

@@ -34,7 +34,8 @@ def tearDownClass(cls) -> None:
             if os.path.exists(path):
                 shutil.rmtree(path)
 
-    def test_add_member(self):        
+    def test_add_member(self):
+        self.maxDiff = None        
         with open("tests/data/add_member/in.md") as f:
             issue_body = f.read()
 
@@ -45,6 +46,9 @@ def test_add_member(self):
             expected = json.load(f)
 
         output = json.loads(json.dumps(out['John Doe']))
+
+        print(output)
+        print(expected)
         self.assertEqual(output, expected)
 
     def test_update_member(self):