Skip to content

Commit b8c5d0b

Browse files
committed
Update DeleteQueryBuilder tests to use double quotes for table and column names in SQL queries, ensuring consistency and compatibility with case-sensitive databases.
1 parent 0885c45 commit b8c5d0b

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

__tests__/delete-builder.test.ts

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -84,8 +84,8 @@ describe("DeleteQueryBuilder", () => {
8484

8585
const result = builder.where(where).build();
8686

87-
expect(result.sql).toContain("DELETE FROM users");
88-
expect(result.sql).toContain("WHERE (users.age > $1 OR users.name = $2)");
87+
expect(result.sql).toContain(`DELETE FROM "users"`);
88+
expect(result.sql).toContain(`WHERE ("users"."age" > $1 OR "users"."name" = $2)`);
8989
expect(result.sql).toContain("RETURNING *");
9090
expect(result.values).toEqual([18, "John"]);
9191
});
@@ -113,9 +113,9 @@ describe("DeleteQueryBuilder", () => {
113113

114114
const result = builder.where(where).build();
115115

116-
expect(result.sql).toContain("DELETE FROM users");
116+
expect(result.sql).toContain(`DELETE FROM "users"`);
117117
expect(result.sql).toContain(
118-
"WHERE (users.age > $1 AND (users.name = $2 OR users.name = $3))"
118+
`WHERE ("users"."age" > $1 AND ("users"."name" = $2 OR "users"."name" = $3))`
119119
);
120120
expect(result.sql).toContain("RETURNING *");
121121
expect(result.values).toEqual([18, "John", "Jane"]);
@@ -149,9 +149,9 @@ describe("DeleteQueryBuilder", () => {
149149
.returning(["id", "name", "email"])
150150
.build();
151151

152-
expect(result.sql).toContain("DELETE FROM users");
152+
expect(result.sql).toContain(`DELETE FROM "users"`);
153153
expect(result.sql).toContain(
154-
"WHERE (users.age > $1 AND (users.name = $2 OR users.name = $3))"
154+
`WHERE ("users"."age" > $1 AND ("users"."name" = $2 OR "users"."name" = $3))`
155155
);
156156
expect(result.sql).toContain("RETURNING id, name, email");
157157
expect(result.values).toEqual([18, "John", "Jane"]);

0 commit comments

Comments
 (0)