Merge pull request #24 from opengisch/integrated-qgisplugci-test

Integrated qgisplugci initialization test

.github/workflows/test.yml

@@ -0,0 +1,68 @@
+name: test pytransifex
+
+concurrency: testing_environment
+
+on:
+  pull_request:
+    branches:
+      - master
+  push:
+    branches:
+      - master
+  workflow_dispatch:
+  workflow_call:
+
+
+jobs:
+  tests:
+    runs-on: ubuntu-latest
+    steps:
+      # Not using strategy.matrix to create multiple jobs
+      # as we do NOT want to test with any form of concurrency
+      # to avoid 'race conditions' against Transifex
+
+      - name: Check out repository code
+        uses: actions/checkout@v3
+
+      - uses: actions/setup-python@v4
+        with:
+          python-version: "3.10"
+          cache: "pip"
+          cache-dependency-path: "requirements/*.txt"
+
+      - name: Install project requirements
+        run: |
+          python -m pip install -U -r requirements/base.txt
+          python -m pip install -U -r requirements/dev.txt
+
+      - name: Test API
+        env:
+          organization: ${{ secrets.ORGANIZATION }}
+          tx_token: ${{ secrets.TX_TOKEN }}
+        run: |
+          TX_TOKEN=$tx_token ORGANIZATION=$organization \
+            python -m unittest ./tests/test_api.py
+
+      - name: Test CLI
+        env:
+          organization: ${{ secrets.ORGANIZATION }}
+          tx_token: ${{ secrets.TX_TOKEN }}
+        run: |
+          TX_TOKEN=$tx_token ORGANIZATION=$organization \
+            python -m unittest ./tests/test_cli.py
+
+      - name: Test with a public project
+        env:
+          organization: ${{ secrets.ORGANIZATION }}
+          tx_token: ${{ secrets.TX_TOKEN }}
+        run: |
+          TX_TOKEN=$tx_token ORGANIZATION=$organization \
+            python -m unittest ./tests/test_public_project.py
+
+      - name: Test with Qgisplugin's 'test_translation'
+        env:
+          organization: ${{ secrets.ORGANIZATION }}
+          tx_token: ${{ secrets.TX_TOKEN }}
+        run: |
+          TX_TOKEN=$tx_token ORGANIZATION=$organization \
+            python -m unittest ./tests/test_translation.py

pytransifex/api.py

@@ -53,11 +53,14 @@ def create_project(
         *,
         project_slug: str,
         project_name: str | None = None,
-        source_language_code: str = "en_GB",
+        source_language_code: str = "en",
         private: bool = False,
         **kwargs,
     ):
         """Create a project."""
+        logger.info(
+            f"Trying to create project from these arguments: project_slug = {project_slug}, "
+        )
         source_language = tx_api.Language.get(code=source_language_code)
         project_name = project_name or project_slug
 
@@ -82,12 +85,19 @@ def delete_project(self, project_slug: str):
     def get_project(self, project_slug: str) -> None | Resource:
         """Fetches the project matching the given slug"""
         if self.projects:
+            logger.info(
+                f"Attempting to get 'o:{self.organization_name}:p:{project_slug}'"
+            )
             try:
                 res = self.projects.get(slug=project_slug)
                 logger.info("Got the project!")
                 return res
             except DoesNotExist:
                 return None
+            """
+            except MultipleObjectsReturned:
+                pass
+            """
 
     @ensure_login
     def list_resources(self, project_slug: str) -> list[Any]:
@@ -247,7 +257,9 @@ def create_language(
     ):
         """Create a new language resource in the remote Transifex repository"""
         if project := self.get_project(project_slug=project_slug):
-            project.add("languages", [tx_api.Language.get(code=language_code)])
+            if language := tx_api.Language.get(code=language_code):
+                logger.debug(f"Adding {language.code} to {project_slug}")
+                project.add("languages", [language])
 
             if coordinators:
                 project.add("coordinators", coordinators)
@@ -260,12 +272,9 @@ def create_language(
     def project_exists(self, project_slug: str) -> bool:
         """Check if the project exists in the remote Transifex repository"""
         try:
-            if not self.projects:
-                return False
-            elif self.get_project(project_slug=project_slug):
+            if self.get_project(project_slug=project_slug):
                 return True
-            else:
-                return False
+            return False
         except DoesNotExist:
             return False
 

requirements/dev.txt

@@ -1 +1,2 @@
 nose2
+PyYAML