Reset internet availability check before each spec

Glyn Normington · Glyn Normington · commit 7bf915be0f14 · 2013-10-23T15:10:53.000+01:00
Some specs run with WebMocks and mock no internet connection while
other specs, notably integration tests, run and access the internet.
Depending on the order of running of specs, an incorrect internet
availability is cached in a DownloadCache class variable. Reset the
cache before each spec.

[#59391214]
diff --git a/lib/java_buildpack/util/download_cache.rb b/lib/java_buildpack/util/download_cache.rb
@@ -164,6 +164,12 @@ def self.store_internet_availability(internet_up)
       internet_up
     end
 
+    def self.clear_internet_availability
+      @@monitor.synchronize do
+        @@internet_checked = false
+      end
+    end
+
     def delete_file(filename)
       File.delete filename if File.exists? filename
     end
diff --git a/spec/spec_helper.rb b/spec/spec_helper.rb
@@ -28,6 +28,7 @@
 require 'fileutils'
 require 'java_buildpack/diagnostics/common'
 require 'java_buildpack/diagnostics/logger_factory'
+require 'java_buildpack/util/download_cache'
 
 RSpec.configure do |config|
   config.treat_symbols_as_metadata_keys_with_true_values = true
@@ -41,6 +42,8 @@
     diagnostics_directory = File.join(tmpdir, JavaBuildpack::Diagnostics::DIAGNOSTICS_DIRECTORY)
     FileUtils.rm_rf diagnostics_directory
     JavaBuildpack::Diagnostics::LoggerFactory.create_logger tmpdir
+    # Reset internet availability check before each test.
+    JavaBuildpack::Util::DownloadCache.send :clear_internet_availability
   end
   config.after(:all) do
     $stderr = STDERR
