Skip update check if DANGER_SWIFT_NO_UPDATE_CHECK is set

Sources/Runner/main.swift

@@ -20,10 +20,7 @@ private func runCommand(_ command: DangerCommand, logger: Logger) throws {
 }
 
 private func checkForUpdate(logger: Logger) {
-    // FIXME: Add another env variable to opt out this check
-    // https://github.com/danger/swift/pull/505#discussion_r799356563
-    guard ProcessInfo.processInfo.environment["DEBUG"] == nil else { return }
-    let versionChecker = VersionChecker(logger: logger)
+    let versionChecker = VersionChecker(logger: logger, env: ProcessInfo.processInfo.environment)
     versionChecker.checkForUpdate(current: DangerVersion)
 }
 

Sources/RunnerLib/VersionChecker.swift

@@ -6,18 +6,22 @@ import Version
 public struct VersionChecker {
     private let shellExecutor: ShellExecuting
     private let logger: Logger
+    private let env: [String: String]
 
     public init(
         shellExecutor: ShellExecuting = ShellExecutor(),
-        logger: Logger
+        logger: Logger,
+        env: [String: String]
     ) {
         self.shellExecutor = shellExecutor
         self.logger = logger
+        self.env = env
     }
 }
 
 public extension VersionChecker {
     func checkForUpdate(current currentVersionString: String) {
+        guard env["DANGER_SWIFT_NO_UPDATE_CHECK"] == nil, env["DEBUG"] == nil else { return }
         guard let latestVersionString = fetchLatestVersion() else { return }
         guard let latestVersion = Version(latestVersionString) else {
             logger.debug("Invalid latestVersionString: (\(latestVersionString)")

Tests/RunnerLibTests/VersionCheckerTests.swift

@@ -27,12 +27,45 @@ final class VersionCheckerTests: XCTestCase {
                 isVerbose: false,
                 isSilent: false,
                 printer: spyPrinter
-            )
+            ),
+            env: [:]
         )
         versionChecker.checkForUpdate(current: "1.0.0")
         XCTAssertTrue(spyPrinter.printedMessages.contains { $0.contains("A new release of danger-swift is available") })
     }
 
+    func testItIgnoresTheCheckIfNoUpdateEnvVariableIsSet() throws {
+        executor.result = mockResult(tagName: "1.0.1")
+
+        let versionChecker = VersionChecker(
+            shellExecutor: executor,
+            logger: Logger(
+                isVerbose: false,
+                isSilent: false,
+                printer: spyPrinter
+            ),
+            env: ["DANGER_SWIFT_NO_UPDATE_CHECK": "*"]
+        )
+        versionChecker.checkForUpdate(current: "1.0.0")
+        XCTAssertFalse(spyPrinter.printedMessages.contains { $0.contains("A new release of danger-swift is available") })
+    }
+
+    func testItIgnoresTheCheckIfDebugEnvVariableIsSet() throws {
+        executor.result = mockResult(tagName: "1.0.1")
+
+        let versionChecker = VersionChecker(
+            shellExecutor: executor,
+            logger: Logger(
+                isVerbose: false,
+                isSilent: false,
+                printer: spyPrinter
+            ),
+            env: ["DEBUG": "*"]
+        )
+        versionChecker.checkForUpdate(current: "1.0.0")
+        XCTAssertFalse(spyPrinter.printedMessages.contains { $0.contains("A new release of danger-swift is available") })
+    }
+
     func testItNotShowNotificationIfRunningIsLatest() throws {
         executor.result = mockResult(tagName: "1.0.0")
 
@@ -42,7 +75,8 @@ final class VersionCheckerTests: XCTestCase {
                 isVerbose: false,
                 isSilent: false,
                 printer: spyPrinter
-            )
+            ),
+            env: [:]
         )
         versionChecker.checkForUpdate(current: "1.0.0")
         XCTAssertFalse(spyPrinter.printedMessages.contains { $0.contains("A new release of danger-swift is available") })

Tests/RunnerLibTests/XCTestManifests.swift

@@ -129,6 +129,8 @@
         //   `swift test --generate-linuxmain`
         // to regenerate.
         static let __allTests__VersionCheckerTests = [
+            ("testItIgnoresTheCheckIfDebugEnvVariableIsSet", testItIgnoresTheCheckIfDebugEnvVariableIsSet),
+            ("testItIgnoresTheCheckIfNoUpdateEnvVariableIsSet", testItIgnoresTheCheckIfNoUpdateEnvVariableIsSet),
             ("testItNotShowNotificationIfRunningIsLatest", testItNotShowNotificationIfRunningIsLatest),
             ("testItShowsNotificationIfNewVersionIsAvailable", testItShowsNotificationIfNewVersionIsAvailable),
         ]