ado-asana-sync 1.23.7__tar.gz → 1.24.0__tar.gz

{ado_asana_sync-1.23.7 → ado_asana_sync-1.24.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ado-asana-sync
-Version: 1.23.7
+Version: 1.24.0
 Summary: Tool to sync work items and pull requests from Azure DevOps to Asana
 Author-email: Dan Anstis <dan@bsod.co.nz>
 License: MIT

{ado_asana_sync-1.23.7 → ado_asana_sync-1.24.0}/ado_asana_sync/database/database.py

@@ -410,6 +410,20 @@ class Database:
 
     def sync_projects_from_json(self, projects_data: List[Dict[str, str]]) -> None:
         """Sync projects from JSON data into the projects table."""
+        # Check for duplicates before DB operations to provide better error messages
+        seen = set()
+        duplicates = []
+        for project in projects_data:
+            key = (project["adoProjectName"], project["adoTeamName"])
+            if key in seen:
+                duplicates.append(f"{project['adoProjectName']} (Team: {project['adoTeamName']})")
+            seen.add(key)
+
+        if duplicates:
+            error_msg = f"Duplicate project configuration found in projects.json for: {', '.join(duplicates)}"
+            _LOGGER.error(error_msg)
+            raise ValueError(error_msg)
+
         with self.get_connection() as conn:
             # Clear existing projects
             conn.execute("DELETE FROM projects")

{ado_asana_sync-1.23.7 → ado_asana_sync-1.24.0}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "ado-asana-sync"
-version = "1.23.7"
+version = "1.24.0"
 description = "Tool to sync work items and pull requests from Azure DevOps to Asana"
 authors = [{name = "Dan Anstis", email = "dan@bsod.co.nz"}]
 license = {text = "MIT"}

{ado_asana_sync-1.23.7 → ado_asana_sync-1.24.0}/tests/database/test_database.py

@@ -280,10 +280,16 @@ class TestDatabaseMigration(unittest.TestCase):
 
         db = Database(self.db_path)
 
-        # This should raise an exception due to the unique constraint violation
-        with self.assertRaises(Exception):
+        # This should raise a ValueError due to the pre-check validation
+        with self.assertRaises(ValueError) as cm:
             db.sync_projects_from_json(projects_data)
 
+        # Verify the error message contains specific details about the duplicate
+        error_msg = str(cm.exception)
+        self.assertIn("Duplicate project configuration found", error_msg)
+        self.assertIn("TestProject", error_msg)
+        self.assertIn("Team1", error_msg)
+
         db.close()
 
     def test_migration_from_old_schema(self):