metasploit-credential 0.7.17 → 0.8.1

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

@@ -43,6 +43,98 @@ describe Metasploit::Credential::Core do
         it { should have_db_index(:public_id) }
         it { should have_db_index(:realm_id) }
         it { should have_db_index(:workspace_id) }
+
+        context 'partial' do
+          #
+          # lets
+          #
+
+          let(:metasploit_credential_origin) {
+            FactoryGirl.create(:metasploit_credential_origin_manual)
+          }
+
+          let(:workspace) {
+            FactoryGirl.create(:mdm_workspace)
+          }
+
+          let(:second_metasploit_credential_core) {
+            FactoryGirl.build(
+                :metasploit_credential_core,
+                origin: metasploit_credential_origin,
+                private: private,
+                public: public,
+                workspace: workspace
+            )
+          }
+
+          #
+          # let!s
+          #
+
+          let!(:first_metasploit_credential_core) {
+            FactoryGirl.create(
+                :metasploit_credential_core,
+                origin: metasploit_credential_origin,
+                private: private,
+                public: public,
+                workspace: workspace
+            )
+          }
+
+          context '#private_id' do
+            context 'with nil' do
+              let(:private) {
+                nil
+              }
+
+              context '#public_id' do
+                context 'without nil' do
+                  let(:public) {
+                    FactoryGirl.create(:metasploit_credential_public)
+                  }
+
+                  it 'does not allow duplicates' do
+                    expect {
+                      second_metasploit_credential_core.save(validate: false)
+                    }.to raise_error(ActiveRecord::RecordNotUnique)
+                  end
+                end
+              end
+            end
+
+            context 'without nil' do
+              let(:private) do
+                FactoryGirl.create(:metasploit_credential_password)
+              end
+
+              context '#public_id' do
+                context 'with nil' do
+                  let(:public) {
+                    nil
+                  }
+
+                  it 'does not allow duplicates' do
+                    expect {
+                      second_metasploit_credential_core.save(validate: false)
+                    }.to raise_error(ActiveRecord::RecordNotUnique)
+                  end
+                end
+
+                context 'without nil' do
+                  let(:public) {
+                    FactoryGirl.create(:metasploit_credential_public)
+                  }
+
+                  it 'does not allow duplicates' do
+                    expect {
+                      second_metasploit_credential_core.save(validate: false)
+                    }.to raise_error(ActiveRecord::RecordNotUnique)
+                  end
+                end
+              end
+            end
+          end
+        end
       end
     end
   end
@@ -388,6 +480,104 @@ describe Metasploit::Credential::Core do
     it { should validate_presence_of :origin }
     it { should validate_presence_of :workspace }
 
+    context 'of uniqueness' do
+      #
+      # lets
+      #
+
+      let(:metasploit_credential_origin) {
+        FactoryGirl.create(:metasploit_credential_origin_manual)
+      }
+
+      let(:workspace) {
+        FactoryGirl.create(:mdm_workspace)
+      }
+
+      let(:second_metasploit_credential_core) {
+        FactoryGirl.build(
+            :metasploit_credential_core,
+            origin: metasploit_credential_origin,
+            private: private,
+            public: public,
+            workspace: workspace
+        )
+      }
+
+      let(:taken_error) {
+        I18n.translate!('activerecord.errors.messages.taken')
+      }
+
+      #
+      # let!s
+      #
+
+      let!(:first_metasploit_credential_core) {
+        FactoryGirl.create(
+            :metasploit_credential_core,
+            origin: metasploit_credential_origin,
+            private: private,
+            public: public,
+            workspace: workspace
+        )
+      }
+
+      #
+      # Callbacks
+      #
+
+      before(:each) do
+        second_metasploit_credential_core.valid?
+      end
+
+      context '#private_id' do
+        context 'with nil' do
+          let(:private) {
+            nil
+          }
+
+          context '#public_id' do
+            context 'without nil' do
+              let(:public) {
+                FactoryGirl.create(:metasploit_credential_public)
+              }
+
+              it 'does not allow duplicates on #public_id' do
+                expect(second_metasploit_credential_core.errors[:public_id]).to include(taken_error)
+              end
+            end
+          end
+        end
+
+        context 'without nil' do
+          let(:private) do
+            FactoryGirl.create(:metasploit_credential_password)
+          end
+
+          context '#public_id' do
+            context 'with nil' do
+              let(:public) {
+                nil
+              }
+
+              it 'does not allow duplicates on #private_id' do
+                expect(second_metasploit_credential_core.errors[:private_id]).to include(taken_error)
+              end
+            end
+
+            context 'without nil' do
+              let(:public) {
+                FactoryGirl.create(:metasploit_credential_public)
+              }
+
+              it 'does not allow duplicates on #private_id' do
+                expect(second_metasploit_credential_core.errors[:private_id]).to include(taken_error)
+              end
+            end
+          end
+        end
+      end
+    end
+
     context '#consistent_workspaces' do
       subject(:workspace_errors) do
         core.errors[:workspace]
@@ -777,9 +967,9 @@ describe Metasploit::Credential::Core do
 
       let(:core) do
         FactoryGirl.build(
-          :metasploit_credential_core,
-          private: FactoryGirl.build(:metasploit_credential_ssh_key),
-          public: FactoryGirl.build(:metasploit_credential_public)
+            :metasploit_credential_core,
+            private: FactoryGirl.build(:metasploit_credential_ssh_key),
+            public: FactoryGirl.build(:metasploit_credential_public)
         )
       end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: metasploit-credential
 version: !ruby/object:Gem::Version
-  version: 0.7.17
+  version: 0.8.1
 platform: ruby
 authors:
 - Luke Imhoff
@@ -152,6 +152,7 @@ files:
 - db/migrate/20140702184622_create_credential_cores_tasks_join_table.rb
 - db/migrate/20140703144541_create_credential_logins_tasks_join_table.rb
 - db/migrate/20140722174919_old_creds_to_new_creds.rb
+- db/migrate/20140728191933_unique_metasploit_credential_cores.rb
 - lib/metasploit/credential.rb
 - lib/metasploit/credential/creation.rb
 - lib/metasploit/credential/engine.rb