Default getAheadCount() and getBehindCount() to 0 instead of null

Author: smashwilson

lib/models/repository-states/state.js

@@ -364,11 +364,11 @@ export default class State {
   }
 
   getAheadCount(branchName) {
-    return Promise.resolve(null);
+    return Promise.resolve(0);
   }
 
   getBehindCount(branchName) {
-    return Promise.resolve(null);
+    return Promise.resolve(0);
   }
 
   getConfig(optionName, options) {

test/models/repository.test.js

@@ -73,12 +73,15 @@ describe('Repository', function() {
       }
 
       // Methods that resolve to null
-      for (const method of [
-        'getAheadCount', 'getBehindCount', 'getLastHistorySnapshots', 'getCache',
-      ]) {
+      for (const method of ['getLastHistorySnapshots', 'getCache']) {
         assert.isNull(await repository[method]());
       }
 
+      // Methods that resolve to 0
+      for (const method of ['getAheadCount', 'getBehindCount']) {
+        assert.strictEqual(await repository[method](), 0);
+      }
+
       // Methods that resolve to an empty array
       for (const method of [
         'getRecentCommits', 'getAuthors', 'getDiscardHistory',