acfs 1.0.0.dev.1.b305 → 1.0.0

data/spec/acfs/configuration_spec.rb

@@ -1,7 +1,6 @@
 require 'spec_helper'
 
 describe Acfs::Configuration do
-
   let(:cfg) { Acfs::Configuration.new }
   before { @configuration = Acfs::Configuration.current.dup }
   after { Acfs::Configuration.set @configuration }
@@ -40,4 +39,13 @@ describe Acfs::Configuration do
       end
     end
   end
+
+  describe '#adapter' do
+    let(:object) { Object.new }
+
+    it 'should be a accessor' do
+      cfg.adapter = object
+      expect(cfg.adapter).to eq object
+    end
+  end
 end

data/spec/acfs/global_spec.rb

@@ -14,11 +14,11 @@ describe ::Acfs::Global do
   let(:adapter) { ::NullAdapter.new }
   let(:runner) { double 'runner' }
   let(:collector) { NotificationCollector.new }
-  let(:acfs) { Object.new.tap { |o| o.extend ::Acfs::Global } }
+  let(:acfs) { Object.new.tap {|o| o.extend ::Acfs::Global } }
 
   describe 'instrumentation' do
     before do
-      #allow(runner).to receive(:start)
+      # allow(runner).to receive(:start)
       allow(acfs).to receive(:runner).and_return runner
     end
 
@@ -48,7 +48,7 @@ describe ::Acfs::Global do
       stub_request(:get, %r{http://users.example.org/users/\d+}).to_return(
         status: 200,
         body: '{}',
-        headers: { 'Content-Type' => 'application/json' })
+        headers: {'Content-Type' => 'application/json'})
     end
 
     it 'should invoke when both resources' do
@@ -72,4 +72,22 @@ describe ::Acfs::Global do
       Acfs.run
     end
   end
+
+  describe '#runner' do
+    it 'returns per-thread runner' do
+      runner1 = Thread.new { acfs.runner } .value
+      runner2 = Thread.new { acfs.runner } .value
+
+      expect(runner1).to_not equal runner2
+    end
+
+    it 'uses configurated adapter' do
+      adapter = double :adapter
+      expect(Acfs::Configuration.current).to receive(:adapter).and_return(-> { adapter })
+
+      runner = Thread.new { acfs.runner }.value
+
+      expect(runner.adapter).to equal adapter
+    end
+  end
 end

data/spec/acfs/middleware/json_spec.rb

@@ -0,0 +1,63 @@
+require 'spec_helper'
+
+describe Acfs::Middleware::JSON do
+  let(:data)     { [{id: 1, name: 'Anon'}, {id: 2, name: 'John', friends: [1]}] }
+  let(:body)     { '' }
+  let(:headers)  { {} }
+  let(:request)  { Acfs::Request.new 'url', method: 'GET', data: data }
+  let(:response) { Acfs::Response.new request, status: 200, headers: headers, body: body }
+  let(:decoder)  { Acfs::Middleware::JSON.new ->(req) { req } }
+
+  before do
+    decoder.call request
+  end
+
+  describe 'encode' do
+    context 'with not serialized request' do
+      it 'should set Content-Type' do
+        expect(request.headers['Content-Type']).to eq 'application/json'
+      end
+
+      it 'should append Accept header' do
+        expect(request.headers['Accept']).to eq 'application/json;q=1'
+      end
+
+      it 'should serialize data to JSON' do
+        expect(JSON.parse(request.body)).to eq data.map(&:stringify_keys)
+      end
+    end
+  end
+
+  describe 'decode' do
+    context 'with JSON response' do
+      let(:headers) { {'Content-Type' => 'application/json; charset=utf-8'} }
+      let(:body)    { data.to_json }
+
+      it 'should decode body data' do
+        request.complete! response
+
+        expect(response.data).to be == data.map(&:stringify_keys)
+      end
+    end
+
+    context 'with invalid JSON response' do
+      let(:headers) { {'Content-Type' => 'application/json'} }
+      let(:body)    { data.to_json[4..-4] }
+
+      it 'should raise an error' do
+        expect { request.complete! response }.to raise_error(MultiJson::LoadError)
+      end
+    end
+
+    context 'without JSON response' do
+      let(:headers) { {'Content-Type' => 'application/text'} }
+      let(:body)    { data.to_json }
+
+      it 'should not decode non-JSON encoded responses' do
+        request.complete! response
+
+        expect(response.data).to be_nil
+      end
+    end
+  end
+end

data/spec/acfs/middleware/msgpack_spec.rb

@@ -0,0 +1,60 @@
+require 'spec_helper'
+
+describe Acfs::Middleware::MessagePack do
+  let(:data)     { [{id: 1, name: 'Anon'}, {id: 2, name: 'John', friends: [1]}] }
+  let(:body)     { '' }
+  let(:headers)  { {} }
+  let(:request)  { Acfs::Request.new 'url', data: data }
+  let(:response) { Acfs::Response.new request, status: 200, headers: headers, body: body }
+  let(:decoder)  { Acfs::Middleware::MessagePack.new ->(req) { req } }
+
+  before do
+    decoder.call request
+  end
+
+  describe 'encode' do
+    context 'with not serialized request' do
+      it 'should set Content-Type' do
+        expect(request.headers['Content-Type']).to eq 'application/x-msgpack'
+      end
+
+      it 'should append Accept header' do
+        expect(request.headers['Accept']).to eq 'application/x-msgpack;q=1'
+      end
+
+      context 'with JSON chained' do
+        let(:decoder) { Acfs::Middleware::JSON.new super(), q: 0.5 }
+
+        it 'should append to Accept header' do
+          expect(request.headers['Accept']).to eq 'application/json;q=0.5,application/x-msgpack;q=1'
+        end
+      end
+
+      it 'should serialize data to MessagePack' do
+        expect(MessagePack.unpack(request.body)).to eq data.map(&:stringify_keys)
+      end
+    end
+  end
+
+  context 'with Message Pack response' do
+    let(:headers) { {'Content-Type' => 'application/x-msgpack'} }
+    let(:body)    { MessagePack.pack data }
+
+    it 'should decode body data' do
+      request.complete! response
+
+      expect(response.data).to be == data.map(&:stringify_keys)
+    end
+  end
+
+  context 'without Message Pack response' do
+    let(:headers) { {'Content-Type' => 'application/text'} }
+    let(:body)    { data.to_json }
+
+    it 'should not decode non-MessagePack encoded responses' do
+      request.complete! response
+
+      expect(response.data).to be_nil
+    end
+  end
+end

data/spec/acfs/operation_spec.rb

@@ -0,0 +1,10 @@
+require 'spec_helper'
+
+describe ::Acfs::Operation do
+  let(:operation) { described_class.new MyUser, :read, params: {id: 0} }
+
+  context '#request' do
+    subject { operation.request }
+    its(:operation) { should eq operation }
+  end
+end

data/spec/acfs/request/callbacks_spec.rb

@@ -1,20 +1,20 @@
 require 'spec_helper'
 
 describe Acfs::Request::Callbacks do
-  let(:callback) { lambda { |res| } }
+  let(:callback) { ->(_res) {} }
   let(:request)  { Acfs::Request.new('fubar') }
 
   describe '#on_complete' do
     it 'should store a given callback' do
-      request.on_complete &callback
+      request.on_complete(&callback)
 
       expect(request.callbacks).to have(1).item
       expect(request.callbacks[0]).to be == callback
     end
 
     it 'should store multiple callback' do
-      request.on_complete { |res| "abc" }
-      request.on_complete &callback
+      request.on_complete {|_res| 'abc' }
+      request.on_complete(&callback)
 
       expect(request.callbacks).to have(2).item
       expect(request.callbacks[0]).to be == callback
@@ -27,16 +27,16 @@ describe Acfs::Request::Callbacks do
     it 'should trigger registered callbacks with given response' do
       expect(callback).to receive(:call).with(response, kind_of(Proc))
 
-      request.on_complete &callback
+      request.on_complete(&callback)
       request.complete! response
     end
 
     it 'should trigger multiple callback in reverted insertion order' do
       check = []
 
-      request.on_complete { |res, nxt| check << 1; nxt.call res }
-      request.on_complete { |res, nxt| check << 2; nxt.call res }
-      request.on_complete { |res, nxt| check << 3; nxt.call res }
+      request.on_complete {|res, nxt| check << 1; nxt.call res }
+      request.on_complete {|res, nxt| check << 2; nxt.call res }
+      request.on_complete {|res, nxt| check << 3; nxt.call res }
 
       request.complete! response
 

data/spec/acfs/request_spec.rb

@@ -15,7 +15,7 @@ describe Acfs::Request do
     end
 
     context 'with parameters' do
-      let(:params) { { id: 10 }}
+      let(:params) { {id: 10} }
 
       it 'should return URL without query' do
         expect(request.url).to be == "#{url}"
@@ -24,7 +24,7 @@ describe Acfs::Request do
   end
 
   describe '#headers' do
-    let(:headers) { { 'Accept' => 'application/json' } }
+    let(:headers) { {'Accept' => 'application/json'} }
 
     it 'should return request headers' do
       expect(request.headers).to be == headers
@@ -46,7 +46,7 @@ describe Acfs::Request do
   end
 
   describe '#params' do
-    let(:params) { { id: 10 }}
+    let(:params) { {id: 10} }
 
     it 'should return request headers' do
       expect(request.params).to be == params
@@ -54,7 +54,7 @@ describe Acfs::Request do
   end
 
   describe '#data' do
-    let(:data) { { id: 10, name: 'Anon' } }
+    let(:data) { {id: 10, name: 'Anon'} }
 
     it 'should return request data' do
       expect(request.data).to be == data
@@ -63,7 +63,7 @@ describe Acfs::Request do
 
   describe '#data' do
     context 'with data' do
-      let(:data) { { id: 10, name: 'Anon' } }
+      let(:data) { {id: 10, name: 'Anon'} }
 
       it { expect(request).to be_data }
     end

data/spec/acfs/resource/attributes/boolean_spec.rb

@@ -3,23 +3,54 @@ require 'spec_helper'
 describe Acfs::Resource::Attributes::Boolean do
   subject { Acfs::Resource::Attributes::Boolean.new }
 
-  describe 'cast' do
-    it 'should preserve boolean values' do
+  describe '#cast' do
+    it 'casts nil' do
+      expect(subject.cast(nil)).to eq nil
+    end
+
+    it 'casts empty string to false' do
+      expect(subject.cast('')).to eq nil
+    end
+
+    it 'casts blank string to false' do
+      expect(subject.cast("  \t")).to eq nil
+    end
+
+    it 'preserves boolean values' do
       expect(subject.cast(false)).to eq false
       expect(subject.cast(true)).to eq true
     end
 
-    it 'should cast TRUE_VALUES to true' do
+    it 'casts falsy values to false' do
+      expect(subject.cast(false)).to eq false
+      expect(subject.cast(0)).to eq false
+      expect(subject.cast('0')).to eq false
+      expect(subject.cast('no')).to eq false
+      expect(subject.cast('NO')).to eq false
+      expect(subject.cast('off')).to eq false
+      expect(subject.cast('OFF')).to eq false
+      expect(subject.cast('false')).to eq false
+      expect(subject.cast('FALSE')).to eq false
+      expect(subject.cast('f')).to eq false
+      expect(subject.cast('F')).to eq false
+    end
+
+    it 'casts any other value to true' do
+      expect(subject.cast(true)).to eq true
+      expect(subject.cast(1)).to eq true
+      expect(subject.cast('1')).to eq true
       expect(subject.cast('yes')).to eq true
+      expect(subject.cast('YES')).to eq true
       expect(subject.cast('on')).to eq true
+      expect(subject.cast('ON')).to eq true
       expect(subject.cast('true')).to eq true
-      expect(subject.cast('1')).to eq true
-    end
+      expect(subject.cast('TRUE')).to eq true
+      expect(subject.cast('t')).to eq true
+      expect(subject.cast('T')).to eq true
 
-    it 'should cast any other value to false' do
-      expect(subject.cast('')).to eq false
-      expect(subject.cast('wrong')).to eq false
-      expect(subject.cast('random')).to eq false
+      expect(subject.cast(2)).to eq true
+      expect(subject.cast('wrong')).to eq true
+      expect(subject.cast('random')).to eq true
     end
   end
 end

data/spec/acfs/resource/attributes/date_time_spec.rb

@@ -1,55 +1,49 @@
 require 'spec_helper'
 
 describe Acfs::Resource::Attributes::DateTime do
-  let(:model) { Class.new Acfs::Resource }
-  let(:params) { {} }
-  subject { Acfs::Resource::Attributes::DateTime.new params }
+  let(:type) { Acfs::Resource::Attributes::DateTime.new }
 
-  describe 'cast' do
-    it 'should return same object, if obj is already of DateTime class' do
-      date_time = DateTime.now
-      retval = subject.cast(date_time)
-      expect(retval).to be == date_time
+  describe '#cast' do
+    subject { -> { type.cast value } }
+
+    context 'with nil' do
+      let(:value) { nil }
+      it { expect(subject.call).to eq nil }
     end
 
-    it 'should return parsed object, if obj is of Time class' do
-      time = Time.now
-      retval = subject.cast(time)
-      expect(retval).to be == DateTime.iso8601(time.iso8601)
+    context 'with empty string' do
+      let(:value) { '' }
+      it { expect(subject.call).to eq nil }
     end
 
-    it 'should return parsed object, if obj is of Date class' do
-      date = Date.today
-      retval = subject.cast(date)
-      expect(retval).to be == DateTime.iso8601(date.iso8601)
+    context 'with blank string' do
+      let(:value) { "  \t" }
+      it { expect(subject.call).to eq nil }
     end
 
-    it 'should return parsed object, if obj is of String class in ISO-8601 format' do
-      date_time_string = DateTime.now.iso8601
-      retval = subject.cast(date_time_string)
-      expect(retval).to be == DateTime.iso8601(date_time_string)
+    context 'with DateTime' do
+      let(:value) { DateTime.now }
+      it { expect(subject.call).to eq value }
     end
 
-    it 'should raise an error if obj is of String class not in valid ISO-8601 format' do
-      malformed_string = 'qwe123'
-      expect {
-        subject.cast(malformed_string)
-      }.to raise_error ArgumentError
+    context 'with Time' do
+      let(:value) { Time.now }
+      it { expect(subject.call).to eq value.to_datetime }
     end
 
-    it 'should raise an error if obj is of wrong class (Fixnum)' do
-      fixnum = 12
-      expect {
-        subject.cast(fixnum)
-      }.to raise_error TypeError
+    context 'with Date' do
+      let(:value) { Date.today }
+      it { expect(subject.call).to eq value.to_datetime }
     end
 
-    context 'with allow_nil option' do
-      let(:params) { {allow_nil: true} }
+    context 'with ISO8601' do
+      let(:value) { DateTime.now.iso8601 }
+      it { expect(subject.call.iso8601).to eq value }
+    end
 
-      it 'should accept empty string as nil' do
-        expect(subject.cast('')).to eq nil
-      end
+    context 'with invalid string' do
+      let(:value) { 'qwe123' }
+      it { is_expected.to raise_error ArgumentError }
     end
   end
 end

data/spec/acfs/resource/attributes/dict_spec.rb

@@ -0,0 +1,75 @@
+require 'spec_helper'
+
+describe Acfs::Resource::Attributes::Dict do
+  let(:type) { Acfs::Resource::Attributes::Dict.new }
+
+  describe '#cast' do
+    subject { -> { type.cast value } }
+
+    context 'with nil' do
+      let(:value) { nil }
+      it { expect(subject.call).to eq nil }
+    end
+
+    context 'with blank string (I)' do
+      let(:value) { '' }
+      it { expect(subject.call).to eq Hash.new }
+    end
+
+    context 'with blank string (II)' do
+      let(:value) { "  \t" }
+      it { expect(subject.call).to eq Hash.new }
+    end
+
+    context 'with hash' do
+      let(:value) { {3 => true, abc: 4} }
+      it { expect(subject.call).to eq value }
+    end
+
+    context 'with non hashable object' do
+      let(:value) { Object.new }
+      it { is_expected.to raise_error TypeError }
+    end
+
+    context 'with hashable object (I)' do
+      let(:value) do
+        Class.new do
+          def to_hash
+            {id: object_id}
+          end
+        end.new
+      end
+
+      it { expect(subject.call).to eq id: value.object_id }
+    end
+
+    context 'with hashable object (II)' do
+      let(:value) do
+        Class.new do
+          def to_h
+            {id: object_id}
+          end
+        end.new
+      end
+
+      it { expect(subject.call).to eq id: value.object_id }
+    end
+
+    context 'with serializable object' do
+      let(:value) do
+        Class.new do
+          def serializable_hash
+            {id: object_id}
+          end
+        end.new
+      end
+
+      it { expect(subject.call).to eq id: value.object_id }
+    end
+
+    context 'with hash subclass object' do
+      let(:value) { HashWithIndifferentAccess.new test: :foo }
+      it { expect(subject.call).to be value }
+    end
+  end
+end