lhs 24.0.0

data/spec/record/first_spec.rb

@@ -0,0 +1,39 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+describe LHS::Record do
+  let(:datastore) { 'http://local.ch/v2' }
+
+  before do
+    LHC.config.placeholder(:datastore, datastore)
+    class Record < LHS::Record
+      endpoint '{+datastore}/content-ads/{campaign_id}/feedbacks'
+      endpoint '{+datastore}/feedbacks'
+    end
+  end
+
+  context 'first' do
+    it 'finds a single record' do
+      stub_request(:get, "#{datastore}/feedbacks?limit=1")
+        .to_return(status: 200, body: load_json(:feedback))
+      record = Record.first
+      expect(record).to be_kind_of Record
+      expect(record.source_id).to be_present
+    end
+
+    it 'returns nil if no record was found' do
+      stub_request(:get, "#{datastore}/feedbacks?limit=1")
+        .to_return(status: 404)
+      expect(Record.first).to be_nil
+    end
+  end
+
+  context 'first!' do
+    it 'raises if nothing was found with parameters' do
+      stub_request(:get, "#{datastore}/feedbacks?limit=1")
+        .to_return(status: 404)
+      expect { Record.first! }.to raise_error LHC::NotFound
+    end
+  end
+end

data/spec/record/force_merge_spec.rb

@@ -0,0 +1,56 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+describe LHS::Record do
+  context 'merge' do
+
+    before do
+      class Change < LHS::Record
+        endpoint 'https://onboarding/places/{id}/change'
+      end
+    end
+
+    it 'merges a given hash' do
+      stub_request(:get, "https://onboarding/places/1/change")
+        .to_return(body: { entry: { name: 'Steve', address: 'Zurich' }, products: ['LBP'] }.to_json)
+      record = Change.find(1)
+      record.merge!(entry: { name: 'Paul' })
+      expect(record.entry.name).to eq 'Paul'
+      expect(record.entry.address).to eq nil
+      expect(record.products.to_a).to eq ['LBP']
+    end
+
+    it 'merges! a given hash' do
+      stub_request(:get, "https://onboarding/places/1/change")
+        .to_return(body: { entry: { name: 'Steve', address: 'Zurich' }, products: ['LBP'] }.to_json)
+      record = Change.find(1)
+      new_record = record.merge(entry: { name: 'Paul' })
+      expect(new_record.entry.name).to eq 'Paul'
+      expect(new_record.entry.address).to eq nil
+      expect(new_record.products.to_a).to eq ['LBP']
+      expect(record.entry.name).to eq 'Steve'
+      expect(record.entry.address).to eq 'Zurich'
+      expect(record.products.to_a).to eq ['LBP']
+    end
+
+    it 'deep_merge! a given hash' do
+      stub_request(:get, "https://onboarding/places/1/change")
+        .to_return(body: { entry: { name: 'Steve', address: 'Zurich' } }.to_json)
+      record = Change.find(1)
+      record.deep_merge!(entry: { name: 'Paul' })
+      expect(record.entry.name).to eq 'Paul'
+      expect(record.entry.address).to eq 'Zurich'
+    end
+
+    it 'deep_merge a given hash' do
+      stub_request(:get, "https://onboarding/places/1/change")
+        .to_return(body: { entry: { name: 'Steve', address: 'Zurich' } }.to_json)
+      record = Change.find(1)
+      new_record = record.deep_merge(entry: { name: 'Paul' })
+      expect(new_record.entry.name).to eq 'Paul'
+      expect(new_record.entry.address).to eq 'Zurich'
+      expect(record.entry.name).to eq 'Steve'
+    end
+  end
+end

data/spec/record/handle_includes_errors_spec.rb

@@ -0,0 +1,32 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+describe LHS::Record do
+  let(:handler) { spy('handler') }
+
+  before do
+    class Record < LHS::Record
+      endpoint 'http://local.ch/v2/records/{id}'
+    end
+    class NestedRecord < LHS::Record
+      endpoint 'http://local.ch/v2/other_records/{id}'
+    end
+    stub_request(:get, "http://local.ch/v2/records/1")
+      .to_return(body: {
+        href: 'http://local.ch/v2/records/1',
+        other: {
+          href: 'http://local.ch/v2/other_records/2'
+        }
+      }.to_json)
+    stub_request(:get, "http://local.ch/v2/other_records/2")
+      .to_return(status: 404)
+  end
+
+  it 'allows to pass error_handling for includes to LHC' do
+    handler = ->(_) { return { deleted: true } }
+    record = Record.includes_first_page(:other).references(other: { rescue: { LHC::NotFound => handler } }).find(id: 1)
+
+    expect(record.other.deleted).to be(true)
+  end
+end

data/spec/record/has_many_spec.rb

@@ -0,0 +1,120 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+describe LHS::Record do
+
+  let(:listing) { location.listings.first }
+
+  before do
+    stub_request(:get, 'http://uberall/locations/1')
+      .to_return(body: {
+        listings: [{
+          directory: { name: 'Instagram' }
+        }]
+      }.to_json)
+  end
+
+  context 'has_many' do
+
+    before do
+      class Location < LHS::Record
+        endpoint 'http://uberall/locations'
+        endpoint 'http://uberall/locations/{id}'
+        has_many :listings
+      end
+
+      class Listing < LHS::Record
+
+        def supported?
+          true
+        end
+      end
+    end
+
+    let(:location) { Location.find(1) }
+
+    it 'casts the relation into the correct type' do
+      expect(listing).to be_kind_of(Listing)
+      expect(listing.supported?).to eq true
+    end
+
+    it 'keeps hirachy when casting it to another class on access' do
+      expect(listing._root._raw).to eq location._raw
+      expect(listing.parent.parent._raw).to eq location._raw
+    end
+  end
+
+  context 'custom class_name' do
+    before do
+      module Uberall
+        class Location < LHS::Record
+          endpoint 'http://uberall/locations'
+          endpoint 'http://uberall/locations/{id}'
+          has_many :listings, class_name: 'Uberall::Listing'
+        end
+      end
+
+      module Uberall
+        class Listing < LHS::Record
+
+          def supported?
+            true
+          end
+        end
+      end
+    end
+
+    let(:location) { Uberall::Location.find(1) }
+
+    it 'casts the relation into the correct type' do
+      expect(listing).to be_kind_of(Uberall::Listing)
+      expect(listing.supported?).to eq true
+    end
+
+    it 'keeps hirachy when casting it to another class on access' do
+      expect(listing._root._raw).to eq location._raw
+      expect(listing.parent.parent._raw).to eq location._raw
+    end
+  end
+
+  context 'explicit association class configuration overrules href class casting' do
+    before do
+      class Place < LHS::Record
+        endpoint 'http://places/places/{id}'
+        has_many :categories, class_name: 'NewCategory'
+      end
+
+      class NewCategory < LHS::Record
+        endpoint 'http://newcategories/newcategories/{id}'
+
+        def name
+          self['category_name']
+        end
+      end
+
+      class Category < LHS::Record
+        endpoint 'http://categories/categories/{id}'
+      end
+
+      stub_request(:get, "http://places/places/1")
+        .to_return(body: {
+          categories: [{
+            href: 'https://categories/categories/1'
+          }]
+        }.to_json)
+
+      stub_request(:get, "https://categories/categories/1")
+        .to_return(body: {
+          href: 'https://categories/categories/1',
+          category_name: 'Pizza'
+        }.to_json)
+    end
+
+    it 'explicit association configuration overrules href class casting' do
+      place = Place.includes_first_page(:categories).find(1)
+      expect(place.categories.first).to be_kind_of NewCategory
+      expect(place.categories.first.name).to eq('Pizza')
+    end
+  end
+end

data/spec/record/has_one_spec.rb

@@ -0,0 +1,116 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+describe LHS::Record do
+
+  let(:transaction) { Transaction.find(1) }
+  let(:user) { transaction.user }
+
+  before do
+    [1, 2].each do |id|
+      stub_request(:get, "http://myservice/transactions/#{id}")
+        .to_return(body: {
+          user: {
+            email_address: 'steve@local.ch',
+            comments: []
+          }
+        }.to_json)
+    end
+  end
+
+  context 'has_one' do
+    before do
+      class Transaction < LHS::Record
+        endpoint 'http://myservice/transactions'
+        endpoint 'http://myservice/transactions/{id}'
+        has_one :user
+      end
+
+      class User < LHS::Record
+        has_many :comments
+
+        def email
+          self[:email_address]
+        end
+      end
+
+      class Comment < LHS::Record
+      end
+    end
+
+    it 'casts the relation into the correct type' do
+      expect(user).to be_kind_of(User)
+      expect(user.email).to eq 'steve@local.ch'
+    end
+
+    it 'keeps hirachy when casting it to another class on access' do
+      expect(user._root._raw).to eq transaction._raw
+      expect(user.parent._raw).to eq transaction._raw
+    end
+  end
+
+  context 'custom class_name' do
+
+    before do
+      class Transaction < LHS::Record
+        endpoint 'http://myservice/transactions'
+        endpoint 'http://myservice/transactions/{id}'
+        has_one :user, class_name: 'Custom::User'
+      end
+
+      module Custom
+        class User < LHS::Record
+          def email
+            self[:email_address]
+          end
+        end
+      end
+    end
+
+    it 'casts the relation into the correct type' do
+      expect(user).to be_kind_of(Custom::User)
+      expect(user.email).to eq 'steve@local.ch'
+    end
+  end
+
+  context 'explicit association class configuration overrules href class casting' do
+    before do
+      class Place < LHS::Record
+        endpoint 'http://places/places/{id}'
+        has_one :category, class_name: 'NewCategory'
+      end
+
+      class NewCategory < LHS::Record
+        endpoint 'http://newcategories/newcategories/{id}'
+
+        def name
+          self['category_name']
+        end
+      end
+
+      class Category < LHS::Record
+        endpoint 'http://categories/categories/{id}'
+      end
+
+      stub_request(:get, "http://places/places/1")
+        .to_return(body: {
+          category: {
+            href: 'https://categories/categories/1'
+          }
+        }.to_json)
+
+      stub_request(:get, "https://categories/categories/1")
+        .to_return(body: {
+          href: 'https://categories/categories/1',
+          category_name: 'Pizza'
+        }.to_json)
+    end
+
+    it 'explicit association configuration overrules href class casting' do
+      place = Place.includes_first_page(:category).find(1)
+      expect(place.category).to be_kind_of NewCategory
+      expect(place.category.name).to eq('Pizza')
+    end
+  end
+end

data/spec/record/href_for_spec.rb

@@ -0,0 +1,25 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+describe LHS::Record do
+  before do
+    class Record < LHS::Record
+      endpoint 'http://datastore/records/'
+      endpoint 'http://datastore/records/{id}'
+    end
+  end
+
+  context 'href_for' do
+
+    it 'injects variables and returns href' do
+      expect(Record.href_for(1)).to eq 'http://datastore/records/1'
+      expect(Record.href_for('vmasd241')).to eq 'http://datastore/records/vmasd241'
+    end
+
+    it 'also works with url_for (alias)' do
+      expect(Record.url_for(1)).to eq 'http://datastore/records/1'
+      expect(Record.url_for('vmasd241')).to eq 'http://datastore/records/vmasd241'
+    end
+  end
+end

data/spec/record/ignore_errors_spec.rb

@@ -0,0 +1,139 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+describe LHS::Record do
+  let(:handler) { spy('handler') }
+  let(:record_json) { { color: 'blue' }.to_json }
+
+  before do
+    class Record < LHS::Record
+      endpoint 'http://local.ch/v2/records'
+      endpoint 'http://local.ch/v2/records/{id}'
+    end
+  end
+
+  context 'ignore errors' do
+    it 'allows to ignore errors' do
+      stub_request(:get, "http://local.ch/v2/records?color=blue").to_return(status: 404)
+      record = Record
+        .where(color: 'blue')
+        .ignore(LHC::NotFound)
+        .fetch
+      expect(record).to eq nil
+    end
+  end
+
+  context 'ignore errors during create' do
+
+    it 'allows to ignore errors during create' do
+      stub_request(:post, 'http://local.ch/v2/records')
+        .to_return(status: 409)
+      record = Record.ignore(LHC::Conflict).create(name: 'Steve')
+      expect(record._raw).to eq(name: 'Steve')
+    end
+  end
+
+  context 'multiple ignored errors' do
+    it 'ignores error if first of them is specified' do
+      stub_request(:get, "http://local.ch/v2/records?color=blue").to_return(status: 401)
+      record = Record
+        .ignore(LHC::Unauthorized)
+        .where(color: 'blue')
+        .ignore(LHC::NotFound)
+        .fetch
+      expect(record).to eq nil
+    end
+
+    it 'ignores error if last of them is specified' do
+      stub_request(:get, "http://local.ch/v2/records?color=blue").to_return(status: 404)
+      record = Record
+        .ignore(LHC::Unauthorized)
+        .where(color: 'blue')
+        .ignore(LHC::NotFound)
+        .fetch
+      expect(record).to eq nil
+    end
+  end
+
+  it 'also can ignore all LHC errors' do
+    stub_request(:get, "http://local.ch/v2/records?color=blue").to_return(status: 401)
+    record = Record
+      .ignore(LHC::Error)
+      .where(color: 'blue')
+      .fetch
+    expect(record).to eq nil
+  end
+
+  it 'can ignore multiple error with one ignore call, on chain start' do
+    stub_request(:get, "http://local.ch/v2/records?color=blue").to_return(status: 401)
+    record = Record
+      .ignore(LHC::Unauthorized, LHC::NotFound)
+      .where(color: 'blue')
+      .fetch
+    expect(record).to eq nil
+  end
+
+  it 'can ignore multiple error with one ignore call, also within the chain' do
+    stub_request(:get, "http://local.ch/v2/records?color=blue").to_return(status: 401)
+    record = Record
+      .where(color: 'blue')
+      .ignore(LHC::Unauthorized, LHC::NotFound)
+      .fetch
+    expect(record).to eq nil
+  end
+
+  it 'returns record when ignoring errors on where' do
+    stub_request(:get, 'http://local.ch/v2/records?color=blue').to_return(status: 200, body: record_json)
+    record = Record
+      .ignore(LHC::Error)
+      .where(color: 'blue')
+    expect(record).not_to eq nil
+  end
+
+  context 'response body' do
+
+    let(:body) { { error_message: 'you are not worthy' }.to_json }
+
+    it 'returns nil also when ignoring errors on find' do
+      stub_request(:get, "http://local.ch/v2/records/1").to_return(status: 500, body: body)
+      record = Record
+        .ignore(LHC::Error)
+        .find(1)
+      expect(record).to eq nil
+    end
+
+    it 'returns nil also when ignoring errors on find with parameters' do
+      stub_request(:get, "http://local.ch/v2/records/1").to_return(status: 500, body: body)
+      record = Record
+        .ignore(LHC::Error)
+        .find(id: 1)
+      expect(record).to eq nil
+    end
+
+    it 'returns nil also when ignoring errors on fetch' do
+      stub_request(:get, "http://local.ch/v2/records?color=blue").to_return(status: 500, body: body)
+      record = Record
+        .ignore(LHC::Error)
+        .where(color: 'blue')
+        .fetch
+      expect(record).to eq nil
+    end
+
+    it 'returns nil also when ignoring errors on find_by' do
+      stub_request(:get, "http://local.ch/v2/records?color=blue&limit=1").to_return(status: 500, body: body)
+      record = Record
+        .ignore(LHC::Error)
+        .find_by(color: 'blue')
+      expect(record).to eq nil
+    end
+
+    it 'returns record when ignoring errors on find' do
+      stub_request(:get, "http://local.ch/v2/records/1").to_return(status: 200, body: record_json)
+      record = Record
+        .ignore(LHC::Error)
+        .find(1)
+      expect(record).not_to eq nil
+    end
+  end
+end