reinstate tests

packages/docmanager/src/savehandler.ts

@@ -104,10 +104,8 @@ class SaveHandler implements IDisposable {
    * Set the timer.
    */
   private _setTimer(): void {
-    console.log('**in settimer', this._interval)
     clearTimeout(this._autosaveTimer);
     if (!this._isActive) {
-      console.log('***not active')
       return;
     }
     this._autosaveTimer = window.setTimeout(() => {
@@ -124,16 +122,13 @@ class SaveHandler implements IDisposable {
     // Trigger the next update.
     this._setTimer();
 
-    console.log('***in _save')
     if (!context) {
-      console.log('***no context')
       return;
     }
 
     // Bail if the model is not dirty or it is read only, or the dialog
     // is already showing.
     if (!context.model.dirty || context.model.readOnly || this._inDialog) {
-      console.log('***bailing')
       return;
     }
 
@@ -156,7 +151,6 @@ class SaveHandler implements IDisposable {
    * Handle a time conflict.
    */
   private _timeConflict(modified: string): Promise<void> {
-    console.log('***in time conflict')
     let localTime = new Date(this._context.contentsModel.last_modified);
     let remoteTime = new Date(modified);
     console.warn(`Last saving peformed ${localTime} ` +
@@ -189,12 +183,9 @@ class SaveHandler implements IDisposable {
    * Perform the save, adjusting the save interval as necessary.
    */
   private _finishSave(): Promise<void> {
-    console.log('**in finish save')
     let start = new Date().getTime();
     return this._context.save().then(() => {
-      console.log('**saved in context')
       if (this.isDisposed) {
-        console.log('***bail dispose')
         return;
       }
       let duration = new Date().getTime() - start;

test/src/docmanager/savehandler.spec.ts

@@ -131,9 +131,7 @@ describe('docregistry/savehandler', () => {
         let called = 0;
         // Lower the duration multiplier.
         (handler as any)._multiplier = 1;
-        console.log('\n\n\n***continue to save')
         context.fileChanged.connect(() => {
-          console.log('***called', called);
           if (called === 0) {
             context.model.fromString('bar');
             called++;

test/src/index.ts

@@ -3,93 +3,93 @@
 
 import '@phosphor/widgets/style/index.css';
 
-// import './application/layoutrestorer.spec';
-// import './application/shell.spec';
-
-// import './apputils/clientsession.spec';
-// import './apputils/commandlinker.spec';
-// import './apputils/dialog.spec';
-// import './apputils/iframe.spec';
-// import './apputils/instancetracker.spec';
-// import './apputils/jsoneditor.spec';
-// import './apputils/sanitizer.spec';
-// import './apputils/statedb.spec';
-// import './apputils/styling.spec';
-// import './apputils/toolbar.spec';
-// import './apputils/vdom.spec';
-
-// import './cells/inputarea.spec';
-// import './cells/model.spec';
-// import './cells/widget.spec';
-
-// import './codeeditor/editor.spec';
-// import './codeeditor/widget.spec';
-
-// import './codemirror/editor.spec';
-// import './codemirror/factory.spec';
-
-// import './completer/handler.spec';
-// import './completer/model.spec';
-// import './completer/widget.spec';
-
-// import './console/foreign.spec';
-// import './console/history.spec';
-// import './console/panel.spec';
-// import './console/widget.spec';
-
-// import './coreutils/activitymonitor.spec';
-// import './coreutils/modeldb.spec';
-// import './coreutils/nbformat.spec';
-// import './coreutils/observablejson.spec';
-// import './coreutils/observablemap.spec';
-// import './coreutils/observablestring.spec';
-// import './coreutils/observablelist.spec';
-// import './coreutils/pageconfig.spec';
-// import './coreutils/path.spec';
-// import './coreutils/settingregistry.spec';
-// import './coreutils/time.spec';
-// import './coreutils/undoablelist.spec';
-// import './coreutils/url.spec';
-// import './coreutils/uuid.spec';
-// import './coreutils/markdowncodeblocks.spec';
-
-// import './csvviewer/toolbar.spec';
-// import './csvviewer/widget.spec';
-
-// import './docmanager/manager.spec';
+import './application/layoutrestorer.spec';
+import './application/shell.spec';
+
+import './apputils/clientsession.spec';
+import './apputils/commandlinker.spec';
+import './apputils/dialog.spec';
+import './apputils/iframe.spec';
+import './apputils/instancetracker.spec';
+import './apputils/jsoneditor.spec';
+import './apputils/sanitizer.spec';
+import './apputils/statedb.spec';
+import './apputils/styling.spec';
+import './apputils/toolbar.spec';
+import './apputils/vdom.spec';
+
+import './cells/inputarea.spec';
+import './cells/model.spec';
+import './cells/widget.spec';
+
+import './codeeditor/editor.spec';
+import './codeeditor/widget.spec';
+
+import './codemirror/editor.spec';
+import './codemirror/factory.spec';
+
+import './completer/handler.spec';
+import './completer/model.spec';
+import './completer/widget.spec';
+
+import './console/foreign.spec';
+import './console/history.spec';
+import './console/panel.spec';
+import './console/widget.spec';
+
+import './coreutils/activitymonitor.spec';
+import './coreutils/modeldb.spec';
+import './coreutils/nbformat.spec';
+import './coreutils/observablejson.spec';
+import './coreutils/observablemap.spec';
+import './coreutils/observablestring.spec';
+import './coreutils/observablelist.spec';
+import './coreutils/pageconfig.spec';
+import './coreutils/path.spec';
+import './coreutils/settingregistry.spec';
+import './coreutils/time.spec';
+import './coreutils/undoablelist.spec';
+import './coreutils/url.spec';
+import './coreutils/uuid.spec';
+import './coreutils/markdowncodeblocks.spec';
+
+import './csvviewer/toolbar.spec';
+import './csvviewer/widget.spec';
+
+import './docmanager/manager.spec';
 import './docmanager/savehandler.spec';
-// import './docmanager/widgetmanager.spec';
+import './docmanager/widgetmanager.spec';
 
-// import './docregistry/context.spec';
-// import './docregistry/default.spec';
-// import './docregistry/registry.spec';
+import './docregistry/context.spec';
+import './docregistry/default.spec';
+import './docregistry/registry.spec';
 
-// import './fileeditor/widget.spec';
+import './fileeditor/widget.spec';
 
-// import './filebrowser/crumbs.spec';
-// import './filebrowser/model.spec';
+import './filebrowser/crumbs.spec';
+import './filebrowser/model.spec';
 
-// import './imageviewer/widget.spec';
+import './imageviewer/widget.spec';
 
-// import './inspector/inspector.spec';
+import './inspector/inspector.spec';
 
-// import './notebook/actions.spec';
-// import './notebook/celltools.spec';
-// import './notebook/default-toolbar.spec';
-// import './notebook/model.spec';
-// import './notebook/modelfactory.spec';
-// import './notebook/panel.spec';
-// import './notebook/tracker.spec';
-// import './notebook/widget.spec';
-// import './notebook/widgetfactory.spec';
+import './notebook/actions.spec';
+import './notebook/celltools.spec';
+import './notebook/default-toolbar.spec';
+import './notebook/model.spec';
+import './notebook/modelfactory.spec';
+import './notebook/panel.spec';
+import './notebook/tracker.spec';
+import './notebook/widget.spec';
+import './notebook/widgetfactory.spec';
 
-// import './outputarea/model.spec';
-// import './outputarea/widget.spec';
+import './outputarea/model.spec';
+import './outputarea/widget.spec';
 
-// import './rendermime/latex.spec';
-// import './rendermime/factories.spec';
-// import './rendermime/mimemodel.spec';
-// import './rendermime/outputmodel.spec';
-// import './rendermime/rendermime.spec';
+import './rendermime/latex.spec';
+import './rendermime/factories.spec';
+import './rendermime/mimemodel.spec';
+import './rendermime/outputmodel.spec';
+import './rendermime/rendermime.spec';
 
-// import './terminal/terminal.spec';
+import './terminal/terminal.spec';