Remove test no longer needed for legacy runSQL API

Author: worksofliam

src/testing/manager.ts

@@ -66,36 +66,6 @@ export const ManagerSuite: TestSuite = {
       assert.strictEqual(JobManager.selectedJob, -1);
     }},
 
-    {name: `runSQL method`, test: async () => {
-      assert.strictEqual(ServerComponent.isInstalled(), true);
-
-      // Ensure we have a blank manager first
-      await JobManager.endAll();
-      assert.strictEqual(JobManager.getRunningJobs().length, 0);
-      assert.strictEqual(JobManager.selectedJob, -1);
-
-      const query = `select * from qiws.qcustcdt`;
-
-      // Run query with no jobs should still work, using the standard API
-      const rowsA = await JobManager.runSQL(query);
-      assert.notStrictEqual(rowsA.length, 0);
-
-      await JobManager.newJob();
-
-      // Check the job exists
-      assert.strictEqual(JobManager.getRunningJobs().length, 1);
-      assert.strictEqual(JobManager.selectedJob, 0);
-
-      // Run query will run the statement using the selected job
-      const rowsB = await JobManager.runSQL(query);
-      assert.notStrictEqual(rowsB.length, 0);
-      
-      // End the jobs
-      await JobManager.endAll();
-      assert.strictEqual(JobManager.getRunningJobs().length, 0);
-      assert.strictEqual(JobManager.selectedJob, -1);
-    }},
-
     {name: `Set selected by name`, test: async () => {
       assert.strictEqual(ServerComponent.isInstalled(), true);