From c06331b12c39048278dbee795507de112ad01a0d Mon Sep 17 00:00:00 2001
From: Jacob Bolda <me@jacobbolda.com>
Date: Wed, 5 Feb 2020 22:46:40 -0600
Subject: [PATCH] only error on actual attempt at using bodyFromFile and
 reorder a bit for tests

---
 dist/index.js         | 19 +++++++++++--------
 src/create-release.js | 19 +++++++++++--------
 2 files changed, 22 insertions(+), 16 deletions(-)

diff --git a/dist/index.js b/dist/index.js
index 5679def..b3a5f25 100644
--- a/dist/index.js
+++ b/dist/index.js
@@ -7863,16 +7863,19 @@ async function run() {
     const tag = tagName.replace('refs/tags/', '');
     const releaseName = core.getInput('release_name', { required: true }).replace('refs/tags/', '');
     const body = core.getInput('body', { required: false });
-    const bodyFromFile = core.getInput('bodyFromFile', { required: false });
-    let bodyFile = null;
-    try {
-      bodyFile = fs.readFileSync(bodyFromFile, { encoding: 'utf8' });
-    } catch (error) {
-      core.setFailed(error.message);
-    }
     const draft = core.getInput('draft', { required: false }) === 'true';
     const prerelease = core.getInput('prerelease', { required: false }) === 'true';
 
+    const bodyFromFile = core.getInput('bodyFromFile', { required: false });
+    let bodyFileContent = null;
+    if (bodyFromFile !== '' && !!bodyFromFile) {
+      try {
+        bodyFileContent = fs.readFileSync(bodyFromFile, { encoding: 'utf8' });
+      } catch (error) {
+        core.setFailed(error.message);
+      }
+    }
+
     // Create a release
     // API Documentation: https://developer.github.com/v3/repos/releases/#create-a-release
     // Octokit Documentation: https://octokit.github.io/rest.js/#octokit-routes-repos-create-release
@@ -7881,7 +7884,7 @@ async function run() {
       repo,
       tag_name: tag,
       name: releaseName,
-      body: bodyFile || body,
+      body: bodyFileContent || body,
       draft,
       prerelease
     });
diff --git a/src/create-release.js b/src/create-release.js
index be1467e..dfc980b 100644
--- a/src/create-release.js
+++ b/src/create-release.js
@@ -17,16 +17,19 @@ async function run() {
     const tag = tagName.replace('refs/tags/', '');
     const releaseName = core.getInput('release_name', { required: true }).replace('refs/tags/', '');
     const body = core.getInput('body', { required: false });
-    const bodyFromFile = core.getInput('bodyFromFile', { required: false });
-    let bodyFile = null;
-    try {
-      bodyFile = fs.readFileSync(bodyFromFile, { encoding: 'utf8' });
-    } catch (error) {
-      core.setFailed(error.message);
-    }
     const draft = core.getInput('draft', { required: false }) === 'true';
     const prerelease = core.getInput('prerelease', { required: false }) === 'true';
 
+    const bodyFromFile = core.getInput('bodyFromFile', { required: false });
+    let bodyFileContent = null;
+    if (bodyFromFile !== '' && !!bodyFromFile) {
+      try {
+        bodyFileContent = fs.readFileSync(bodyFromFile, { encoding: 'utf8' });
+      } catch (error) {
+        core.setFailed(error.message);
+      }
+    }
+
     // Create a release
     // API Documentation: https://developer.github.com/v3/repos/releases/#create-a-release
     // Octokit Documentation: https://octokit.github.io/rest.js/#octokit-routes-repos-create-release
@@ -35,7 +38,7 @@ async function run() {
       repo,
       tag_name: tag,
       name: releaseName,
-      body: bodyFile || body,
+      body: bodyFileContent || body,
       draft,
       prerelease
     });