Labels management for Test Cases

tests/test_api_client.py

@@ -1,4 +1,5 @@
 import pytest
+from unittest.mock import patch, MagicMock
 from trcli.constants import FAULT_MAPPING
 from trcli.cli import Environment
 from trcli.api.api_client import APIClient
@@ -280,3 +281,150 @@ def test_timeout_is_parsed_and_validated(
         else:
             with pytest.raises(AssertionError):
                 environment.log.assert_has_calls([mocker.call(TIMEOUT_PARSE_ERROR)])
+
+    @pytest.mark.api_client
+    @patch('requests.post')
+    def test_send_post_with_json_default(self, mock_post, api_resources_maker):
+        """Test that send_post uses JSON by default"""
+        # Mock response
+        mock_response = MagicMock()
+        mock_response.status_code = 200
+        mock_response.json.return_value = {"id": 1, "title": "Test"}
+        mock_response.content = b'{"id": 1, "title": "Test"}'
+        mock_post.return_value = mock_response
+
+        # Create API client
+        api_client = api_resources_maker()
+
+        # Call send_post without as_form_data parameter (should default to JSON)
+        result = api_client.send_post("test_endpoint", {":title": "Test Label"})
+
+        # Verify the result
+        assert result.status_code == 200
+        assert result.response_text == {"id": 1, "title": "Test"}
+
+        # Verify JSON was used
+        mock_post.assert_called_once()
+        call_args = mock_post.call_args
+
+        # Should use json parameter, not data
+        assert 'json' in call_args[1]
+        assert 'data' not in call_args[1]
+        assert call_args[1]['json'] == {":title": "Test Label"}
+
+        # Should have JSON content type header
+        headers = call_args[1]['headers']
+        assert headers.get('Content-Type') == 'application/json'
+
+    @pytest.mark.api_client
+    @patch('requests.post')
+    def test_send_post_with_form_data_true(self, mock_post, api_resources_maker):
+        """Test that send_post uses form-data when as_form_data=True"""
+        # Mock response
+        mock_response = MagicMock()
+        mock_response.status_code = 200
+        mock_response.json.return_value = {"id": 1, "title": "Test"}
+        mock_response.content = b'{"id": 1, "title": "Test"}'
+        mock_post.return_value = mock_response
+
+        # Create API client
+        api_client = api_resources_maker()
+
+        # Call send_post with as_form_data=True
+        result = api_client.send_post("test_endpoint", {":title": "Test Label"}, as_form_data=True)
+
+        # Verify the result
+        assert result.status_code == 200
+        assert result.response_text == {"id": 1, "title": "Test"}
+
+        # Verify form-data was used
+        mock_post.assert_called_once()
+        call_args = mock_post.call_args
+
+        # Should use data parameter, not json
+        assert 'data' in call_args[1]
+        assert 'json' not in call_args[1]
+        assert call_args[1]['data'] == {":title": "Test Label"}
+
+        # Should NOT have files parameter (uses application/x-www-form-urlencoded)
+        assert 'files' not in call_args[1] or call_args[1]['files'] is None
+
+        # Should NOT have JSON content type header when using form-data
+        headers = call_args[1]['headers']
+        assert headers.get('Content-Type') != 'application/json'
+
+    @pytest.mark.api_client
+    @patch('requests.post')
+    def test_send_post_with_form_data_false(self, mock_post, api_resources_maker):
+        """Test that send_post uses JSON when as_form_data=False explicitly"""
+        # Mock response
+        mock_response = MagicMock()
+        mock_response.status_code = 200
+        mock_response.json.return_value = {"id": 1, "title": "Test"}
+        mock_response.content = b'{"id": 1, "title": "Test"}'
+        mock_post.return_value = mock_response
+
+        # Create API client
+        api_client = api_resources_maker()
+
+        # Call send_post with as_form_data=False
+        result = api_client.send_post("test_endpoint", {":title": "Test Label"}, as_form_data=False)
+
+        # Verify the result
+        assert result.status_code == 200
+        assert result.response_text == {"id": 1, "title": "Test"}
+
+        # Verify JSON was used
+        mock_post.assert_called_once()
+        call_args = mock_post.call_args
+
+        # Should use json parameter, not data
+        assert 'json' in call_args[1]
+        assert 'data' not in call_args[1]
+        assert call_args[1]['json'] == {":title": "Test Label"}
+
+        # Should have JSON content type header
+        headers = call_args[1]['headers']
+        assert headers.get('Content-Type') == 'application/json'
+
+    @pytest.mark.api_client
+    @patch('requests.post')
+    def test_send_post_with_files_and_form_data(self, mock_post, api_resources_maker):
+        """Test that send_post handles files parameter with form-data"""
+        # Mock response
+        mock_response = MagicMock()
+        mock_response.status_code = 200
+        mock_response.json.return_value = {"id": 1, "title": "Test"}
+        mock_response.content = b'{"id": 1, "title": "Test"}'
+        mock_post.return_value = mock_response
+
+        # Create API client
+        api_client = api_resources_maker()
+
+        # Call send_post with files and form-data
+        files = {"file1": "/path/to/file"}
+        result = api_client.send_post(
+            "test_endpoint", 
+            {":title": "Test Label"}, 
+            files=files, 
+            as_form_data=True
+        )
+
+        # Verify the result
+        assert result.status_code == 200
+        assert result.response_text == {"id": 1, "title": "Test"}
+
+        # Verify form-data was used
+        mock_post.assert_called_once()
+        call_args = mock_post.call_args
+
+        # Should use data parameter, not json
+        assert 'data' in call_args[1]
+        assert 'json' not in call_args[1]
+        assert call_args[1]['data'] == {":title": "Test Label"}
+        # Files should be passed through as provided (not replaced with empty dict)
+        assert call_args[1]['files'] == files
+
+        # Should NOT have JSON content type header when using files
+        headers = call_args[1]['headers']
+        assert headers.get('Content-Type') != 'application/json'