metasploit-model 1.0.0.pre.rails.pre.4.0 → 1.0.0

data/spec/lib/metasploit/model/search/association_spec.rb

@@ -1,6 +1,4 @@
-require 'spec_helper'
-
-describe Metasploit::Model::Search::Association do
+RSpec.describe Metasploit::Model::Search::Association do
   subject(:base_class) do
     described_class = self.described_class
 
@@ -56,7 +54,7 @@ describe Metasploit::Model::Search::Association do
 
   context 'search_associations' do
     subject(:search_associations) {
-      base_class.search_associations *associations
+      base_class.search_associations(*associations)
     }
 
     let(:associations) {

data/spec/lib/metasploit/model/search/attribute_spec.rb

@@ -1,6 +1,4 @@
-require 'spec_helper'
-
-describe Metasploit::Model::Search::Attribute do
+RSpec.describe Metasploit::Model::Search::Attribute do
   subject(:base_class) do
     described_class = self.described_class
 
@@ -26,7 +24,7 @@ describe Metasploit::Model::Search::Attribute do
           end
 
           it 'should call search_with' do
-            base_class.should_receive(:search_with).with(
+            expect(base_class).to receive(:search_with).with(
                 Metasploit::Model::Search::Operator::Attribute,
                 hash_including(
                     :attribute => attribute,
@@ -40,7 +38,7 @@ describe Metasploit::Model::Search::Attribute do
           it 'should be in search_attribute_operator_by_attribute' do
             # grab operator first since it calls search_attribute and populates search_attribute_operator_by_attribute
             cached = operator
-            base_class.search_with_operator_by_name[attribute].should == cached
+            expect(base_class.search_with_operator_by_name[attribute]).to eq(cached)
           end
 
           context 'attribute' do
@@ -49,7 +47,7 @@ describe Metasploit::Model::Search::Attribute do
             end
 
             it 'should be the attribute passed to search_attribute' do
-              operator_attribute.should == attribute
+              expect(operator_attribute).to eq(attribute)
             end
           end
 
@@ -59,7 +57,7 @@ describe Metasploit::Model::Search::Attribute do
             end
 
             it 'should be class on which search_attribute was called' do
-              klass.should == base_class
+              expect(klass).to eq(base_class)
             end
           end
 
@@ -69,7 +67,7 @@ describe Metasploit::Model::Search::Attribute do
             end
 
             it 'should be type passed to search_attribute' do
-              operator_type.should == type
+              expect(operator_type).to eq(type)
             end
           end
         end

data/spec/lib/metasploit/model/search/operation/value/integer_spec.rb

@@ -1,6 +1,4 @@
-require 'spec_helper'
-
-describe Metasploit::Model::Search::Operation::Value::Integer do
+RSpec.describe Metasploit::Model::Search::Operation::Value::Integer do
   it_should_behave_like 'Metasploit::Model::Search::Operation::Value::Integer' do
     let(:operation_class) do
       described_class = self.described_class

data/spec/lib/metasploit/model/search/operation/value/string_spec.rb

@@ -1,6 +1,4 @@
-require 'spec_helper'
-
-describe Metasploit::Model::Search::Operation::Value::String do
+RSpec.describe Metasploit::Model::Search::Operation::Value::String do
   it_should_behave_like 'Metasploit::Model::Search::Operation::Value::String' do
     let(:operation_class) do
       described_class = self.described_class

data/spec/lib/metasploit/model/search/operation_spec.rb

@@ -1,6 +1,4 @@
-require 'spec_helper'
-
-describe Metasploit::Model::Search::Operation do
+RSpec.describe Metasploit::Model::Search::Operation do
   context 'parse' do
     subject(:parse) do
       described_class.parse(options)
@@ -48,8 +46,8 @@ describe Metasploit::Model::Search::Operation do
 
         context "with at least one ':' in :formatted_operation" do
           before(:each) do
-            query.stub(:parse_operator).with(formatted_operator).and_return(operator)
-            operator.stub(:operate_on).with(formatted_value).and_return(operation)
+            allow(query).to receive(:parse_operator).with(formatted_operator).and_return(operator)
+            allow(operator).to receive(:operate_on).with(formatted_value).and_return(operation)
           end
 
           context "with multiple ':' in :formatted_operation" do
@@ -58,13 +56,13 @@ describe Metasploit::Model::Search::Operation do
             end
 
             it "should treat portion before first ':' as formatted operator" do
-              query.should_receive(:parse_operator).with(formatted_operator)
+              expect(query).to receive(:parse_operator).with(formatted_operator)
 
               parse
             end
 
             it "should treat portion after first ':' as formatted value including later ':'" do
-              operator.should_receive(:operate_on).with(formatted_value)
+              expect(operator).to receive(:operate_on).with(formatted_value)
 
               parse
             end
@@ -76,13 +74,13 @@ describe Metasploit::Model::Search::Operation do
             end
 
             it "should use portion before ':' as formatted operator" do
-              query.should_receive(:parse_operator).with(formatted_operator)
+              expect(query).to receive(:parse_operator).with(formatted_operator)
 
               parse
             end
 
             it "should use portion after ':' as formatted value" do
-              operator.should_receive(:operate_on).with(formatted_value)
+              expect(operator).to receive(:operate_on).with(formatted_value)
 
               parse
             end
@@ -103,17 +101,17 @@ describe Metasploit::Model::Search::Operation do
           end
 
           it "should use entirety as formatted operator" do
-            operator.stub(:operate_on).and_return(operation)
+            allow(operator).to receive(:operate_on).and_return(operation)
 
-            query.should_receive(:parse_operator).with(formatted_operator).and_return(operator)
+            expect(query).to receive(:parse_operator).with(formatted_operator).and_return(operator)
 
             parse
           end
 
           it "should use '' as formatted value instead of nil" do
-            query.stub(:parse_operator).and_return(operator)
+            allow(query).to receive(:parse_operator).and_return(operator)
 
-            operator.should_receive(:operate_on).with('')
+            expect(operator).to receive(:operate_on).with('')
 
             parse
           end

data/spec/lib/metasploit/model/search/operator/help_spec.rb

@@ -1,6 +1,4 @@
-require 'spec_helper'
-
-describe Metasploit::Model::Search::Operator::Help do
+RSpec.describe Metasploit::Model::Search::Operator::Help do
   it_should_behave_like 'Metasploit::Model::Search::Operator::Help' do
     let(:operator) do
       operator_class.new(

data/spec/lib/metasploit/model/search/with_spec.rb

@@ -1,6 +1,4 @@
-require 'spec_helper'
-
-describe Metasploit::Model::Search::With do
+RSpec.describe Metasploit::Model::Search::With do
   let(:base_class) do
     described_class = self.described_class
 
@@ -37,7 +35,7 @@ describe Metasploit::Model::Search::With do
     end
 
     it 'should pass given options to operator_class.new' do
-      operator_class.should_receive(:new).with(
+      expect(operator_class).to receive(:new).with(
           hash_including(options)
       ).and_return(operator)
 
@@ -45,7 +43,7 @@ describe Metasploit::Model::Search::With do
     end
 
     it 'should merge :klass into options passed to operator.new' do
-      operator_class.should_receive(:new).with(
+      expect(operator_class).to receive(:new).with(
           hash_including(
               :klass => base_class
           )
@@ -55,9 +53,9 @@ describe Metasploit::Model::Search::With do
     end
 
     it 'should validate operator' do
-      operator_class.stub(:new).and_return(operator)
+      allow(operator_class).to receive(:new).and_return(operator)
 
-      operator.should_receive(:valid!)
+      expect(operator).to receive(:valid!)
 
       search_with_operator
     end
@@ -65,7 +63,7 @@ describe Metasploit::Model::Search::With do
     it 'should add operator to search_with_operator_by_name' do
       search_with_operator
 
-      base_class.search_with_operator_by_name[operator.name].should == operator
+      expect(base_class.search_with_operator_by_name[operator.name]).to eq(operator)
     end
   end
 
@@ -75,7 +73,7 @@ describe Metasploit::Model::Search::With do
     end
 
     it 'should default to empty Hash' do
-      search_with_operator_by_name.should == {}
+      expect(search_with_operator_by_name).to eq({})
     end
   end
 end

data/spec/lib/metasploit/model/search_spec.rb

@@ -1,6 +1,4 @@
-require 'spec_helper'
-
-describe Metasploit::Model::Search do
+RSpec.describe Metasploit::Model::Search do
   subject(:base_instance) do
     base_class.new
   end
@@ -16,9 +14,9 @@ describe Metasploit::Model::Search do
     base_class.send(:include, described_class)
   end
 
-  it { should be_a Metasploit::Model::Search::Association }
-  it { should be_a Metasploit::Model::Search::Attribute }
-  it { should be_a Metasploit::Model::Search::With }
+  it { is_expected.to be_a Metasploit::Model::Search::Association }
+  it { is_expected.to be_a Metasploit::Model::Search::Attribute }
+  it { is_expected.to be_a Metasploit::Model::Search::With }
 
   context 'search_operator_by_name' do
     subject(:search_operator_by_name) do
@@ -45,7 +43,7 @@ describe Metasploit::Model::Search do
           end
 
           it 'should be same as the attribute' do
-            name.should == attribute
+            expect(name).to eq(attribute)
           end
         end
       end
@@ -90,7 +88,7 @@ describe Metasploit::Model::Search do
           "#{association}.#{associated_attribute}".to_sym
         end
 
-        it { should be_a Metasploit::Model::Search::Operator::Association }
+        it { is_expected.to be_a Metasploit::Model::Search::Operator::Association }
 
         context 'association' do
           subject(:operator_association) do
@@ -98,7 +96,7 @@ describe Metasploit::Model::Search do
           end
 
           it 'should be the registered association' do
-            operator_association.should == association
+            expect(operator_association).to eq(association)
           end
         end
 
@@ -112,7 +110,7 @@ describe Metasploit::Model::Search do
           end
 
           it 'should be operator from associated class' do
-            source_operator.should == direct_attribute_operator
+            expect(source_operator).to eq(direct_attribute_operator)
           end
         end
 
@@ -122,7 +120,7 @@ describe Metasploit::Model::Search do
           end
 
           it 'should be class that called search_operator_by_name' do
-            klass.should == base_class
+            expect(klass).to eq(base_class)
           end
         end
       end
@@ -155,7 +153,7 @@ describe Metasploit::Model::Search do
         end
 
         it 'should be in search_operator_by_name' do
-          named_operator.should == operator
+          expect(named_operator).to eq(operator)
         end
       end
     end
@@ -163,7 +161,7 @@ describe Metasploit::Model::Search do
     context 'without search attribute' do
       context 'without search association' do
         context 'without search with' do
-          it { should be_empty }
+          it { is_expected.to be_empty }
         end
       end
     end

data/spec/lib/metasploit/model/spec/error_spec.rb

@@ -1,5 +1,3 @@
-require 'spec_helper'
-
-describe Metasploit::Model::Spec::Error do
-  it { should be_a StandardError }
+RSpec.describe Metasploit::Model::Spec::Error do
+  it { is_expected.to be_a StandardError }
 end

data/spec/lib/metasploit/model/spec/i18n_exception_handler_spec.rb

@@ -1,6 +1,4 @@
-require 'spec_helper'
-
-describe Metasploit::Model::Spec::I18nExceptionHandler do
+RSpec.describe Metasploit::Model::Spec::I18nExceptionHandler do
   subject(:i18n_exception_handler) do
     described_class.new
   end
@@ -32,10 +30,10 @@ describe Metasploit::Model::Spec::I18nExceptionHandler do
       expect {
         call
       }.to raise_error(converted_exception.class) do |actual_exception|
-        actual_exception.class == converted_exception.class
-        actual_exception.key.should == converted_exception.key
-        actual_exception.locale.should == converted_exception.locale
-        actual_exception.options.should == converted_exception.options
+        expect(actual_exception.class).to eq(converted_exception.class)
+        expect(actual_exception.key).to eq(converted_exception.key)
+        expect(actual_exception.locale).to eq(converted_exception.locale)
+        expect(actual_exception.options).to eq(converted_exception.options)
       end
     end
   end

data/spec/lib/metasploit/model/spec/pathname_collision_spec.rb

@@ -1,6 +1,4 @@
-require 'spec_helper'
-
-describe Metasploit::Model::Spec::PathnameCollision do
+RSpec.describe Metasploit::Model::Spec::PathnameCollision do
   let(:pathname) do
     Metasploit::Model::Spec.temporary_pathname.join('pathname')
   end
@@ -9,7 +7,7 @@ describe Metasploit::Model::Spec::PathnameCollision do
     described_class.new(pathname)
   end
 
-  it { should be_a Metasploit::Model::Spec::Error }
+  it { is_expected.to be_a Metasploit::Model::Spec::Error }
 
   context 'check!' do
     subject(:check!) do
@@ -44,11 +42,11 @@ describe Metasploit::Model::Spec::PathnameCollision do
       end
 
       it 'should include pathname' do
-        message.should include("#{pathname} already exists.")
+        expect(message).to include("#{pathname} already exists.")
       end
 
       it 'should include potential cause' do
-        message.should include('Metasploit::Model::Spec.remove_temporary_pathname was not called after the previous spec.')
+        expect(message).to include('Metasploit::Model::Spec.remove_temporary_pathname was not called after the previous spec.')
       end
     end
   end

data/spec/lib/metasploit/model/spec_spec.rb

@@ -1,6 +1,4 @@
-require 'spec_helper'
-
-describe Metasploit::Model::Spec do
+RSpec.describe Metasploit::Model::Spec do
   before(:each) do
     @before_temporary_pathname = described_class.send(:remove_instance_variable, :@temporary_pathname)
   end
@@ -35,11 +33,11 @@ describe Metasploit::Model::Spec do
         end
 
         it 'should remove file tree' do
-          pathname.exist?.should be_true
+          expect(pathname.exist?).to eq(true)
 
           remove_temporary_pathname
 
-          pathname.exist?.should be_false
+          expect(pathname.exist?).to eq(false)
         end
       end
 
@@ -82,7 +80,7 @@ describe Metasploit::Model::Spec do
       end
 
       it 'should return set pathname' do
-        temporary_pathname.should == pathname
+        expect(temporary_pathname).to eq(pathname)
       end
     end
 
@@ -107,7 +105,11 @@ describe Metasploit::Model::Spec do
       expect {
         described_class.temporary_pathname = temporary_pathname
       }.to change {
-        described_class.instance_variable_get(:@temporary_pathname)
+                   if described_class.instance_variable_defined? :@temporary_pathname
+                     described_class.instance_variable_get(:@temporary_pathname)
+                   else
+                     nil
+                   end
       }.to(temporary_pathname)
     end
   end

data/spec/lib/metasploit/model/translation_spec.rb

@@ -1,6 +1,4 @@
-require 'spec_helper'
-
-describe Metasploit::Model::Translation do
+RSpec.describe Metasploit::Model::Translation do
   let(:base_class) do
     described_class = self.described_class
 
@@ -56,21 +54,21 @@ describe Metasploit::Model::Translation do
     end
 
     it 'should have named and unnamed ancestors' do
-      base_class.ancestors.should include(named_class)
-      base_class.ancestors.should include(named_module)
-      base_class.ancestors.should include(unnamed_class)
-      base_class.ancestors.should include(unnamed_module)
+      expect(base_class.ancestors).to include(named_class)
+      expect(base_class.ancestors).to include(named_module)
+      expect(base_class.ancestors).to include(unnamed_class)
+      expect(base_class.ancestors).to include(unnamed_module)
     end
 
     it 'should return all ancestors that respond to model_name' do
-      lookup_ancestors.should include(base_class)
-      lookup_ancestors.should include(named_class)
-      lookup_ancestors.should include(named_module)
+      expect(lookup_ancestors).to include(base_class)
+      expect(lookup_ancestors).to include(named_class)
+      expect(lookup_ancestors).to include(named_module)
     end
 
     it 'should not return ancestors that do not respond to model_name' do
-      lookup_ancestors.should_not include(unnamed_class)
-      lookup_ancestors.should_not include(unnamed_module)
+      expect(lookup_ancestors).not_to include(unnamed_class)
+      expect(lookup_ancestors).not_to include(unnamed_module)
     end
   end
 

data/spec/lib/metasploit/model/version_spec.rb

@@ -1,139 +1,3 @@
-require 'spec_helper'
-
-describe Metasploit::Model::Version do
-  context 'CONSTANTS' do
-    context 'MAJOR' do
-      subject(:major) do
-        described_class::MAJOR
-      end
-
-      it { should be_a Integer }
-    end
-
-    context 'MINOR' do
-      subject(:minor) do
-        described_class::MINOR
-      end
-
-      it { should be_a Integer }
-    end
-
-    context 'PATCH' do
-      subject(:patch) do
-        described_class::PATCH
-      end
-
-      it { should be_a Integer }
-    end
-
-    pull_request = ENV['TRAVIS_PULL_REQUEST']
-
-    # a pull request cannot check PRERELEASE because it will be tested in the target branch, but the source itself
-    # is from the source branch and so has the source branch PRERELEASE.
-    #
-    # PRERELEASE can only be set appropriately for a merge by merging to the target branch and then updating PRERELEASE
-    # on the target branch before committing and/or pushing to github and travis-ci.
-    if pull_request.nil? || pull_request == 'false'
-      context 'PREPRELEASE' do
-        subject(:prerelease) do
-          described_class::PRERELEASE
-        end
-
-        branch = ENV['TRAVIS_BRANCH']
-
-        if branch.blank?
-          branch = `git rev-parse --abbrev-ref HEAD`.strip
-        end
-
-        if branch == 'master'
-          it 'does not have a PRERELEASE' do
-            expect(defined? described_class::PRERELEASE).to be_nil
-          end
-        else
-          branch_regex = /\A(?<type>bug|chore|feature|staging)(\/(?<story>[^\/]+))?\/(?<prerelease>[^\/]+)\z/
-          match = branch.match(branch_regex)
-
-          if match
-            it 'matches the branch relative name' do
-              expect(prerelease).to eq(match[:prerelease])
-            end
-          else
-            tag_regex = /\Av(?<major>\d+).(?<minor>\d+).(?<patch>\d+)(\.pre\.(?<prerelease>.*))?\z/
-            # travis-ci sets TRAVIS_BRANCH to the tag name for tag builds
-            match = branch.match(tag_regex)
-
-            if match
-              tag_prerelease = match[:prerelease]
-
-              if tag_prerelease
-                it 'matches the tag prerelease converted from a gem version to a VERSION' do
-                  expect(prerelease).to eq(tag_prerelease.gsub('.pre.', '-'))
-                end
-              else
-                it 'does not have a PRERELEASE' do
-                  expect(defined? described_class::PRERELEASE).to be_nil
-                end
-              end
-            else
-              it 'has a abbreviated reference that can be parsed for prerelease' do
-                fail "Do not know how to parse #{branch.inspect} for PRERELEASE"
-              end
-            end
-          end
-        end
-      end
-    end
-  end
-
-  context 'full' do
-    subject(:full) do
-      described_class.full
-    end
-
-    #
-    # lets
-    #
-
-    let(:major) do
-      1
-    end
-
-    let(:minor) do
-      2
-    end
-
-    let(:patch) do
-      3
-    end
-
-    before(:each) do
-      stub_const("#{described_class}::MAJOR", major)
-      stub_const("#{described_class}::MINOR", minor)
-      stub_const("#{described_class}::PATCH", patch)
-    end
-
-    context 'with PRERELEASE' do
-      let(:prerelease) do
-        'prerelease'
-      end
-
-      before(:each) do
-        stub_const("#{described_class}::PRERELEASE", prerelease)
-      end
-
-      it 'is <major>.<minor>.<patch>-<prerelease>' do
-        expect(full).to eq("#{major}.#{minor}.#{patch}-#{prerelease}")
-      end
-    end
-
-    context 'without PRERELEASE' do
-      before(:each) do
-        hide_const("#{described_class}::PRERELEASE")
-      end
-
-      it 'is <major>.<minor>.<patch>' do
-        expect(full).to eq("#{major}.#{minor}.#{patch}")
-      end
-    end
-  end
-end
+RSpec.describe Metasploit::Model::Version do
+  it_should_behave_like 'Metasploit::Version Version Module'
+end