metasploit-credential 0.14.7 → 0.14.8

data/spec/lib/metasploit/credential_spec.rb

@@ -1,4 +1,15 @@
-RSpec.describe Metasploit::Credential do
-  it_should_behave_like 'Metasploit::Version GEM_VERSION constant'
-  it_should_behave_like 'Metasploit::Version VERSION constant'
-end
+require 'spec_helper'
+
+describe Metasploit::Credential do
+  context 'CONSTANTS' do
+    context 'VERSION' do
+      subject(:version) do
+        described_class::VERSION
+      end
+
+      it 'is Metasploit::Credential::Version.full' do
+        expect(version).to eq(Metasploit::Credential::Version.full)
+      end
+    end
+  end
+end

data/spec/models/mdm/service_spec.rb

@@ -1,6 +1,8 @@
-RSpec.describe Mdm::Service, type: :model do
+require 'spec_helper'
+
+describe Mdm::Service do
   context 'associations' do
-    it { is_expected.to have_many(:credential_origins).class_name('Metasploit::Credential::Origin::Service').dependent(:destroy) }
-    it { is_expected.to have_many(:logins).class_name('Metasploit::Credential::Login').dependent(:destroy) }
+    it { should have_many(:credential_origins).class_name('Metasploit::Credential::Origin::Service').dependent(:destroy) }
+    it { should have_many(:logins).class_name('Metasploit::Credential::Login').dependent(:destroy) }
   end
 end

data/spec/models/mdm/session_spec.rb

@@ -1,5 +1,7 @@
-RSpec.describe Mdm::Session, type: :model do
+require 'spec_helper'
+
+describe Mdm::Session do
   context 'associations' do
-    it { is_expected.to have_many(:credential_origins).class_name('Metasploit::Credential::Origin::Session').dependent(:destroy) }
+    it { should have_many(:credential_origins).class_name('Metasploit::Credential::Origin::Session').dependent(:destroy) }
   end
 end

data/spec/models/mdm/task_spec.rb

@@ -1,7 +1,9 @@
-RSpec.describe Mdm::Task, type: :model do
+require 'spec_helper'
+
+describe Mdm::Task do
   context 'associations' do
-    it { is_expected.to have_many(:import_credential_origins).class_name('Metasploit::Credential::Origin::Import').dependent(:destroy) }
-    it { is_expected.to have_and_belong_to_many(:credential_cores).class_name('Metasploit::Credential::Core') }
-    it { is_expected.to have_and_belong_to_many(:credential_logins).class_name('Metasploit::Credential::Login') }
+    it { should have_many(:import_credential_origins).class_name('Metasploit::Credential::Origin::Import').dependent(:destroy) }
+    it { should have_and_belong_to_many(:credential_cores).class_name('Metasploit::Credential::Core') }
+    it { should have_and_belong_to_many(:credential_logins).class_name('Metasploit::Credential::Login') }
   end
 end

data/spec/models/mdm/user_spec.rb

@@ -1,5 +1,7 @@
-RSpec.describe Mdm::User, type: :model do
+require 'spec_helper'
+
+describe Mdm::User do
   context 'associations' do
-    it { is_expected.to have_many(:credential_origins).class_name('Metasploit::Credential::Origin::Manual').dependent(:destroy) }
+    it { should have_many(:credential_origins).class_name('Metasploit::Credential::Origin::Manual').dependent(:destroy) }
   end
 end

data/spec/models/mdm/workspace_spec.rb

@@ -1,5 +1,7 @@
-RSpec.describe Mdm::Workspace, type: :model do
+require 'spec_helper'
+
+describe Mdm::Workspace do
   context 'associations' do
-    it { is_expected.to have_many(:core_credentials).class_name('Metasploit::Credential::Core').dependent(:destroy) }
+    it { should have_many(:core_credentials).class_name('Metasploit::Credential::Core').dependent(:destroy) }
   end
 end

data/spec/models/metasploit/credential/blank_username_spec.rb

@@ -1,23 +1,25 @@
-RSpec.describe Metasploit::Credential::BlankUsername, type: :model do
+require 'spec_helper'
+
+describe Metasploit::Credential::BlankUsername do
   it_should_behave_like 'Metasploit::Concern.run'
 
   context 'database' do
     context 'columns' do
       it_should_behave_like 'timestamp database columns'
 
-      it { is_expected.to have_db_column(:username).of_type(:string).with_options(null: false) }
-      it { is_expected.to have_db_column(:type).of_type(:string).with_options(null: false) }
+      it { should have_db_column(:username).of_type(:string).with_options(null: false) }
+      it { should have_db_column(:type).of_type(:string).with_options(null: false) }
     end
 
     context 'indices' do
-      it { is_expected.to have_db_index(:username).unique(true) }
+      it { should have_db_index(:username).unique(true) }
     end
   end
 
   context 'mass assignment security' do
     it { should_not allow_mass_assignment_of(:created_at) }
     it { should_not allow_mass_assignment_of(:updated_at) }
-    it { is_expected.to allow_mass_assignment_of(:username) }
+    it { should allow_mass_assignment_of(:username) }
   end
 
 

data/spec/models/metasploit/credential/core_spec.rb

@@ -1,3 +1,5 @@
+require 'spec_helper'
+
 # Test plan for unique indexes and uniqueness validators
 #
 #    Index        |  First Metasploit::Credential::Core  |           |           |           |  Second Metasploit::Credential::Core  |             |             |             |  Collision  |
@@ -52,7 +54,7 @@
 #    complete     |  non-nil                             |  non-nil  |  non-nil  |  non-nil  |  different                            |  different  |  different  |  same       |  FALSE      |
 #    complete     |  non-nil                             |  non-nil  |  non-nil  |  non-nil  |  different                            |  different  |  different  |  different  |  FALSE      |
 #
-RSpec.describe Metasploit::Credential::Core, type: :model do
+describe Metasploit::Credential::Core do
   include_context 'Mdm::Workspace'
 
   subject(:core) do
@@ -69,27 +71,27 @@ RSpec.describe Metasploit::Credential::Core, type: :model do
   it_should_behave_like 'Metasploit::Concern.run'
 
   context 'associations' do
-    it { is_expected.to have_and_belong_to_many(:tasks).class_name('Mdm::Task') }
-    it { is_expected.to have_many(:logins).class_name('Metasploit::Credential::Login').dependent(:destroy) }
-    it { is_expected.to belong_to(:origin) }
-    it { is_expected.to belong_to(:private).class_name('Metasploit::Credential::Private') }
-    it { is_expected.to belong_to(:public).class_name('Metasploit::Credential::Public') }
-    it { is_expected.to belong_to(:realm).class_name('Metasploit::Credential::Realm') }
-    it { is_expected.to belong_to(:workspace).class_name('Mdm::Workspace') }
+    it { should have_and_belong_to_many(:tasks).class_name('Mdm::Task') }
+    it { should have_many(:logins).class_name('Metasploit::Credential::Login').dependent(:destroy) }
+    it { should belong_to(:origin) }
+    it { should belong_to(:private).class_name('Metasploit::Credential::Private') }
+    it { should belong_to(:public).class_name('Metasploit::Credential::Public') }
+    it { should belong_to(:realm).class_name('Metasploit::Credential::Realm') }
+    it { should belong_to(:workspace).class_name('Mdm::Workspace') }
   end
 
   context 'database' do
     context 'columns' do
       context 'foreign keys' do
         context 'polymorphic origin' do
-          it { is_expected.to have_db_column(:origin_id).of_type(:integer).with_options(null: false) }
-          it { is_expected.to have_db_column(:origin_type).of_type(:string).with_options(null: false) }
+          it { should have_db_column(:origin_id).of_type(:integer).with_options(null: false) }
+          it { should have_db_column(:origin_type).of_type(:string).with_options(null: false) }
         end
 
-        it { is_expected.to have_db_column(:private_id).of_type(:integer).with_options(null: true) }
-        it { is_expected.to have_db_column(:public_id).of_type(:integer).with_options(null: true) }
-        it { is_expected.to have_db_column(:realm_id).of_type(:integer).with_options(null: true) }
-        it { is_expected.to have_db_column(:workspace_id).of_type(:integer).with_options(null: false) }
+        it { should have_db_column(:private_id).of_type(:integer).with_options(null: true) }
+        it { should have_db_column(:public_id).of_type(:integer).with_options(null: true) }
+        it { should have_db_column(:realm_id).of_type(:integer).with_options(null: true) }
+        it { should have_db_column(:workspace_id).of_type(:integer).with_options(null: false) }
       end
 
       it_should_behave_like 'timestamp database columns'
@@ -98,11 +100,11 @@ RSpec.describe Metasploit::Credential::Core, type: :model do
     context 'indices' do
       context 'foreign keys' do
 
-        it { is_expected.to have_db_index([:origin_type, :origin_id]) }
-        it { is_expected.to have_db_index(:private_id) }
-        it { is_expected.to have_db_index(:public_id) }
-        it { is_expected.to have_db_index(:realm_id) }
-        it { is_expected.to have_db_index(:workspace_id) }
+        it { should have_db_index([:origin_type, :origin_id]) }
+        it { should have_db_index(:private_id) }
+        it { should have_db_index(:public_id) }
+        it { should have_db_index(:realm_id) }
+        it { should have_db_index(:workspace_id) }
 
 
       end
@@ -293,7 +295,7 @@ RSpec.describe Metasploit::Credential::Core, type: :model do
         metasploit_credential_core.origin
       end
 
-      it { is_expected.to be_valid }
+      it { should be_valid }
 
       context 'with origin_factory' do
         subject(:metasploit_credential_core) do
@@ -308,7 +310,7 @@ RSpec.describe Metasploit::Credential::Core, type: :model do
             :metasploit_credential_origin_import
           end
 
-          it { is_expected.to be_valid }
+          it { should be_valid }
         end
 
         context ':metasploit_credential_origin_manual' do
@@ -316,14 +318,14 @@ RSpec.describe Metasploit::Credential::Core, type: :model do
             :metasploit_credential_origin_manual
           end
 
-          it { is_expected.to be_valid }
+          it { should be_valid }
 
           context '#origin' do
             subject(:origin) do
               metasploit_credential_core.origin
             end
 
-            it { is_expected.to be_a Metasploit::Credential::Origin::Manual }
+            it { should be_a Metasploit::Credential::Origin::Manual }
           end
 
           context '#workspace' do
@@ -340,7 +342,7 @@ RSpec.describe Metasploit::Credential::Core, type: :model do
             :metasploit_credential_origin_service
           end
 
-          it { is_expected.to be_valid }
+          it { should be_valid }
 
           context '#workspace' do
             subject(:workspace) do
@@ -359,7 +361,7 @@ RSpec.describe Metasploit::Credential::Core, type: :model do
             :metasploit_credential_origin_session
           end
 
-          it { is_expected.to be_valid }
+          it { should be_valid }
 
           context '#workspace' do
             subject(:workspace) do
@@ -380,7 +382,7 @@ RSpec.describe Metasploit::Credential::Core, type: :model do
         FactoryGirl.build(:metasploit_credential_core_import)
       end
 
-      it { is_expected.to be_valid }
+      it { should be_valid }
     end
 
     context 'metasploit_credential_core_manual' do
@@ -388,7 +390,7 @@ RSpec.describe Metasploit::Credential::Core, type: :model do
         FactoryGirl.build(:metasploit_credential_core_manual)
       end
 
-      it { is_expected.to be_valid }
+      it { should be_valid }
 
       context '#workspace' do
         subject(:workspace) do
@@ -404,7 +406,7 @@ RSpec.describe Metasploit::Credential::Core, type: :model do
         FactoryGirl.build(:metasploit_credential_core_service)
       end
 
-      it { is_expected.to be_valid }
+      it { should be_valid }
 
       context '#workspace' do
         subject(:workspace) do
@@ -427,7 +429,7 @@ RSpec.describe Metasploit::Credential::Core, type: :model do
         FactoryGirl.build(:metasploit_credential_core_session)
       end
 
-      it { is_expected.to be_valid }
+      it { should be_valid }
 
       context '#workspace' do
         subject(:workspace) do
@@ -447,7 +449,7 @@ RSpec.describe Metasploit::Credential::Core, type: :model do
   end
 
   context 'validations' do
-    it { is_expected.to validate_presence_of :origin }
+    it { should validate_presence_of :origin }
 
 
     context '#consistent_workspaces' do
@@ -536,7 +538,7 @@ RSpec.describe Metasploit::Credential::Core, type: :model do
               end
 
               context 'without #workspace in Mdm::User#workspaces' do
-                it { is_expected.to include error }
+                it { should include error }
               end
             end
           end
@@ -546,7 +548,7 @@ RSpec.describe Metasploit::Credential::Core, type: :model do
               nil
             end
 
-            it { is_expected.to include error }
+            it { should include error }
           end
         end
 
@@ -591,7 +593,7 @@ RSpec.describe Metasploit::Credential::Core, type: :model do
                   FactoryGirl.create(:mdm_workspace)
                 end
 
-                it { is_expected.to include error }
+                it { should include error }
               end
             end
 
@@ -600,7 +602,7 @@ RSpec.describe Metasploit::Credential::Core, type: :model do
                 nil
               end
 
-              it { is_expected.to include error }
+              it { should include error }
             end
           end
 
@@ -609,7 +611,7 @@ RSpec.describe Metasploit::Credential::Core, type: :model do
               nil
             end
 
-            it { is_expected.to include error }
+            it { should include error }
           end
         end
 
@@ -655,7 +657,7 @@ RSpec.describe Metasploit::Credential::Core, type: :model do
                     FactoryGirl.create(:mdm_workspace)
                   end
 
-                  it { is_expected.to include error }
+                  it { should include error }
                 end
               end
 
@@ -664,7 +666,7 @@ RSpec.describe Metasploit::Credential::Core, type: :model do
                   nil
                 end
 
-                it { is_expected.to include error }
+                it { should include error }
               end
             end
 
@@ -673,7 +675,7 @@ RSpec.describe Metasploit::Credential::Core, type: :model do
                 nil
               end
 
-              it { is_expected.to include error }
+              it { should include error }
             end
           end
 
@@ -682,7 +684,7 @@ RSpec.describe Metasploit::Credential::Core, type: :model do
               nil
             end
 
-            it { is_expected.to include error }
+            it { should include error }
           end
         end
       end
@@ -837,7 +839,7 @@ RSpec.describe Metasploit::Credential::Core, type: :model do
         I18n.translate!('activerecord.errors.models.metasploit/credential/core.attributes.base.public_for_ssh_key')
       end
 
-      subject(:core) do
+      let(:core) do
         FactoryGirl.build(
             :metasploit_credential_core,
             private: FactoryGirl.build(:metasploit_credential_ssh_key),
@@ -845,7 +847,7 @@ RSpec.describe Metasploit::Credential::Core, type: :model do
         )
       end
 
-      it { is_expected.to be_valid }
+      it { core.should be_valid }
 
       context "when the Public is missing" do
         before(:each) do
@@ -853,12 +855,12 @@ RSpec.describe Metasploit::Credential::Core, type: :model do
         end
 
         it 'should not be valid if Private is an SSHKey and Public is missing' do
-          expect(core).not_to be_valid
+          core.should_not be_valid
         end
 
         it 'should show the proper error' do
           core.valid?
-          expect(core.errors[:base]).to include(error)
+          core.errors[:base].should include(error)
         end
       end
 

data/spec/models/metasploit/credential/login/status_spec.rb

@@ -1,16 +1,18 @@
-RSpec.describe Metasploit::Model::Login::Status, type: :model do
+require 'spec_helper'
+
+describe Metasploit::Model::Login::Status do
   context 'CONSTANTS' do
     context 'ALL' do
       subject(:all) do
         described_class::ALL
       end
 
-      it { is_expected.to include described_class::DENIED_ACCESS }
-      it { is_expected.to include described_class::DISABLED }
-      it { is_expected.to include described_class::LOCKED_OUT }
-      it { is_expected.to include described_class::SUCCESSFUL }
-      it { is_expected.to include described_class::UNABLE_TO_CONNECT }
-      it { is_expected.to include described_class::UNTRIED }
+      it { should include described_class::DENIED_ACCESS }
+      it { should include described_class::DISABLED }
+      it { should include described_class::LOCKED_OUT }
+      it { should include described_class::SUCCESSFUL }
+      it { should include described_class::UNABLE_TO_CONNECT }
+      it { should include described_class::UNTRIED }
     end
 
     context 'DENIED_ACCESS' do
@@ -18,8 +20,8 @@ RSpec.describe Metasploit::Model::Login::Status, type: :model do
         described_class::DENIED_ACCESS
       end
 
-      it { is_expected.to eq 'Denied Access' }
-      it { is_expected.to be_in described_class::ALL }
+      it { should == 'Denied Access' }
+      it { should be_in described_class::ALL }
     end
 
     context 'DISABLED' do
@@ -27,8 +29,8 @@ RSpec.describe Metasploit::Model::Login::Status, type: :model do
         described_class::DISABLED
       end
 
-      it { is_expected.to eq 'Disabled' }
-      it { is_expected.to be_in described_class::ALL }
+      it { should == 'Disabled' }
+      it { should be_in described_class::ALL }
     end
 
     context 'LOCKED_OUT' do
@@ -36,8 +38,8 @@ RSpec.describe Metasploit::Model::Login::Status, type: :model do
         described_class::LOCKED_OUT
       end
 
-      it { is_expected.to eq 'Locked Out' }
-      it { is_expected.to be_in described_class::ALL }
+      it { should == 'Locked Out' }
+      it { should be_in described_class::ALL }
     end
 
     context 'SUCCESSFUL' do
@@ -45,8 +47,8 @@ RSpec.describe Metasploit::Model::Login::Status, type: :model do
         described_class::SUCCESSFUL
       end
 
-      it { is_expected.to eq 'Successful' }
-      it { is_expected.to be_in described_class::ALL }
+      it { should == 'Successful' }
+      it { should be_in described_class::ALL }
     end
 
     context 'UNABLE_TO_CONNECT' do
@@ -54,8 +56,8 @@ RSpec.describe Metasploit::Model::Login::Status, type: :model do
         described_class::UNABLE_TO_CONNECT
       end
 
-      it { is_expected.to eq 'Unable to Connect' }
-      it { is_expected.to be_in described_class::ALL }
+      it { should == 'Unable to Connect' }
+      it { should be_in described_class::ALL }
     end
 
     context 'UNTRIED' do
@@ -63,8 +65,8 @@ RSpec.describe Metasploit::Model::Login::Status, type: :model do
         described_class::UNTRIED
       end
 
-      it { is_expected.to eq 'Untried' }
-      it { is_expected.to be_in described_class::ALL }
+      it { should == 'Untried' }
+      it { should be_in described_class::ALL }
     end
   end
 end