From 89663d3fcfed34923289b12d4b2b5c509f4db321 Mon Sep 17 00:00:00 2001
From: Pierre Vanduynslager <pierre.denis.vanduynslager@gmail.com>
Date: Sun, 16 Dec 2018 15:54:21 -0500
Subject: [PATCH] fix: correctly handle skipped releases

---
 lib/definitions/plugins.js | 12 ++++++------
 test/index.test.js         |  6 +++++-
 2 files changed, 11 insertions(+), 7 deletions(-)

diff --git a/lib/definitions/plugins.js b/lib/definitions/plugins.js
index 8fc5f143..69651eb2 100644
--- a/lib/definitions/plugins.js
+++ b/lib/definitions/plugins.js
@@ -71,9 +71,9 @@ module.exports = {
     pipelineConfig: () => ({
       // Add `nextRelease` and plugin properties to published release
       transform: (release, step, {nextRelease}) => ({
-        ...nextRelease,
-        ...(release || {}),
-        ...(release === false ? {} : step),
+        ...(release === false ? {} : nextRelease),
+        ...release,
+        ...step,
       }),
     }),
   },
@@ -84,9 +84,9 @@ module.exports = {
     pipelineConfig: () => ({
       // Add `nextRelease` and plugin properties to published release
       transform: (release, step, {nextRelease}) => ({
-        ...nextRelease,
-        ...(release || {}),
-        ...(release === false ? {} : step),
+        ...(release === false ? {} : nextRelease),
+        ...release,
+        ...step,
       }),
     }),
   },
diff --git a/test/index.test.js b/test/index.test.js
index d1099d4b..0afdfe53 100644
--- a/test/index.test.js
+++ b/test/index.test.js
@@ -1165,6 +1165,7 @@ test('Accept "undefined" value returned by "generateNotes" and "false" by "publi
   const generateNotes2 = stub().resolves(notes2);
   const publish = stub().resolves(false);
   const addChannel = stub().resolves(false);
+  const success = stub().resolves();
 
   const options = {
     branches: ['master', 'next'],
@@ -1176,7 +1177,7 @@ test('Accept "undefined" value returned by "generateNotes" and "false" by "publi
     addChannel,
     prepare: stub().resolves(),
     publish,
-    success: stub().resolves(),
+    success,
     fail: stub().resolves(),
   };
 
@@ -1199,7 +1200,10 @@ test('Accept "undefined" value returned by "generateNotes" and "false" by "publi
   t.is(generateNotes2.callCount, 2);
   t.is(addChannel.callCount, 1);
   t.is(publish.callCount, 1);
+  t.is(success.callCount, 2);
   t.deepEqual(publish.args[0][1].nextRelease, {...nextRelease, notes: notes2});
+  t.deepEqual(success.args[0][1].releases, [{pluginName: '[Function: proxy]'}]);
+  t.deepEqual(success.args[1][1].releases, [{pluginName: '[Function: proxy]'}, {pluginName: '[Function: proxy]'}]);
 });
 
 test('Returns false if triggered by a PR', async t => {