test(thrift): adapt metricViewMetadata tests to post-abstraction DBSQLSession

Rebasing onto main brought in the #378 backend-abstraction, which changed
the DBSQLSession constructor from `{ handle, context }` to `{ backend }`.
The metricViewMetadata tests still assert the right thing (the driver's
confOverlay carries the metric-view conf key, now built in
ThriftSessionBackend), so switch their construction to the
`createSessionForTest({ handle, context })` helper the rest of the suite
uses post-abstraction.

Co-authored-by: Isaac
Signed-off-by: Madhavendra Rathore <madhavendra.rathore@databricks.com>

Author: msrathore-db

tests/unit/DBSQLSession.test.ts

@@ -304,7 +304,7 @@ describe('DBSQLSession', () => {
     it('should forward the metric-view conf via confOverlay when metricViewMetadata is true', async () => {
       const context = new ClientContextStub();
       const driver = sinon.spy(context.driver);
-      const session = new DBSQLSession({ handle: sessionHandleStub, context });
+      const session = createSessionForTest({ handle: sessionHandleStub, context });
 
       await session.executeStatement('SELECT * FROM my_metric_view', { metricViewMetadata: true });
 
@@ -316,7 +316,7 @@ describe('DBSQLSession', () => {
     it('should not set the metric-view conf when metricViewMetadata is omitted', async () => {
       const context = new ClientContextStub();
       const driver = sinon.spy(context.driver);
-      const session = new DBSQLSession({ handle: sessionHandleStub, context });
+      const session = createSessionForTest({ handle: sessionHandleStub, context });
 
       await session.executeStatement('SELECT 1');
 
@@ -328,7 +328,7 @@ describe('DBSQLSession', () => {
     it('should not set the metric-view conf when metricViewMetadata is false', async () => {
       const context = new ClientContextStub();
       const driver = sinon.spy(context.driver);
-      const session = new DBSQLSession({ handle: sessionHandleStub, context });
+      const session = createSessionForTest({ handle: sessionHandleStub, context });
 
       await session.executeStatement('SELECT 1', { metricViewMetadata: false });
 
@@ -340,7 +340,7 @@ describe('DBSQLSession', () => {
     it('should coexist with queryTags in the same confOverlay', async () => {
       const context = new ClientContextStub();
       const driver = sinon.spy(context.driver);
-      const session = new DBSQLSession({ handle: sessionHandleStub, context });
+      const session = createSessionForTest({ handle: sessionHandleStub, context });
 
       await session.executeStatement('SELECT 1', {
         metricViewMetadata: true,