metasploit-credential 0.14.7 → 0.14.8

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

@@ -1,10 +1,12 @@
-RSpec.describe Metasploit::Credential::Login, type: :model do
+require 'spec_helper'
+
+describe Metasploit::Credential::Login do
   it_should_behave_like 'Metasploit::Concern.run'
 
   context 'associations' do
-    it { is_expected.to belong_to(:core).class_name('Metasploit::Credential::Core') }
-    it { is_expected.to have_one(:host).class_name('Mdm::Host') }
-    it { is_expected.to belong_to(:service).class_name('Mdm::Service')}
+    it { should belong_to(:core).class_name('Metasploit::Credential::Core') }
+    it { should have_one(:host).class_name('Mdm::Host') }
+    it { should belong_to(:service).class_name('Mdm::Service')}
   end
 
   context 'callbacks' do
@@ -37,7 +39,7 @@ RSpec.describe Metasploit::Credential::Login, type: :model do
               ''
             end
 
-            it { is_expected.to be_nil }
+            it { should be_nil }
           end
 
           context 'with nil' do
@@ -45,7 +47,7 @@ RSpec.describe Metasploit::Credential::Login, type: :model do
               nil
             end
 
-            it { is_expected.to be_nil }
+            it { should be_nil }
           end
 
           context 'with present' do
@@ -64,21 +66,21 @@ RSpec.describe Metasploit::Credential::Login, type: :model do
 
   context 'database' do
     context 'columns' do
-      it { is_expected.to have_db_column(:access_level).of_type(:string).with_options(null: true) }
-      it { is_expected.to have_db_column(:last_attempted_at).of_type(:datetime).with_options(null: true) }
-      it { is_expected.to have_db_column(:status).of_type(:string).with_options(null: false) }
+      it { should have_db_column(:access_level).of_type(:string).with_options(null: true) }
+      it { should have_db_column(:last_attempted_at).of_type(:datetime).with_options(null: true) }
+      it { should have_db_column(:status).of_type(:string).with_options(null: false) }
 
       it_should_behave_like 'timestamp database columns'
 
       context 'foreign keys' do
-        it { is_expected.to have_db_column(:core_id).of_type(:integer).with_options(null: false) }
-        it { is_expected.to have_db_column(:service_id).of_type(:integer).with_options(null: false) }
+        it { should have_db_column(:core_id).of_type(:integer).with_options(null: false) }
+        it { should have_db_column(:service_id).of_type(:integer).with_options(null: false) }
       end
     end
 
     context 'indices' do
-      it { is_expected.to have_db_index([:core_id, :service_id]).unique(true) }
-      it { is_expected.to have_db_index([:service_id, :core_id]).unique(true) }
+      it { should have_db_index([:core_id, :service_id]).unique(true) }
+      it { should have_db_index([:service_id, :core_id]).unique(true) }
     end
   end
 
@@ -90,7 +92,7 @@ RSpec.describe Metasploit::Credential::Login, type: :model do
         FactoryGirl.build(:metasploit_credential_login)
       end
 
-      it { is_expected.to be_valid }
+      it { should be_valid }
 
       context '#status' do
         subject(:metasploit_credential_login) do
@@ -105,7 +107,7 @@ RSpec.describe Metasploit::Credential::Login, type: :model do
             Metasploit::Model::Login::Status::DENIED_ACCESS
           end
 
-          it { is_expected.to be_valid }
+          it { should be_valid }
         end
 
         context 'with Metasploit::Model::Login::Status::DISABLED' do
@@ -113,7 +115,7 @@ RSpec.describe Metasploit::Credential::Login, type: :model do
             Metasploit::Model::Login::Status::DISABLED
           end
 
-          it { is_expected.to be_valid }
+          it { should be_valid }
         end
 
         context 'with Metasploit::Model::Login::Status::LOCKED_OUT' do
@@ -121,7 +123,7 @@ RSpec.describe Metasploit::Credential::Login, type: :model do
             Metasploit::Model::Login::Status::LOCKED_OUT
           end
 
-          it { is_expected.to be_valid }
+          it { should be_valid }
         end
 
         context 'with Metasploit::Model::Login::Status::SUCCESSFUL' do
@@ -129,7 +131,7 @@ RSpec.describe Metasploit::Credential::Login, type: :model do
             Metasploit::Model::Login::Status::SUCCESSFUL
           end
 
-          it { is_expected.to be_valid }
+          it { should be_valid }
         end
 
         context 'with Metasploit::Model::Login::Status::UNABLE_TO_CONNECT' do
@@ -137,7 +139,7 @@ RSpec.describe Metasploit::Credential::Login, type: :model do
             Metasploit::Model::Login::Status::UNABLE_TO_CONNECT
           end
 
-          it { is_expected.to be_valid }
+          it { should be_valid }
         end
 
         context 'with Metasploit::Model::Login::Status::UNTRIED' do
@@ -145,21 +147,21 @@ RSpec.describe Metasploit::Credential::Login, type: :model do
             Metasploit::Model::Login::Status::UNTRIED
           end
 
-          it { is_expected.to be_valid }
+          it { should be_valid }
         end
       end
     end
   end
 
   context 'mass assignment security' do
-    it { is_expected.to allow_mass_assignment_of(:access_level) }
+    it { should allow_mass_assignment_of(:access_level) }
     it { should_not allow_mass_assignment_of(:core) }
     it { should_not allow_mass_assignment_of(:core_id) }
     it { should_not allow_mass_assignment_of(:created_at) }
-    it { is_expected.to allow_mass_assignment_of(:last_attempted_at) }
+    it { should allow_mass_assignment_of(:last_attempted_at) }
     it { should_not allow_mass_assignment_of(:service) }
     it { should_not allow_mass_assignment_of(:service_id) }
-    it { is_expected.to allow_mass_assignment_of(:status) }
+    it { should allow_mass_assignment_of(:status) }
     it { should_not allow_mass_assignment_of(:updated_at) }
   end
 
@@ -189,7 +191,7 @@ RSpec.describe Metasploit::Credential::Login, type: :model do
   end
 
   context 'validations' do
-    it { is_expected.to validate_presence_of :core }
+    it { should validate_presence_of :core }
 
     context 'with existent Metasploit::Credential::Login' do
       include_context 'Mdm::Workspace'
@@ -202,11 +204,11 @@ RSpec.describe Metasploit::Credential::Login, type: :model do
         )
       end
 
-      it { is_expected.to validate_uniqueness_of(:core_id).scoped_to(:service_id) }
+      it { should validate_uniqueness_of(:core_id).scoped_to(:service_id) }
     end
 
-    it { is_expected.to validate_presence_of :service }
-    it { is_expected.to ensure_inclusion_of(:status).in_array(Metasploit::Model::Login::Status::ALL) }
+    it { should validate_presence_of :service }
+    it { should ensure_inclusion_of(:status).in_array(Metasploit::Model::Login::Status::ALL) }
 
     context '#consistent_last_attempted_at' do
       include_context 'Mdm::Workspace'
@@ -250,7 +252,7 @@ RSpec.describe Metasploit::Credential::Login, type: :model do
               DateTime.now.utc
             end
 
-            it { is_expected.to include(error) }
+            it { should include(error) }
           end
 
           context 'without #last_attempted' do
@@ -288,7 +290,7 @@ RSpec.describe Metasploit::Credential::Login, type: :model do
               nil
             end
 
-            it { is_expected.to include(error) }
+            it { should include(error) }
           end
         end
       end
@@ -361,7 +363,7 @@ RSpec.describe Metasploit::Credential::Login, type: :model do
                     FactoryGirl.build(:mdm_workspace)
                   end
 
-                  it { is_expected.to include(error) }
+                  it { should include(error) }
                 end
               end
 
@@ -370,7 +372,7 @@ RSpec.describe Metasploit::Credential::Login, type: :model do
                   nil
                 end
 
-                it { is_expected.to include(error) }
+                it { should include(error) }
               end
             end
 
@@ -379,7 +381,7 @@ RSpec.describe Metasploit::Credential::Login, type: :model do
                 nil
               end
 
-              it { is_expected.to include(error) }
+              it { should include(error) }
             end
           end
 
@@ -388,7 +390,7 @@ RSpec.describe Metasploit::Credential::Login, type: :model do
               nil
             end
 
-            it { is_expected.to include(error) }
+            it { should include(error) }
           end
         end
 
@@ -420,7 +422,7 @@ RSpec.describe Metasploit::Credential::Login, type: :model do
                   FactoryGirl.build(:mdm_workspace)
                 end
 
-                it { is_expected.to include(error) }
+                it { should include(error) }
               end
 
               context 'without Mdm::Host#workspace' do
@@ -477,7 +479,7 @@ RSpec.describe Metasploit::Credential::Login, type: :model do
                 FactoryGirl.build(:mdm_workspace)
               end
 
-              it { is_expected.to include(error) }
+              it { should include(error) }
             end
 
             context 'without Mdm::Host#workspace' do
@@ -520,7 +522,7 @@ RSpec.describe Metasploit::Credential::Login, type: :model do
       subject(:login){ FactoryGirl.create :metasploit_credential_login, core: core}
 
       it 'should find the right objects' do
-        expect(Metasploit::Credential::Login.in_workspace_including_hosts_and_services(service.host.workspace)).to include(login)
+        Metasploit::Credential::Login.in_workspace_including_hosts_and_services(service.host.workspace).should include(login)
       end
     end
   end

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

@@ -1,7 +1,9 @@
-RSpec.describe Metasploit::Credential::NonreplayableHash, type: :model do
+require 'spec_helper'
+
+describe Metasploit::Credential::NonreplayableHash do
   it_should_behave_like 'Metasploit::Concern.run'
 
-  it { is_expected.to be_a Metasploit::Credential::PasswordHash }
+  it { should be_a Metasploit::Credential::PasswordHash }
 
   context 'factories' do
     context 'metasploit_credential_nonreplayable_hash' do
@@ -9,7 +11,7 @@ RSpec.describe Metasploit::Credential::NonreplayableHash, type: :model do
         FactoryGirl.build(:metasploit_credential_nonreplayable_hash)
       end
 
-      it { is_expected.to be_valid }
+      it { should be_valid }
     end
   end
 end

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

@@ -1,7 +1,9 @@
-RSpec.describe Metasploit::Credential::NTLMHash, type: :model do
+require 'spec_helper'
+
+describe Metasploit::Credential::NTLMHash do
   it_should_behave_like 'Metasploit::Concern.run'
 
-  it { is_expected.to be_a Metasploit::Credential::ReplayableHash }
+  it { should be_a Metasploit::Credential::ReplayableHash }
 
   context 'CONSTANTS' do
     context 'DATA_REGEXP' do
@@ -36,7 +38,7 @@ RSpec.describe Metasploit::Credential::NTLMHash, type: :model do
         described_class::LAN_MANAGER_MAX_CHARACTERS
       end
 
-      it { is_expected.to eq 14 }
+      it { should == 14 }
     end
 
     context 'LAN_MANAGER_HEX_DIGEST_REGEXP' do
@@ -99,7 +101,7 @@ RSpec.describe Metasploit::Credential::NTLMHash, type: :model do
             nil
           end
 
-          it { is_expected.to be_nil }
+          it { should be_nil }
         end
 
         context 'with upper case characters' do
@@ -131,7 +133,7 @@ RSpec.describe Metasploit::Credential::NTLMHash, type: :model do
         FactoryGirl.build(:metasploit_credential_ntlm_hash)
       end
 
-      it { is_expected.to be_valid }
+      it { should be_valid }
     end
   end
 
@@ -189,7 +191,7 @@ RSpec.describe Metasploit::Credential::NTLMHash, type: :model do
           super().gsub(':', '')
         end
 
-        it { is_expected.to include(error) }
+        it { should include(error) }
       end
 
       context 'without LAN Manager hex_digest' do
@@ -197,7 +199,7 @@ RSpec.describe Metasploit::Credential::NTLMHash, type: :model do
           ":#{nt_lan_manager_hex_digest}"
         end
 
-        it { is_expected.to include(error) }
+        it { should include(error) }
       end
 
       context 'with incorrect hash length(s)' do
@@ -205,7 +207,7 @@ RSpec.describe Metasploit::Credential::NTLMHash, type: :model do
           "123456:abcdef"
         end
 
-        it { is_expected.to include(error) }
+        it { should include(error) }
       end
     end
   end
@@ -373,26 +375,26 @@ RSpec.describe Metasploit::Credential::NTLMHash, type: :model do
     context 'blank_password?' do
 
       it 'returns true if the hash is for a blank password' do
-        expect(blank_password_hash.blank_password?).to eq(true)
+        expect(blank_password_hash.blank_password?).to be_true
       end
 
       it 'returns false if the hash is not for a blank password' do
-        expect(non_blank_password.blank_password?).to eq(false)
+        expect(non_blank_password.blank_password?).to be_false
       end
 
       it 'returns false if the nt hash is not blank but the lm hash is' do
-        expect(no_lm_hash.blank_password?).to eq(false)
+        expect(no_lm_hash.blank_password?).to be_false
       end
     end
 
     context 'lm_hash_present?' do
 
       it 'returns false if the lm_hash is blank' do
-        expect(no_lm_hash.lm_hash_present?).to eq(false)
+        expect(no_lm_hash.lm_hash_present?).to be_false
       end
 
       it 'returns true if the lm_hash is not blank' do
-        expect(non_blank_password.lm_hash_present?).to eq(true)
+        expect(non_blank_password.lm_hash_present?).to be_true
       end
     end
   end

data/spec/models/metasploit/credential/origin/cracked_password_spec.rb

@@ -1,9 +1,11 @@
-RSpec.describe Metasploit::Credential::Origin::CrackedPassword, type: :model do
+require 'spec_helper'
+
+describe Metasploit::Credential::Origin::CrackedPassword do
   it_should_behave_like 'Metasploit::Concern.run'
 
   context 'associations' do
-    it { is_expected.to have_many(:cores).class_name('Metasploit::Credential::Core').dependent(:destroy) }
-    it { is_expected.to belong_to(:originating_core).class_name('Metasploit::Credential::Core') }
+    it { should have_many(:cores).class_name('Metasploit::Credential::Core').dependent(:destroy) }
+    it { should belong_to(:originating_core).class_name('Metasploit::Credential::Core') }
   end
 
   context 'database' do
@@ -11,13 +13,13 @@ RSpec.describe Metasploit::Credential::Origin::CrackedPassword, type: :model do
       it_should_behave_like 'timestamp database columns'
 
       context 'foreign keys' do
-        it { is_expected.to have_db_column(:metasploit_credential_core_id).of_type(:integer).with_options(null: false) }
+        it { should have_db_column(:metasploit_credential_core_id).of_type(:integer).with_options(null: false) }
       end
     end
 
     context 'indices' do
       context 'foreign keys' do
-        it { is_expected.to have_db_index(:metasploit_credential_core_id) }
+        it { should have_db_index(:metasploit_credential_core_id) }
       end
     end
   end

data/spec/models/metasploit/credential/origin/import_spec.rb

@@ -1,25 +1,27 @@
-RSpec.describe Metasploit::Credential::Origin::Import, type: :model do
+require 'spec_helper'
+
+describe Metasploit::Credential::Origin::Import do
   it_should_behave_like 'Metasploit::Concern.run'
 
   context 'associations' do
-    it { is_expected.to have_many(:cores).class_name('Metasploit::Credential::Core').dependent(:destroy) }
-    it { is_expected.to belong_to(:task).class_name('Mdm::Task') }
+    it { should have_many(:cores).class_name('Metasploit::Credential::Core').dependent(:destroy) }
+    it { should belong_to(:task).class_name('Mdm::Task') }
   end
 
   context 'database' do
     context 'columns' do
-      it { is_expected.to have_db_column(:filename).of_type(:text).with_options(null: false) }
+      it { should have_db_column(:filename).of_type(:text).with_options(null: false) }
 
       it_should_behave_like 'timestamp database columns'
 
       context 'foreign keys' do
-        it { is_expected.to have_db_column(:task_id).of_type(:integer) }
+        it { should have_db_column(:task_id).of_type(:integer) }
       end
     end
 
     context 'indices' do
       context 'foreign keys' do
-        it { is_expected.to have_db_index(:task_id) }
+        it { should have_db_index(:task_id) }
       end
     end
   end
@@ -31,12 +33,12 @@ RSpec.describe Metasploit::Credential::Origin::Import, type: :model do
       FactoryGirl.build(:metasploit_credential_origin_import)
     end
 
-    it { is_expected.to be_valid }
+    it { should be_valid }
   end
 
   context 'mass assignment security' do
     it { should_not allow_mass_assignment_of :created_at }
-    it { is_expected.to allow_mass_assignment_of :filename }
+    it { should allow_mass_assignment_of :filename }
     it { should_not allow_mass_assignment_of :task }
     it { should_not allow_mass_assignment_of :task_id }
     it { should_not allow_mass_assignment_of :updated_at }

data/spec/models/metasploit/credential/origin/manual_spec.rb

@@ -1,15 +1,17 @@
-RSpec.describe Metasploit::Credential::Origin::Manual, type: :model do
+require 'spec_helper'
+
+describe Metasploit::Credential::Origin::Manual do
   it_should_behave_like 'Metasploit::Concern.run'
 
   context 'associations' do
-    it { is_expected.to have_many(:cores).class_name('Metasploit::Credential::Core').dependent(:destroy) }
-    it { is_expected.to belong_to(:user).class_name('Mdm::User') }
+    it { should have_many(:cores).class_name('Metasploit::Credential::Core').dependent(:destroy) }
+    it { should belong_to(:user).class_name('Mdm::User') }
   end
 
   context 'database' do
     context 'columns' do
       context 'foreign keys' do
-        it { is_expected.to have_db_column(:user_id).of_type(:integer).with_options(null: false) }
+        it { should have_db_column(:user_id).of_type(:integer).with_options(null: false) }
       end
 
       it_should_behave_like 'timestamp database columns'
@@ -17,7 +19,7 @@ RSpec.describe Metasploit::Credential::Origin::Manual, type: :model do
 
     context 'indices' do
       context 'foreign keys' do
-        it { is_expected.to have_db_index(:user_id) }
+        it { should have_db_index(:user_id) }
       end
     end
   end
@@ -28,11 +30,11 @@ RSpec.describe Metasploit::Credential::Origin::Manual, type: :model do
         FactoryGirl.build(:metasploit_credential_origin_manual)
       end
 
-      it { is_expected.to be_valid }
+      it { should be_valid }
     end
   end
 
   context 'validations' do
-    it { is_expected.to validate_presence_of :user }
+    it { should validate_presence_of :user }
   end
 end