Test JDBC-compatible catalog backslash handling

Signed-off-by: Vu Anh Phung <vu.phung@databricks.com>

Author: vuanhphung

lib/utils/escapeCatalogName.ts

@@ -1,4 +1,6 @@
-/** Escape catalog underscores without double-escaping existing `\_` sequences. */
+/**
+ * Match JDBC by preserving immediate `\_` pairs, even after another backslash.
+ */
 export default function escapeCatalogName(catalogName: string | undefined): string | undefined {
   if (catalogName == null) {
     return catalogName;

tests/unit/kernel/execution.test.ts

@@ -1102,6 +1102,7 @@ describe('KernelSessionBackend', () => {
       ['main', 'main'],
       ['comparator_tests', 'comparator\\_tests'],
       ['a\\_b_c', 'a\\_b\\_c'],
+      [String.raw`my\\_cat`, String.raw`my\\_cat`],
       ['a_b%c', 'a\\_b%c'],
       ['%', '%'],
     ]) {

tests/unit/thrift-backend/ThriftCatalogEscaping.test.ts

@@ -17,6 +17,7 @@ describe('Thrift metadata catalog escaping', () => {
       ['_a__b_', '\\_a\\_\\_b\\_'],
       ['comparator\\_tests', 'comparator\\_tests'],
       ['a\\_b_c', 'a\\_b\\_c'],
+      [String.raw`my\\_cat`, String.raw`my\\_cat`],
       ['a_b%c', 'a\\_b%c'],
       ['%', '%'],
     ]) {

tests/unit/utils/escapeCatalogName.test.ts

@@ -0,0 +1,9 @@
+import { expect } from 'chai';
+import escapeCatalogName from '../../../lib/utils/escapeCatalogName';
+
+describe('escapeCatalogName JDBC compatibility', () => {
+  it('preserves two backslashes before an underscore, matching JDBC', () => {
+    const catalogName = String.raw`my\\_cat`;
+    expect(escapeCatalogName(catalogName)).to.equal(catalogName);
+  });
+});