Fix for job added when already on pile

Author: k1o0

lib.js

@@ -413,7 +413,7 @@ function getBadgeData(data) {
       report['color'] = 'orange';
       // Check test isn't already on the pile
       let onPile = false;
-      for (let job of queue.pile) { if (job.id === id) { onPile = true; break; } }
+      for (let job of queue.pile) { if (job.data.sha === id) { onPile = true; break; } }
       if (!onPile) { // Add test to queue
          data['skipPost'] = true
          queue.add(data);

test/lib.test.js

@@ -318,6 +318,11 @@ describe("getBadgeData function", () => {
       data = lib.getBadgeData(input);
       expect(data).to.deep.equal(expected);
       sandbox.assert.calledOnce(queue.add);
+
+      // Shouldn't add as job already queued
+      data = lib.getBadgeData(input);
+      expect(data).to.deep.equal(expected);
+      sandbox.assert.calledOnce(queue.add);
    });
 
    it('Check force flag', function () {