Remove more unused code

Author: krassowski

packages/jupyterlab-go-to-definition/package.json

@@ -26,8 +26,7 @@
     "build": "tsc -b",
     "bundle": "npm pack .",
     "clean": "rimraf lib",
-    "lab:link": "jupyter labextension link . --no-build",
-    "test": "jlpm jest --coverage --coverageReporters=cobertura --coverageReporters=html --coverageReporters=text-summary"
+    "lab:link": "jupyter labextension link . --no-build"
   },
   "peerDependencies": {
     "@jupyterlab/application": "~2.2.0",

packages/jupyterlab-go-to-definition/src/jumpers/fileeditor.ts

@@ -5,11 +5,9 @@ import { JumpHistory } from '../history';
 import { IDocumentManager } from '@jupyterlab/docmanager';
 import { IDocumentWidget } from '@jupyterlab/docregistry';
 import { CodeEditor } from '@jupyterlab/codeeditor';
-import { PathExt } from '@jupyterlab/coreutils';
 
 export class FileEditorJumper extends CodeJumper {
   editor: FileEditor;
-  language: string;
   widget: IDocumentWidget;
 
   constructor(
@@ -21,32 +19,12 @@ export class FileEditorJumper extends CodeJumper {
     this.document_manager = document_manager;
     this.editor = editor_widget.content;
     this.history = new JumpHistory(this.editor.model.modelDB);
-    this.setLanguageFromMime(this.editor.model.mimeType);
-
-    this.editor.model.mimeTypeChanged.connect((session, mimeChanged) => {
-      this.setLanguageFromMime(mimeChanged.newValue);
-    });
   }
 
   get path() {
     return this.widget.context.path;
   }
 
-  get cwd() {
-    return PathExt.dirname(this.path);
-  }
-
-  setLanguageFromMime(mime: string) {
-    let type = mime.replace('text/x-', '');
-    switch (type) {
-      case 'rsrc':
-        this.language = 'R';
-        break;
-      default:
-        this.language = type;
-    }
-  }
-
   get editors() {
     return [this.editor.editor];
   }

packages/jupyterlab-go-to-definition/src/jumpers/jumper.ts

@@ -38,8 +38,6 @@ const system_keys = [
 ];
 
 export abstract class CodeJumper {
-  abstract language: string;
-
   document_manager: IDocumentManager;
   widget: IDocumentWidget;
 
@@ -76,7 +74,6 @@ export abstract class CodeJumper {
       .then(() => {
         this.go_to_position(
           document_widget,
-          // TODO - not anymore!
           position.contents_path.endsWith('.ipynb') ? 'notebook' : 'fileeditor',
           position.column,
           position.line,
@@ -137,8 +134,6 @@ export abstract class CodeJumper {
     );
   }
 
-  abstract get cwd(): string;
-
   protected abstract jump(position: ILocalPosition): void;
 
   global_jump_back() {

packages/jupyterlab-go-to-definition/src/jumpers/notebook.ts

@@ -1,5 +1,4 @@
 import { Notebook, NotebookPanel } from '@jupyterlab/notebook';
-import * as nbformat from '@jupyterlab/nbformat';
 import { IDocumentManager } from '@jupyterlab/docmanager';
 
 import { CodeJumper, jumpers } from './jumper';
@@ -23,22 +22,10 @@ export class NotebookJumper extends CodeJumper {
     this.document_manager = document_manager;
   }
 
-  get cwd() {
-    return this.widget.model.modelDB.basePath.split('/').slice(0, -1).join('/');
-  }
-
   get editors() {
     return this.notebook.widgets.map(cell => cell.editor);
   }
 
-  get language() {
-    let languageInfo = this.notebook.model.metadata.get(
-      'language_info'
-    ) as nbformat.ILanguageInfoMetadata;
-    // TODO: consider version of the language as well
-    return languageInfo.name;
-  }
-
   jump(position: ILocalPosition) {
     let { token, index } = position;
 
@@ -66,7 +53,7 @@ export class NotebookJumper extends CodeJumper {
     let position = this.editors[
       this.notebook.activeCellIndex
     ].getCursorPosition();
-    console.log('file path: ', this.widget.context.path);
+
     return {
       editor_index: this.notebook.activeCellIndex,
       line: position.line,