test: refactor failing test

Author: jescalada

test/processors/writePack.test.js

@@ -6,26 +6,28 @@ const { Action, Step } = require('../../src/proxy/actions');
 chai.should();
 const expect = chai.expect;
 
-describe('writePack', () => {
+describe.only('writePack', () => {
   let exec;
+  let readdirSyncStub;
   let spawnSyncStub;
   let stepLogSpy;
   let stepSetContentSpy;
   let stepSetErrorSpy;
 
   beforeEach(() => {
-    spawnSyncStub = sinon.stub().returns({
-      stdout: 'git receive-pack output',
-      stderr: '',
-      status: 0
-    });
+    spawnSyncStub = sinon.stub();
+    readdirSyncStub = sinon.stub();
+
+    readdirSyncStub.onFirstCall().returns(['old1.idx']);
+    readdirSyncStub.onSecondCall().returns(['old1.idx', 'new1.idx']);
 
     stepLogSpy = sinon.spy(Step.prototype, 'log');
     stepSetContentSpy = sinon.spy(Step.prototype, 'setContent');
     stepSetErrorSpy = sinon.spy(Step.prototype, 'setError');
 
     const writePack = proxyquire('../../src/proxy/processors/push-action/writePack', {
-      'child_process': { spawnSync: spawnSyncStub }
+      'child_process': { spawnSync: spawnSyncStub },
+      'fs': { readdirSync: readdirSyncStub },
     });
 
     exec = writePack.exec;
@@ -50,28 +52,34 @@ describe('writePack', () => {
         1234567890,
         'test/repo'
       );
-      action.proxyGitPath = '/path/to/repo';
+      action.proxyGitPath = '/path/to';
+      action.repoName = 'repo';
     });
 
     it('should execute git receive-pack with correct parameters', async () => {
+      const dummySpawnOutput = { stdout: 'git receive-pack output', stderr: '', status: 0 };
+      spawnSyncStub.returns(dummySpawnOutput);
+
       const result = await exec(req, action);
 
-      expect(spawnSyncStub.calledOnce).to.be.true;
+      expect(spawnSyncStub.callCount).to.equal(2);
       expect(spawnSyncStub.firstCall.args[0]).to.equal('git');
-      expect(spawnSyncStub.firstCall.args[1]).to.deep.equal(['receive-pack', 'repo']);
-      expect(spawnSyncStub.firstCall.args[2]).to.deep.equal({
-        cwd: '/path/to/repo',
-        input: 'pack data',
-        encoding: 'utf-8'
+      expect(spawnSyncStub.firstCall.args[1]).to.deep.equal(['config', 'receive.unpackLimit', '0']);
+      expect(spawnSyncStub.firstCall.args[2]).to.include({ cwd: '/path/to/repo' });
+
+      expect(spawnSyncStub.secondCall.args[0]).to.equal('git');
+      expect(spawnSyncStub.secondCall.args[1]).to.deep.equal(['receive-pack', 'repo']);
+      expect(spawnSyncStub.secondCall.args[2]).to.include({
+        cwd: '/path/to',
+        input: 'pack data'
       });
 
-      expect(stepLogSpy.calledWith('executing git receive-pack repo')).to.be.true;
-      expect(stepLogSpy.calledWith('git receive-pack output')).to.be.true;
-
-      expect(stepSetContentSpy.calledWith('git receive-pack output')).to.be.true;
+      expect(stepLogSpy.calledWith('new idx files: new1.idx')).to.be.true;
+      expect(stepSetContentSpy.calledWith(dummySpawnOutput)).to.be.true;
 
       expect(result.steps).to.have.lengthOf(1);
       expect(result.steps[0].error).to.be.false;
+      expect(result.newIdxFiles).to.deep.equal(['new1.idx']);
     });
 
     it('should handle errors from git receive-pack', async () => {