Delete commented code

Author: julioyildo

packages/jupyterlab-lsp/src/transclusions/ipython-bigquery/extractors.spec.ts

@@ -1,10 +1,9 @@
 import { VirtualDocument } from '../../virtual/document';
 import { expect } from 'chai';
-import { foreign_code_extractors, SQL_URL_PATTERN } from './extractors';
+import { foreign_code_extractors } from './extractors';
 import {
   extract_code,
-  get_the_only_virtual,
-  wrap_in_python_lines
+  get_the_only_virtual
 } from '../../extractors/testutils';
 
 describe('Bigquery SQL extractors', () => {
@@ -30,55 +29,6 @@ describe('Bigquery SQL extractors', () => {
     document.clear();
   });
 
-    /*
-  describe('SQL url pattern', () => {
-    it('matches connection strings', () => {
-      const correct_urls = [
-        'mysql+pymysql://scott:tiger@localhost/foo',
-        'oracle://scott:[email protected]:1521/sidname',
-        'sqlite://',
-        'sqlite:///foo.db',
-        'mssql+pyodbc://username:password@host/database?driver=SQL+Server+Native+Client+11.0',
-        'impala://hserverhost:port/default?kerberos_service_name=hive&auth_mechanism=GSSAPI'
-      ];
-      const pattern = new RegExp(SQL_URL_PATTERN);
-      for (let url of correct_urls) {
-        expect(pattern.test(url)).to.equal(true);
-      }
-    });
-  });
-*/
-/*  describe('%bigquery line magic', () => {
-    it('extracts simple commands', () => {
-      let code = wrap_in_python_lines('%bigquery select * from work');
-      let { cell_code_kept, foreign_document_map } = extract(code);
-
-      // should not be removed, but left for the static analysis (using magic overrides)
-      expect(cell_code_kept).to.equal(code);
-      let document = get_the_only_virtual(foreign_document_map);
-      expect(document.language).to.equal('sql');
-      expect(document.value).to.equal('select * from work\n');
-    });*/
-/*
-    it('leaves out the connection specification', () => {
-      let code = wrap_in_python_lines(
-        '%sql postgresql://will:longliveliz@localhost/shakes'
-      );
-      let foreign_document_map = extract(code).foreign_document_map;
-      let document = get_the_only_virtual(foreign_document_map);
-      expect(document.language).to.equal('sql');
-      expect(document.value).to.equal('\n');
-    });
-
-    it('leaves out options', () => {
-      let code = wrap_in_python_lines('%bigquery -l');
-      let foreign_document_map = extract(code).foreign_document_map;
-      let document = get_the_only_virtual(foreign_document_map);
-      expect(document.language).to.equal('sql');
-      expect(document.value).to.equal('\n');
-    });
-  });
-*/
   describe('%%bigquery cell magic', () => {
     it('extracts simple commands', () => {
       let code = "%%bigquery\nselect * from character\nwhere abbrev = 'ALICE'";
@@ -91,46 +41,7 @@ describe('Bigquery SQL extractors', () => {
         "select * from character\nwhere abbrev = 'ALICE'\n"
       );
     });
-/*
-    it('leaves out the connection specification', () => {
-      let code =
-        "%%sql postgresql://will:longliveliz@localhost/shakes\nselect * from character\nwhere abbrev = 'ALICE'";
-      let { foreign_document_map } = extract(code);
 
-      let document = get_the_only_virtual(foreign_document_map);
-      expect(document.language).to.equal('sql');
-      expect(document.value).to.equal(
-        "select * from character\nwhere abbrev = 'ALICE'\n"
-      );
-    });
-
-      
-    it('leaves out the variable assignment', () => {
-      let code = '%%sql works << SELECT title, year\nFROM work';
-      let { foreign_document_map } = extract(code);
-
-      let document = get_the_only_virtual(foreign_document_map);
-      expect(document.language).to.equal('sql');
-      expect(document.value).to.equal('SELECT title, year\nFROM work\n');
-    });
-
-   it('leaves out existing connection references', () => {
-      let code = '%%sql will@shakes\nSELECT title, year\nFROM work';
-      let { foreign_document_map } = extract(code);
-
-      let document = get_the_only_virtual(foreign_document_map);
-      expect(document.language).to.equal('sql');
-      expect(document.value).to.equal('SELECT title, year\nFROM work\n');
-    });
-*/
-/*    it('leaves out persist option', () => {
-      let code = '%%bigquery --persist dataframe\nSELECT * FROM dataframe;';
-      let { foreign_document_map } = extract(code);
-      let document = get_the_only_virtual(foreign_document_map);
-      expect(document.language).to.equal('sql');
-      expect(document.value).to.equal('SELECT * FROM dataframe;\n');
-    });
-*/    
 
   });
 });