test: address CodeRabbit review feedback

- `dbt-helpers.test.ts`: use `toBe` (reference equality) for symlink cache test
- `status.test.ts`: add presence assertion before relative-path loop
- `project-scan.test.ts`: add `connection_string` masking assertion for `clickhouse+http/https` schemes

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: anandgupta42

packages/opencode/test/altimate/dbt-helpers.test.ts

@@ -354,7 +354,7 @@ describe("loadRawManifest", () => {
 
     const viaReal = loadRawManifest(realPath)
     const viaSym = loadRawManifest(symPath)
-    expect(viaSym).toEqual(viaReal)
+    expect(viaSym).toBe(viaReal)
     expect(viaSym.nodes.sym).toBe(true)
   })
 })

packages/opencode/test/file/status.test.ts

@@ -112,6 +112,7 @@ describe("File.status()", () => {
       directory: tmp.path,
       fn: async () => {
         const changed = await File.status()
+        expect(changed.length).toBeGreaterThan(0)
         for (const file of changed) {
           // All paths should be relative, not absolute
           expect(path.isAbsolute(file.path)).toBe(false)

packages/opencode/test/tool/project-scan.test.ts

@@ -557,6 +557,7 @@ describe("detectEnvVars", () => {
       expect(ch).toBeDefined()
       expect(ch!.signal).toBe("DATABASE_URL")
       expect(ch!.type).toBe("clickhouse")
+      expect(ch!.config.connection_string).toBe("***")
     }
   })