History

Simplify tenant isolation assertions

Rhys Sullivan committed 6 months ago

Commit f7e6ca41 parent b039b6e1 file · +10 −14Browse files

Unified · Split

apps/cloud/src/services/tenant-isolation.node.test.ts+10−14
@@ -76,15 +76,13 @@ describe("tenant isolation (HTTP)", () => { · 75 unmodified lines
7676 }),
7777 );
7878
79− const result = yield* asOrg(orgB, (client) =>
80− client.openapi
81− .getSource({ params: { scopeId: ScopeId.make(orgB), namespace: namespaceA } })
82− .pipe(Effect.result),
79+ const source = yield* asOrg(orgB, (client) =>
80+ client.openapi.getSource({
81+ params: { scopeId: ScopeId.make(orgB), namespace: namespaceA },
82+ }),
8383 );
8484
85− expect(result._tag).toBe("Success");
86− if (result._tag !== "Success") return;
87− expect(result.success).toBeNull();
85+ expect(source).toBeNull();
8886 }),
8987 );
9088
@@ -121,15 +119,13 @@ describe("tenant isolation (HTTP)", () => { · 30 unmodified lines
121119 }),
122120 );
123121
124− const result = yield* asOrg(orgB, (client) =>
125− client.secrets
126− .status({ params: { scopeId: ScopeId.make(orgB), secretId: SecretId.make(secretIdA) } })
127− .pipe(Effect.result),
122+ const status = yield* asOrg(orgB, (client) =>
123+ client.secrets.status({
124+ params: { scopeId: ScopeId.make(orgB), secretId: SecretId.make(secretIdA) },
125+ }),
128126 );
129127
130− expect(result._tag).toBe("Success");
131− if (result._tag !== "Success") return;
132− expect(result.success.status).toBe("missing");
128+ expect(status.status).toBe("missing");
133129 }),
134130 );
135131