Update staging operation tests for the new patch comparison logic

atom · Nov 14, 2018 · caeaa47 · caeaa47
1 parent 2ce9ee1
commit caeaa47
Showing 1 changed file with 12 additions and 3 deletions.
diff --git a/test/controllers/file-patch-controller.test.js b/test/controllers/file-patch-controller.test.js
@@ -4,6 +4,7 @@ import React from 'react';
 import {shallow} from 'enzyme';
 
 import FilePatchController from '../../lib/controllers/file-patch-controller';
+import FilePatch from '../../lib/models/patch/file-patch';
 import * as reporterProxy from '../../lib/reporter-proxy';
 import {cloneRepository, buildRepository} from '../helpers';
 
@@ -151,14 +152,22 @@ describe('FilePatchController', function() {
       const wrapper = shallow(buildApp({relPath: 'a.txt', stagingStatus: 'unstaged'}));
       assert.strictEqual(await wrapper.find('FilePatchView').prop('toggleFile')(), 'staged');
 
-      wrapper.setProps({stagingStatus: 'staged'});
+      // No-op
       assert.isNull(await wrapper.find('FilePatchView').prop('toggleFile')());
 
-      const promise = wrapper.instance().patchChangePromise;
+      // Simulate an identical patch arriving too soon
       wrapper.setProps({filePatch: filePatch.clone()});
+
+      // Still a no-op
+      assert.isNull(await wrapper.find('FilePatchView').prop('toggleFile')());
+
+      // Simulate updated patch arrival
+      const promise = wrapper.instance().patchChangePromise;
+      wrapper.setProps({filePatch: FilePatch.createNull()});
       await promise;
 
-      assert.strictEqual(await wrapper.find('FilePatchView').prop('toggleFile')(), 'unstaged');
+      // Performs an operation again
+      assert.strictEqual(await wrapper.find('FilePatchView').prop('toggleFile')(), 'staged');
     });
   });