Clean up fixtures

Author: savannahostrowski

src/test/analyzer.test.ts

@@ -12,9 +12,9 @@ const getWasmPaths = () => {
   }
 }
 
-// Fixtures are in src/test/fixtures/python
+// Fixtures are in src/test/fixtures
 const getFixturesPath = () => {
-  return path.join(__dirname, "..", "..", "src", "test", "fixtures", "python")
+  return path.join(__dirname, "..", "..", "src", "test", "fixtures")
 }
 
 suite("analyzer", () => {
@@ -130,7 +130,8 @@ import os
 
   suite("analyzeFile", () => {
     test("analyzes main.py fixture", () => {
-      const mainPyPath = path.join(fixturesPath, "main.py")
+      const standardPath = path.join(fixturesPath, "standard")
+      const mainPyPath = path.join(standardPath, "app", "main.py")
       const result = analyzeFile(mainPyPath, parser)
 
       assert.ok(result)
@@ -141,7 +142,7 @@ import os
       assert.ok(fastApiRouter)
       assert.strictEqual(fastApiRouter.variableName, "app")
 
-      // Should find include_router call
+      // Should find include_router calls
       assert.ok(result.includeRouters.length > 0)
 
       // Should find health check route
@@ -151,13 +152,8 @@ import os
     })
 
     test("analyzes users.py fixture", () => {
-      const usersPath = path.join(
-        fixturesPath,
-        "app",
-        "api",
-        "routes",
-        "users.py",
-      )
+      const standardPath = path.join(fixturesPath, "standard")
+      const usersPath = path.join(standardPath, "app", "routes", "users.py")
       const result = analyzeFile(usersPath, parser)
 
       assert.ok(result)
@@ -166,15 +162,13 @@ import os
       const apiRouter = result.routers.find((r) => r.type === "APIRouter")
       assert.ok(apiRouter)
 
-      // Should find multiple routes
-      assert.ok(result.routes.length >= 5)
+      // Should find routes (users.py has 3 routes: list, get, create)
+      assert.ok(result.routes.length >= 3)
 
-      // Check specific routes
+      // Check specific routes exist
       const methods = result.routes.map((r) => r.method)
       assert.ok(methods.includes("get"))
       assert.ok(methods.includes("post"))
-      assert.ok(methods.includes("put"))
-      assert.ok(methods.includes("delete"))
     })
 
     test("returns null for non-existent file", () => {

src/test/fixtures/flat/main.py

@@ -0,0 +1,12 @@
+from fastapi import FastAPI
+
+from routes import router
+
+app = FastAPI(title="Flat Layout")
+
+app.include_router(router)
+
+
+@app.get("/")
+def root():
+    return {"message": "Hello from flat layout"}

src/test/fixtures/flat/routes.py

@@ -0,0 +1,13 @@
+from fastapi import APIRouter
+
+router = APIRouter(prefix="/api", tags=["api"])
+
+
+@router.get("/users")
+def list_users():
+    return [{"id": 1, "name": "Alice"}]
+
+
+@router.get("/items")
+def list_items():
+    return [{"id": 1, "name": "Widget"}]

src/test/fixtures/namespace/app/__init__.py

@@ -0,0 +1,14 @@
+from fastapi import FastAPI
+
+# Import from namespace package (no __init__.py in namespace_routes)
+from .namespace_routes import items, users
+
+app = FastAPI(title="Namespace Package Layout")
+
+app.include_router(users.router)
+app.include_router(items.router)
+
+
+@app.get("/")
+def root():
+    return {"message": "Hello from namespace package layout"}

src/test/fixtures/namespace/app/main.py

@@ -0,0 +1,9 @@
+# Note: namespace_routes has NO __init__.py (namespace package)
+from fastapi import APIRouter
+
+router = APIRouter(prefix="/items", tags=["items"])
+
+
+@router.get("/")
+def list_items():
+    return [{"id": 1, "name": "Widget"}]

src/test/fixtures/namespace/app/namespace_routes/items.py

@@ -0,0 +1,14 @@
+# Note: namespace_routes has NO __init__.py (namespace package)
+from fastapi import APIRouter
+
+router = APIRouter(prefix="/users", tags=["users"])
+
+
+@router.get("/")
+def list_users():
+    return [{"id": 1, "name": "Alice"}]
+
+
+@router.get("/{user_id}")
+def get_user(user_id: int):
+    return {"id": user_id, "name": "Alice"}