fix(ci): restore test files to maintain 100% coverage

Accidentally removed real test files that were mimicking the naming convention of the scratch files. Restoring them brings test coverage back to 100% and resolves the pre-commit badge hook failure in CI.

Author: SamuelMarks

.gitignore

@@ -169,5 +169,3 @@ celerybeat.pid
 /update_tests*.py
 /fix_*.py
 
-# Scratch numbered tests
-tests/*[0-9].py

tests/test_cli_extra2.py

@@ -0,0 +1,164 @@
+import json
+from openapi_client.cli import process_from_openapi
+
+
+def test_cli_output_file(tmp_path):
+    spec = {
+        "openapi": "3.0.0",
+        "info": {"title": "Test API", "version": "1.0"},
+        "paths": {},
+    }
+    spec_path = tmp_path / "openapi.json"
+    spec_path.write_text(json.dumps(spec))
+
+    out_file = tmp_path / "out" / "client.py"
+    process_from_openapi("to_sdk", str(spec_path), None, str(out_file))
+
+    assert (tmp_path / "out" / "src" / "client.py").exists()
+
+
+def test_cli_to_server_with_models(tmp_path):
+    spec = {
+        "openapi": "3.0.0",
+        "info": {"title": "Test API", "version": "1.0"},
+        "paths": {},
+        "components": {
+            "schemas": {
+                "Pet": {
+                    "type": "object",
+                    "properties": {
+                        "id": {"type": "integer"},
+                        "name": {"type": "string"},
+                    },
+                }
+            }
+        },
+    }
+    spec_path = tmp_path / "openapi.json"
+    spec_path.write_text(json.dumps(spec))
+
+    out_dir = tmp_path / "out_server"
+    process_from_openapi("to_server", str(spec_path), None, str(out_dir))
+
+    assert (out_dir / "main.py").exists()
+    assert (out_dir / "models.py").exists()
+
+
+def test_cli_to_sdk_cli_cdd(tmp_path, monkeypatch):
+    import ast
+
+    spec = {
+        "openapi": "3.0.0",
+        "info": {"title": "Test API", "version": "1.0"},
+        "paths": {
+            "/pets": {
+                "get": {
+                    "operationId": "getPets",
+                    "summary": "Get all pets",
+                    "parameters": [
+                        {
+                            "name": "limit",
+                            "in": "query",
+                            "schema_": {"type": ["integer", "null"]},
+                            "required": True,
+                        }
+                    ],
+                }
+            }
+        },
+    }
+    spec_path = tmp_path / "openapi.json"
+    spec_path.write_text(json.dumps(spec))
+
+    out_dir = tmp_path / "out_sdk_cli"
+    process_from_openapi("to_sdk_cli", str(spec_path), None, str(out_dir))
+
+    assert (out_dir / "src" / "cli_main.py").exists()
+
+    # Now make ast.unparse fail to cover the except block in cli_sdk_cdd/emit.py
+    def mock_unparse(*args, **kwargs):
+        raise Exception("Mock error")
+
+    monkeypatch.setattr(ast, "unparse", mock_unparse)
+    process_from_openapi("to_sdk_cli", str(spec_path), None, str(out_dir))
+
+
+def test_cli_to_server_sqlalchemy_ast_fail(tmp_path, monkeypatch):
+    import ast
+
+    spec = {
+        "openapi": "3.0.0",
+        "info": {"title": "Test API", "version": "1.0"},
+        "paths": {},
+        "components": {
+            "schemas": {
+                "Pet": {
+                    "type": "object",
+                    "properties": {
+                        "id": {"type": ["integer"]},
+                        "name": {"type": ["string", "null"]},
+                    },
+                }
+            }
+        },
+    }
+    spec_path = tmp_path / "openapi.json"
+    spec_path.write_text(json.dumps(spec))
+
+    out_dir = tmp_path / "out_server2"
+
+    def mock_unparse(*args, **kwargs):
+        raise Exception("Mock error")
+
+    monkeypatch.setattr(ast, "unparse", mock_unparse)
+    process_from_openapi("to_server", str(spec_path), None, str(out_dir))
+
+
+def test_fastapi_emit(tmp_path):
+    from openapi_client.fastapi.emit import emit_fastapi
+    from openapi_client.models import OpenAPI, Info, PathItem, Operation
+
+    spec = OpenAPI(
+        openapi="3.0.0",
+        info=Info(title="Test", version="1.0", description="Desc"),
+        paths={
+            "/test": PathItem(
+                get=Operation(summary="sum", description="desc"),
+                post=Operation(operationId="post_test"),
+            )
+        },
+    )
+    code = emit_fastapi(spec)
+    assert "@app.get('/test')" in code
+    assert "def get_test():" in code
+    assert "def post_test():" in code
+
+
+def test_fastapi_parse():
+    import libcst as cst
+    from openapi_client.fastapi.parse import extract_fastapi_from_ast
+    from openapi_client.models import OpenAPI
+
+    code = """
+from fastapi import FastAPI
+app = FastAPI()
+
+@app.get("/hello")
+def hello():
+    pass
+"""
+    module = cst.parse_module(code)
+    spec = OpenAPI(openapi="3.0.0", info={"title": "T", "version": "1"})
+    extract_fastapi_from_ast(module, spec)
+    assert "/hello" in spec.paths
+    assert "get" in spec.paths["/hello"]
+    assert spec.paths["/hello"]["get"]["operationId"] == "get_hello"
+
+
+def test_sqlalchemy_cdd_emit_no_components(tmp_path):
+    from openapi_client.sqlalchemy_cdd.emit import emit_sqlalchemy
+    from openapi_client.models import OpenAPI
+
+    spec = OpenAPI(openapi="3.0.0", info={"title": "Test", "version": "1.0"})
+    res = emit_sqlalchemy(spec)
+    assert res == ""

tests/test_cli_extra3.py

@@ -0,0 +1,41 @@
+import json
+from openapi_client.cli import process_from_openapi, sync_to_openapi
+
+
+def test_snapshot_roundtrip(tmp_path):
+    """Test that a snapshot is created during to_sdk and correctly parsed during from_sdk."""
+    spec_json = {
+        "openapi": "3.2.0",
+        "info": {"title": "Test", "version": "1.0"},
+        "paths": {},
+        "components": {"schemas": {}},
+        "webhooks": {
+            "myWebhook": {
+                "post": {
+                    "requestBody": {
+                        "content": {"application/json": {"schema": {"type": "object"}}}
+                    },
+                    "responses": {"200": {"description": "OK"}},
+                }
+            }
+        },
+    }
+
+    in_file = tmp_path / "in.json"
+    in_file.write_text(json.dumps(spec_json))
+
+    out_sdk_dir = tmp_path / "sdk"
+    out_sdk_dir.mkdir()
+
+    # Run to_sdk
+    process_from_openapi("to_sdk", str(in_file), None, str(out_sdk_dir))
+
+    assert (out_sdk_dir / "src" / "openapi.snapshot.json").exists()
+
+    # Run sync_to_openapi (to_openapi)
+    out_spec = tmp_path / "out.json"
+    sync_to_openapi(out_sdk_dir, out_spec)
+
+    out_spec_parsed = json.loads(out_spec.read_text())
+    assert "webhooks" in out_spec_parsed
+    assert "myWebhook" in out_spec_parsed["webhooks"]

tests/test_coverage_emit_test2.py

@@ -0,0 +1,22 @@
+def test_emit_tests_other_param_exception_real():
+    from openapi_client.tests.emit import emit_operation_test
+    from openapi_client.models import Operation, Parameter
+
+    op = Operation(
+        operationId="test_other_param_exception",
+        parameters=[Parameter(name="other_param_3", in_="query")],
+    )
+    func = emit_operation_test("post", "/test", op)
+    assert func.name.value == "test_test_other_param_exception"
+
+
+def test_emit_tests_body_param_exception_real():
+    from openapi_client.tests.emit import emit_operation_test
+    from openapi_client.models import Operation, Parameter
+
+    op = Operation(
+        operationId="test_body_param_exception_real",
+        parameters=[Parameter(name="body_param_real", in_="body")],
+    )
+    func = emit_operation_test("post", "/test", op)
+    assert func.name.value == "test_test_body_param_exception_real"