climate_control 0.2.0 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 21d2f4ce1d74f8b2e4650cd7ab4efd3d40dac9d6
-  data.tar.gz: d14c7f5bc3b4ae309682f529c0ada07a9a874d9b
+SHA256:
+  metadata.gz: '00584062039c6b9161ff0e4a1b61ca51e4d8742225fc611180c4effcc43959c5'
+  data.tar.gz: 99135f45c57f58d87e17480896045c00eecebd632ed64eaf18bd1964ef7d454d
 SHA512:
-  metadata.gz: 1c533460213a9e6a4285c3daa7fade227a4a5a5e4eb9a23e41d6bb06362c60468739476dd916be6b9ec3447b3e155dc1594e0fd89597ae58935acef27b850940
-  data.tar.gz: 1a9aefaad845f0cdcb147b77cd176e81c0004d28fa17979aab312d27fe1e5251653902bc6fe05581ccd1ecab49075f122b7d029953c708af350e6ad0809f575d
+  metadata.gz: 01ba4ad3241cea796d408f0adb02f8c23b85771d1ce8338eedf973b95228a7cac9978218f08921915918c90a0f8f292dbb38f9401ca29de8d7a6ef3dc76d99b5
+  data.tar.gz: 9b20ce4affbcf22b6204016d31b7b54465ad5ba2a143fd2e6578d3dc5d4c80ac167c02c9ccc47d91c2a62441c5ce3037ebb13c445407876c163edaba73eb484d

data/.github/workflows/ci.yml

@@ -0,0 +1,23 @@
+name: CI
+on: [pull_request]
+jobs:
+  tests:
+    name: Tests
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        ruby-version: [3.1, '3.0', 2.7, 2.6, 2.5]
+    steps:
+      - uses: actions/checkout@v2
+      - name: Set up Ruby ${{ matrix.ruby-version }}
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: ${{ matrix.ruby-version }}
+      - name: Install gems
+        run: |
+          bundle config path vendor/bundle
+          bundle install --jobs 4 --retry 3
+      - name: Run tests
+        run: bundle exec rake
+      - name: Run StandardRB
+        run: bundle exec standardrb

data/CHANGELOG.md

@@ -0,0 +1,53 @@
+# Changelog
+
+All notable changes to this project will be documented in this file.
+
+The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
+and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
+
+## 1.2.0 / 2022-07-15
+
+- Added: `ClimateControl.unsafe_modify` for a thread-unsafe version of
+  `ClimateControl.modify` (useful for minitest-around for instance)
+- Deprecates `ClimateControl.env`, `ENV` should be used instead
+
+## 1.1.1 / 2022-05-28
+
+- Fixed: ENV was not restored if an error was thrown when assigning ENV
+
+## 1.1.0 / 2022-05-26
+
+- Refactor to use `Monitor` instead of `Mutex`
+- Add documentation about thread-safety
+- Allow ClimateControl.modify to be called without environment variables
+- Add test for concurrent access needed to be inside block
+- Relax development dependencies
+
+## 1.0.1 / 2021-05-26
+
+- Require minimum Ruby version of 2.5.0
+
+# 1.0.0 / 2021-03-06
+
+- Commit to supporting latest patch versions of Ruby 2.5+
+- Improve documentation
+- Format code with StandardRB
+- Bump gem dependencies
+
+# 0.2.0 / 2017-05-12
+
+- Allow nested environment changes in the same thread
+
+# 0.1.0 / 2017-01-07
+
+- Remove ActiveSupport dependency
+
+# 0.0.4 / 2017-01-06
+
+- Improved thread safety
+- Handle TypeErrors during assignment
+- Improve documentation
+
+# 0.0.1 / 2012-11-28
+
+- Initial release

data/CODEOWNERS

@@ -0,0 +1 @@
+* @dorianmariefr

data/CODE_OF_CONDUCT.md

@@ -0,0 +1,6 @@
+# Code of Conduct
+
+By participating in this project, you agree to abide by the
+[thoughtbot code of conduct][1].
+
+[1]: https://thoughtbot.com/open-source-code-of-conduct

data/{LICENSE.txt → LICENSE}

@@ -1,4 +1,4 @@
-Copyright (c) 2012-2014 Joshua Clayton and thoughtbot, inc.
+Copyright (c) 2012-2021 Joshua Clayton and thoughtbot, inc.
 
 MIT License
 

data/README.md

@@ -1,5 +1,7 @@
 # Climate Control
 
+![GitHub Actions CI](https://github.com/thoughtbot/climate_control/actions/workflows/ci.yml/badge.svg)
+
 Easily manage your environment.
 
 ## Installation
@@ -24,15 +26,31 @@ within a block:
 ```ruby
 ClimateControl.modify CONFIRMATION_INSTRUCTIONS_BCC: 'confirmation_bcc@example.com' do
   sign_up_as 'john@example.com'
+
+  confirm_account_for_email 'john@example.com'
+
+  expect(current_email).to bcc_to('confirmation_bcc@example.com')
+end
+```
+
+To modify multiple environment variables:
+
+```ruby
+ClimateControl.modify CONFIRMATION_INSTRUCTIONS_BCC: 'confirmation_bcc@example.com',
+                      MAIL_FROM: 'us@example.com' do
+  sign_up_as 'john@example.com'
+
   confirm_account_for_email 'john@example.com'
-  current_email.should bcc_to('confirmation_bcc@example.com')
+
+  expect(current_email).to bcc_to('confirmation_bcc@example.com')
+  expect(current_email).to be_from('us@example.com')
 end
 ```
 
 To use with RSpec, you could define this in your spec:
 
 ```ruby
-def with_modified_env(options, &block)
+def with_modified_env(options = {}, &block)
   ClimateControl.modify(options, &block)
 end
 ```
@@ -89,6 +107,39 @@ manner becomes more difficult:
 Climate Control modifies environment variables only within the context of the
 block, ensuring values are managed properly and consistently.
 
+## Thread-safety
+
+When using threads, for instance when running tests concurrently in the same
+process, you may need to wrap your code inside `ClimateControl.modify` blocks,
+e.g.:
+
+```ruby
+first_thread = Thread.new do
+  ClimateControl.modify(SECRET: "1") do
+    p ENV["SECRET"] # => "1"
+    sleep 2
+    p ENV["SECRET"] # => "1"
+  end
+end
+
+second_thread = Thread.new do
+  ClimateControl.modify({}) do
+    sleep 1
+    p ENV["SECRET"] # => nil
+    sleep 1
+    p ENV["SECRET"] # => nil
+  end
+end
+
+first_thread.join
+second_thread.join
+```
+
+> The modification wraps ENV in a mutex. If there's contention (the env being used - including potentially mutating values), it blocks until the value is freed (we shift out of the Ruby block).
+>
+> <cite><a href="https://github.com/thoughtbot/climate_control/issues/32#issuecomment-800713686">Josh Clayton</a></cite>
+
+
 ## Contributing
 
 1. Fork it
@@ -97,6 +148,23 @@ block, ensuring values are managed properly and consistently.
 4. Push to the branch (`git push origin my-new-feature`)
 5. Create new Pull Request
 
+This project uses [StandardRB](https://github.com/testdouble/standard) to ensure formatting.
+
 ## License
 
-climate_control is copyright 2012-2017 Joshua Clayton and thoughtbot, inc. It is free software and may be redistributed under the terms specified in the [LICENSE.txt](https://github.com/thoughtbot/climate_control/blob/master/LICENSE.txt) file.
+climate_control is copyright 2012-2021 Joshua Clayton and thoughtbot, inc. It is free software and may be redistributed under the terms specified in the [LICENSE](https://github.com/thoughtbot/climate_control/blob/main/LICENSE) file.
+
+About thoughtbot
+----------------
+
+![thoughtbot](https://thoughtbot.com/brand_assets/93:44.svg)
+
+climate_control is maintained and funded by thoughtbot, inc.
+The names and logos for thoughtbot are trademarks of thoughtbot, inc.
+
+We love open source software!
+See [our other projects][community] or
+[hire us][hire] to design, develop, and grow your product.
+
+[community]: https://thoughtbot.com/community?utm_source=github
+[hire]: https://thoughtbot.com/hire-us?utm_source=github

data/climate_control.gemspec

@@ -1,23 +1,25 @@
-# -*- encoding: utf-8 -*-
 lib = File.expand_path("../lib", __FILE__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require "climate_control/version"
 
 Gem::Specification.new do |gem|
-  gem.name          = "climate_control"
-  gem.version       = ClimateControl::VERSION
-  gem.authors       = ["Joshua Clayton"]
-  gem.email         = ["joshua.clayton@gmail.com"]
-  gem.description   = %q{Modify your ENV}
-  gem.summary       = %q{Modify your ENV easily with ClimateControl}
-  gem.homepage      = "https://github.com/thoughtbot/climate_control"
-  gem.license       = "MIT"
+  gem.name = "climate_control"
+  gem.version = ClimateControl::VERSION
+  gem.authors = ["Joshua Clayton", "Dorian Marié"]
+  gem.email = ["joshua.clayton@gmail.com", "dorian@dorianmarie.fr"]
+  gem.description = "Modify your ENV"
+  gem.summary = "Modify your ENV easily with ClimateControl"
+  gem.homepage = "https://github.com/thoughtbot/climate_control"
+  gem.license = "MIT"
 
-  gem.files         = `git ls-files`.split($/)
-  gem.test_files    = gem.files.grep(%r{^(test|spec|features)/})
+  gem.files = `git ls-files`.split($/)
+  gem.test_files = gem.files.grep(%r{^(test|spec|features)/})
   gem.require_paths = ["lib"]
 
-  gem.add_development_dependency "rspec", "~> 3.1.0"
-  gem.add_development_dependency "rake", "~> 10.3.2"
-  gem.add_development_dependency "simplecov", "~> 0.9.1"
+  gem.required_ruby_version = ">= 2.5.0"
+
+  gem.add_development_dependency "rspec"
+  gem.add_development_dependency "rake"
+  gem.add_development_dependency "simplecov"
+  gem.add_development_dependency "standard"
 end

data/lib/climate_control/version.rb

@@ -1,3 +1,3 @@
 module ClimateControl
-  VERSION = "0.2.0".freeze
+  VERSION = "1.2.0".freeze
 end

data/lib/climate_control.rb

@@ -1,16 +1,72 @@
-require "climate_control/environment"
 require "climate_control/errors"
-require "climate_control/modifier"
 require "climate_control/version"
+require "monitor"
 
 module ClimateControl
-  @@env = ClimateControl::Environment.new
+  extend self
+  extend Gem::Deprecate
 
-  def self.modify(environment_overrides, &block)
-    Modifier.new(env, environment_overrides, &block).process
+  SEMAPHORE = Monitor.new
+  private_constant :SEMAPHORE
+
+  def modify(environment_overrides = {}, &block)
+    environment_overrides = environment_overrides.transform_keys(&:to_s)
+
+    SEMAPHORE.synchronize do
+      previous = ENV.to_hash
+
+      begin
+        copy environment_overrides
+      ensure
+        middle = ENV.to_hash
+      end
+
+      block.call
+    ensure
+      after = ENV
+      (previous.keys | middle.keys | after.keys).each do |key|
+        if previous[key] != after[key] && middle[key] == after[key]
+          ENV[key] = previous[key]
+        end
+      end
+    end
   end
 
-  def self.env
-    @@env
+  def unsafe_modify(environment_overrides = {}, &block)
+    environment_overrides = environment_overrides.transform_keys(&:to_s)
+
+    previous = ENV.to_hash
+
+    begin
+      copy environment_overrides
+    ensure
+      middle = ENV.to_hash
+    end
+
+    block.call
+  ensure
+    after = ENV
+    (previous.keys | middle.keys | after.keys).each do |key|
+      if previous[key] != after[key] && middle[key] == after[key]
+        ENV[key] = previous[key]
+      end
+    end
+  end
+
+  def env
+    ENV
+  end
+
+  deprecate :env, "ENV", 2022, 10
+
+  private
+
+  def copy(overrides)
+    overrides.each do |key, value|
+      ENV[key] = value
+    rescue TypeError => e
+      raise UnassignableValueError,
+        "attempted to assign #{value} to #{key} but failed (#{e.message})"
+    end
   end
 end

data/spec/acceptance/climate_control_spec.rb

@@ -1,9 +1,11 @@
 require "spec_helper"
 
+Thing = Class.new
+
 describe "Climate control" do
   it "allows modification of the environment" do
     block_run = false
-    ClimateControl.modify FOO: "bar" do
+    with_modified_env FOO: "bar" do
       expect(ENV["FOO"]).to eq "bar"
       block_run = true
     end
@@ -87,22 +89,22 @@ describe "Climate control" do
     # 0.25s passes
     # [other_thread] thread resolves, FOO is removed, BAZ is copied back to ENV
 
-    thread_removing_env = Thread.new do
+    thread_removing_env = Thread.new {
       with_modified_env BAZ: "buzz" do
         sleep 0.5
       end
 
       expect(ENV["BAZ"]).to be_nil
-    end
+    }
 
-    other_thread = Thread.new do
+    other_thread = Thread.new {
       sleep 0.25
       with_modified_env FOO: "bar" do
         sleep 0.5
       end
 
       expect(ENV["FOO"]).to be_nil
-    end
+    }
 
     thread_removing_env.join
     other_thread.join
@@ -111,12 +113,45 @@ describe "Climate control" do
     expect(ENV["BAZ"]).to be_nil
   end
 
+  it "handles threads accessing the same key wrapped in a block" do
+    first_thread = Thread.new {
+      with_modified_env do
+        with_modified_env CONFLICTING_KEY: "1" do
+          sleep 0.5
+          expect(ENV["CONFLICTING_KEY"]).to eq("1")
+        end
+
+        expect(ENV["CONFLICTING_KEY"]).to be_nil
+      end
+    }
+
+    second_thread = Thread.new {
+      with_modified_env do
+        sleep 0.25
+        expect(ENV["CONFLICTING_KEY"]).to be_nil
+
+        with_modified_env CONFLICTING_KEY: "2" do
+          expect(ENV["CONFLICTING_KEY"]).to eq("2")
+          sleep 0.5
+          expect(ENV["CONFLICTING_KEY"]).to eq("2")
+        end
+
+        expect(ENV["CONFLICTING_KEY"]).to be_nil
+      end
+    }
+
+    first_thread.join
+    second_thread.join
+
+    expect(ENV["CONFLICTING_KEY"]).to be_nil
+  end
+
   it "is re-entrant" do
-    ret = with_modified_env(FOO: "foo") do
+    ret = with_modified_env(FOO: "foo") {
       with_modified_env(BAR: "bar") do
         "bar"
       end
-    end
+    }
 
     expect(ret).to eq("bar")
 
@@ -125,15 +160,39 @@ describe "Climate control" do
   end
 
   it "raises when the value cannot be assigned properly" do
-    Thing = Class.new
     message = generate_type_error_for_object(Thing.new)
 
-    expect do
+    expect {
       with_modified_env(FOO: Thing.new)
-    end.to raise_error ClimateControl::UnassignableValueError, /attempted to assign .*Thing.* to FOO but failed \(#{message}\)$/
+    }.to raise_error ClimateControl::UnassignableValueError, /attempted to assign .*Thing.* to FOO but failed \(#{message}\)$/
+  end
+
+  it "restores the ENV even when an error was raised when assigning values" do
+    ENV["KEY_TO_OVERRIDE"] = "initial_value_1"
+    ENV["KEY_THAT_WILL_ERROR_OUT"] = "initial_value_2"
+
+    expect {
+      with_modified_env(
+        KEY_TO_OVERRIDE: "overwriten_value_1",
+        KEY_THAT_WILL_ERROR_OUT: :value_that_will_error_out
+      ) {}
+    }.to raise_error ClimateControl::UnassignableValueError
+
+    expect(ENV["KEY_TO_OVERRIDE"]).to eq("initial_value_1")
+    expect(ENV["KEY_THAT_WILL_ERROR_OUT"]).to eq("initial_value_2")
+  end
+
+  it "doesn't block on nested modify calls" do
+    with_modified_env(SMS_DEFAULT_COUNTRY_CODE: nil) do
+      with_modified_env(SMS_DEFAULT_COUNTRY_CODE: "++56") do
+        expect(ENV.fetch("SMS_DEFAULT_COUNTRY_CODE", "++41")).to eq("++56")
+      end
+
+      expect(ENV.fetch("SMS_DEFAULT_COUNTRY_CODE", "++41")).to eq("++41")
+    end
   end
 
-  def with_modified_env(options, &block)
+  def with_modified_env(options = {}, &block)
     ClimateControl.modify(options, &block)
   end
 

data/spec/acceptance/unsafe_modify_spec.rb

@@ -0,0 +1,186 @@
+require "spec_helper"
+
+describe "ClimateControl#unsafe_modify" do
+  it "allows modification of the environment" do
+    block_run = false
+    with_modified_env FOO: "bar" do
+      expect(ENV["FOO"]).to eq "bar"
+      block_run = true
+    end
+
+    expect(ENV["FOO"]).to be_nil
+    expect(block_run).to be true
+  end
+
+  it "modifies the environment" do
+    with_modified_env VARIABLE_1: "bar", VARIABLE_2: "qux" do
+      expect(ENV["VARIABLE_1"]).to eq "bar"
+      expect(ENV["VARIABLE_2"]).to eq "qux"
+    end
+
+    expect(ENV["VARIABLE_1"]).to be_nil
+    expect(ENV["VARIABLE_2"]).to be_nil
+  end
+
+  it "allows for environment variables to be assigned within the block" do
+    with_modified_env VARIABLE_1: "modified" do
+      ENV["ASSIGNED_IN_BLOCK"] = "assigned"
+    end
+
+    expect(ENV["ASSIGNED_IN_BLOCK"]).to eq "assigned"
+  end
+
+  it "reassigns previously set environment variables" do
+    ENV["VARIABLE_ASSIGNED_BEFORE_MODIFYING_ENV"] = "original"
+    expect(ENV["VARIABLE_ASSIGNED_BEFORE_MODIFYING_ENV"]).to eq "original"
+
+    with_modified_env VARIABLE_ASSIGNED_BEFORE_MODIFYING_ENV: "overridden" do
+      expect(ENV["VARIABLE_ASSIGNED_BEFORE_MODIFYING_ENV"]).to eq "overridden"
+    end
+
+    expect(ENV["VARIABLE_ASSIGNED_BEFORE_MODIFYING_ENV"]).to eq "original"
+  end
+
+  it "persists the change when overriding the variable in the block" do
+    with_modified_env VARIABLE_MODIFIED_AND_THEN_ASSIGNED: "modified" do
+      ENV["VARIABLE_MODIFIED_AND_THEN_ASSIGNED"] = "assigned value"
+    end
+
+    expect(ENV["VARIABLE_MODIFIED_AND_THEN_ASSIGNED"]).to eq "assigned value"
+  end
+
+  it "resets environment variables even if the block raises" do
+    expect {
+      with_modified_env FOO: "bar" do
+        raise "broken"
+      end
+    }.to raise_error("broken")
+
+    expect(ENV["FOO"]).to be_nil
+  end
+
+  it "preserves environment variables set within the block" do
+    ENV["CHANGED"] = "old value"
+
+    with_modified_env IRRELEVANT: "ignored value" do
+      ENV["CHANGED"] = "new value"
+    end
+
+    expect(ENV["CHANGED"]).to eq "new value"
+  end
+
+  it "returns the value of the block" do
+    value = with_modified_env VARIABLE_1: "bar" do
+      "value inside block"
+    end
+
+    expect(value).to eq "value inside block"
+  end
+
+  it "handles threads correctly" do
+    # failure path without mutex
+    # [thread_removing_env] BAZ is assigned
+    # 0.25s passes
+    # [other_thread] FOO is assigned and ENV is copied (which includes BAZ)
+    # 0.25s passes
+    # [thread_removing_env] thread resolves and BAZ is removed from env; other_thread still retains knowledge of BAZ
+    # 0.25s passes
+    # [other_thread] thread resolves, FOO is removed, BAZ is copied back to ENV
+
+    thread_removing_env = Thread.new {
+      with_modified_env BAZ: "buzz" do
+        sleep 0.5
+      end
+
+      expect(ENV["BAZ"]).to be_nil
+    }
+
+    other_thread = Thread.new {
+      sleep 0.25
+      with_modified_env FOO: "bar" do
+        sleep 0.5
+      end
+
+      expect(ENV["FOO"]).to be_nil
+    }
+
+    thread_removing_env.join
+    other_thread.join
+
+    expect(ENV["FOO"]).to be_nil
+    expect(ENV["BAZ"]).to be_nil
+  end
+
+  it "is re-entrant" do
+    ret = with_modified_env(FOO: "foo") {
+      with_modified_env(BAR: "bar") do
+        "bar"
+      end
+    }
+
+    expect(ret).to eq("bar")
+
+    expect(ENV["FOO"]).to be_nil
+    expect(ENV["BAR"]).to be_nil
+  end
+
+  it "raises when the value cannot be assigned properly" do
+    message = generate_type_error_for_object(Thing.new)
+
+    expect {
+      with_modified_env(FOO: Thing.new)
+    }.to raise_error ClimateControl::UnassignableValueError, /attempted to assign .*Thing.* to FOO but failed \(#{message}\)$/
+  end
+
+  it "restores the ENV even when an error was raised when assigning values" do
+    ENV["KEY_TO_OVERRIDE"] = "initial_value_1"
+    ENV["KEY_THAT_WILL_ERROR_OUT"] = "initial_value_2"
+
+    expect {
+      with_modified_env(
+        KEY_TO_OVERRIDE: "overwriten_value_1",
+        KEY_THAT_WILL_ERROR_OUT: :value_that_will_error_out
+      ) {}
+    }.to raise_error ClimateControl::UnassignableValueError
+
+    expect(ENV["KEY_TO_OVERRIDE"]).to eq("initial_value_1")
+    expect(ENV["KEY_THAT_WILL_ERROR_OUT"]).to eq("initial_value_2")
+  end
+
+  it "doesn't block on nested modify calls" do
+    with_modified_env(SMS_DEFAULT_COUNTRY_CODE: nil) do
+      with_modified_env(SMS_DEFAULT_COUNTRY_CODE: "++56") do
+        expect(ENV.fetch("SMS_DEFAULT_COUNTRY_CODE", "++41")).to eq("++56")
+      end
+
+      expect(ENV.fetch("SMS_DEFAULT_COUNTRY_CODE", "++41")).to eq("++41")
+    end
+  end
+
+  def with_modified_env(options = {}, &block)
+    ClimateControl.unsafe_modify(options, &block)
+  end
+
+  def generate_type_error_for_object(object)
+    message = nil
+
+    begin
+      "1" + object
+    rescue TypeError => e
+      message = e.message
+    end
+
+    message
+  end
+
+  around do |example|
+    old_env = ENV.to_hash
+
+    example.run
+
+    ENV.clear
+    old_env.each do |key, value|
+      ENV[key] = value
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -1,5 +1,5 @@
 begin
-  require 'simplecov'
+  require "simplecov"
   SimpleCov.start
 rescue LoadError
   warn "warning: simplecov gem not found; skipping coverage"

metadata

@@ -1,84 +1,101 @@
 --- !ruby/object:Gem::Specification
 name: climate_control
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 1.2.0
 platform: ruby
 authors:
 - Joshua Clayton
-autorequire: 
+- Dorian Marié
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2017-05-12 00:00:00.000000000 Z
+date: 2022-07-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 3.1.0
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 3.1.0
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 10.3.2
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 10.3.2
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 0.9.1
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 0.9.1
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: standard
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: Modify your ENV
 email:
 - joshua.clayton@gmail.com
+- dorian@dorianmarie.fr
 executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/workflows/ci.yml"
 - ".gitignore"
-- ".travis.yml"
+- CHANGELOG.md
+- CODEOWNERS
+- CODE_OF_CONDUCT.md
 - Gemfile
-- LICENSE.txt
-- NEWS
+- LICENSE
 - README.md
 - Rakefile
 - climate_control.gemspec
 - lib/climate_control.rb
-- lib/climate_control/environment.rb
 - lib/climate_control/errors.rb
-- lib/climate_control/modifier.rb
 - lib/climate_control/version.rb
 - spec/acceptance/climate_control_spec.rb
+- spec/acceptance/unsafe_modify_spec.rb
 - spec/spec_helper.rb
 homepage: https://github.com/thoughtbot/climate_control
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -86,18 +103,18 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 2.5.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.5.2
-signing_key: 
+rubygems_version: 3.3.13
+signing_key:
 specification_version: 4
 summary: Modify your ENV easily with ClimateControl
 test_files:
 - spec/acceptance/climate_control_spec.rb
+- spec/acceptance/unsafe_modify_spec.rb
 - spec/spec_helper.rb

data/.travis.yml

@@ -1,11 +0,0 @@
-rvm:
-  - 2.2.6
-  - 2.3.3
-  - 2.4.0
-before_install:
-  - gem update --system
-  - gem update bundler
-install: "bundle install"
-branches:
-  only:
-    - master

data/NEWS

@@ -1,13 +0,0 @@
-0.2.0 (May 12, 2017)
-  Allow nested environment changes in the same thread
-
-0.1.0 (January 7, 2017)
-  Remove ActiveSupport dependency
-
-0.0.4 (January 6, 2017)
-  Improved thread safety
-  Handle TypeErrors during assignment
-  Improve documentation
-
-0.0.1 (November 28, 2012)
-  Initial release

data/lib/climate_control/environment.rb

@@ -1,36 +0,0 @@
-require "thread"
-require "forwardable"
-
-module ClimateControl
-  class Environment
-    extend Forwardable
-
-    def initialize
-      @semaphore = Mutex.new
-      @owner = nil
-    end
-
-    def_delegators :env, :[]=, :to_hash, :[], :delete
-
-    def synchronize
-      if @owner == Thread.current
-        return yield if block_given?
-      end
-
-      @semaphore.synchronize do
-        begin
-          @owner = Thread.current
-          yield if block_given?
-        ensure
-          @owner = nil
-        end
-      end
-    end
-
-    private
-
-    def env
-      ENV
-    end
-  end
-end

data/lib/climate_control/modifier.rb

@@ -1,96 +0,0 @@
-module ClimateControl
-  class Modifier
-    def initialize(env, environment_overrides = {}, &block)
-      @environment_overrides = stringify_keys(environment_overrides)
-      @block = block
-      @env = env
-    end
-
-    def process
-      @env.synchronize do
-        begin
-          prepare_environment_for_block
-          run_block
-        ensure
-          cache_environment_after_block
-          delete_keys_that_do_not_belong
-          revert_changed_keys
-        end
-      end
-    end
-
-    private
-
-    def prepare_environment_for_block
-      @original_env = clone_environment
-      copy_overrides_to_environment
-      @env_with_overrides_before_block = clone_environment
-    end
-
-    def run_block
-      @block.call
-    end
-
-    def copy_overrides_to_environment
-      @environment_overrides.each do |key, value|
-        begin
-          @env[key] = value
-        rescue TypeError => e
-          raise UnassignableValueError,
-            "attempted to assign #{value} to #{key} but failed (#{e.message})"
-        end
-      end
-    end
-
-    def keys_to_remove
-      @environment_overrides.keys
-    end
-
-    def keys_changed_by_block
-      @keys_changed_by_block ||= OverlappingKeysWithChangedValues.new(@env_with_overrides_before_block, @env_after_block).keys
-    end
-
-    def cache_environment_after_block
-      @env_after_block = clone_environment
-    end
-
-    def delete_keys_that_do_not_belong
-      (keys_to_remove - keys_changed_by_block).each {|key| @env.delete(key) }
-    end
-
-    def revert_changed_keys
-      (@original_env.keys - keys_changed_by_block).each do |key|
-        @env[key] = @original_env[key]
-      end
-    end
-
-    def clone_environment
-      @env.to_hash
-    end
-
-    def stringify_keys(env)
-      env.each_with_object({}) do |(key, value), hash|
-        hash[key.to_s] = value
-      end
-    end
-
-    class OverlappingKeysWithChangedValues
-      def initialize(hash_1, hash_2)
-        @hash_1 = hash_1 || {}
-        @hash_2 = hash_2
-      end
-
-      def keys
-        overlapping_keys.select do |overlapping_key|
-          @hash_1[overlapping_key] != @hash_2[overlapping_key]
-        end
-      end
-
-      private
-
-      def overlapping_keys
-        @hash_2.keys & @hash_1.keys
-      end
-    end
-  end
-end