Use a slightly cleaner approach to stub ENV

Signed-off-by: Rémy Coutable <remy@rymai.me>

spec/support/stub_env.rb

@@ -1,15 +1,33 @@
+# Inspired by https://github.com/ljkbennett/stub_env/blob/master/lib/stub_env/helpers.rb
 module StubENV
-  def stub_env(key, value)
+  def stub_env(key_or_hash, value = nil)
-    allow(ENV).to receive(:[]).and_call_original unless @env_already_stubbed
+    init_stub unless env_stubbed?
-    @env_already_stubbed ||= true
+    if key_or_hash.is_a? Hash
-    allow(ENV).to receive(:[]).with(key).and_return(value)
+      key_or_hash.each { |k, v| add_stubbed_value(k, v) }
+    else
+      add_stubbed_value key_or_hash, value
+    end
   end
-end
 
-# It's possible that the state of the class variables are not reset across
+  private
-# test runs.
+
-RSpec.configure do |config|
+  STUBBED_KEY = '__STUBBED__'.freeze
-  config.after(:each) do
+
-    @env_already_stubbed = nil
+  def add_stubbed_value(key, value)
+    allow(ENV).to receive(:[]).with(key).and_return(value)
+    allow(ENV).to receive(:fetch).with(key).and_return(value)
+    allow(ENV).to receive(:fetch).with(key, anything()) do |_, default_val|
+      value || default_val
+    end
+  end
+
+  def env_stubbed?
+    ENV[STUBBED_KEY]
+  end
+
+  def init_stub
+    allow(ENV).to receive(:[]).and_call_original
+    allow(ENV).to receive(:fetch).and_call_original
+    add_stubbed_value(STUBBED_KEY, true)
   end
 end