youtrack-cli 0.2.0__tar.gz → 0.2.1__tar.gz

{youtrack_cli-0.2.0 → youtrack_cli-0.2.1}/.claude/settings.local.json

@@ -34,7 +34,9 @@
       "Bash(tree:*)",
       "Bash(rm:*)",
       "Bash(yt:*)",
-      "Bash(git branch:*)"
+      "Bash(git branch:*)",
+      "Bash(yt:*)",
+      "WebFetch(domain:pypi.org)"
     ],
     "deny": []
   }

{youtrack_cli-0.2.0 → youtrack_cli-0.2.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: youtrack-cli
-Version: 0.2.0
+Version: 0.2.1
 Summary: YouTrack CLI - Command line interface for JetBrains YouTrack issue tracking system
 Project-URL: Homepage, https://github.com/ryan-murphy/yt-cli
 Project-URL: Documentation, https://yt-cli.readthedocs.io/
@@ -243,8 +243,10 @@ uv pip install -e .
 
 ### Testing
 
+This project uses `pytest` with randomized test execution to ensure test reliability:
+
 ```bash
-# Run tests
+# Run tests (automatically randomized order)
 uv run pytest
 
 # Run tests with coverage
@@ -252,8 +254,16 @@ uv run pytest --cov=yt_cli
 
 # Run tests on multiple Python versions
 uv run tox
+
+# Run tests with specific random seed for reproducibility
+uv run pytest --randomly-seed=12345
+
+# Disable randomization if needed
+uv run pytest --randomly-dont-shuffle
 ```
 
+**Randomized Testing**: Tests run in random order by default using `pytest-randomly` to catch order-dependent bugs and improve test reliability. Each test run displays the random seed used, which can be reused to reproduce specific test failures.
+
 ### Code Quality
 
 This project uses comprehensive pre-commit hooks for code quality:

{youtrack_cli-0.2.0 → youtrack_cli-0.2.1}/README.md

@@ -195,8 +195,10 @@ uv pip install -e .
 
 ### Testing
 
+This project uses `pytest` with randomized test execution to ensure test reliability:
+
 ```bash
-# Run tests
+# Run tests (automatically randomized order)
 uv run pytest
 
 # Run tests with coverage
@@ -204,8 +206,16 @@ uv run pytest --cov=yt_cli
 
 # Run tests on multiple Python versions
 uv run tox
+
+# Run tests with specific random seed for reproducibility
+uv run pytest --randomly-seed=12345
+
+# Disable randomization if needed
+uv run pytest --randomly-dont-shuffle
 ```
 
+**Randomized Testing**: Tests run in random order by default using `pytest-randomly` to catch order-dependent bugs and improve test reliability. Each test run displays the random seed used, which can be reused to reproduce specific test failures.
+
 ### Code Quality
 
 This project uses comprehensive pre-commit hooks for code quality:

{youtrack_cli-0.2.0 → youtrack_cli-0.2.1}/docs/development.rst

@@ -225,7 +225,9 @@ Tests are organized into categories:
 Running Tests
 ~~~~~~~~~~~~~
 
-Run all tests:
+The project uses pytest with randomized test execution to improve test reliability and catch order-dependent bugs.
+
+Run all tests (automatically randomized order):
 
 .. code-block:: bash
 
@@ -244,6 +246,21 @@ Run with coverage:
 
    uv run pytest --cov=youtrack_cli --cov-report=html
 
+**Randomized Testing Options:**
+
+.. code-block:: bash
+
+   # Run with specific random seed for reproducibility
+   uv run pytest --randomly-seed=12345
+
+   # Disable randomization if needed
+   uv run pytest --randomly-dont-shuffle
+
+   # Show current random seed
+   uv run pytest --randomly-seed=last
+
+The test suite uses ``pytest-randomly`` to randomize test execution order. Each test run displays the random seed used, which can be reused to reproduce specific test failures. This helps identify and eliminate order-dependent test bugs.
+
 Multi-version Testing
 ~~~~~~~~~~~~~~~~~~~~~
 

{youtrack_cli-0.2.0 → youtrack_cli-0.2.1}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "youtrack-cli"
-version = "0.2.0"
+version = "0.2.1"
 description = "YouTrack CLI - Command line interface for JetBrains YouTrack issue tracking system"
 readme = "README.md"
 requires-python = ">=3.9, <3.14"
@@ -66,7 +66,7 @@ requires = ["hatchling"]
 build-backend = "hatchling.build"
 
 [tool.ruff]
-target-version = "py39"
+target-version = "0.2.1"
 line-length = 88
 
 [tool.ruff.lint]
@@ -118,6 +118,7 @@ dev = [
     "pytest>=8.4.1",
     "pytest-asyncio>=1.0.0",
     "pytest-cov>=6.2.1",
+    "pytest-randomly>=3.0.0",
     "ruff>=0.12.1",
     "tox>=4.27.0",
     "zizmor>=1.11.0",

youtrack_cli-0.2.1/tests/conftest.py

@@ -0,0 +1,40 @@
+"""Global test configuration and fixtures."""
+
+import os
+from typing import Optional
+from unittest.mock import patch
+
+import pytest
+
+
+@pytest.fixture(scope="function", autouse=True)
+def isolate_environment():
+    """Ensure test isolation by saving and restoring environment variables."""
+    # YouTrack environment variables that could cause test contamination
+    youtrack_env_vars = ["YOUTRACK_BASE_URL", "YOUTRACK_TOKEN", "YOUTRACK_USERNAME"]
+
+    # Store original values
+    original_env: dict[str, Optional[str]] = {}
+    for key in youtrack_env_vars:
+        original_env[key] = os.environ.get(key)
+
+    # Clear YouTrack environment variables before each test
+    for key in youtrack_env_vars:
+        if key in os.environ:
+            del os.environ[key]
+
+    yield
+
+    # Restore original environment after test
+    for key, value in original_env.items():
+        if value is not None:
+            os.environ[key] = value
+        elif key in os.environ:
+            del os.environ[key]
+
+
+@pytest.fixture(scope="function", autouse=True)
+def mock_dotenv_loading():
+    """Mock dotenv loading to prevent real config files from loading."""
+    with patch("youtrack_cli.auth.load_dotenv") as mock_load_dotenv:
+        yield mock_load_dotenv

{youtrack_cli-0.2.0 → youtrack_cli-0.2.1}/tests/test_articles.py

@@ -1,6 +1,6 @@
 """Tests for article management functionality."""
 
-from unittest.mock import AsyncMock, MagicMock, patch
+from unittest.mock import MagicMock, Mock, patch
 
 import pytest
 from click.testing import CliRunner
@@ -46,9 +46,11 @@ class TestArticleManager:
         }
 
         with patch("httpx.AsyncClient") as mock_client:
-            mock_resp = AsyncMock()
+            mock_resp = Mock()
             mock_resp.status_code = 200
-            mock_resp.json = lambda: mock_response
+            mock_resp.json.return_value = mock_response
+            mock_resp.text = '{"id": "123", "summary": "Test Article"}'
+            mock_resp.headers = {"content-type": "application/json"}
             mock_resp.raise_for_status.return_value = None
             mock_client.return_value.__aenter__.return_value.post.return_value = (
                 mock_resp  # noqa: E501
@@ -67,9 +69,9 @@ class TestArticleManager:
     async def test_create_article_failure(self, article_manager):
         """Test article creation failure."""
         with patch("httpx.AsyncClient") as mock_client:
-            mock_resp = AsyncMock()
+            mock_resp = Mock()
             mock_resp.status_code = 400
-            mock_resp.text.return_value = "Bad Request"
+            mock_resp.text = "Bad Request"
             mock_client.return_value.__aenter__.return_value.post.return_value = (
                 mock_resp  # noqa: E501
             )
@@ -99,9 +101,11 @@ class TestArticleManager:
         ]
 
         with patch("httpx.AsyncClient") as mock_client:
-            mock_resp = AsyncMock()
+            mock_resp = Mock()
             mock_resp.status_code = 200
-            mock_resp.json = lambda: mock_response
+            mock_resp.json.return_value = mock_response
+            mock_resp.text = '[{"id": "123", "summary": "Article 1"}]'
+            mock_resp.headers = {"content-type": "application/json"}
             mock_resp.raise_for_status.return_value = None
             mock_client.return_value.__aenter__.return_value.get.return_value = (
                 mock_resp  # noqa: E501
@@ -123,9 +127,11 @@ class TestArticleManager:
         }
 
         with patch("httpx.AsyncClient") as mock_client:
-            mock_resp = AsyncMock()
+            mock_resp = Mock()
             mock_resp.status_code = 200
-            mock_resp.json = lambda: mock_response
+            mock_resp.json.return_value = mock_response
+            mock_resp.text = '{"mock": "response"}'
+            mock_resp.headers = {"content-type": "application/json"}
             mock_resp.raise_for_status.return_value = None
             mock_client.return_value.__aenter__.return_value.get.return_value = (
                 mock_resp  # noqa: E501
@@ -146,9 +152,11 @@ class TestArticleManager:
         }
 
         with patch("httpx.AsyncClient") as mock_client:
-            mock_resp = AsyncMock()
+            mock_resp = Mock()
             mock_resp.status_code = 200
-            mock_resp.json = lambda: mock_response
+            mock_resp.json.return_value = mock_response
+            mock_resp.text = '{"mock": "response"}'
+            mock_resp.headers = {"content-type": "application/json"}
             mock_resp.raise_for_status.return_value = None
             mock_client.return_value.__aenter__.return_value.post.return_value = (
                 mock_resp  # noqa: E501
@@ -176,7 +184,7 @@ class TestArticleManager:
     async def test_delete_article_success(self, article_manager):
         """Test successful article deletion."""
         with patch("httpx.AsyncClient") as mock_client:
-            mock_resp = AsyncMock()
+            mock_resp = Mock()
             mock_resp.status_code = 200
             mock_resp.raise_for_status.return_value = None
             mock_client.return_value.__aenter__.return_value.delete.return_value = (
@@ -198,9 +206,11 @@ class TestArticleManager:
         }
 
         with patch("httpx.AsyncClient") as mock_client:
-            mock_resp = AsyncMock()
+            mock_resp = Mock()
             mock_resp.status_code = 200
-            mock_resp.json = lambda: mock_response
+            mock_resp.json.return_value = mock_response
+            mock_resp.text = '{"mock": "response"}'
+            mock_resp.headers = {"content-type": "application/json"}
             mock_resp.raise_for_status.return_value = None
             mock_client.return_value.__aenter__.return_value.post.return_value = (
                 mock_resp  # noqa: E501
@@ -224,9 +234,11 @@ class TestArticleManager:
         ]
 
         with patch("httpx.AsyncClient") as mock_client:
-            mock_resp = AsyncMock()
+            mock_resp = Mock()
             mock_resp.status_code = 200
-            mock_resp.json = lambda: mock_response
+            mock_resp.json.return_value = mock_response
+            mock_resp.text = '{"mock": "response"}'
+            mock_resp.headers = {"content-type": "application/json"}
             mock_resp.raise_for_status.return_value = None
             mock_client.return_value.__aenter__.return_value.get.return_value = (
                 mock_resp  # noqa: E501
@@ -250,9 +262,11 @@ class TestArticleManager:
         ]
 
         with patch("httpx.AsyncClient") as mock_client:
-            mock_resp = AsyncMock()
+            mock_resp = Mock()
             mock_resp.status_code = 200
-            mock_resp.json = lambda: mock_response
+            mock_resp.json.return_value = mock_response
+            mock_resp.text = '{"mock": "response"}'
+            mock_resp.headers = {"content-type": "application/json"}
             mock_resp.raise_for_status.return_value = None
             mock_client.return_value.__aenter__.return_value.get.return_value = (
                 mock_resp  # noqa: E501
@@ -273,9 +287,11 @@ class TestArticleManager:
         }
 
         with patch("httpx.AsyncClient") as mock_client:
-            mock_resp = AsyncMock()
+            mock_resp = Mock()
             mock_resp.status_code = 200
-            mock_resp.json = lambda: mock_response
+            mock_resp.json.return_value = mock_response
+            mock_resp.text = '{"mock": "response"}'
+            mock_resp.headers = {"content-type": "application/json"}
             mock_resp.raise_for_status.return_value = None
             mock_client.return_value.__aenter__.return_value.post.return_value = (
                 mock_resp  # noqa: E501
@@ -300,9 +316,11 @@ class TestArticleManager:
         ]
 
         with patch("httpx.AsyncClient") as mock_client:
-            mock_resp = AsyncMock()
+            mock_resp = Mock()
             mock_resp.status_code = 200
-            mock_resp.json = lambda: mock_response
+            mock_resp.json.return_value = mock_response
+            mock_resp.text = '{"mock": "response"}'
+            mock_resp.headers = {"content-type": "application/json"}
             mock_resp.raise_for_status.return_value = None
             mock_client.return_value.__aenter__.return_value.get.return_value = (
                 mock_resp  # noqa: E501

{youtrack_cli-0.2.0 → youtrack_cli-0.2.1}/tests/test_auth.py

@@ -49,6 +49,24 @@ class TestAuthManager:
         self.config_path = os.path.join(self.temp_dir, ".env")
         self.auth_manager = AuthManager(self.config_path)
 
+        # Store original environment variables to restore later
+        self.original_env = {
+            key: os.environ.get(key)
+            for key in ["YOUTRACK_BASE_URL", "YOUTRACK_TOKEN", "YOUTRACK_USERNAME"]
+        }
+
+    def teardown_method(self):
+        """Clean up test fixtures and restore environment."""
+        # Remove any YouTrack environment variables that were set during tests
+        for key in ["YOUTRACK_BASE_URL", "YOUTRACK_TOKEN", "YOUTRACK_USERNAME"]:
+            if key in os.environ:
+                del os.environ[key]
+
+        # Restore original environment variables
+        for key, value in self.original_env.items():
+            if value is not None:
+                os.environ[key] = value
+
     def test_default_config_path(self):
         """Test default config path generation."""
         manager = AuthManager()

{youtrack_cli-0.2.0 → youtrack_cli-0.2.1}/tests/test_boards.py

@@ -1,6 +1,6 @@
 """Tests for board management functionality."""
 
-from unittest.mock import AsyncMock, MagicMock, patch
+from unittest.mock import MagicMock, Mock, patch
 
 import pytest
 
@@ -48,9 +48,11 @@ class TestBoardManager:
         ]
 
         with patch("httpx.AsyncClient") as mock_client:
-            mock_resp = AsyncMock()
+            mock_resp = Mock()
             mock_resp.status_code = 200
-            mock_resp.json = lambda: mock_response
+            mock_resp.json.return_value = mock_response
+            mock_resp.text = '{"mock": "response"}'
+            mock_resp.headers = {"content-type": "application/json"}
             mock_resp.raise_for_status.return_value = None
 
             mock_client.return_value.__aenter__.return_value.get.return_value = (
@@ -75,9 +77,11 @@ class TestBoardManager:
         ]
 
         with patch("httpx.AsyncClient") as mock_client:
-            mock_resp = AsyncMock()
+            mock_resp = Mock()
             mock_resp.status_code = 200
-            mock_resp.json = lambda: mock_response
+            mock_resp.json.return_value = mock_response
+            mock_resp.text = '{"mock": "response"}'
+            mock_resp.headers = {"content-type": "application/json"}
             mock_resp.raise_for_status.return_value = None
 
             mock_client.return_value.__aenter__.return_value.get.return_value = (
@@ -113,9 +117,11 @@ class TestBoardManager:
         }
 
         with patch("httpx.AsyncClient") as mock_client:
-            mock_resp = AsyncMock()
+            mock_resp = Mock()
             mock_resp.status_code = 200
-            mock_resp.json = lambda: mock_response
+            mock_resp.json.return_value = mock_response
+            mock_resp.text = '{"mock": "response"}'
+            mock_resp.headers = {"content-type": "application/json"}
             mock_resp.raise_for_status.return_value = None
 
             mock_client.return_value.__aenter__.return_value.get.return_value = (
@@ -148,9 +154,11 @@ class TestBoardManager:
         }
 
         with patch("httpx.AsyncClient") as mock_client:
-            mock_resp = AsyncMock()
+            mock_resp = Mock()
             mock_resp.status_code = 200
-            mock_resp.json = lambda: mock_response
+            mock_resp.json.return_value = mock_response
+            mock_resp.text = '{"mock": "response"}'
+            mock_resp.headers = {"content-type": "application/json"}
             mock_resp.raise_for_status.return_value = None
 
             mock_client.return_value.__aenter__.return_value.post.return_value = (