nenv 0.0.3 → 0.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: bf78f1e4b9c0b4b0836dcb88c546d8d917506442
-  data.tar.gz: e18492bf894e4951f9e57bee5c343e507cbf7c06
+  metadata.gz: d917265f8d1d8e39066ac71d48d6b3f5893b1d55
+  data.tar.gz: 97282240ee65114e25585c6c69d2049c10de1b65
 SHA512:
-  metadata.gz: 2370b04d5e91d4817da56f343c36f0e94ababa43073316695c4f3e0f9365073f3031aca40956a4866d0e4b7a8ff6e790a6d3c9dee0596cabbbd56a820e1aa746
-  data.tar.gz: 23245f176a801613181ab0443aad7c1770d4cf879e83ba2cdbfdb392e0abf37a8a43013952a42d686a47de4c6f7cf57d8f71e9f0ab5a8ec2fdaa0ea1df2c608a
+  metadata.gz: aa347c7f19065e60988782362dff2b8b219f48fd5bf0baba29851f6f73b80c1f66fb42ea3f38a90d8afc41f084790367a36f0b19fde2bfd35b47ef35ad2bdde8
+  data.tar.gz: 921f77d78cb85ecaca8d2c4e073e8033bb24385b11ef00f936ea978c71489a83a88a597321d46c332089e7046303ba27abfdcb22fc4eacf3343625a3d59d35d0

data/Gemfile

@@ -8,4 +8,5 @@ gem 'coveralls', require: false
 group :development do
   gem 'guard-rspec', '~> 4.5', require: false
   gem 'guard-rubocop', require: false
+  gem 'pry'
 end

data/README.md

@@ -71,9 +71,9 @@ Nenv.debug = true
 
 ```ruby
 # Without Nenv
-puts ENV['GIT_BROWSER`]
-puts ENV['GIT_PAGER`]
-puts ENV['GIT_EDITOR`]
+puts ENV['GIT_BROWSER']
+puts ENV['GIT_PAGER']
+puts ENV['GIT_EDITOR']
 ```
 
 now becomes:
@@ -186,17 +186,29 @@ Nenv.path += Pathname.pwd + "foo"
 
 ```
 
-### Your own class
+### Your own class (recommended version for simpler unit tests)
+
+```ruby
+MyEnv = Nenv::Builder.build do
+  create_method(:foo?)
+end
+
+MyEnv.new('my').foo? # same as ENV['MY_FOO'][/^(?:false|no|n|0)/i,1].nil?
+
+```
+
+
+### Your own class (dynamic version - not recommended because harder to test)
 
 ```ruby
 class MyEnv < Nenv::Environment
   def initialize
-    super("my_env")
+    super("my")
     create_method(:foo?)
   end
 end
 
-MyEnv.new.foo? # same as ENV['MY_ENV_FOO'][/^(?:false|no|n|0)/i,1].nil?
+MyEnv.new.foo? # same as ENV['MY_FOO'][/^(?:false|no|n|0)/i,1].nil?
 
 ```
 

data/lib/nenv.rb

@@ -1,6 +1,7 @@
 require 'nenv/version'
 
 require 'nenv/autoenvironment'
+require 'nenv/builder'
 
 def Nenv(namespace = nil)
   Nenv::AutoEnvironment.new(namespace)

data/lib/nenv/builder.rb

@@ -0,0 +1,9 @@
+require 'nenv/environment'
+
+module Nenv
+  module Builder
+    def self.build(&block)
+      Class.new(Nenv::Environment, &block)
+    end
+  end
+end

data/lib/nenv/environment.rb

@@ -22,26 +22,37 @@ module Nenv
       @namespace = (namespace ? namespace.upcase : nil)
     end
 
+    def self.create_method(meth, &block)
+      _create_env_method(self, meth, &block)
+    end
+
     def create_method(meth, &block)
-      fail(AlreadyExistsError, meth) if respond_to?(meth)
+      self.class._create_env_method(class << self; self; end, meth, &block)
+    end
+
+    private
+
+    def _sanitize(meth)
+      meth.to_s[/^([^=?]*)[=?]?$/, 1].upcase
+    end
 
-      (class << self; self; end).send(:define_method, meth) do |*args|
-        raw_value = args.first
+    def self._create_env_method(instance, meth, &block)
+      _fail_if_exists(instance, meth)
+
+      instance.send(:define_method, meth) do |*args|
         env_name = [@namespace, _sanitize(meth)].compact.join('_')
 
-        callback = block
         if args.size == 1
-          ENV[env_name] = Dumper.new.dump(raw_value, &callback)
+          raw_value = args.first
+          ENV[env_name] = Dumper.new.dump(raw_value, &block)
         else
-          Loader.new(meth).load(ENV[env_name], &callback)
+          Loader.new(meth).load(ENV[env_name], &block)
         end
       end
     end
 
-    private
-
-    def _sanitize(meth)
-      meth.to_s[/^([^=?]*)[=?]?$/, 1].upcase
+    def self._fail_if_exists(instance, meth)
+      fail(AlreadyExistsError, meth) if instance.instance_methods.include?(meth)
     end
   end
 end

data/lib/nenv/version.rb

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

data/spec/lib/nenv/builder_spec.rb

@@ -0,0 +1,27 @@
+require 'nenv/builder'
+
+RSpec.describe Nenv::Builder do
+  describe '#build' do
+    before do
+      allow(ENV).to receive(:[]).with('FOO')
+    end
+
+    it 'returns a class with the given methods' do
+      FooEnv = Nenv::Builder.build do
+        create_method(:foo?)
+      end
+      FooEnv.new.foo?
+    end
+
+    context 'with duplicate methods' do
+      it 'fails' do
+        expect do
+          FooEnv = Nenv::Builder.build do
+            create_method(:foo?)
+            create_method(:foo?)
+          end
+        end.to raise_error(Nenv::Environment::AlreadyExistsError)
+      end
+    end
+  end
+end

data/spec/lib/nenv_spec.rb

@@ -16,7 +16,7 @@ RSpec.describe Nenv do
     end
   end
 
-  describe 'Nenv() module' do
+  describe 'Nenv module' do
     it 'reads from env' do
       expect(ENV).to receive(:[]).with('CI').and_return('true')
       Nenv.ci?
@@ -50,4 +50,17 @@ RSpec.describe Nenv do
       end
     end
   end
+
+  # Test added here to properly test if builder is required
+  describe 'Nenv builder' do
+    before do
+      allow(ENV).to receive(:[]).with('FOO').and_return('false')
+    end
+    it 'is required and works' do
+      FooEnv = Nenv::Builder.build do
+        create_method(:foo?)
+      end
+      FooEnv.new.foo?
+    end
+  end
 end

data/spec/spec_helper.rb

@@ -10,7 +10,7 @@ RSpec.configure do |config|
     mocks.verify_partial_doubles = true
   end
 
-  config.filter_run :focus
+  config.filter_run focus: ENV['CI'] != 'true'
   config.run_all_when_everything_filtered = true
 
   config.disable_monkey_patching!
@@ -33,6 +33,11 @@ RSpec.configure do |config|
     allow(ENV).to receive(:[]=) do |key, value|
       fail "stub me: ENV[#{key.inspect}] = #{value.inspect}"
     end
+
+    allow(ENV).to receive(:[]).with('PRYRC').and_call_original
+    allow(ENV).to receive(:[]).with('DISABLE_PRY').and_call_original
+    allow(ENV).to receive(:[]).with('ANSICON').and_call_original
+    allow(ENV).to receive(:[]).with('TERM').and_call_original
   end
 
   config.after do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: nenv
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.1.0
 platform: ruby
 authors:
 - Cezary Baginski
@@ -72,11 +72,13 @@ files:
 - Rakefile
 - lib/nenv.rb
 - lib/nenv/autoenvironment.rb
+- lib/nenv/builder.rb
 - lib/nenv/environment.rb
 - lib/nenv/environment/dumper.rb
 - lib/nenv/environment/loader.rb
 - lib/nenv/version.rb
 - nenv.gemspec
+- spec/lib/nenv/builder_spec.rb
 - spec/lib/nenv/environment/dumper_spec.rb
 - spec/lib/nenv/environment/loader_spec.rb
 - spec/lib/nenv/environment_spec.rb
@@ -107,6 +109,7 @@ signing_key:
 specification_version: 4
 summary: Convenience wrapper for Ruby's ENV
 test_files:
+- spec/lib/nenv/builder_spec.rb
 - spec/lib/nenv/environment/dumper_spec.rb
 - spec/lib/nenv/environment/loader_spec.rb
 - spec/lib/nenv/environment_spec.rb