chewy 0.8.4 → 5.1.0

data/spec/chewy/index/settings_spec.rb

@@ -6,77 +6,128 @@ describe Chewy::Index::Settings do
     before { allow(Chewy).to receive_messages(repository: Chewy::Repository.send(:new)) }
 
     specify { expect(described_class.new.to_hash).to eq({}) }
-    specify { expect(described_class.new(number_of_nodes: 3).to_hash).to eq({settings: {number_of_nodes: 3}}) }
-    specify { expect(described_class.new(number_of_nodes: 3, analysis: {}).to_hash)
-      .to eq({settings: {number_of_nodes: 3, analysis: {}}}) }
-    specify { expect(described_class.new(number_of_nodes: 3, analysis: {filter: {filter1: {}}}).to_hash)
-      .to eq({settings: {number_of_nodes: 3, analysis: {filter: {filter1: {}}}}}) }
-    specify { expect(described_class.new(number_of_nodes: 3, analysis: {analyzer: {analyzer1: {}}}).to_hash)
-      .to eq({settings: {number_of_nodes: 3, analysis: {analyzer: {analyzer1: {}}}}}) }
-    specify { expect(described_class.new(number_of_nodes: 3, analysis: {
-      analyzer: {analyzer1: {tokenizer: 'tokenizer1', filter: ['filter1', 'filter2']}}
-    }).to_hash)
-      .to eq({settings: {number_of_nodes: 3, analysis: {
-        analyzer: {analyzer1: {tokenizer: 'tokenizer1', filter: ['filter1', 'filter2']}}
-      }}}) }
-    specify { expect(described_class.new(number_of_nodes: 3, analysis: {analyser: ['analyzer1']}).to_hash)
-      .to eq({settings: {number_of_nodes: 3, analysis: {}}}) }
+    specify { expect(described_class.new(number_of_nodes: 3).to_hash).to eq(settings: {number_of_nodes: 3}) }
+    specify do
+      expect(described_class.new(number_of_nodes: 3, analysis: {}).to_hash)
+        .to eq(settings: {number_of_nodes: 3, analysis: {}})
+    end
+    specify do
+      expect(described_class.new(number_of_nodes: 3, analysis: {filter: {filter1: {}}}).to_hash)
+        .to eq(settings: {number_of_nodes: 3, analysis: {filter: {filter1: {}}}})
+    end
+    specify do
+      expect(described_class.new(number_of_nodes: 3, analysis: {analyzer: {analyzer1: {}}}).to_hash)
+        .to eq(settings: {number_of_nodes: 3, analysis: {analyzer: {analyzer1: {}}}})
+    end
+    specify do
+      expect(described_class.new(number_of_nodes: 3, analysis: {
+        analyzer: {analyzer1: {tokenizer: 'tokenizer1', filter: %w[filter1 filter2]}}
+      }).to_hash)
+        .to eq(settings: {number_of_nodes: 3, analysis: {
+          analyzer: {analyzer1: {tokenizer: 'tokenizer1', filter: %w[filter1 filter2]}}
+        }})
+    end
+    specify do
+      expect(described_class.new(number_of_nodes: 3, analysis: {analyzer: ['analyzer1']}).to_hash)
+        .to eq(settings: {number_of_nodes: 3, analysis: {}})
+    end
+    specify do
+      expect(described_class.new(number_of_nodes: 3, analysis: {analyzer: {analyzer1: {}}, normalizer: {}}).to_hash)
+        .to eq(settings: {number_of_nodes: 3, analysis: {analyzer: {analyzer1: {}}, normalizer: {}}})
+    end
+    specify do
+      expect(described_class.new(number_of_nodes: 3, analysis: {analyzer: ['analyzer1'], normalizer: {}}).to_hash)
+        .to eq(settings: {number_of_nodes: 3, analysis: {normalizer: {}}})
+    end
+
+    specify do
+      stub_const('Synonyms', Class.new do
+        def self.synonyms
+          ['kaftan => dress']
+        end
+      end)
+      expect(
+        described_class.new do
+          {
+            analysis: {filter: {synonym: {type: 'synonym', synonyms: Synonyms.synonyms}}}
+          }
+        end.to_hash
+      ).to eq(settings: {
+        analysis: {filter: {
+          synonym: {
+            type: 'synonym', synonyms: ['kaftan => dress']
+          }
+        }}
+      })
+    end
 
     context do
-      before { Chewy.tokenizer :tokenizer1, {options: 42} }
+      before { Chewy.tokenizer :tokenizer1, options: 42 }
 
-      specify { expect(described_class.new(number_of_nodes: 3, analysis: {
-        analyzer: {analyzer1: {tokenizer: 'tokenizer1', filter: ['filter1', 'filter2']}}
-      }).to_hash)
-        .to eq({settings: {number_of_nodes: 3, analysis: {
-          analyzer: {analyzer1: {tokenizer: 'tokenizer1', filter: ['filter1', 'filter2']}},
-          tokenizer: {tokenizer1: {options: 42}}
-        }}}) }
+      specify do
+        expect(described_class.new(number_of_nodes: 3, analysis: {
+          analyzer: {analyzer1: {tokenizer: 'tokenizer1', filter: %w[filter1 filter2]}}
+        }).to_hash)
+          .to eq(settings: {number_of_nodes: 3, analysis: {
+            analyzer: {analyzer1: {tokenizer: 'tokenizer1', filter: %w[filter1 filter2]}},
+            tokenizer: {tokenizer1: {options: 42}}
+          }})
+      end
     end
 
     context do
       before do
-        Chewy.filter :filter2, {options: 42}
-        Chewy.filter :filter3, {options: 43}
-        Chewy.filter :filter5, {options: 44}
+        Chewy.filter :filter2, options: 42
+        Chewy.filter :filter3, options: 43
+        Chewy.filter :filter5, options: 44
       end
 
-      specify { expect(described_class.new(number_of_nodes: 3, analysis: {
-        analyzer: {analyzer1: {tokenizer: 'tokenizer1', filter: ['filter1', 'filter2']}},
-        filter: ['filter3', {filter4: {options: 45}}]
-      }).to_hash)
-        .to eq({settings: {number_of_nodes: 3, analysis: {
-          analyzer: {analyzer1: {tokenizer: 'tokenizer1', filter: ['filter1', 'filter2']}},
-          filter: {filter2: {options: 42}, filter3: {options: 43}, filter4: {options: 45}}
-        }}}) }
+      specify do
+        expect(described_class.new(number_of_nodes: 3, analysis: {
+          analyzer: {analyzer1: {tokenizer: 'tokenizer1', filter: %w[filter1 filter2]}},
+          filter: ['filter3', {filter4: {options: 45}}]
+        }).to_hash)
+          .to eq(settings: {number_of_nodes: 3, analysis: {
+            analyzer: {analyzer1: {tokenizer: 'tokenizer1', filter: %w[filter1 filter2]}},
+            filter: {filter2: {options: 42}, filter3: {options: 43}, filter4: {options: 45}}
+          }})
+      end
     end
 
     context do
       before do
-        Chewy.analyzer :analyzer1, {options: 42, tokenizer: 'tokenizer1'}
-        Chewy.tokenizer :tokenizer1, {options: 43}
+        Chewy.analyzer :analyzer1, options: 42, tokenizer: 'tokenizer1'
+        Chewy.tokenizer :tokenizer1, options: 43
       end
 
-      specify { expect(described_class.new(number_of_nodes: 3, analysis: {
-        analyzer: ['analyzer1', {analyzer2: {options: 44}}]
-      }).to_hash)
-        .to eq({settings: {number_of_nodes: 3, analysis: {
-          analyzer: {analyzer1: {options: 42, tokenizer: 'tokenizer1'}, analyzer2: {options: 44}},
-          tokenizer: {tokenizer1: {options: 43}}
-        }}}) }
+      specify do
+        expect(described_class.new(number_of_nodes: 3, analysis: {
+          analyzer: ['analyzer1', {analyzer2: {options: 44}}]
+        }).to_hash)
+          .to eq(settings: {number_of_nodes: 3, analysis: {
+            analyzer: {analyzer1: {options: 42, tokenizer: 'tokenizer1'}, analyzer2: {options: 44}},
+            tokenizer: {tokenizer1: {options: 43}}
+          }})
+      end
     end
 
     context ':index' do
-      specify { expect(described_class.new(index: {number_of_shards: 3}).to_hash)
-        .to eq({settings: {index: {number_of_shards: 3}}}) }
+      specify do
+        expect(described_class.new(index: {number_of_shards: 3}).to_hash)
+          .to eq(settings: {index: {number_of_shards: 3}})
+      end
 
       context do
         before { allow(Chewy).to receive_messages(configuration: {index: {number_of_shards: 7, number_of_replicas: 2}}) }
 
-        specify { expect(described_class.new.to_hash)
-          .to eq({settings: {index: {number_of_shards: 7, number_of_replicas: 2}}}) }
-        specify { expect(described_class.new(index: {number_of_shards: 3}).to_hash)
-          .to eq({settings: {index: {number_of_shards: 3, number_of_replicas: 2}}}) }
+        specify do
+          expect(described_class.new.to_hash)
+            .to eq(settings: {index: {number_of_shards: 7, number_of_replicas: 2}})
+        end
+        specify do
+          expect(described_class.new(index: {number_of_shards: 3}).to_hash)
+            .to eq(settings: {index: {number_of_shards: 3, number_of_replicas: 2}})
+        end
       end
     end
   end

data/spec/chewy/index/specification_spec.rb

@@ -0,0 +1,169 @@
+require 'spec_helper'
+
+describe Chewy::Index::Specification do
+  before { Chewy.massacre }
+
+  let(:index1) do
+    stub_index(:places) do
+      define_type(:city) do
+        field :founded_on, type: 'date'
+      end
+    end
+  end
+
+  let(:index2) do
+    stub_index(:places) do
+      settings analyzer: {}
+      define_type(:city) do
+        field :founded_on, type: 'date'
+      end
+    end
+  end
+
+  let(:index3) do
+    stub_index(:places) do
+      define_type(:city) do
+        field :founded_on, type: 'date'
+        field :population, type: 'integer'
+      end
+    end
+  end
+
+  let(:index4) do
+    stub_index(:places) do
+      define_type(:city) do
+        field :population, type: 'integer'
+        field :founded_on, type: 'date'
+      end
+    end
+  end
+
+  let(:index5) do
+    stub_index('namespace/cities') do
+      define_type(:city) do
+        field :population, type: 'integer'
+      end
+    end
+  end
+
+  let(:specification1) { described_class.new(index1) }
+  let(:specification2) { described_class.new(index2) }
+  let(:specification3) { described_class.new(index3) }
+  let(:specification4) { described_class.new(index4) }
+  let(:specification5) { described_class.new(index5) }
+
+  describe '#lock!' do
+    specify do
+      expect { specification1.lock! }.to change { Chewy::Stash::Specification.all.hits }.from([]).to([{
+        '_index' => 'chewy_specifications',
+        '_type' => 'specification',
+        '_id' => 'places',
+        '_score' => 1.0,
+        '_source' => {'specification' => Base64.encode64({
+          'settings' => {'index' => {'number_of_shards' => 1, 'number_of_replicas' => 0}},
+          'mappings' => {'city' => {'properties' => {'founded_on' => {'type' => 'date'}}}}
+        }.to_json)}
+      }])
+    end
+
+    context do
+      before { specification1.lock! }
+
+      specify do
+        expect { specification5.lock! }.to change { Chewy::Stash::Specification.all.hits }.to([{
+          '_index' => 'chewy_specifications',
+          '_type' => 'specification',
+          '_id' => 'places',
+          '_score' => 1.0,
+          '_source' => {'specification' => Base64.encode64({
+            'settings' => {'index' => {'number_of_shards' => 1, 'number_of_replicas' => 0}},
+            'mappings' => {'city' => {'properties' => {'founded_on' => {'type' => 'date'}}}}
+          }.to_json)}
+        }, {
+          '_index' => 'chewy_specifications',
+          '_type' => 'specification',
+          '_id' => 'namespace/cities',
+          '_score' => 1.0,
+          '_source' => {'specification' => Base64.encode64({
+            'settings' => {'index' => {'number_of_shards' => 1, 'number_of_replicas' => 0}},
+            'mappings' => {'city' => {'properties' => {'population' => {'type' => 'integer'}}}}
+          }.to_json)}
+        }])
+      end
+    end
+  end
+
+  describe '#locked' do
+    specify do
+      expect { specification1.lock! }.to change { specification1.locked }.from({}).to(
+        'settings' => {'index' => {'number_of_shards' => 1, 'number_of_replicas' => 0}},
+        'mappings' => {'city' => {'properties' => {'founded_on' => {'type' => 'date'}}}}
+      )
+    end
+
+    specify do
+      expect { specification5.lock! }.to change { specification5.locked }.from({}).to(
+        'settings' => {'index' => {'number_of_shards' => 1, 'number_of_replicas' => 0}},
+        'mappings' => {'city' => {'properties' => {'population' => {'type' => 'integer'}}}}
+      )
+    end
+
+    context do
+      before { specification1.lock! }
+
+      specify do
+        expect { specification2.lock! }.to change { specification2.locked }.from(
+          'settings' => {'index' => {'number_of_shards' => 1, 'number_of_replicas' => 0}},
+          'mappings' => {'city' => {'properties' => {'founded_on' => {'type' => 'date'}}}}
+        ).to(
+          'settings' => {'analyzer' => {}, 'index' => {'number_of_shards' => 1, 'number_of_replicas' => 0}},
+          'mappings' => {'city' => {'properties' => {'founded_on' => {'type' => 'date'}}}}
+        )
+      end
+
+      specify do
+        expect { specification3.lock! }.to change { specification3.locked }.from(
+          'settings' => {'index' => {'number_of_shards' => 1, 'number_of_replicas' => 0}},
+          'mappings' => {'city' => {'properties' => {'founded_on' => {'type' => 'date'}}}}
+        ).to(
+          'settings' => {'index' => {'number_of_shards' => 1, 'number_of_replicas' => 0}},
+          'mappings' => {'city' => {'properties' => {'founded_on' => {'type' => 'date'}, 'population' => {'type' => 'integer'}}}}
+        )
+      end
+    end
+  end
+
+  describe '#current' do
+    specify do
+      expect(specification2.current).to eq(
+        'mappings' => {'city' => {'properties' => {'founded_on' => {'type' => 'date'}}}},
+        'settings' => {'analyzer' => {}, 'index' => {'number_of_shards' => 1, 'number_of_replicas' => 0}}
+      )
+    end
+  end
+
+  describe '#changed?' do
+    specify { expect(specification1.changed?).to eq(true) }
+    specify { expect { specification1.lock! }.to change { specification1.changed? }.to(false) }
+
+    context do
+      before { specification1.lock! }
+      specify { expect { specification2.lock! }.to change { specification2.changed? }.from(true).to(false) }
+    end
+
+    context do
+      before { specification1.lock! }
+      specify { expect { specification3.lock! }.to change { specification3.changed? }.from(true).to(false) }
+    end
+
+    context do
+      before { specification1.lock! }
+      specify { expect { specification5.lock! }.to change { specification5.changed? }.from(true) }
+    end
+
+    context do
+      before { specification3.lock! }
+      specify { expect { specification4.lock! }.not_to change { specification4.changed? }.from(false) }
+    end
+  end
+end