metasploit_data_models 0.24.4 → 0.24.5

data/spec/app/models/metasploit_data_models/search/operator/ip_address_spec.rb

@@ -1,4 +1,6 @@
-RSpec.describe MetasploitDataModels::Search::Operator::IPAddress, type: :model do
+require 'spec_helper'
+
+describe MetasploitDataModels::Search::Operator::IPAddress do
   subject(:operator) {
     described_class.new
   }
@@ -12,6 +14,6 @@ RSpec.describe MetasploitDataModels::Search::Operator::IPAddress, type: :model d
       nil
     }
 
-    it { is_expected.to be_a MetasploitDataModels::Search::Operation::IPAddress }
+    it { should be_a MetasploitDataModels::Search::Operation::IPAddress }
   end
 end

data/spec/app/models/metasploit_data_models/search/operator/multitext_spec.rb

@@ -1,4 +1,6 @@
-RSpec.describe MetasploitDataModels::Search::Operator::Multitext, type: :model do
+require 'spec_helper'
+
+describe MetasploitDataModels::Search::Operator::Multitext do
   subject(:multitext_operator) {
     described_class.new(
         attributes
@@ -10,8 +12,8 @@ RSpec.describe MetasploitDataModels::Search::Operator::Multitext, type: :model d
   }
 
   context 'validations' do
-    it { is_expected.to ensure_length_of(:operator_names).is_at_least(2) }
-    it { is_expected.to validate_presence_of :name }
+    it { should ensure_length_of(:operator_names).is_at_least(2) }
+    it { should validate_presence_of :name }
   end
 
   context '#children' do
@@ -43,7 +45,7 @@ RSpec.describe MetasploitDataModels::Search::Operator::Multitext, type: :model d
         nil
       }
 
-      it { is_expected.to eq([]) }
+      it { should == [] }
     end
 
     context 'with empty String' do
@@ -51,7 +53,7 @@ RSpec.describe MetasploitDataModels::Search::Operator::Multitext, type: :model d
         ''
       }
 
-      it { is_expected.to eq([]) }
+      it { should == [] }
     end
 
     context 'without quotes' do
@@ -80,7 +82,7 @@ RSpec.describe MetasploitDataModels::Search::Operator::Multitext, type: :model d
       }
 
       it 'generates a single union for quoted words as a single argument' do
-        expect(children.length).to eq(1)
+        expect(children).to have(1).items
 
         child = children.first
 
@@ -99,7 +101,7 @@ RSpec.describe MetasploitDataModels::Search::Operator::Multitext, type: :model d
     }
 
     context 'default' do
-      it { is_expected.to be_nil }
+      it { should be_nil }
     end
 
     context 'setter' do
@@ -121,7 +123,7 @@ RSpec.describe MetasploitDataModels::Search::Operator::Multitext, type: :model d
     }
 
     context 'default' do
-      it { is_expected.to eq([]) }
+      it { should == [] }
     end
   end
 

data/spec/app/models/metasploit_data_models/search/operator/port/list_spec.rb

@@ -1,4 +1,6 @@
-RSpec.describe MetasploitDataModels::Search::Operator::Port::List, type: :model do
+require 'spec_helper'
+
+describe MetasploitDataModels::Search::Operator::Port::List do
   subject(:port_list_operator) {
     described_class.new(
         klass: klass
@@ -15,7 +17,7 @@ RSpec.describe MetasploitDataModels::Search::Operator::Port::List, type: :model
         described_class::SEPARATOR
       }
 
-      it { is_expected.to eq(',') }
+      it { should == ',' }
     end
   end
 
@@ -25,7 +27,7 @@ RSpec.describe MetasploitDataModels::Search::Operator::Port::List, type: :model
     }
 
     context 'default' do
-      it { is_expected.to eq(:port) }
+      it { should == :port }
     end
 
     context 'setter' do
@@ -85,7 +87,7 @@ RSpec.describe MetasploitDataModels::Search::Operator::Port::List, type: :model
               operation_range.value
             }
 
-            it { is_expected.to be_a Range }
+            it { should be_a Range }
           end
         end
       end
@@ -119,7 +121,7 @@ RSpec.describe MetasploitDataModels::Search::Operator::Port::List, type: :model
               operation_range.value
             }
 
-            it { is_expected.to be_an Integer }
+            it { should be_an Integer }
           end
         end
       end
@@ -157,6 +159,6 @@ RSpec.describe MetasploitDataModels::Search::Operator::Port::List, type: :model
       described_class.operator_name
     }
 
-    it { is_expected.to eq('port_list') }
+    it { should == 'port_list' }
   end
 end

data/spec/app/models/metasploit_data_models/search/visitor/attribute_spec.rb

@@ -1,4 +1,6 @@
-RSpec.describe MetasploitDataModels::Search::Visitor::Attribute, type: :model do
+require 'spec_helper'
+
+describe MetasploitDataModels::Search::Visitor::Attribute do
   subject(:visitor) do
     described_class.new
   end
@@ -20,7 +22,7 @@ RSpec.describe MetasploitDataModels::Search::Visitor::Attribute, type: :model do
       node_class = options.fetch(:node_class)
 
       context "with #{node_class}" do
-        it { is_expected.to be_a Arel::Attributes::Attribute }
+        it { should be_a Arel::Attributes::Attribute }
 
         context '#name' do
           subject(:name) do
@@ -28,7 +30,7 @@ RSpec.describe MetasploitDataModels::Search::Visitor::Attribute, type: :model do
           end
 
           it "should be #{node_class}#attribute" do
-            expect(name).to eq(node.attribute)
+            name.should == node.attribute
           end
         end
 
@@ -38,7 +40,7 @@ RSpec.describe MetasploitDataModels::Search::Visitor::Attribute, type: :model do
           end
 
           it "should be Class#arel_table for #{node_class}#klass" do
-            expect(relation).to eq(node.klass.arel_table)
+            relation.should == node.klass.arel_table
           end
         end
       end
@@ -56,19 +58,19 @@ RSpec.describe MetasploitDataModels::Search::Visitor::Attribute, type: :model do
       end
 
       it 'should visit Metasploit::Model::Search::Operator::Association#source_operator' do
-        expect(visitor).to receive(:visit).with(node).and_call_original
-        expect(visitor).to receive(:visit).with(source_operator)
+        visitor.should_receive(:visit).with(node).and_call_original
+        visitor.should_receive(:visit).with(source_operator)
 
         visit
       end
 
       it 'should return visit of Metasploit::Model::Search::Operator::Association#source_operator' do
-        expect(visitor).to receive(:visit).with(node).and_call_original
+        visitor.should_receive(:visit).with(node).and_call_original
 
         visited = double('Source Operator Visited')
-        allow(visitor).to receive(:visit).with(source_operator).and_return(visited)
+        visitor.stub(:visit).with(source_operator).and_return(visited)
 
-        expect(visit).to eq(visited)
+        visit.should == visited
       end
     end
 

data/spec/app/models/metasploit_data_models/search/visitor/includes_spec.rb

@@ -1,4 +1,6 @@
-RSpec.describe MetasploitDataModels::Search::Visitor::Includes, type: :model do
+require 'spec_helper'
+
+describe MetasploitDataModels::Search::Visitor::Includes do
   subject(:visitor) do
     described_class.new
   end
@@ -120,7 +122,7 @@ RSpec.describe MetasploitDataModels::Search::Visitor::Includes, type: :model do
         Metasploit::Model::Search::Operator::Attribute.new
       end
 
-      it { is_expected.to eq([]) }
+      it { should == [] }
     end
 
     context 'with MetasploitDataModels::Search::Operator::Port::List' do
@@ -128,7 +130,7 @@ RSpec.describe MetasploitDataModels::Search::Visitor::Includes, type: :model do
         MetasploitDataModels::Search::Operator::Port::List.new
       end
 
-      it { is_expected.to eq([]) }
+      it { should == [] }
     end
 
     context 'with Metasploit::Model::Search::Query#tree' do
@@ -158,7 +160,7 @@ RSpec.describe MetasploitDataModels::Search::Visitor::Includes, type: :model do
               "name:\"#{name}\""
             end
 
-            it { is_expected.to be_empty }
+            it { should be_empty }
           end
 
           context 'with services.name' do
@@ -170,7 +172,7 @@ RSpec.describe MetasploitDataModels::Search::Visitor::Includes, type: :model do
               "services.name:\"#{name}\""
             end
 
-            it { is_expected.to include :services }
+            it { should include :services }
           end
         end
       end

data/spec/app/models/metasploit_data_models/search/visitor/joins_spec.rb

@@ -1,4 +1,6 @@
-RSpec.describe MetasploitDataModels::Search::Visitor::Joins, type: :model do
+require 'spec_helper'
+
+describe MetasploitDataModels::Search::Visitor::Joins do
   subject(:visitor) do
     described_class.new
   end
@@ -31,10 +33,10 @@ RSpec.describe MetasploitDataModels::Search::Visitor::Joins, type: :model do
 
         it 'should visit each child' do
           # needed for call to visit subject
-          expect(visitor).to receive(:visit).with(node).and_call_original
+          visitor.should_receive(:visit).with(node).and_call_original
 
           children.each do |child|
-            expect(visitor).to receive(:visit).with(child).and_return([])
+            visitor.should_receive(:visit).with(child).and_return([])
           end
 
           visit
@@ -43,15 +45,15 @@ RSpec.describe MetasploitDataModels::Search::Visitor::Joins, type: :model do
         it 'should return Array of all child visits' do
           child_visits = []
 
-          expect(visitor).to receive(:visit).with(node).and_call_original
+          visitor.should_receive(:visit).with(node).and_call_original
 
           children.each_with_index do |child, i|
             child_visit = ["Visited Child #{i}"]
-            allow(visitor).to receive(:visit).with(child).and_return(child_visit)
+            visitor.stub(:visit).with(child).and_return(child_visit)
             child_visits.concat(child_visit)
           end
 
-          expect(visit).to eq(child_visits)
+          visit.should == child_visits
         end
       end
     end
@@ -79,7 +81,7 @@ RSpec.describe MetasploitDataModels::Search::Visitor::Joins, type: :model do
               }
             end
 
-            it { is_expected.to eq([]) }
+            it { should == [] }
           end
 
           context 'with association and attribute' do
@@ -115,7 +117,7 @@ RSpec.describe MetasploitDataModels::Search::Visitor::Joins, type: :model do
               ]
             end
 
-            it { is_expected.to eq([]) }
+            it { should == [] }
           end
 
           context 'with the same child join for all' do
@@ -143,7 +145,7 @@ RSpec.describe MetasploitDataModels::Search::Visitor::Joins, type: :model do
             end
 
             it 'should include association' do
-              expect(visit).to include association
+              visit.should include association
             end
           end
 
@@ -213,13 +215,13 @@ RSpec.describe MetasploitDataModels::Search::Visitor::Joins, type: :model do
 
               it 'should include common associations' do
                 common_associations.each do |association|
-                  expect(visit).to include(association)
+                  visit.should include(association)
                 end
               end
 
               it 'should not include disjoint associations' do
                 disjoint_associations.each do |association|
-                  expect(visit).not_to include(association)
+                  visit.should_not include(association)
                 end
               end
             end
@@ -229,7 +231,7 @@ RSpec.describe MetasploitDataModels::Search::Visitor::Joins, type: :model do
                 []
               end
 
-              it { is_expected.to eq([]) }
+              it { should == [] }
             end
           end
         end
@@ -239,7 +241,7 @@ RSpec.describe MetasploitDataModels::Search::Visitor::Joins, type: :model do
             []
           end
 
-          it { is_expected.to eq([]) }
+          it { should == [] }
         end
       end
     end
@@ -337,7 +339,7 @@ RSpec.describe MetasploitDataModels::Search::Visitor::Joins, type: :model do
         Metasploit::Model::Search::Operator::Attribute.new
       end
 
-      it { is_expected.to eq([]) }
+      it { should == [] }
     end
 
     context 'with MetasploitDataModels::Search::Operator::Port::List' do
@@ -345,7 +347,7 @@ RSpec.describe MetasploitDataModels::Search::Visitor::Joins, type: :model do
         MetasploitDataModels::Search::Operator::Port::List.new
       end
 
-      it { is_expected.to eq([]) }
+      it { should == [] }
     end
 
     context 'with Metasploit::Model::Search::Query#tree' do
@@ -375,7 +377,7 @@ RSpec.describe MetasploitDataModels::Search::Visitor::Joins, type: :model do
               "name:\"#{name}\""
             end
 
-            it { is_expected.to be_empty }
+            it { should be_empty }
           end
 
           context 'with services.name' do
@@ -387,7 +389,7 @@ RSpec.describe MetasploitDataModels::Search::Visitor::Joins, type: :model do
               "services.name:\"#{name}\""
             end
 
-            it { is_expected.to include :services }
+            it { should include :services }
           end
         end
       end

data/spec/app/models/metasploit_data_models/search/visitor/method_spec.rb

@@ -1,4 +1,6 @@
-RSpec.describe MetasploitDataModels::Search::Visitor::Method, type: :model do
+require 'spec_helper'
+
+describe MetasploitDataModels::Search::Visitor::Method do
   subject(:visitor) do
     described_class.new
   end
@@ -19,7 +21,7 @@ RSpec.describe MetasploitDataModels::Search::Visitor::Method, type: :model do
         Metasploit::Model::Search::Group::Intersection
       end
 
-      it { is_expected.to eq(:and) }
+      it { should == :and }
     end
 
     context 'with Metasploit::Model::Search::Operation::Group::Intersection' do
@@ -27,7 +29,7 @@ RSpec.describe MetasploitDataModels::Search::Visitor::Method, type: :model do
         Metasploit::Model::Search::Operation::Group::Intersection
       end
 
-      it { is_expected.to eq(:and) }
+      it { should == :and }
     end
 
     context 'with Metasploit::Model::Search::Group::Union' do
@@ -35,7 +37,7 @@ RSpec.describe MetasploitDataModels::Search::Visitor::Method, type: :model do
         Metasploit::Model::Search::Group::Union
       end
 
-      it { is_expected.to eq(:or) }
+      it { should == :or }
     end
 
     context 'with Metasploit::Model::Search::Operation::Group::Union' do
@@ -43,7 +45,7 @@ RSpec.describe MetasploitDataModels::Search::Visitor::Method, type: :model do
         Metasploit::Model::Search::Operation::Group::Union
       end
 
-      it { is_expected.to eq(:or) }
+      it { should == :or }
     end
   end
 end

data/spec/app/models/metasploit_data_models/search/visitor/relation_spec.rb

@@ -1,4 +1,6 @@
-RSpec.describe MetasploitDataModels::Search::Visitor::Relation, type: :model do
+require 'spec_helper'
+
+describe MetasploitDataModels::Search::Visitor::Relation do
   subject(:visitor) do
     described_class.new(
         :query => query
@@ -29,7 +31,7 @@ RSpec.describe MetasploitDataModels::Search::Visitor::Relation, type: :model do
 
   context 'validations' do
     context 'query' do
-      it { is_expected.to validate_presence_of(:query) }
+      it { should validate_presence_of(:query) }
 
       context 'valid' do
         let(:error) do
@@ -46,7 +48,7 @@ RSpec.describe MetasploitDataModels::Search::Visitor::Relation, type: :model do
           end
 
           before(:each) do
-            allow(query).to receive(:valid?).and_return(query)
+            query.stub(:valid? => query)
 
             visitor.valid?
           end
@@ -57,7 +59,7 @@ RSpec.describe MetasploitDataModels::Search::Visitor::Relation, type: :model do
             end
 
             it 'should not record error' do
-              expect(errors).not_to include(error)
+              errors.should_not include(error)
             end
           end
 
@@ -67,7 +69,7 @@ RSpec.describe MetasploitDataModels::Search::Visitor::Relation, type: :model do
             end
 
             it 'should record error' do
-              expect(errors).not_to include(error)
+              errors.should_not include(error)
             end
           end
         end
@@ -78,7 +80,7 @@ RSpec.describe MetasploitDataModels::Search::Visitor::Relation, type: :model do
           end
 
           it 'should not record error' do
-            expect(errors).not_to include(error)
+            errors.should_not include(error)
           end
         end
       end
@@ -96,16 +98,16 @@ RSpec.describe MetasploitDataModels::Search::Visitor::Relation, type: :model do
       end
 
       it 'should visit Metasploit::Model::Search::Query#tree' do
-        expect(includes_visitor).to receive(:visit).with(query.tree)
+        includes_visitor.should_receive(:visit).with(query.tree)
 
         visit
       end
 
       it 'should pass visited to ActiveRecord::Relation#includes' do
         visited = double('Visited')
-        allow(includes_visitor).to receive(:visit).with(query.tree).and_return(visited)
+        includes_visitor.stub(:visit).with(query.tree).and_return(visited)
 
-        expect_any_instance_of(ActiveRecord::Relation).to receive(:includes).with(visited).and_return(query.klass.scoped)
+        ActiveRecord::Relation.any_instance.should_receive(:includes).with(visited).and_return(query.klass.scoped)
 
         visit
       end
@@ -117,16 +119,16 @@ RSpec.describe MetasploitDataModels::Search::Visitor::Relation, type: :model do
       end
 
       it 'should visit Metasploit::Model::Search::Query#tree' do
-        expect(joins_visitor).to receive(:visit).with(query.tree)
+        joins_visitor.should_receive(:visit).with(query.tree)
 
         visit
       end
 
       it 'should pass visited to ActiveRecord::Relation#joins' do
         visited = double('Visited')
-        allow(joins_visitor).to receive(:visit).with(query.tree).and_return(visited)
+        joins_visitor.stub(:visit).with(query.tree).and_return(visited)
 
-        expect_any_instance_of(ActiveRecord::Relation).to receive(:joins).with(visited).and_return(query.klass.scoped)
+        ActiveRecord::Relation.any_instance.should_receive(:joins).with(visited).and_return(query.klass.scoped)
 
         visit
       end
@@ -138,16 +140,16 @@ RSpec.describe MetasploitDataModels::Search::Visitor::Relation, type: :model do
       end
 
       it 'should visit Metasploit::Model::Search::Query#tree' do
-        expect(where_visitor).to receive(:visit).with(query.tree)
+        where_visitor.should_receive(:visit).with(query.tree)
 
         visit
       end
 
       it 'should pass visited to ActiveRecord::Relation#includes' do
         visited = double('Visited')
-        allow(where_visitor).to receive(:visit).with(query.tree).and_return(visited)
+        where_visitor.stub(:visit).with(query.tree).and_return(visited)
 
-        expect_any_instance_of(ActiveRecord::Relation).to receive(:where).with(visited).and_return(query.klass.scoped)
+        ActiveRecord::Relation.any_instance.should_receive(:where).with(visited).and_return(query.klass.scoped)
 
         visit
       end
@@ -868,29 +870,9 @@ RSpec.describe MetasploitDataModels::Search::Visitor::Relation, type: :model do
       visitor.visitor_by_relation_method
     end
 
-    context 'joins' do
-      subject(:joins) {
-        visitor_by_relation_method[:joins]
-      }
-
-      it { is_expected.to be_a MetasploitDataModels::Search::Visitor::Joins }
-    end
-
-    context 'includes' do
-      subject(:includes) {
-        visitor_by_relation_method[:includes]
-      }
-
-      it { is_expected.to be_a MetasploitDataModels::Search::Visitor::Includes }
-    end
-
-    context 'where' do
-      subject(:where) {
-        visitor_by_relation_method[:where]
-      }
-
-      it { is_expected.to be_a MetasploitDataModels::Search::Visitor::Where }
-    end
+    its([:joins]) { should be_a MetasploitDataModels::Search::Visitor::Joins }
+    its([:includes]) { should be_a MetasploitDataModels::Search::Visitor::Includes }
+    its([:where]) { should be_a MetasploitDataModels::Search::Visitor::Where }
   end
 
   context 'visitor_class_by_relation_method' do
@@ -898,28 +880,8 @@ RSpec.describe MetasploitDataModels::Search::Visitor::Relation, type: :model do
       described_class.visitor_class_by_relation_method
     end
 
-    context 'joins' do
-      subject(:joins) {
-        visitor_class_by_relation_method[:joins]
-      }
-
-      it { is_expected.to eq(MetasploitDataModels::Search::Visitor::Joins) }
-    end
-
-    context 'includes' do
-      subject(:includes) {
-        visitor_class_by_relation_method[:includes]
-      }
-
-      it { is_expected.to eq(MetasploitDataModels::Search::Visitor::Includes) }
-    end
-
-    context 'where' do
-      subject(:where) {
-        visitor_class_by_relation_method[:where]
-      }
-
-      it { is_expected.to eq(MetasploitDataModels::Search::Visitor::Where) }
-    end
+    its([:joins]) { should == MetasploitDataModels::Search::Visitor::Joins }
+    its([:includes]) { should == MetasploitDataModels::Search::Visitor::Includes }
+    its([:where]) { should == MetasploitDataModels::Search::Visitor::Where }
   end
 end