nenv 0.0.2 → 0.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 4aa66c0bda93185b1177fda8e93484e004f4e802
-  data.tar.gz: 8b360b9cb96f49e49ade892326f1ab979ab4ab64
+  metadata.gz: bf78f1e4b9c0b4b0836dcb88c546d8d917506442
+  data.tar.gz: e18492bf894e4951f9e57bee5c343e507cbf7c06
 SHA512:
-  metadata.gz: 872463708e4ba5ea45e565f5cf2de586900e702abd3996037388947f3e9e69da949e62d1a60dfa41550400427ee27e6cc8c736516f17278dc2a606903d7f81bb
-  data.tar.gz: b793da1678ae9e09d164f0121286f6f35874c8721da2351aab457918188d086c318dcd84a75deaa67c9f7c1f270057275d26d5f02fabdf1929e1fe336c202882
+  metadata.gz: 2370b04d5e91d4817da56f343c36f0e94ababa43073316695c4f3e0f9365073f3031aca40956a4866d0e4b7a8ff6e790a6d3c9dee0596cabbbd56a820e1aa746
+  data.tar.gz: 23245f176a801613181ab0443aad7c1770d4cf879e83ba2cdbfdb392e0abf37a8a43013952a42d686a47de4c6f7cf57d8f71e9f0ab5a8ec2fdaa0ea1df2c608a

data/.rubocop.yml

@@ -0,0 +1 @@
+inherit_from: .rubocop_todo.yml

data/.rubocop_todo.yml

@@ -0,0 +1,33 @@
+# This configuration was generated by `rubocop --auto-gen-config`
+# on 2014-12-13 13:35:53 +0100 using RuboCop version 0.28.0.
+# The point is for the user to remove these configuration records
+# one by one as the offenses are removed from the code base.
+# Note that changes in the inspected code, or installation of new
+# versions of RuboCop, may require this file to be generated again.
+
+# Offense count: 1
+Lint/HandleExceptions:
+  Enabled: false
+
+# Offense count: 1
+Metrics/AbcSize:
+  Max: 16
+
+# Offense count: 1
+# Configuration parameters: AllowURI, URISchemes.
+Metrics/LineLength:
+  Max: 112
+
+# Offense count: 1
+# Configuration parameters: CountComments.
+Metrics/MethodLength:
+  Max: 11
+
+# Offense count: 8
+Style/Documentation:
+  Enabled: false
+
+# Offense count: 1
+# Configuration parameters: EnforcedStyle, SupportedStyles.
+Style/MethodName:
+  Enabled: false

data/.travis.yml

@@ -0,0 +1,13 @@
+language: ruby
+bundler_args: --without development
+rvm:
+  - 1.9.3
+  - 2.0.0
+  - 2.1.5
+  - ruby-head
+  - jruby
+  - rbx
+matrix:
+  allow_failures:
+    - rvm: rbx
+    - rvm: jruby

data/Gemfile

@@ -3,6 +3,9 @@ source 'https://rubygems.org'
 # Specify your gem's dependencies in nenv.gemspec
 gemspec development_group: :test
 
+gem 'coveralls', require: false
+
 group :development do
-  gem 'guard-rspec', '~> 4.5.0'
+  gem 'guard-rspec', '~> 4.5', require: false
+  gem 'guard-rubocop', require: false
 end

data/Guardfile

@@ -7,28 +7,26 @@ directories(%w(lib spec))
 ## Uncomment to clear the screen before every task
 # clearing :on
 
-# Note: The cmd option is now required due to the increasing number of ways
-#       rspec may be run, below are examples of the most common uses.
-#  * bundler: 'bundle exec rspec'
-#  * bundler binstubs: 'bin/rspec'
-#  * spring: 'bin/rspec' (This will use spring if running and you have
-#                          installed the spring binstubs per the docs)
-#  * zeus: 'zeus rspec' (requires the server to be started separately)
-#  * 'just' rspec: 'rspec'
+group :spec, halt_on_fail: true do
+  guard :rspec, cmd: 'bundle exec rspec' do
+    require 'guard/rspec/dsl'
+    dsl = Guard::RSpec::Dsl.new(self)
 
-guard :rspec, cmd: "bundle exec rspec" do
-  require "guard/rspec/dsl"
-  dsl = Guard::RSpec::Dsl.new(self)
+    # Feel free to open issues for suggestions and improvements
 
-  # Feel free to open issues for suggestions and improvements
+    # RSpec files
+    rspec = dsl.rspec
+    watch(rspec.spec_helper) { rspec.spec_dir }
+    watch(rspec.spec_support) { rspec.spec_dir }
+    watch(rspec.spec_files)
 
-  # RSpec files
-  rspec = dsl.rspec
-  watch(rspec.spec_helper) { rspec.spec_dir }
-  watch(rspec.spec_support) { rspec.spec_dir }
-  watch(rspec.spec_files)
+    # Ruby files
+    ruby = dsl.ruby
+    dsl.watch_spec_files_for(ruby.lib_files)
+  end
 
-  # Ruby files
-  ruby = dsl.ruby
-  dsl.watch_spec_files_for(ruby.lib_files)
+  guard :rubocop do
+    watch(/.+\.rb$/)
+    watch(/(?:.+\/)?\.rubocop(?:_todo)?\.yml$/) { |m| File.dirname(m[0]) }
+  end
 end

data/README.md

@@ -1,3 +1,9 @@
+[![Build Status](https://travis-ci.org/e2/nenv.png?branch=master)](https://travis-ci.org/e2/nenv)
+[![Gem Version](http://img.shields.io/gem/v/nenv.svg)](http://badge.fury.io/rb/nenv)
+[![Dependency Status](https://gemnasium.com/e2/nenv.svg)](https://gemnasium.com/e2/nenv)
+[![Code Climate](https://codeclimate.com/github/e2/nenv/badges/gpa.svg)](https://codeclimate.com/github/e2/nenv)
+[![Coverage Status](https://coveralls.io/repos/e2/nenv/badge.png)](https://coveralls.io/r/e2/nenv)
+
 # Nenv
 
 Using ENV in Ruby is like using raw SQL statements - it feels wrong, because it is.
@@ -16,6 +22,10 @@ If you agree, this gem is for you.
 - strict mode saves you from doing validation yourself
 - and there's more to come...
 
+Other benefits (and compared to other solutions):
+- should still work with Ruby 1.8 (in case anyone is still stuck with it)
+- it's designed to be as lightweight and as fast as possible compared to ENV
+- designed to be both hackable and convenient
 
 ## Installation
 
@@ -42,6 +52,7 @@ Or install it yourself as:
 You no longer have to care whether the value is "0" or "false" or "no" or "FALSE" or ... whatever
 
 ```ruby
+# Without Nenv
 t.verbose = (ENV['CI'] == 'true')
 ok = ENV['RUBYGEMS_GEMDEPS'] == "1" || ENV.key?('BUNDLE_GEMFILE']
 ENV['DEBUG'] = "true"
@@ -59,6 +70,7 @@ Nenv.debug = true
 ### "Namespaces"
 
 ```ruby
+# Without Nenv
 puts ENV['GIT_BROWSER`]
 puts ENV['GIT_PAGER`]
 puts ENV['GIT_EDITOR`]
@@ -76,6 +88,7 @@ puts git.editor
 ### Custom type handling
 
 ```ruby
+# Code without Nenv
 paths = [ENV['GEM_HOME`]] + ENV['GEM_PATH'].split(':')
 enable_logging if Integer(ENV['WEB_CONCURRENCY']) > 1
 mydata = YAML.load(ENV['MY_DATA'])
@@ -108,10 +121,10 @@ Nenv.verbose = debug
 ### Automatic conversion to string
 
 ```ruby
-ENV['RUBYGEMS_GEMDEPS'] = 1  # TypeError: no implicit conversion of Fixnum into String
+ENV['RUBYGEMS_GEMDEPS'] = 1  # TypeError: no implicit conversion of Fixnum (...)
 ```
 
-No automatically uses `to_s`:
+Nenv automatically uses `to_s`:
 
 ```ruby
 Nenv.rubygems_gemdeps = 1  # no problem here
@@ -129,7 +142,7 @@ ENV['MY_DATA'] = YAML.dump(data)
 can now become:
 
 ```ruby
-my = Nenv(:my)
+my = Nenv :my
 my.instance.create_method(:data) { |d| YAML.load(d) }
 my.instance.create_method(:data=) { |d| YAML.dump(d) }
 
@@ -141,6 +154,7 @@ my.data = data
 ### Strict mode
 
 ```ruby
+# Without Nenv
 fail 'home not allowed' if ENV['HOME'] = Dir.pwd  # BUG! Assignment instead of comparing!
 puts ENV['HOME'] # Now contains clobbered value
 ```
@@ -172,6 +186,21 @@ Nenv.path += Pathname.pwd + "foo"
 
 ```
 
+### Your own class
+
+```ruby
+class MyEnv < Nenv::Environment
+  def initialize
+    super("my_env")
+    create_method(:foo?)
+  end
+end
+
+MyEnv.new.foo? # same as ENV['MY_ENV_FOO'][/^(?:false|no|n|0)/i,1].nil?
+
+```
+
+
 ## NOTES
 
 Still, avoid using environment variables if you can.
@@ -196,13 +225,13 @@ Well sure, having ENV act like a Hash is much better than calling "getenv".
 
 Unfortunately, the advantages of using ENV make no sense:
 
-1) it's faster but ... environment variables are rarely used thousands of times in tight loops
-2) it's already an object ... but there's not much you can do with it
-3) it's globally available ... but you can't isolate it in tests (you need to reset it every time)
-4) you can use it to set variables ... but it's named like a const
-5) it allows you to use keys regardless of case ... but by convention lowercase shouldn't be used except for local variables (which are only really used by shell scripts)
-6) it's supposed to look ugly to discourage use ... but often your app/gem is forced to use them anyway
-7) it's a simple class ... but either you encapsulate it in your own classes - or all the value mapping/validation happens everywhere you want the data
+- it's faster but ... environment variables are rarely used thousands of times in tight loops
+- it's already an object ... but there's not much you can do with it (try ENV.class)
+- it's globally available ... but you can't isolate it in tests (you need to reset it every time)
+- you can use it to set variables ... but it's named like a const
+- it allows you to use keys regardless of case ... but by convention lowercase shouldn't be used except for local variables (which are only really used by shell scripts)
+- it's supposed to look ugly to discourage use ... but often your app/gem is forced to use them anyway
+- it's a simple class ... but either you encapsulate it in your own classes - or all the value mapping/validation happens everywhere you want the data (yuck!)
 
 
 But the BIGGEST disadvantage is in specs, e.g.:

data/Rakefile

@@ -1,8 +1,17 @@
-require "bundler/gem_tasks"
+require 'bundler/gem_tasks'
+
+require 'nenv'
 
 default_tasks = []
 
-require "rspec/core/rake_task"
-default_tasks  << RSpec::Core::RakeTask.new(:spec)
+require 'rspec/core/rake_task'
+default_tasks  << RSpec::Core::RakeTask.new(:spec) do |t|
+  t.verbose = Nenv.ci?
+end
+
+unless Nenv.ci?
+  require 'rubocop/rake_task'
+  default_tasks << RuboCop::RakeTask.new(:rubocop)
+end
 
 task default: default_tasks.map(&:name)

data/lib/nenv.rb

@@ -1,8 +1,8 @@
-require "nenv/version"
+require 'nenv/version'
 
-require "nenv/autoenvironment"
+require 'nenv/autoenvironment'
 
-def Nenv(namespace=nil)
+def Nenv(namespace = nil)
   Nenv::AutoEnvironment.new(namespace)
 end
 

data/lib/nenv/autoenvironment.rb

@@ -1,9 +1,9 @@
-require "nenv/environment"
+require 'nenv/environment'
 module Nenv
   class AutoEnvironment < Nenv::Environment
     def method_missing(meth, *args)
       create_method(meth) unless respond_to?(meth)
-      self.send(meth, *args)
+      send(meth, *args)
     end
   end
 end

data/lib/nenv/environment.rb

@@ -1,5 +1,5 @@
-require "nenv/environment/dumper"
-require "nenv/environment/loader"
+require 'nenv/environment/dumper'
+require 'nenv/environment/loader'
 
 module Nenv
   class Environment
@@ -14,7 +14,7 @@ module Nenv
 
     class AlreadyExistsError < MethodError
       def message
-        format("Method %s already exists", @meth.inspect)
+        format('Method %s already exists', @meth.inspect)
       end
     end
 
@@ -41,7 +41,7 @@ module Nenv
     private
 
     def _sanitize(meth)
-      meth[/^([^=?]*)[=?]?$/, 1].upcase
+      meth.to_s[/^([^=?]*)[=?]?$/, 1].upcase
     end
   end
 end

data/lib/nenv/environment/loader.rb

@@ -2,7 +2,7 @@ module Nenv
   class Environment
     class Loader
       def initialize(meth)
-        @bool = meth.to_s.end_with?("?")
+        @bool = meth.to_s.end_with?('?')
       end
 
       def load(raw_value, &callback)
@@ -16,9 +16,9 @@ module Nenv
         case raw_value
         when nil
           nil
-        when ""
+        when ''
           fail ArgumentError, "Can't convert empty string into Bool"
-        when "0", "false", "n", "no", "NO", "FALSE"
+        when '0', 'false', 'n', 'no', 'NO', 'FALSE'
           false
         else
           true

data/lib/nenv/version.rb

@@ -1,3 +1,3 @@
 module Nenv
-  VERSION = "0.0.2"
+  VERSION = '0.0.3'
 end

data/nenv.gemspec

@@ -4,21 +4,21 @@ $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require 'nenv/version'
 
 Gem::Specification.new do |spec|
-  spec.name          = "nenv"
+  spec.name          = 'nenv'
   spec.version       = Nenv::VERSION
-  spec.authors       = ["Cezary Baginski"]
-  spec.email         = ["cezary@chronomantic.net"]
-  spec.summary       = %q{Convenience wrapper for Ruby's ENV}
-  spec.description   = %q{Using ENV is like using raw SQL statements in your code. Well all know how that ends...}
-  spec.homepage      = ""
-  spec.license       = "MIT"
+  spec.authors       = ['Cezary Baginski']
+  spec.email         = ['cezary@chronomantic.net']
+  spec.summary       = "Convenience wrapper for Ruby's ENV"
+  spec.description   = 'Using ENV is like using raw SQL statements in your code. Well all know how that ends...'
+  spec.homepage      = 'https://github.com/e2/nenv'
+  spec.license       = 'MIT'
 
   spec.files         = `git ls-files -z`.split("\x0")
-  spec.executables   = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
-  spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
-  spec.require_paths = ["lib"]
+  spec.executables   = spec.files.grep(/^bin\//) { |f| File.basename(f) }
+  spec.test_files    = spec.files.grep(/^(test|spec|features)\//)
+  spec.require_paths = ['lib']
 
-  spec.add_development_dependency "bundler", "~> 1.7"
-  spec.add_development_dependency "rspec", "~> 3.1"
-  spec.add_development_dependency "rake", "~> 10.0"
+  spec.add_development_dependency 'bundler', '~> 1.7'
+  spec.add_development_dependency 'rspec', '~> 3.1'
+  spec.add_development_dependency 'rake', '~> 10.0'
 end

data/spec/lib/nenv/environment/dumper_spec.rb

@@ -1,31 +1,31 @@
 require 'yaml'
 
-require "nenv/environment/dumper"
+require 'nenv/environment/dumper'
 
 RSpec.describe Nenv::Environment::Dumper do
   subject { described_class.new.dump(value) }
 
   context "with \"abc\"" do
-    let(:value) { "abc" }
-    it { is_expected.to eq("abc") }
+    let(:value) { 'abc' }
+    it { is_expected.to eq('abc') }
   end
 
-  context "with 123" do
+  context 'with 123' do
     let(:value) { 123 }
-    it { is_expected.to eq("123") }
+    it { is_expected.to eq('123') }
   end
 
-  context "with nil" do
+  context 'with nil' do
     let(:value) { nil }
     it { is_expected.to eq(nil) }
   end
 
-  context "with a block" do
+  context 'with a block' do
     subject do
-      described_class.new.dump(value) { |data| YAML::dump(data) }
+      described_class.new.dump(value) { |data| YAML.dump(data) }
     end
 
-    context "with a yaml string" do
+    context 'with a yaml string' do
       let(:value) { { foo: 3 } }
       let(:yaml) { "---\n:foo: 3\n" }
       it { is_expected.to eq(yaml) }

data/spec/lib/nenv/environment/loader_spec.rb

@@ -1,20 +1,20 @@
 require 'yaml'
-require "nenv/environment/loader"
+require 'nenv/environment/loader'
 
 RSpec.describe Nenv::Environment::Loader do
-  context "with no block" do
+  context 'with no block' do
     subject { described_class.new(meth).load(value) }
 
-    context "with a normal method" do
+    context 'with a normal method' do
       let(:meth) { :foo }
 
       context "with \"abc\"" do
-        let(:value) { "abc" }
-        it { is_expected.to eq("abc") }
+        let(:value) { 'abc' }
+        it { is_expected.to eq('abc') }
       end
     end
 
-    context "with a bool method" do
+    context 'with a bool method' do
       let(:meth) { :foo? }
 
       %w(1 true y yes TRUE YES foobar).each do |data|
@@ -31,13 +31,13 @@ RSpec.describe Nenv::Environment::Loader do
         end
       end
 
-      context "with nil" do
+      context 'with nil' do
         let(:value) { nil }
         it { is_expected.to eq(nil) }
       end
 
-      context "when empty string" do
-        let(:value) { "" }
+      context 'when empty string' do
+        let(:value) { '' }
         it do
           expect { subject }.to raise_error(
             ArgumentError, /Can't convert empty string into Bool/
@@ -47,13 +47,13 @@ RSpec.describe Nenv::Environment::Loader do
     end
   end
 
-  context "with a block" do
+  context 'with a block' do
     subject do
-      described_class.new(:foo).load(value) { |data| YAML::load(data) }
+      described_class.new(:foo).load(value) { |data| YAML.load(data) }
     end
-    context "with a yaml string" do
+    context 'with a yaml string' do
       let(:value) { "--- foo\n...\n" }
-      it { is_expected.to eq("foo") }
+      it { is_expected.to eq('foo') }
     end
   end
 end

data/spec/lib/nenv/environment_spec.rb

@@ -1,13 +1,12 @@
-require "yaml"
+require 'yaml'
 
-require "nenv/environment"
+require 'nenv/environment'
 
 RSpec.describe Nenv::Environment do
   let(:env) { instance_double(Hash) } # a hash is close enough
-  before(:each) { stub_const("ENV", env) }
-
-  context "without integration" do
+  before(:each) { stub_const('ENV', env) }
 
+  context 'without integration' do
     let(:dumper) { instance_double(described_class::Dumper) }
     let(:loader) { instance_double(described_class::Loader) }
 
@@ -16,27 +15,27 @@ RSpec.describe Nenv::Environment do
       allow(described_class::Loader).to receive(:new).and_return(loader)
     end
 
-    context "with no namespace" do
+    context 'with no namespace' do
       let(:instance) { described_class.new }
 
-      context "with an existing method" do
+      context 'with an existing method' do
         before do
           subject.create_method(:foo?)
         end
 
-        it "uses the name as full key" do
-          expect(ENV).to receive(:[]).with("FOO").and_return("true")
-          expect(loader).to receive(:load).with("true").and_return(true)
+        it 'uses the name as full key' do
+          expect(ENV).to receive(:[]).with('FOO').and_return('true')
+          expect(loader).to receive(:load).with('true').and_return(true)
           expect(subject.foo?).to eq(true)
         end
       end
     end
 
-    context "with any namespace" do
-      let(:namespace) { "bar" }
+    context 'with any namespace' do
+      let(:namespace) { 'bar' }
       let(:instance) { described_class.new(namespace) }
 
-      describe "creating a method" do
+      describe 'creating a method' do
         subject { instance }
 
         before do
@@ -45,9 +44,9 @@ RSpec.describe Nenv::Environment do
 
         it { is_expected.to respond_to(:foo) }
 
-        context "when the method already exists" do
+        context 'when the method already exists' do
           let(:error) { described_class::AlreadyExistsError }
-          let(:message) { "Method :foo already exists" }
+          let(:message) { 'Method :foo already exists' }
           specify do
             expect do
               subject.create_method(:foo)
@@ -56,53 +55,52 @@ RSpec.describe Nenv::Environment do
         end
       end
 
-      describe "calling" do
+      describe 'calling' do
         subject { instance }
 
-        context "when method does not exist" do
+        context 'when method does not exist' do
           let(:error) { NoMethodError }
           let(:message) { /undefined method `foo' for/ }
           it { expect { subject.foo }.to raise_error(error, message) }
         end
 
-        context "with a reader method" do
-
-          context "with no block" do
+        context 'with a reader method' do
+          context 'with no block' do
             before { instance.create_method(meth) }
 
-            context "with a normal method" do
+            context 'with a normal method' do
               let(:meth) { :foo }
               before do
-                allow(loader).to receive(:load).with("123").and_return(123)
+                allow(loader).to receive(:load).with('123').and_return(123)
               end
 
-              it "returns unmarshalled stored value" do
-                expect(ENV).to receive(:[]).with("BAR_FOO").and_return("123")
+              it 'returns unmarshalled stored value' do
+                expect(ENV).to receive(:[]).with('BAR_FOO').and_return('123')
                 expect(subject.foo).to eq 123
               end
             end
 
-            context "with a bool method" do
+            context 'with a bool method' do
               let(:meth) { :foo? }
 
-              it "references the proper ENV variable" do
-                allow(loader).to receive(:load).with("false").and_return(false)
-                expect(ENV).to receive(:[]).with("BAR_FOO").and_return("false")
+              it 'references the proper ENV variable' do
+                allow(loader).to receive(:load).with('false').and_return(false)
+                expect(ENV).to receive(:[]).with('BAR_FOO').and_return('false')
                 expect(subject.foo?).to eq false
               end
             end
           end
 
-          context "with a block" do
+          context 'with a block' do
             before do
-              instance.create_method(:foo) { |data| YAML::load(data) }
+              instance.create_method(:foo) { |data| YAML.load(data) }
             end
 
             let(:value) { "---\n:foo: 5\n" }
 
-            it "unmarshals using the block" do
-              allow(ENV).to receive(:[]).with("BAR_FOO").
-                and_return(value)
+            it 'unmarshals using the block' do
+              allow(ENV).to receive(:[]).with('BAR_FOO')
+                .and_return(value)
 
               allow(loader).to receive(:load).with(value) do |arg, &block|
                 expect(block).to be
@@ -112,43 +110,42 @@ RSpec.describe Nenv::Environment do
               expect(subject.foo).to eq(foo: 5)
             end
           end
-
         end
 
-        context "with a writer method" do
+        context 'with a writer method' do
           before { instance.create_method(:foo=) }
 
-          it "set the environment variable" do
-            expect(ENV).to receive(:[]=).with("BAR_FOO", "123")
-            allow(dumper).to receive(:dump).with(123).and_return("123")
+          it 'set the environment variable' do
+            expect(ENV).to receive(:[]=).with('BAR_FOO', '123')
+            allow(dumper).to receive(:dump).with(123).and_return('123')
             subject.foo = 123
           end
 
-          it "marshals and stores the value" do
-            expect(ENV).to receive(:[]=).with("BAR_FOO", "123")
-            allow(dumper).to receive(:dump).with(123).and_return("123")
+          it 'marshals and stores the value' do
+            expect(ENV).to receive(:[]=).with('BAR_FOO', '123')
+            allow(dumper).to receive(:dump).with(123).and_return('123')
             subject.foo = 123
           end
         end
 
-        context "with a method containing underscores" do
+        context 'with a method containing underscores' do
           before { instance.create_method(:foo_baz) }
-          it "reads the correct variable" do
-            expect(ENV).to receive(:[]).with("BAR_FOO_BAZ").and_return("123")
-            allow(loader).to receive(:load).with("123").and_return(123)
+          it 'reads the correct variable' do
+            expect(ENV).to receive(:[]).with('BAR_FOO_BAZ').and_return('123')
+            allow(loader).to receive(:load).with('123').and_return(123)
             subject.foo_baz
           end
         end
 
-        context "with a block" do
+        context 'with a block' do
           before do
-            instance.create_method(:foo=) { |data| YAML::dump(data) }
+            instance.create_method(:foo=) { |data| YAML.dump(data) }
           end
 
           let(:result) { "---\n:foo: 5\n" }
 
-          it "marshals using the block" do
-            allow(ENV).to receive(:[]=).with("BAR_FOO", result)
+          it 'marshals using the block' do
+            allow(ENV).to receive(:[]=).with('BAR_FOO', result)
 
             allow(dumper).to receive(:dump).with(foo: 5) do |arg, &block|
               expect(block).to be
@@ -159,60 +156,60 @@ RSpec.describe Nenv::Environment do
           end
         end
 
-        context "with an unsanitized name" do
+        context 'with an unsanitized name' do
           pending
         end
       end
     end
   end
 
-  describe "with integration" do
-    context "with any namespace" do
-      let(:namespace) { "baz" }
+  describe 'with integration' do
+    context 'with any namespace' do
+      let(:namespace) { 'baz' }
       let(:instance) { described_class.new(namespace) }
       subject { instance }
 
-      context "with a reader method" do
-        context "with no block" do
+      context 'with a reader method' do
+        context 'with no block' do
           before { instance.create_method(:foo) }
 
-          it "returns the stored value" do
-            allow(ENV).to receive(:[]).with("BAZ_FOO").and_return("123")
-            expect(subject.foo).to eq "123"
+          it 'returns the stored value' do
+            allow(ENV).to receive(:[]).with('BAZ_FOO').and_return('123')
+            expect(subject.foo).to eq '123'
           end
         end
 
-        context "with a block" do
+        context 'with a block' do
           before do
-            instance.create_method(:foo) { |data| YAML::load(data) }
+            instance.create_method(:foo) { |data| YAML.load(data) }
           end
 
-          it "unmarshals the value" do
-            expect(ENV).to receive(:[]).with("BAZ_FOO").
-              and_return("---\n:foo: 5\n")
+          it 'unmarshals the value' do
+            expect(ENV).to receive(:[]).with('BAZ_FOO')
+              .and_return("---\n:foo: 5\n")
 
             expect(subject.foo).to eq(foo: 5)
           end
         end
       end
 
-      context "with a writer method" do
-        context "with no block" do
+      context 'with a writer method' do
+        context 'with no block' do
           before { instance.create_method(:foo=) }
 
-          it "marshals and stores the value" do
-            expect(ENV).to receive(:[]=).with("BAZ_FOO", "123")
+          it 'marshals and stores the value' do
+            expect(ENV).to receive(:[]=).with('BAZ_FOO', '123')
             subject.foo = 123
           end
         end
 
-        context "with a block" do
+        context 'with a block' do
           before do
-            instance.create_method(:foo=) { |data| YAML::dump(data) }
+            instance.create_method(:foo=) { |data| YAML.dump(data) }
           end
 
-          it "nmarshals the value" do
-            expect(ENV).to receive(:[]=).with("BAZ_FOO", "---\n:foo: 5\n")
+          it 'nmarshals the value' do
+            expect(ENV).to receive(:[]=).with('BAZ_FOO', "---\n:foo: 5\n")
 
             subject.foo = { foo: 5 }
           end

data/spec/lib/nenv_spec.rb

@@ -1,51 +1,51 @@
-require "nenv"
+require 'nenv'
 
 RSpec.describe Nenv do
   let(:env) { instance_double(Hash) } # Hash is close enough
-  before { stub_const("ENV", env) }
+  before { stub_const('ENV', env) }
 
-  describe "Nenv() helper method" do
-    it "reads from env" do
+  describe 'Nenv() helper method' do
+    it 'reads from env' do
       expect(ENV).to receive(:[]).with('GIT_BROWSER').and_return('chrome')
       Nenv('git').browser
     end
 
-    it "return the value from env" do
+    it 'return the value from env' do
       allow(ENV).to receive(:[]).with('GIT_BROWSER').and_return('firefox')
       expect(Nenv('git').browser).to eq('firefox')
     end
   end
 
-  describe "Nenv() module" do
-    it "reads from env" do
+  describe 'Nenv() module' do
+    it 'reads from env' do
       expect(ENV).to receive(:[]).with('CI').and_return('true')
       Nenv.ci?
     end
 
-    it "return the value from env" do
+    it 'return the value from env' do
       allow(ENV).to receive(:[]).with('CI').and_return('false')
       expect(Nenv.ci?).to be(false)
     end
 
-    context "with no method" do
-      it "automatically creates the method" do
+    context 'with no method' do
+      it 'automatically creates the method' do
         expect(ENV).to receive(:[]).with('FOO').and_return('true')
         Nenv.foo?
       end
     end
 
-    context "with existing method" do
+    context 'with existing method' do
       before do
         Nenv.instance.create_method(:foo?)
       end
 
-      it "reads from env" do
-        expect(ENV).to receive(:[]).with('FOO').and_return("true")
+      it 'reads from env' do
+        expect(ENV).to receive(:[]).with('FOO').and_return('true')
         Nenv.foo?
       end
 
-      it "return the value from env" do
-        expect(ENV).to receive(:[]).with('FOO').and_return("true")
+      it 'return the value from env' do
+        expect(ENV).to receive(:[]).with('FOO').and_return('true')
         expect(Nenv.foo?).to be(true)
       end
     end

data/spec/spec_helper.rb

@@ -1,3 +1,6 @@
+require 'coveralls'
+Coveralls.wear!
+
 RSpec.configure do |config|
   config.expect_with :rspec do |expectations|
     expectations.include_chain_clauses_in_custom_matcher_descriptions = true
@@ -14,9 +17,7 @@ RSpec.configure do |config|
 
   # config.warnings = true
 
-  if config.files_to_run.one?
-    config.default_formatter = 'doc'
-  end
+  config.default_formatter = 'doc' if config.files_to_run.one?
 
   # config.profile_examples = 10
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: nenv
 version: !ruby/object:Gem::Version
-  version: 0.0.2
+  version: 0.0.3
 platform: ruby
 authors:
 - Cezary Baginski
@@ -62,6 +62,9 @@ extra_rdoc_files: []
 files:
 - ".gitignore"
 - ".rspec"
+- ".rubocop.yml"
+- ".rubocop_todo.yml"
+- ".travis.yml"
 - Gemfile
 - Guardfile
 - LICENSE.txt
@@ -79,7 +82,7 @@ files:
 - spec/lib/nenv/environment_spec.rb
 - spec/lib/nenv_spec.rb
 - spec/spec_helper.rb
-homepage: ''
+homepage: https://github.com/e2/nenv
 licenses:
 - MIT
 metadata: {}