From 48830d985c7bc5430db3eb463bb81575c9dc8c01 Mon Sep 17 00:00:00 2001
From: Suzanne Aitchison <suzanne.aitchison@netlify.com>
Date: Thu, 13 Apr 2023 13:25:43 +0100
Subject: [PATCH] rename run on success env var

---
 package.json      | 2 +-
 src/index.js      | 5 +++--
 src/index.test.js | 4 ++--
 3 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/package.json b/package.json
index cb6cb007..e09fcc7e 100644
--- a/package.json
+++ b/package.json
@@ -5,7 +5,7 @@
   "main": "src/index.js",
   "scripts": {
     "local": "node -e 'import(\"./src/index.js\").then(index => index.default()).then(events => events.onPostBuild());'",
-    "local-onsuccess": "RUN_ON_SUCCESS=true node -e 'import(\"./src/index.js\").then(index => index.default()).then(events => events.onSuccess());'",
+    "local-onsuccess": "LIGHTHOUSE_RUN_ON_SUCCESS=true node -e 'import(\"./src/index.js\").then(index => index.default()).then(events => events.onSuccess());'",
     "lint": "eslint 'src/**/*.js'",
     "format": "prettier --write 'src/**/*.js'",
     "test": "node --experimental-vm-modules node_modules/jest/bin/jest.js --collect-coverage",
diff --git a/src/index.js b/src/index.js
index 8dbaf858..e1bbfa96 100644
--- a/src/index.js
+++ b/src/index.js
@@ -6,9 +6,10 @@ import getUtils from './lib/get-utils/index.js';
 dotenv.config();
 
 export default function lighthousePlugin(inputs) {
-  // Run onPostBuild by default, unless RUN_ON_SUCCESS is set to true
+  // Run onPostBuild by default, unless LIGHTHOUSE_RUN_ON_SUCCESS env var is set to true, or run_on_success is specified in plugin inputs
   const defaultEvent =
-    inputs?.run_on_success === 'true' || process.env.RUN_ON_SUCCESS === 'true'
+    inputs?.run_on_success === 'true' ||
+    process.env.LIGHTHOUSE_RUN_ON_SUCCESS === 'true'
       ? 'onSuccess'
       : 'onPostBuild';
 
diff --git a/src/index.test.js b/src/index.test.js
index e2e157d6..8162cb54 100644
--- a/src/index.test.js
+++ b/src/index.test.js
@@ -12,10 +12,10 @@ describe('lighthousePlugin plugin events', () => {
 
   describe('onSuccess', () => {
     beforeEach(() => {
-      process.env.RUN_ON_SUCCESS = 'true';
+      process.env.LIGHTHOUSE_RUN_ON_SUCCESS = 'true';
     });
     afterEach(() => {
-      delete process.env.RUN_ON_SUCCESS;
+      delete process.env.LIGHTHOUSE_RUN_ON_SUCCESS;
     });
     it('should return only the expected event function', async () => {
       const events = lighthousePlugin();