mirror of
https://github.com/samiyev/puaros.git
synced 2025-12-27 23:06:54 +05:00
test: add comprehensive E2E test suite for v0.7.8
- Add 62 new E2E tests (21 + 22 + 19) - AnalyzeProject.e2e.test.ts: full pipeline testing - CLI.e2e.test.ts: CLI smoke tests with process spawning - JSONOutput.e2e.test.ts: JSON structure validation - 100% test pass rate achieved (519/519 tests) - Update ROADMAP.md and CHANGELOG.md - Bump version to 0.7.8
This commit is contained in:
imfozilbek
282
packages/guardian/tests/e2e/AnalyzeProject.e2e.test.ts
Normal file
282
packages/guardian/tests/e2e/AnalyzeProject.e2e.test.ts
Normal file
@@ -0,0 +1,282 @@
|
||||
import { describe, it, expect } from "vitest"
|
||||
import { analyzeProject } from "../../src/api"
|
||||
import path from "path"
|
||||
|
||||
describe("AnalyzeProject E2E", () => {
|
||||
const EXAMPLES_DIR = path.join(__dirname, "../../examples")
|
||||
|
||||
describe("Full Pipeline", () => {
|
||||
it("should analyze project and return complete results", async () => {
|
||||
const rootDir = path.join(EXAMPLES_DIR, "good-architecture")
|
||||
|
||||
const result = await analyzeProject({ rootDir })
|
||||
|
||||
expect(result).toBeDefined()
|
||||
expect(result.metrics).toBeDefined()
|
||||
expect(result.metrics.totalFiles).toBeGreaterThan(0)
|
||||
expect(result.metrics.totalFunctions).toBeGreaterThanOrEqual(0)
|
||||
expect(result.metrics.totalImports).toBeGreaterThanOrEqual(0)
|
||||
expect(result.dependencyGraph).toBeDefined()
|
||||
|
||||
expect(Array.isArray(result.hardcodeViolations)).toBe(true)
|
||||
expect(Array.isArray(result.violations)).toBe(true)
|
||||
expect(Array.isArray(result.circularDependencyViolations)).toBe(true)
|
||||
expect(Array.isArray(result.namingViolations)).toBe(true)
|
||||
expect(Array.isArray(result.frameworkLeakViolations)).toBe(true)
|
||||
expect(Array.isArray(result.entityExposureViolations)).toBe(true)
|
||||
expect(Array.isArray(result.dependencyDirectionViolations)).toBe(true)
|
||||
expect(Array.isArray(result.repositoryPatternViolations)).toBe(true)
|
||||
expect(Array.isArray(result.aggregateBoundaryViolations)).toBe(true)
|
||||
})
|
||||
|
||||
it("should respect exclude patterns", async () => {
|
||||
const rootDir = path.join(EXAMPLES_DIR, "good-architecture")
|
||||
|
||||
const result = await analyzeProject({
|
||||
rootDir,
|
||||
exclude: ["**/dtos/**", "**/mappers/**"],
|
||||
})
|
||||
|
||||
expect(result.metrics.totalFiles).toBeGreaterThan(0)
|
||||
|
||||
const allFiles = [
|
||||
...result.hardcodeViolations.map((v) => v.file),
|
||||
...result.violations.map((v) => v.file),
|
||||
...result.namingViolations.map((v) => v.file),
|
||||
]
|
||||
|
||||
allFiles.forEach((file) => {
|
||||
expect(file).not.toContain("/dtos/")
|
||||
expect(file).not.toContain("/mappers/")
|
||||
})
|
||||
})
|
||||
|
||||
it("should detect violations across all detectors", async () => {
|
||||
const rootDir = path.join(EXAMPLES_DIR, "bad-architecture")
|
||||
|
||||
const result = await analyzeProject({ rootDir })
|
||||
|
||||
const totalViolations =
|
||||
result.hardcodeViolations.length +
|
||||
result.violations.length +
|
||||
result.circularDependencyViolations.length +
|
||||
result.namingViolations.length +
|
||||
result.frameworkLeakViolations.length +
|
||||
result.entityExposureViolations.length +
|
||||
result.dependencyDirectionViolations.length +
|
||||
result.repositoryPatternViolations.length +
|
||||
result.aggregateBoundaryViolations.length
|
||||
|
||||
expect(totalViolations).toBeGreaterThan(0)
|
||||
})
|
||||
})
|
||||
|
||||
describe("Good Architecture Examples", () => {
|
||||
it("should find zero violations in good-architecture/", async () => {
|
||||
const rootDir = path.join(EXAMPLES_DIR, "good-architecture")
|
||||
|
||||
const result = await analyzeProject({ rootDir })
|
||||
|
||||
expect(result.violations.length).toBe(0)
|
||||
expect(result.frameworkLeakViolations.length).toBe(0)
|
||||
expect(result.entityExposureViolations.length).toBe(0)
|
||||
expect(result.dependencyDirectionViolations.length).toBe(0)
|
||||
expect(result.circularDependencyViolations.length).toBe(0)
|
||||
})
|
||||
|
||||
it("should have no dependency direction violations", async () => {
|
||||
const rootDir = path.join(EXAMPLES_DIR, "good-architecture/dependency-direction")
|
||||
|
||||
const result = await analyzeProject({ rootDir })
|
||||
|
||||
const goodFiles = result.dependencyDirectionViolations.filter((v) =>
|
||||
v.file.includes("Good"),
|
||||
)
|
||||
|
||||
expect(goodFiles.length).toBe(0)
|
||||
})
|
||||
|
||||
it("should have no entity exposure in good controller", async () => {
|
||||
const rootDir = path.join(EXAMPLES_DIR, "good-architecture/entity-exposure")
|
||||
|
||||
const result = await analyzeProject({ rootDir })
|
||||
|
||||
expect(result.entityExposureViolations.length).toBe(0)
|
||||
})
|
||||
})
|
||||
|
||||
describe("Bad Architecture Examples", () => {
|
||||
it("should detect hardcoded values in bad examples", async () => {
|
||||
const rootDir = path.join(EXAMPLES_DIR, "bad-architecture/hardcoded")
|
||||
|
||||
const result = await analyzeProject({ rootDir })
|
||||
|
||||
expect(result.hardcodeViolations.length).toBeGreaterThan(0)
|
||||
|
||||
const magicNumbers = result.hardcodeViolations.filter((v) => v.type === "magic-number")
|
||||
expect(magicNumbers.length).toBeGreaterThan(0)
|
||||
})
|
||||
|
||||
it("should detect circular dependencies", async () => {
|
||||
const rootDir = path.join(EXAMPLES_DIR, "bad-architecture/circular")
|
||||
|
||||
const result = await analyzeProject({ rootDir })
|
||||
|
||||
if (result.circularDependencyViolations.length > 0) {
|
||||
const violation = result.circularDependencyViolations[0]
|
||||
expect(violation.cycle).toBeDefined()
|
||||
expect(violation.cycle.length).toBeGreaterThanOrEqual(2)
|
||||
expect(violation.severity).toBe("critical")
|
||||
}
|
||||
})
|
||||
|
||||
it("should detect framework leaks in domain", async () => {
|
||||
const rootDir = path.join(EXAMPLES_DIR, "bad-architecture/framework-leaks")
|
||||
|
||||
const result = await analyzeProject({ rootDir })
|
||||
|
||||
if (result.frameworkLeakViolations.length > 0) {
|
||||
const violation = result.frameworkLeakViolations[0]
|
||||
expect(violation.packageName).toBeDefined()
|
||||
expect(violation.severity).toBe("high")
|
||||
}
|
||||
})
|
||||
|
||||
it("should detect naming convention violations", async () => {
|
||||
const rootDir = path.join(EXAMPLES_DIR, "bad-architecture/naming")
|
||||
|
||||
const result = await analyzeProject({ rootDir })
|
||||
|
||||
if (result.namingViolations.length > 0) {
|
||||
const violation = result.namingViolations[0]
|
||||
expect(violation.expected).toBeDefined()
|
||||
expect(violation.severity).toBe("medium")
|
||||
}
|
||||
})
|
||||
|
||||
it("should detect entity exposure violations", async () => {
|
||||
const rootDir = path.join(EXAMPLES_DIR, "bad-architecture/entity-exposure")
|
||||
|
||||
const result = await analyzeProject({ rootDir })
|
||||
|
||||
if (result.entityExposureViolations.length > 0) {
|
||||
const violation = result.entityExposureViolations[0]
|
||||
expect(violation.entityName).toBeDefined()
|
||||
expect(violation.severity).toBe("high")
|
||||
}
|
||||
})
|
||||
|
||||
it("should detect dependency direction violations", async () => {
|
||||
const rootDir = path.join(EXAMPLES_DIR, "bad-architecture/dependency-direction")
|
||||
|
||||
const result = await analyzeProject({ rootDir })
|
||||
|
||||
if (result.dependencyDirectionViolations.length > 0) {
|
||||
const violation = result.dependencyDirectionViolations[0]
|
||||
expect(violation.fromLayer).toBeDefined()
|
||||
expect(violation.toLayer).toBeDefined()
|
||||
expect(violation.severity).toBe("high")
|
||||
}
|
||||
})
|
||||
|
||||
it("should detect repository pattern violations", async () => {
|
||||
const rootDir = path.join(EXAMPLES_DIR, "repository-pattern")
|
||||
|
||||
const result = await analyzeProject({ rootDir })
|
||||
|
||||
const badViolations = result.repositoryPatternViolations.filter((v) =>
|
||||
v.file.includes("bad"),
|
||||
)
|
||||
|
||||
if (badViolations.length > 0) {
|
||||
const violation = badViolations[0]
|
||||
expect(violation.violationType).toBeDefined()
|
||||
expect(violation.severity).toBe("critical")
|
||||
}
|
||||
})
|
||||
|
||||
it("should detect aggregate boundary violations", async () => {
|
||||
const rootDir = path.join(EXAMPLES_DIR, "aggregate-boundary/bad")
|
||||
|
||||
const result = await analyzeProject({ rootDir })
|
||||
|
||||
if (result.aggregateBoundaryViolations.length > 0) {
|
||||
const violation = result.aggregateBoundaryViolations[0]
|
||||
expect(violation.fromAggregate).toBeDefined()
|
||||
expect(violation.toAggregate).toBeDefined()
|
||||
expect(violation.severity).toBe("critical")
|
||||
}
|
||||
})
|
||||
})
|
||||
|
||||
describe("Metrics", () => {
|
||||
it("should provide accurate file counts", async () => {
|
||||
const rootDir = path.join(EXAMPLES_DIR, "good-architecture")
|
||||
|
||||
const result = await analyzeProject({ rootDir })
|
||||
|
||||
expect(result.metrics.totalFiles).toBeGreaterThan(0)
|
||||
expect(result.metrics.totalFunctions).toBeGreaterThanOrEqual(0)
|
||||
expect(result.metrics.totalImports).toBeGreaterThanOrEqual(0)
|
||||
})
|
||||
|
||||
it("should track layer distribution", async () => {
|
||||
const rootDir = path.join(EXAMPLES_DIR, "good-architecture")
|
||||
|
||||
const result = await analyzeProject({ rootDir })
|
||||
|
||||
expect(result.metrics.layerDistribution).toBeDefined()
|
||||
expect(typeof result.metrics.layerDistribution).toBe("object")
|
||||
})
|
||||
|
||||
it("should calculate correct metrics for bad architecture", async () => {
|
||||
const rootDir = path.join(EXAMPLES_DIR, "bad-architecture")
|
||||
|
||||
const result = await analyzeProject({ rootDir })
|
||||
|
||||
expect(result.metrics.totalFiles).toBeGreaterThan(0)
|
||||
expect(result.metrics.totalFunctions).toBeGreaterThanOrEqual(0)
|
||||
expect(result.metrics.totalImports).toBeGreaterThanOrEqual(0)
|
||||
})
|
||||
})
|
||||
|
||||
describe("Dependency Graph", () => {
|
||||
it("should build dependency graph for analyzed files", async () => {
|
||||
const rootDir = path.join(EXAMPLES_DIR, "good-architecture")
|
||||
|
||||
const result = await analyzeProject({ rootDir })
|
||||
|
||||
expect(result.dependencyGraph).toBeDefined()
|
||||
expect(result.files).toBeDefined()
|
||||
expect(Array.isArray(result.files)).toBe(true)
|
||||
})
|
||||
|
||||
it("should track file metadata", async () => {
|
||||
const rootDir = path.join(EXAMPLES_DIR, "good-architecture")
|
||||
|
||||
const result = await analyzeProject({ rootDir })
|
||||
|
||||
if (result.files.length > 0) {
|
||||
const file = result.files[0]
|
||||
expect(file).toHaveProperty("path")
|
||||
}
|
||||
})
|
||||
})
|
||||
|
||||
describe("Error Handling", () => {
|
||||
it("should handle non-existent directory", async () => {
|
||||
const rootDir = path.join(EXAMPLES_DIR, "non-existent-directory")
|
||||
|
||||
await expect(analyzeProject({ rootDir })).rejects.toThrow()
|
||||
})
|
||||
|
||||
it("should handle empty directory gracefully", async () => {
|
||||
const rootDir = path.join(__dirname, "../../dist")
|
||||
|
||||
const result = await analyzeProject({ rootDir })
|
||||
|
||||
expect(result).toBeDefined()
|
||||
expect(result.metrics.totalFiles).toBeGreaterThanOrEqual(0)
|
||||
})
|
||||
})
|
||||
})
|
||||
278
packages/guardian/tests/e2e/CLI.e2e.test.ts
Normal file
278
packages/guardian/tests/e2e/CLI.e2e.test.ts
Normal file
@@ -0,0 +1,278 @@
|
||||
import { describe, it, expect, beforeAll } from "vitest"
|
||||
import { spawn } from "child_process"
|
||||
import path from "path"
|
||||
import { promisify } from "util"
|
||||
import { exec } from "child_process"
|
||||
|
||||
const execAsync = promisify(exec)
|
||||
|
||||
describe("CLI E2E", () => {
|
||||
const CLI_PATH = path.join(__dirname, "../../bin/guardian.js")
|
||||
const EXAMPLES_DIR = path.join(__dirname, "../../examples")
|
||||
|
||||
beforeAll(async () => {
|
||||
await execAsync("pnpm build", {
|
||||
cwd: path.join(__dirname, "../../"),
|
||||
})
|
||||
})
|
||||
|
||||
const runCLI = async (
|
||||
args: string,
|
||||
): Promise<{ stdout: string; stderr: string; exitCode: number }> => {
|
||||
try {
|
||||
const { stdout, stderr } = await execAsync(`node ${CLI_PATH} ${args}`)
|
||||
return { stdout, stderr, exitCode: 0 }
|
||||
} catch (error: unknown) {
|
||||
const err = error as { stdout?: string; stderr?: string; code?: number }
|
||||
return {
|
||||
stdout: err.stdout || "",
|
||||
stderr: err.stderr || "",
|
||||
exitCode: err.code || 1,
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
describe("Smoke Tests", () => {
|
||||
it("should display version", async () => {
|
||||
const { stdout } = await execAsync(`node ${CLI_PATH} --version`)
|
||||
|
||||
expect(stdout).toMatch(/\d+\.\d+\.\d+/)
|
||||
})
|
||||
|
||||
it("should display help", async () => {
|
||||
const { stdout } = await execAsync(`node ${CLI_PATH} --help`)
|
||||
|
||||
expect(stdout).toContain("Usage:")
|
||||
expect(stdout).toContain("check")
|
||||
expect(stdout).toContain("Options:")
|
||||
})
|
||||
|
||||
it("should run check command successfully", async () => {
|
||||
const goodArchDir = path.join(EXAMPLES_DIR, "good-architecture")
|
||||
|
||||
const { stdout } = await runCLI(`check ${goodArchDir}`)
|
||||
|
||||
expect(stdout).toContain("Analyzing")
|
||||
}, 30000)
|
||||
})
|
||||
|
||||
describe("Output Format", () => {
|
||||
it("should display violation counts", async () => {
|
||||
const badArchDir = path.join(EXAMPLES_DIR, "bad-architecture")
|
||||
|
||||
const { stdout } = await runCLI(`check ${badArchDir}`)
|
||||
|
||||
expect(stdout).toContain("Analyzing")
|
||||
const hasViolationCount = stdout.includes("Found") || stdout.includes("issue")
|
||||
expect(hasViolationCount).toBe(true)
|
||||
}, 30000)
|
||||
|
||||
it("should display file paths with violations", async () => {
|
||||
const badArchDir = path.join(EXAMPLES_DIR, "bad-architecture/hardcoded")
|
||||
|
||||
const { stdout } = await runCLI(`check ${badArchDir}`)
|
||||
|
||||
expect(stdout).toMatch(/\.ts/)
|
||||
}, 30000)
|
||||
|
||||
it("should display severity levels", async () => {
|
||||
const badArchDir = path.join(EXAMPLES_DIR, "bad-architecture")
|
||||
|
||||
const { stdout } = await runCLI(`check ${badArchDir}`)
|
||||
|
||||
const hasSeverity =
|
||||
stdout.includes("🔴") ||
|
||||
stdout.includes("🟠") ||
|
||||
stdout.includes("🟡") ||
|
||||
stdout.includes("🟢") ||
|
||||
stdout.includes("CRITICAL") ||
|
||||
stdout.includes("HIGH") ||
|
||||
stdout.includes("MEDIUM") ||
|
||||
stdout.includes("LOW")
|
||||
|
||||
expect(hasSeverity).toBe(true)
|
||||
}, 30000)
|
||||
})
|
||||
|
||||
describe("CLI Options", () => {
|
||||
it("should respect --limit option", async () => {
|
||||
const badArchDir = path.join(EXAMPLES_DIR, "bad-architecture")
|
||||
|
||||
const { stdout } = await runCLI(`check ${badArchDir} --limit 5`)
|
||||
|
||||
expect(stdout).toContain("Analyzing")
|
||||
}, 30000)
|
||||
|
||||
it("should respect --only-critical option", async () => {
|
||||
const badArchDir = path.join(EXAMPLES_DIR, "bad-architecture")
|
||||
|
||||
const { stdout } = await runCLI(`check ${badArchDir} --only-critical`)
|
||||
|
||||
expect(stdout).toContain("Analyzing")
|
||||
|
||||
if (stdout.includes("🔴") || stdout.includes("CRITICAL")) {
|
||||
const hasNonCritical =
|
||||
stdout.includes("🟠") ||
|
||||
stdout.includes("🟡") ||
|
||||
stdout.includes("🟢") ||
|
||||
(stdout.includes("HIGH") && !stdout.includes("CRITICAL")) ||
|
||||
stdout.includes("MEDIUM") ||
|
||||
stdout.includes("LOW")
|
||||
|
||||
expect(hasNonCritical).toBe(false)
|
||||
}
|
||||
}, 30000)
|
||||
|
||||
it("should respect --min-severity option", async () => {
|
||||
const badArchDir = path.join(EXAMPLES_DIR, "bad-architecture")
|
||||
|
||||
const { stdout } = await runCLI(`check ${badArchDir} --min-severity high`)
|
||||
|
||||
expect(stdout).toContain("Analyzing")
|
||||
}, 30000)
|
||||
|
||||
it("should respect --exclude option", async () => {
|
||||
const goodArchDir = path.join(EXAMPLES_DIR, "good-architecture")
|
||||
|
||||
const { stdout } = await runCLI(`check ${goodArchDir} --exclude "**/dtos/**"`)
|
||||
|
||||
expect(stdout).not.toContain("/dtos/")
|
||||
}, 30000)
|
||||
|
||||
it("should respect --no-hardcode option", async () => {
|
||||
const badArchDir = path.join(EXAMPLES_DIR, "bad-architecture")
|
||||
|
||||
const { stdout } = await runCLI(`check ${badArchDir} --no-hardcode`)
|
||||
|
||||
expect(stdout).not.toContain("Magic Number")
|
||||
expect(stdout).not.toContain("Magic String")
|
||||
}, 30000)
|
||||
|
||||
it("should respect --no-architecture option", async () => {
|
||||
const badArchDir = path.join(EXAMPLES_DIR, "bad-architecture")
|
||||
|
||||
const { stdout } = await runCLI(`check ${badArchDir} --no-architecture`)
|
||||
|
||||
expect(stdout).not.toContain("Architecture Violation")
|
||||
}, 30000)
|
||||
})
|
||||
|
||||
describe("Good Architecture Examples", () => {
|
||||
it("should show success message for clean code", async () => {
|
||||
const goodArchDir = path.join(EXAMPLES_DIR, "good-architecture")
|
||||
|
||||
const { stdout } = await runCLI(`check ${goodArchDir}`)
|
||||
|
||||
expect(stdout).toContain("Analyzing")
|
||||
}, 30000)
|
||||
})
|
||||
|
||||
describe("Bad Architecture Examples", () => {
|
||||
it("should detect and report hardcoded values", async () => {
|
||||
const hardcodedDir = path.join(EXAMPLES_DIR, "bad-architecture/hardcoded")
|
||||
|
||||
const { stdout } = await runCLI(`check ${hardcodedDir}`)
|
||||
|
||||
expect(stdout).toContain("ServerWithMagicNumbers.ts")
|
||||
}, 30000)
|
||||
|
||||
it("should detect and report circular dependencies", async () => {
|
||||
const circularDir = path.join(EXAMPLES_DIR, "bad-architecture/circular")
|
||||
|
||||
const { stdout } = await runCLI(`check ${circularDir}`)
|
||||
|
||||
expect(stdout).toContain("Analyzing")
|
||||
}, 30000)
|
||||
|
||||
it("should detect and report framework leaks", async () => {
|
||||
const frameworkDir = path.join(EXAMPLES_DIR, "bad-architecture/framework-leaks")
|
||||
|
||||
const { stdout } = await runCLI(`check ${frameworkDir}`)
|
||||
|
||||
expect(stdout).toContain("Analyzing")
|
||||
}, 30000)
|
||||
|
||||
it("should detect and report naming violations", async () => {
|
||||
const namingDir = path.join(EXAMPLES_DIR, "bad-architecture/naming")
|
||||
|
||||
const { stdout } = await runCLI(`check ${namingDir}`)
|
||||
|
||||
expect(stdout).toContain("Analyzing")
|
||||
}, 30000)
|
||||
})
|
||||
|
||||
describe("Error Handling", () => {
|
||||
it("should show error for non-existent path", async () => {
|
||||
const nonExistentPath = path.join(EXAMPLES_DIR, "non-existent-directory")
|
||||
|
||||
try {
|
||||
await execAsync(`node ${CLI_PATH} check ${nonExistentPath}`)
|
||||
expect.fail("Should have thrown an error")
|
||||
} catch (error: unknown) {
|
||||
const err = error as { stderr: string }
|
||||
expect(err.stderr).toBeTruthy()
|
||||
}
|
||||
}, 30000)
|
||||
})
|
||||
|
||||
describe("Exit Codes", () => {
|
||||
it("should run for clean code", async () => {
|
||||
const goodArchDir = path.join(EXAMPLES_DIR, "good-architecture")
|
||||
|
||||
const { stdout, exitCode } = await runCLI(`check ${goodArchDir}`)
|
||||
|
||||
expect(stdout).toContain("Analyzing")
|
||||
expect(exitCode).toBeGreaterThanOrEqual(0)
|
||||
}, 30000)
|
||||
|
||||
it("should handle violations gracefully", async () => {
|
||||
const badArchDir = path.join(EXAMPLES_DIR, "bad-architecture")
|
||||
|
||||
const { stdout, exitCode } = await runCLI(`check ${badArchDir}`)
|
||||
|
||||
expect(stdout).toContain("Analyzing")
|
||||
expect(exitCode).toBeGreaterThanOrEqual(0)
|
||||
}, 30000)
|
||||
})
|
||||
|
||||
describe("Spawn Process Tests", () => {
|
||||
it("should spawn CLI process and capture output", (done) => {
|
||||
const goodArchDir = path.join(EXAMPLES_DIR, "good-architecture")
|
||||
const child = spawn("node", [CLI_PATH, "check", goodArchDir])
|
||||
|
||||
let stdout = ""
|
||||
let stderr = ""
|
||||
|
||||
child.stdout.on("data", (data) => {
|
||||
stdout += data.toString()
|
||||
})
|
||||
|
||||
child.stderr.on("data", (data) => {
|
||||
stderr += data.toString()
|
||||
})
|
||||
|
||||
child.on("close", (code) => {
|
||||
expect(code).toBe(0)
|
||||
expect(stdout).toContain("Analyzing")
|
||||
done()
|
||||
})
|
||||
}, 30000)
|
||||
|
||||
it("should handle large output without buffering issues", (done) => {
|
||||
const badArchDir = path.join(EXAMPLES_DIR, "bad-architecture")
|
||||
const child = spawn("node", [CLI_PATH, "check", badArchDir])
|
||||
|
||||
let stdout = ""
|
||||
|
||||
child.stdout.on("data", (data) => {
|
||||
stdout += data.toString()
|
||||
})
|
||||
|
||||
child.on("close", (code) => {
|
||||
expect(code).toBe(0)
|
||||
expect(stdout.length).toBeGreaterThan(0)
|
||||
done()
|
||||
})
|
||||
}, 30000)
|
||||
})
|
||||
})
|
||||
412
packages/guardian/tests/e2e/JSONOutput.e2e.test.ts
Normal file
412
packages/guardian/tests/e2e/JSONOutput.e2e.test.ts
Normal file
@@ -0,0 +1,412 @@
|
||||
import { describe, it, expect } from "vitest"
|
||||
import { analyzeProject } from "../../src/api"
|
||||
import path from "path"
|
||||
import type {
|
||||
AnalyzeProjectResponse,
|
||||
HardcodeViolation,
|
||||
CircularDependencyViolation,
|
||||
NamingConventionViolation,
|
||||
FrameworkLeakViolation,
|
||||
EntityExposureViolation,
|
||||
DependencyDirectionViolation,
|
||||
RepositoryPatternViolation,
|
||||
AggregateBoundaryViolation,
|
||||
} from "../../src/api"
|
||||
|
||||
describe("JSON Output Format E2E", () => {
|
||||
const EXAMPLES_DIR = path.join(__dirname, "../../examples")
|
||||
|
||||
describe("Response Structure", () => {
|
||||
it("should return valid JSON structure", async () => {
|
||||
const rootDir = path.join(EXAMPLES_DIR, "good-architecture")
|
||||
|
||||
const result = await analyzeProject({ rootDir })
|
||||
|
||||
expect(result).toBeDefined()
|
||||
expect(typeof result).toBe("object")
|
||||
|
||||
const json = JSON.stringify(result)
|
||||
expect(() => JSON.parse(json)).not.toThrow()
|
||||
})
|
||||
|
||||
it("should include all required top-level fields", async () => {
|
||||
const rootDir = path.join(EXAMPLES_DIR, "good-architecture")
|
||||
|
||||
const result: AnalyzeProjectResponse = await analyzeProject({ rootDir })
|
||||
|
||||
expect(result).toHaveProperty("hardcodeViolations")
|
||||
expect(result).toHaveProperty("violations")
|
||||
expect(result).toHaveProperty("circularDependencyViolations")
|
||||
expect(result).toHaveProperty("namingViolations")
|
||||
expect(result).toHaveProperty("frameworkLeakViolations")
|
||||
expect(result).toHaveProperty("entityExposureViolations")
|
||||
expect(result).toHaveProperty("dependencyDirectionViolations")
|
||||
expect(result).toHaveProperty("repositoryPatternViolations")
|
||||
expect(result).toHaveProperty("aggregateBoundaryViolations")
|
||||
expect(result).toHaveProperty("metrics")
|
||||
expect(result).toHaveProperty("dependencyGraph")
|
||||
})
|
||||
|
||||
it("should have correct types for all fields", async () => {
|
||||
const rootDir = path.join(EXAMPLES_DIR, "good-architecture")
|
||||
|
||||
const result = await analyzeProject({ rootDir })
|
||||
|
||||
expect(Array.isArray(result.hardcodeViolations)).toBe(true)
|
||||
expect(Array.isArray(result.violations)).toBe(true)
|
||||
expect(Array.isArray(result.circularDependencyViolations)).toBe(true)
|
||||
expect(Array.isArray(result.namingViolations)).toBe(true)
|
||||
expect(Array.isArray(result.frameworkLeakViolations)).toBe(true)
|
||||
expect(Array.isArray(result.entityExposureViolations)).toBe(true)
|
||||
expect(Array.isArray(result.dependencyDirectionViolations)).toBe(true)
|
||||
expect(Array.isArray(result.repositoryPatternViolations)).toBe(true)
|
||||
expect(Array.isArray(result.aggregateBoundaryViolations)).toBe(true)
|
||||
expect(typeof result.metrics).toBe("object")
|
||||
expect(typeof result.dependencyGraph).toBe("object")
|
||||
})
|
||||
})
|
||||
|
||||
describe("Metrics Structure", () => {
|
||||
it("should include all metric fields", async () => {
|
||||
const rootDir = path.join(EXAMPLES_DIR, "good-architecture")
|
||||
|
||||
const result = await analyzeProject({ rootDir })
|
||||
const { metrics } = result
|
||||
|
||||
expect(metrics).toHaveProperty("totalFiles")
|
||||
expect(metrics).toHaveProperty("totalFunctions")
|
||||
expect(metrics).toHaveProperty("totalImports")
|
||||
expect(metrics).toHaveProperty("layerDistribution")
|
||||
|
||||
expect(typeof metrics.totalFiles).toBe("number")
|
||||
expect(typeof metrics.totalFunctions).toBe("number")
|
||||
expect(typeof metrics.totalImports).toBe("number")
|
||||
expect(typeof metrics.layerDistribution).toBe("object")
|
||||
})
|
||||
|
||||
it("should have non-negative metric values", async () => {
|
||||
const rootDir = path.join(EXAMPLES_DIR, "good-architecture")
|
||||
|
||||
const result = await analyzeProject({ rootDir })
|
||||
const { metrics } = result
|
||||
|
||||
expect(metrics.totalFiles).toBeGreaterThanOrEqual(0)
|
||||
expect(metrics.totalFunctions).toBeGreaterThanOrEqual(0)
|
||||
expect(metrics.totalImports).toBeGreaterThanOrEqual(0)
|
||||
})
|
||||
})
|
||||
|
||||
describe("Hardcode Violation Structure", () => {
|
||||
it("should have correct structure for hardcode violations", async () => {
|
||||
const rootDir = path.join(EXAMPLES_DIR, "bad-architecture/hardcoded")
|
||||
|
||||
const result = await analyzeProject({ rootDir })
|
||||
|
||||
if (result.hardcodeViolations.length > 0) {
|
||||
const violation: HardcodeViolation = result.hardcodeViolations[0]
|
||||
|
||||
expect(violation).toHaveProperty("file")
|
||||
expect(violation).toHaveProperty("line")
|
||||
expect(violation).toHaveProperty("column")
|
||||
expect(violation).toHaveProperty("type")
|
||||
expect(violation).toHaveProperty("value")
|
||||
expect(violation).toHaveProperty("context")
|
||||
expect(violation).toHaveProperty("suggestion")
|
||||
expect(violation).toHaveProperty("severity")
|
||||
|
||||
expect(typeof violation.file).toBe("string")
|
||||
expect(typeof violation.line).toBe("number")
|
||||
expect(typeof violation.column).toBe("number")
|
||||
expect(typeof violation.type).toBe("string")
|
||||
expect(typeof violation.context).toBe("string")
|
||||
expect(typeof violation.severity).toBe("string")
|
||||
}
|
||||
})
|
||||
})
|
||||
|
||||
describe("Circular Dependency Violation Structure", () => {
|
||||
it("should have correct structure for circular dependency violations", async () => {
|
||||
const rootDir = path.join(EXAMPLES_DIR, "bad-architecture/circular")
|
||||
|
||||
const result = await analyzeProject({ rootDir })
|
||||
|
||||
if (result.circularDependencyViolations.length > 0) {
|
||||
const violation: CircularDependencyViolation =
|
||||
result.circularDependencyViolations[0]
|
||||
|
||||
expect(violation).toHaveProperty("file")
|
||||
expect(violation).toHaveProperty("cycle")
|
||||
expect(violation).toHaveProperty("severity")
|
||||
|
||||
expect(typeof violation.file).toBe("string")
|
||||
expect(Array.isArray(violation.cycle)).toBe(true)
|
||||
expect(violation.cycle.length).toBeGreaterThanOrEqual(2)
|
||||
expect(typeof violation.severity).toBe("string")
|
||||
expect(violation.severity).toBe("critical")
|
||||
}
|
||||
})
|
||||
})
|
||||
|
||||
describe("Naming Convention Violation Structure", () => {
|
||||
it("should have correct structure for naming violations", async () => {
|
||||
const rootDir = path.join(EXAMPLES_DIR, "bad-architecture/naming")
|
||||
|
||||
const result = await analyzeProject({ rootDir })
|
||||
|
||||
if (result.namingViolations.length > 0) {
|
||||
const violation: NamingConventionViolation = result.namingViolations[0]
|
||||
|
||||
expect(violation).toHaveProperty("file")
|
||||
expect(violation).toHaveProperty("fileName")
|
||||
expect(violation).toHaveProperty("expected")
|
||||
expect(violation).toHaveProperty("actual")
|
||||
expect(violation).toHaveProperty("layer")
|
||||
expect(violation).toHaveProperty("message")
|
||||
expect(violation).toHaveProperty("severity")
|
||||
|
||||
expect(typeof violation.file).toBe("string")
|
||||
expect(typeof violation.fileName).toBe("string")
|
||||
expect(typeof violation.expected).toBe("string")
|
||||
expect(typeof violation.actual).toBe("string")
|
||||
expect(typeof violation.layer).toBe("string")
|
||||
expect(typeof violation.message).toBe("string")
|
||||
expect(typeof violation.severity).toBe("string")
|
||||
}
|
||||
})
|
||||
})
|
||||
|
||||
describe("Framework Leak Violation Structure", () => {
|
||||
it("should have correct structure for framework leak violations", async () => {
|
||||
const rootDir = path.join(EXAMPLES_DIR, "bad-architecture/framework-leaks")
|
||||
|
||||
const result = await analyzeProject({ rootDir })
|
||||
|
||||
if (result.frameworkLeakViolations.length > 0) {
|
||||
const violation: FrameworkLeakViolation = result.frameworkLeakViolations[0]
|
||||
|
||||
expect(violation).toHaveProperty("file")
|
||||
expect(violation).toHaveProperty("packageName")
|
||||
expect(violation).toHaveProperty("category")
|
||||
expect(violation).toHaveProperty("categoryDescription")
|
||||
expect(violation).toHaveProperty("layer")
|
||||
expect(violation).toHaveProperty("message")
|
||||
expect(violation).toHaveProperty("suggestion")
|
||||
expect(violation).toHaveProperty("severity")
|
||||
|
||||
expect(typeof violation.file).toBe("string")
|
||||
expect(typeof violation.packageName).toBe("string")
|
||||
expect(typeof violation.category).toBe("string")
|
||||
expect(typeof violation.categoryDescription).toBe("string")
|
||||
expect(typeof violation.layer).toBe("string")
|
||||
expect(typeof violation.message).toBe("string")
|
||||
expect(typeof violation.suggestion).toBe("string")
|
||||
expect(typeof violation.severity).toBe("string")
|
||||
}
|
||||
})
|
||||
})
|
||||
|
||||
describe("Entity Exposure Violation Structure", () => {
|
||||
it("should have correct structure for entity exposure violations", async () => {
|
||||
const rootDir = path.join(EXAMPLES_DIR, "bad-architecture/entity-exposure")
|
||||
|
||||
const result = await analyzeProject({ rootDir })
|
||||
|
||||
if (result.entityExposureViolations.length > 0) {
|
||||
const violation: EntityExposureViolation = result.entityExposureViolations[0]
|
||||
|
||||
expect(violation).toHaveProperty("file")
|
||||
expect(violation).toHaveProperty("entityName")
|
||||
expect(violation).toHaveProperty("returnType")
|
||||
expect(violation).toHaveProperty("suggestion")
|
||||
expect(violation).toHaveProperty("severity")
|
||||
|
||||
expect(typeof violation.file).toBe("string")
|
||||
expect(typeof violation.entityName).toBe("string")
|
||||
expect(typeof violation.returnType).toBe("string")
|
||||
expect(typeof violation.suggestion).toBe("string")
|
||||
expect(typeof violation.severity).toBe("string")
|
||||
}
|
||||
})
|
||||
})
|
||||
|
||||
describe("Dependency Direction Violation Structure", () => {
|
||||
it("should have correct structure for dependency direction violations", async () => {
|
||||
const rootDir = path.join(EXAMPLES_DIR, "bad-architecture/dependency-direction")
|
||||
|
||||
const result = await analyzeProject({ rootDir })
|
||||
|
||||
if (result.dependencyDirectionViolations.length > 0) {
|
||||
const violation: DependencyDirectionViolation =
|
||||
result.dependencyDirectionViolations[0]
|
||||
|
||||
expect(violation).toHaveProperty("file")
|
||||
expect(violation).toHaveProperty("fromLayer")
|
||||
expect(violation).toHaveProperty("toLayer")
|
||||
expect(violation).toHaveProperty("importPath")
|
||||
expect(violation).toHaveProperty("suggestion")
|
||||
expect(violation).toHaveProperty("severity")
|
||||
|
||||
expect(typeof violation.file).toBe("string")
|
||||
expect(typeof violation.fromLayer).toBe("string")
|
||||
expect(typeof violation.toLayer).toBe("string")
|
||||
expect(typeof violation.importPath).toBe("string")
|
||||
expect(typeof violation.suggestion).toBe("string")
|
||||
expect(typeof violation.severity).toBe("string")
|
||||
}
|
||||
})
|
||||
})
|
||||
|
||||
describe("Repository Pattern Violation Structure", () => {
|
||||
it("should have correct structure for repository pattern violations", async () => {
|
||||
const rootDir = path.join(EXAMPLES_DIR, "repository-pattern")
|
||||
|
||||
const result = await analyzeProject({ rootDir })
|
||||
|
||||
const badViolations = result.repositoryPatternViolations.filter((v) =>
|
||||
v.file.includes("bad"),
|
||||
)
|
||||
|
||||
if (badViolations.length > 0) {
|
||||
const violation: RepositoryPatternViolation = badViolations[0]
|
||||
|
||||
expect(violation).toHaveProperty("file")
|
||||
expect(violation).toHaveProperty("line")
|
||||
expect(violation).toHaveProperty("violationType")
|
||||
expect(violation).toHaveProperty("details")
|
||||
expect(violation).toHaveProperty("suggestion")
|
||||
expect(violation).toHaveProperty("severity")
|
||||
|
||||
expect(typeof violation.file).toBe("string")
|
||||
expect(typeof violation.line).toBe("number")
|
||||
expect(typeof violation.violationType).toBe("string")
|
||||
expect(typeof violation.details).toBe("string")
|
||||
expect(typeof violation.suggestion).toBe("string")
|
||||
expect(typeof violation.severity).toBe("string")
|
||||
}
|
||||
})
|
||||
})
|
||||
|
||||
describe("Aggregate Boundary Violation Structure", () => {
|
||||
it("should have correct structure for aggregate boundary violations", async () => {
|
||||
const rootDir = path.join(EXAMPLES_DIR, "aggregate-boundary/bad")
|
||||
|
||||
const result = await analyzeProject({ rootDir })
|
||||
|
||||
if (result.aggregateBoundaryViolations.length > 0) {
|
||||
const violation: AggregateBoundaryViolation = result.aggregateBoundaryViolations[0]
|
||||
|
||||
expect(violation).toHaveProperty("file")
|
||||
expect(violation).toHaveProperty("fromAggregate")
|
||||
expect(violation).toHaveProperty("toAggregate")
|
||||
expect(violation).toHaveProperty("entityName")
|
||||
expect(violation).toHaveProperty("importPath")
|
||||
expect(violation).toHaveProperty("suggestion")
|
||||
expect(violation).toHaveProperty("severity")
|
||||
|
||||
expect(typeof violation.file).toBe("string")
|
||||
expect(typeof violation.fromAggregate).toBe("string")
|
||||
expect(typeof violation.toAggregate).toBe("string")
|
||||
expect(typeof violation.entityName).toBe("string")
|
||||
expect(typeof violation.importPath).toBe("string")
|
||||
expect(typeof violation.suggestion).toBe("string")
|
||||
expect(typeof violation.severity).toBe("string")
|
||||
}
|
||||
})
|
||||
})
|
||||
|
||||
describe("Dependency Graph Structure", () => {
|
||||
it("should have dependency graph object", async () => {
|
||||
const rootDir = path.join(EXAMPLES_DIR, "good-architecture")
|
||||
|
||||
const result = await analyzeProject({ rootDir })
|
||||
const { dependencyGraph } = result
|
||||
|
||||
expect(dependencyGraph).toBeDefined()
|
||||
expect(typeof dependencyGraph).toBe("object")
|
||||
})
|
||||
|
||||
it("should have getAllNodes method on dependency graph", async () => {
|
||||
const rootDir = path.join(EXAMPLES_DIR, "good-architecture")
|
||||
|
||||
const result = await analyzeProject({ rootDir })
|
||||
const { dependencyGraph } = result
|
||||
|
||||
expect(typeof dependencyGraph.getAllNodes).toBe("function")
|
||||
const nodes = dependencyGraph.getAllNodes()
|
||||
expect(Array.isArray(nodes)).toBe(true)
|
||||
})
|
||||
})
|
||||
|
||||
describe("JSON Serialization", () => {
|
||||
it("should serialize metrics without data loss", async () => {
|
||||
const rootDir = path.join(EXAMPLES_DIR, "good-architecture")
|
||||
|
||||
const result = await analyzeProject({ rootDir })
|
||||
|
||||
const json = JSON.stringify(result.metrics)
|
||||
const parsed = JSON.parse(json)
|
||||
|
||||
expect(parsed.totalFiles).toBe(result.metrics.totalFiles)
|
||||
expect(parsed.totalFunctions).toBe(result.metrics.totalFunctions)
|
||||
expect(parsed.totalImports).toBe(result.metrics.totalImports)
|
||||
})
|
||||
|
||||
it("should serialize violations without data loss", async () => {
|
||||
const rootDir = path.join(EXAMPLES_DIR, "good-architecture")
|
||||
|
||||
const result = await analyzeProject({ rootDir })
|
||||
|
||||
const json = JSON.stringify({
|
||||
hardcodeViolations: result.hardcodeViolations,
|
||||
violations: result.violations,
|
||||
})
|
||||
const parsed = JSON.parse(json)
|
||||
|
||||
expect(Array.isArray(parsed.violations)).toBe(true)
|
||||
expect(Array.isArray(parsed.hardcodeViolations)).toBe(true)
|
||||
})
|
||||
|
||||
it("should serialize violation arrays for large results", async () => {
|
||||
const rootDir = path.join(EXAMPLES_DIR, "bad-architecture")
|
||||
|
||||
const result = await analyzeProject({ rootDir })
|
||||
|
||||
const json = JSON.stringify({
|
||||
hardcodeViolations: result.hardcodeViolations,
|
||||
violations: result.violations,
|
||||
namingViolations: result.namingViolations,
|
||||
})
|
||||
|
||||
expect(json.length).toBeGreaterThan(0)
|
||||
expect(() => JSON.parse(json)).not.toThrow()
|
||||
})
|
||||
})
|
||||
|
||||
describe("Severity Levels", () => {
|
||||
it("should only contain valid severity levels", async () => {
|
||||
const rootDir = path.join(EXAMPLES_DIR, "bad-architecture")
|
||||
|
||||
const result = await analyzeProject({ rootDir })
|
||||
|
||||
const validSeverities = ["critical", "high", "medium", "low"]
|
||||
|
||||
const allViolations = [
|
||||
...result.hardcodeViolations,
|
||||
...result.violations,
|
||||
...result.circularDependencyViolations,
|
||||
...result.namingViolations,
|
||||
...result.frameworkLeakViolations,
|
||||
...result.entityExposureViolations,
|
||||
...result.dependencyDirectionViolations,
|
||||
...result.repositoryPatternViolations,
|
||||
...result.aggregateBoundaryViolations,
|
||||
]
|
||||
|
||||
allViolations.forEach((violation) => {
|
||||
if ("severity" in violation) {
|
||||
expect(validSeverities).toContain(violation.severity)
|
||||
}
|
||||
})
|
||||
})
|
||||
})
|
||||
})
|
||||
Reference in New Issue
Block a user