Return a Disposable from mockWorkspace.observeTextEditors

lib/models/workdir-context.js

@@ -20,7 +20,6 @@ export default class WorkdirContext {
 
   /*
    * Available options:
-   * - `options.savedResolutionProgress`: Serialized state used to initialize the ResolutionProgress.
    * - `options.window`: Browser window global, used on Linux by the WorkspaceChangeObserver.
    * - `options.workspace`: Atom's workspace singleton, used on Linux by the WorkspaceChangeObserver.
    * - `options.promptCallback`: Callback used to collect information interactively through Atom.

lib/models/workspace-change-observer.js

@@ -22,7 +22,7 @@ export default class WorkspaceChangeObserver {
     const focusHandler = event => {
       if (this.repository) {
         this.logger.showEvents([event]);
-        this.emitter.emit('did-change');
+        this.didChange();
       }
     };
     this.window.addEventListener('focus', focusHandler);
@@ -80,11 +80,11 @@ export default class WorkspaceChangeObserver {
           });
           if (filteredEvents.length) {
             this.logger.showEvents(filteredEvents);
-            this.emitter.emit('did-change');
+            this.didChange(filteredEvents);
             const workdirOrHeadEvent = filteredEvents.filter(e => !['config', 'index'].includes(e.file || e.newFile));
             if (workdirOrHeadEvent) {
               this.logger.showWorkdirOrHeadEvents();
-              this.emitter.emit('did-change-workdir-or-head');
+              this.didChangeWorkdirOrHead();
             }
           }
         },

test/models/workdir-context-pool.test.js

@@ -1,4 +1,5 @@
 import temp from 'temp';
+import {Disposable} from 'event-kit';
 
 import {cloneRepository} from '../helpers';
 
@@ -15,7 +16,7 @@ describe('WorkdirContextPool', function() {
     };
 
     mockWorkspace = {
-      observeTextEditors: sinon.spy(),
+      observeTextEditors: sinon.stub().returns(new Disposable()),
     };
 
     pool = new WorkdirContextPool({