Merge pull request #1452 from imaginaris/fix/dry_run-on-ci

fix: always use `LocalOnlyGitRepo` source with `dry_run` command

Author: manicmaniac

CHANGELOG.md

@@ -11,6 +11,7 @@
 ## master
 
 <!-- Your comment below here -->
+* Always use `LocalOnlyGitRepo` source with `dry_run` command - [@imaginaris](https://github.com/imaginaris) [#1452](https://github.com/danger/danger/pull/1452)
 <!-- Your comment above here -->
 
 ## 9.4.3

lib/danger/danger_core/environment_manager.rb

@@ -7,6 +7,8 @@ class EnvironmentManager
 
     # Finds a Danger::CI class based on the ENV
     def self.local_ci_source(env)
+      return Danger::LocalOnlyGitRepo if LocalOnlyGitRepo.validates_as_ci? env
+
       CI.available_ci_sources.find { |ci| ci.validates_as_ci? env }
     end
 

spec/lib/danger/danger_core/environment_manager_spec.rb

@@ -115,6 +115,23 @@
         expect(Danger::EnvironmentManager.local_ci_source(system_env)).to eq nil
       end
     end
+
+    context "when Local Only Git Repo is valid" do
+      it "loads Local Only Git Repo" do
+        with_localonlygitrepo_setup do |system_env|
+          expect(described_class.local_ci_source(system_env)).to eq Danger::LocalOnlyGitRepo
+        end
+      end
+
+      it "loads Local Only Git Repo ignoring other valid sources" do
+        with_bitrise_setup_and_is_a_pull_request do |system_env_bitrise|
+          with_localonlygitrepo_setup do |system_env_local_only|
+            system_env = system_env_bitrise.merge(system_env_local_only)
+            expect(described_class.local_ci_source(system_env)).to eq Danger::LocalOnlyGitRepo
+          end
+        end
+      end
+    end
   end
 
   describe ".pr?" do

spec/support/ci_helper.rb

@@ -145,6 +145,14 @@ def with_localgitrepo_setup
         yield(system_env)
       end
 
+      def with_localonlygitrepo_setup
+        system_env = {
+          "DANGER_USE_LOCAL_ONLY_GIT" => "true"
+        }
+
+        yield(system_env)
+      end
+
       def with_screwdriver_setup_and_is_a_pull_request
         system_env = {
           "SCREWDRIVER" => "true",