metasploit_data_models 2.0.17 → 3.0.0

data/spec/app/models/mdm/service_spec.rb

@@ -41,29 +41,29 @@ RSpec.describe Mdm::Service, type: :model do
   context 'scopes' do
     context "inactive" do
       it "should exclude open services" do
-        open_service = FactoryGirl.create(:mdm_service, :state => 'open')
+        open_service = FactoryBot.create(:mdm_service, :state => 'open')
         expect(Mdm::Service.inactive).not_to include(open_service)
       end
     end
 
     context "with_state open" do
       it "should exclude closed services" do
-        closed_service = FactoryGirl.create(:mdm_service, :state => 'closed')
+        closed_service = FactoryBot.create(:mdm_service, :state => 'closed')
         expect(Mdm::Service.with_state('open')).not_to include(closed_service)
       end
     end
 
     context 'search' do
       it 'should find only services that match for \'tcp\'' do
-        tcp_service   = FactoryGirl.create(:mdm_service, proto: 'tcp')
-        udp_service    =  FactoryGirl.create(:mdm_service, proto: 'udp')
+        tcp_service   = FactoryBot.create(:mdm_service, proto: 'tcp')
+        udp_service    =  FactoryBot.create(:mdm_service, proto: 'udp')
         search_results = Mdm::Service.search('tcp')
         expect(search_results).to     include(tcp_service)
         expect(search_results).not_to include(udp_service)
       end
 
       it 'should query host name of services' do
-        service = FactoryGirl.create(:mdm_service)
+        service = FactoryBot.create(:mdm_service)
         host_name = service.host.name
         expect(Mdm::Service.search(host_name)).to include(service)
       end
@@ -75,14 +75,14 @@ RSpec.describe Mdm::Service, type: :model do
       include_context 'Rex::Text'
 
       it 'should call #normalize_host_os' do
-        svc = FactoryGirl.create(:mdm_service)
+        svc = FactoryBot.create(:mdm_service)
         expect(svc).to receive(:normalize_host_os)
         svc.run_callbacks(:save)
       end
 
       it 'should include recog data when there is a match' do
-        host = FactoryGirl.create(:mdm_host)
-        FactoryGirl.create(
+        host = FactoryBot.create(:mdm_host)
+        FactoryBot.create(
           :mdm_service,
           :host => host,
           :name => 'ftp',
@@ -93,8 +93,8 @@ RSpec.describe Mdm::Service, type: :model do
       end
 
       it 'should not include recog data when there is not a match' do
-        host = FactoryGirl.create(:mdm_host)
-        FactoryGirl.create(
+        host = FactoryBot.create(:mdm_host)
+        FactoryBot.create(
           :mdm_service,
           :host => host,
           :name => 'ftp',
@@ -107,14 +107,14 @@ RSpec.describe Mdm::Service, type: :model do
 
   context 'factory' do
     it 'should be valid' do
-      service = FactoryGirl.build(:mdm_service)
+      service = FactoryBot.build(:mdm_service)
       expect(service).to be_valid
     end
   end
 
   context '#destroy' do
     it 'should successfully destroy the object' do
-      service = FactoryGirl.create(:mdm_service)
+      service = FactoryBot.create(:mdm_service)
       expect {
         service.destroy
       }.to_not raise_error
@@ -179,38 +179,38 @@ RSpec.describe Mdm::Service, type: :model do
 
     context 'port' do
       it 'should require a port' do
-        portless_service= FactoryGirl.build(:mdm_service, :port => nil)
+        portless_service= FactoryBot.build(:mdm_service, :port => nil)
         expect(portless_service).not_to be_valid
         expect(portless_service.errors[:port]).to include("is not a number")
       end
 
       it 'should not be valid for out-of-range numbers' do
-        out_of_range = FactoryGirl.build(:mdm_service, :port => 70000)
+        out_of_range = FactoryBot.build(:mdm_service, :port => 70000)
         expect(out_of_range).not_to be_valid
         expect(out_of_range.errors[:port]).to include("is not included in the list")
       end
 
       it 'should not be valid for port 0' do
-        out_of_range = FactoryGirl.build(:mdm_service, :port => 0)
+        out_of_range = FactoryBot.build(:mdm_service, :port => 0)
         expect(out_of_range).not_to be_valid
         expect(out_of_range.errors[:port]).to include("is not included in the list")
       end
 
       it 'should not be valid for decimal numbers' do
-        out_of_range = FactoryGirl.build(:mdm_service, :port => 5.67)
+        out_of_range = FactoryBot.build(:mdm_service, :port => 5.67)
         expect(out_of_range).not_to be_valid
         expect(out_of_range.errors[:port]).to include("must be an integer")
       end
 
       it 'should not be valid for a negative number' do
-        out_of_range = FactoryGirl.build(:mdm_service, :port => -8)
+        out_of_range = FactoryBot.build(:mdm_service, :port => -8)
         expect(out_of_range).not_to be_valid
         expect(out_of_range.errors[:port]).to include("is not included in the list")
       end
     end
 
     subject(:mdm_service) {
-      FactoryGirl.build(:mdm_service)
+      FactoryBot.build(:mdm_service)
     }
 
     it 'validate port is only an integer' do
@@ -220,8 +220,8 @@ RSpec.describe Mdm::Service, type: :model do
     it { is_expected.to validate_inclusion_of(:proto).in_array(described_class::PROTOS) }
 
     context 'when a duplicate service already exists' do
-      let(:service1) { FactoryGirl.create(:mdm_service)}
-      let(:service2) { FactoryGirl.build(:mdm_service, :host => service1.host, :port => service1.port, :proto => service1.proto )}
+      let(:service1) { FactoryBot.create(:mdm_service)}
+      let(:service2) { FactoryBot.build(:mdm_service, :host => service1.host, :port => service1.port, :proto => service1.proto )}
       it 'is not valid' do
         expect(service2).to_not be_valid
       end

data/spec/app/models/mdm/session_event_spec.rb

@@ -7,14 +7,14 @@ RSpec.describe Mdm::SessionEvent, type: :model do
 
   context 'factory' do
     it 'should be valid' do
-      session_event = FactoryGirl.build(:mdm_session_event)
+      session_event = FactoryBot.build(:mdm_session_event)
       expect(session_event).to be_valid
     end
   end
 
   context '#destroy' do
     it 'should successfully destroy the object' do
-      session_event = FactoryGirl.create(:mdm_session_event)
+      session_event = FactoryBot.create(:mdm_session_event)
       expect {
         session_event.destroy
       }.to_not raise_error
@@ -39,4 +39,4 @@ RSpec.describe Mdm::SessionEvent, type: :model do
     end
   end
 
-end
+end

data/spec/app/models/mdm/session_spec.rb

@@ -3,14 +3,14 @@ RSpec.describe Mdm::Session, type: :model do
 
   context 'factory' do
     it 'should be valid' do
-      session = FactoryGirl.build(:mdm_session)
+      session = FactoryBot.build(:mdm_session)
       expect(session).to be_valid
     end
   end
 
   context '#destroy' do
     it 'should successfully destroy the object' do
-      session = FactoryGirl.create(:mdm_session)
+      session = FactoryBot.create(:mdm_session)
       expect {
         session.destroy
       }.to_not raise_error
@@ -56,8 +56,8 @@ RSpec.describe Mdm::Session, type: :model do
   context 'scopes' do
     context 'alive' do
       it 'should return sessions that have not been closed' do
-        alive_session = FactoryGirl.create(:mdm_session)
-        dead_session = FactoryGirl.create(:mdm_session, :closed_at => Time.now)
+        alive_session = FactoryBot.create(:mdm_session)
+        dead_session = FactoryBot.create(:mdm_session, :closed_at => Time.now)
         alive_set = Mdm::Session.alive
         expect(alive_set).to include(alive_session)
         expect(alive_set).not_to include(dead_session)
@@ -66,8 +66,8 @@ RSpec.describe Mdm::Session, type: :model do
 
     context 'dead'  do
       it 'should return sessions that have been closed' do
-        alive_session = FactoryGirl.create(:mdm_session)
-        dead_session = FactoryGirl.create(:mdm_session, :closed_at => Time.now)
+        alive_session = FactoryBot.create(:mdm_session)
+        dead_session = FactoryBot.create(:mdm_session, :closed_at => Time.now)
         dead_set = Mdm::Session.dead
         expect(dead_set).not_to include(alive_session)
         expect(dead_set).to include(dead_session)
@@ -76,9 +76,9 @@ RSpec.describe Mdm::Session, type: :model do
 
     context 'upgradeable' do
       it 'should return sessions that can be upgraded to meterpreter' do
-        win_shell = FactoryGirl.create(:mdm_session, :stype => 'shell', :platform => 'Windows')
-        linux_shell = FactoryGirl.create(:mdm_session, :stype => 'shell', :platform => 'Linux')
-        win_meterp = FactoryGirl.create(:mdm_session, :stype => 'meterpreter', :platform => 'Windows')
+        win_shell = FactoryBot.create(:mdm_session, :stype => 'shell', :platform => 'Windows')
+        linux_shell = FactoryBot.create(:mdm_session, :stype => 'shell', :platform => 'Linux')
+        win_meterp = FactoryBot.create(:mdm_session, :stype => 'meterpreter', :platform => 'Windows')
         upgrade_set = Mdm::Session.upgradeable
         expect(upgrade_set).to include(win_shell)
         expect(upgrade_set).not_to include(linux_shell)
@@ -90,7 +90,7 @@ RSpec.describe Mdm::Session, type: :model do
   context 'callbacks' do
     context 'before_destroy' do
       it 'should call #stop' do
-        mysession = FactoryGirl.create(:mdm_session)
+        mysession = FactoryBot.create(:mdm_session)
         expect(mysession).to receive(:stop)
         mysession.destroy
       end
@@ -100,19 +100,19 @@ RSpec.describe Mdm::Session, type: :model do
   context 'methods' do
     context '#upgradeable?' do
       it 'should return true for windows shells' do
-        win_shell = FactoryGirl.create(:mdm_session, :stype => 'shell', :platform => 'Windows')
+        win_shell = FactoryBot.create(:mdm_session, :stype => 'shell', :platform => 'Windows')
         expect(win_shell.upgradeable?).to eq(true)
       end
 
       it 'should return false for non-windows shells' do
-        linux_shell = FactoryGirl.create(:mdm_session, :stype => 'shell', :platform => 'Linux')
+        linux_shell = FactoryBot.create(:mdm_session, :stype => 'shell', :platform => 'Linux')
         expect(linux_shell.upgradeable?).to eq(false)
       end
 
       it 'should return false for Windows Meterpreter Sessions' do
-        win_meterp = FactoryGirl.create(:mdm_session, :stype => 'meterpreter', :platform => 'Windows')
+        win_meterp = FactoryBot.create(:mdm_session, :stype => 'meterpreter', :platform => 'Windows')
         expect(win_meterp.upgradeable?).to eq(false)
       end
     end
   end
-end
+end

data/spec/app/models/mdm/tag_spec.rb

@@ -30,7 +30,7 @@ RSpec.describe Mdm::Tag, type: :model do
     context 'desc'  do
       it 'should not ba valid for a length over 8k' do
         desc = SecureRandom.hex(9001) #over 9000?!
-        large_tag = FactoryGirl.build(:mdm_tag, :desc => desc)
+        large_tag = FactoryBot.build(:mdm_tag, :desc => desc)
         expect(large_tag).not_to be_valid
         expect(large_tag.errors[:desc]).to include('desc must be less than 8k.')
       end
@@ -39,28 +39,28 @@ RSpec.describe Mdm::Tag, type: :model do
     context 'name' do
       let(:error_msg) {I18n.t('activerecord.ancestors.mdm/tag.model.errors.messages.character')}
       it 'must be present' do
-        nameless_tag = FactoryGirl.build(:mdm_tag, :name => nil)
+        nameless_tag = FactoryBot.build(:mdm_tag, :name => nil)
         expect(nameless_tag).not_to be_valid
         expect(nameless_tag.errors[:name]).to include("can't be blank")
       end
 
       it 'may only contain alphanumerics, dot, dashes, and underscores' do
-        mytag = FactoryGirl.build(:mdm_tag, :name => 'A.1-B_2')
+        mytag = FactoryBot.build(:mdm_tag, :name => 'A.1-B_2')
         expect(mytag).to be_valid
         #Test for various bad inputs we should never allow
-        mytag = FactoryGirl.build(:mdm_tag, :name => "A'1")
+        mytag = FactoryBot.build(:mdm_tag, :name => "A'1")
         expect(mytag).not_to be_valid
         expect(mytag.errors[:name]).to include(error_msg)
-        mytag = FactoryGirl.build(:mdm_tag, :name => "A;1")
+        mytag = FactoryBot.build(:mdm_tag, :name => "A;1")
         expect(mytag).not_to be_valid
         expect(mytag.errors[:name]).to include(error_msg)
-        mytag = FactoryGirl.build(:mdm_tag, :name => "A%1")
+        mytag = FactoryBot.build(:mdm_tag, :name => "A%1")
         expect(mytag).not_to be_valid
         expect(mytag.errors[:name]).to include(error_msg)
-        mytag = FactoryGirl.build(:mdm_tag, :name => "A=1")
+        mytag = FactoryBot.build(:mdm_tag, :name => "A=1")
         expect(mytag).not_to be_valid
         expect(mytag.errors[:name]).to include(error_msg)
-        mytag = FactoryGirl.build(:mdm_tag, :name => "#A1")
+        mytag = FactoryBot.build(:mdm_tag, :name => "#A1")
         expect(mytag).not_to be_valid
         expect(mytag.errors[:name]).to include(error_msg)
       end
@@ -70,7 +70,7 @@ RSpec.describe Mdm::Tag, type: :model do
   context 'instance methods' do
     context '#to_s' do
       it 'should return the name of the tag as a string' do
-        mytag = FactoryGirl.build(:mdm_tag, :name => 'mytag')
+        mytag = FactoryBot.build(:mdm_tag, :name => 'mytag')
         expect(mytag.to_s).to eq('mytag')
       end
     end
@@ -79,7 +79,7 @@ RSpec.describe Mdm::Tag, type: :model do
   context 'factories' do
     context 'mdm_tag' do
       subject(:mdm_tag) do
-        FactoryGirl.build(:mdm_tag)
+        FactoryBot.build(:mdm_tag)
       end
 
       it { is_expected.to be_valid }
@@ -88,7 +88,7 @@ RSpec.describe Mdm::Tag, type: :model do
 
   context '#destroy' do
     let!(:tag) do
-      FactoryGirl.create(:mdm_tag)
+      FactoryBot.create(:mdm_tag)
     end
 
     it 'should successfully destroy the object' do
@@ -113,4 +113,4 @@ RSpec.describe Mdm::Tag, type: :model do
                             type: :string
     end
   end
-end
+end

data/spec/app/models/mdm/task_cred_spec.rb

@@ -3,7 +3,7 @@ RSpec.describe Mdm::TaskCred, type: :model do
 
   context 'factory' do
     it 'should be valid' do
-      task_cred = FactoryGirl.build(:mdm_task_cred)
+      task_cred = FactoryBot.build(:mdm_task_cred)
       expect(task_cred).to be_valid
     end
   end
@@ -23,7 +23,7 @@ RSpec.describe Mdm::TaskCred, type: :model do
 
   context '#destroy' do
     it 'should successfully destroy the object' do
-      task_cred = FactoryGirl.create(:mdm_task_cred)
+      task_cred = FactoryBot.create(:mdm_task_cred)
       expect {
         task_cred.destroy
       }.to_not raise_error
@@ -40,10 +40,10 @@ RSpec.describe Mdm::TaskCred, type: :model do
 
   context "validations" do
     it "should not allow duplicate associations" do
-      task = FactoryGirl.build(:mdm_task)
-      cred = FactoryGirl.build(:mdm_cred)
-      FactoryGirl.create(:mdm_task_cred, :task => task, :cred => cred)
-      task_cred2 = FactoryGirl.build(:mdm_task_cred, :task => task, :cred => cred)
+      task = FactoryBot.build(:mdm_task)
+      cred = FactoryBot.build(:mdm_cred)
+      FactoryBot.create(:mdm_task_cred, :task => task, :cred => cred)
+      task_cred2 = FactoryBot.build(:mdm_task_cred, :task => task, :cred => cred)
       expect(task_cred2).not_to be_valid
     end
   end

data/spec/app/models/mdm/task_host_spec.rb

@@ -3,7 +3,7 @@ RSpec.describe Mdm::TaskHost, type: :model do
 
   context 'factory' do
     it 'should be valid' do
-      task_host = FactoryGirl.build(:mdm_task_host)
+      task_host = FactoryBot.build(:mdm_task_host)
       expect(task_host).to be_valid
     end
   end
@@ -23,7 +23,7 @@ RSpec.describe Mdm::TaskHost, type: :model do
 
   context '#destroy' do
     it 'should successfully destroy the object' do
-      task_host = FactoryGirl.create(:mdm_task_host)
+      task_host = FactoryBot.create(:mdm_task_host)
       expect {
         task_host.destroy
       }.to_not raise_error
@@ -40,10 +40,10 @@ RSpec.describe Mdm::TaskHost, type: :model do
 
   context "validations" do
     it "should not allow duplicate associations" do
-      task = FactoryGirl.build(:mdm_task)
-      host = FactoryGirl.build(:mdm_host)
-      FactoryGirl.create(:mdm_task_host, :task => task, :host => host)
-      task_host2 = FactoryGirl.build(:mdm_task_host, :task => task, :host => host)
+      task = FactoryBot.build(:mdm_task)
+      host = FactoryBot.build(:mdm_host)
+      FactoryBot.create(:mdm_task_host, :task => task, :host => host)
+      task_host2 = FactoryBot.build(:mdm_task_host, :task => task, :host => host)
       expect(task_host2).not_to be_valid
     end
   end

data/spec/app/models/mdm/task_service_spec.rb

@@ -3,7 +3,7 @@ RSpec.describe Mdm::TaskService, type: :model do
 
   context 'factory' do
     it 'should be valid' do
-      task_service = FactoryGirl.build(:mdm_task_service)
+      task_service = FactoryBot.build(:mdm_task_service)
       expect(task_service).to be_valid
     end
   end
@@ -23,7 +23,7 @@ RSpec.describe Mdm::TaskService, type: :model do
 
   context '#destroy' do
     it 'should successfully destroy the object' do
-      task_service = FactoryGirl.create(:mdm_task_service)
+      task_service = FactoryBot.create(:mdm_task_service)
       expect {
         task_service.destroy
       }.to_not raise_error
@@ -40,10 +40,10 @@ RSpec.describe Mdm::TaskService, type: :model do
 
   context "validations" do
     it "should not allow duplicate associations" do
-      task = FactoryGirl.build(:mdm_task)
-      service = FactoryGirl.build(:mdm_service)
-      FactoryGirl.create(:mdm_task_service, :task => task, :service => service)
-      task_service2 = FactoryGirl.build(:mdm_task_service, :task => task, :service => service)
+      task = FactoryBot.build(:mdm_task)
+      service = FactoryBot.build(:mdm_service)
+      FactoryBot.create(:mdm_task_service, :task => task, :service => service)
+      task_service2 = FactoryBot.build(:mdm_task_service, :task => task, :service => service)
       expect(task_service2).not_to be_valid
     end
   end

data/spec/app/models/mdm/task_session_spec.rb

@@ -3,7 +3,7 @@ RSpec.describe Mdm::TaskSession, type: :model do
 
   context 'factory' do
     it 'should be valid' do
-      task_session = FactoryGirl.build(:mdm_task_session)
+      task_session = FactoryBot.build(:mdm_task_session)
       expect(task_session).to be_valid
     end
   end
@@ -23,7 +23,7 @@ RSpec.describe Mdm::TaskSession, type: :model do
 
   context '#destroy' do
     it 'should successfully destroy the object' do
-      task_session = FactoryGirl.create(:mdm_task_session)
+      task_session = FactoryBot.create(:mdm_task_session)
       expect {
         task_session.destroy
       }.to_not raise_error
@@ -35,10 +35,10 @@ RSpec.describe Mdm::TaskSession, type: :model do
 
   context "validations" do
     it "should not allow duplicate associations" do
-      task = FactoryGirl.build(:mdm_task)
-      session = FactoryGirl.build(:mdm_session)
-      FactoryGirl.create(:mdm_task_session, :task => task, :session => session)
-      task_session2 = FactoryGirl.build(:mdm_task_session, :task => task, :session => session)
+      task = FactoryBot.build(:mdm_task)
+      session = FactoryBot.build(:mdm_session)
+      FactoryBot.create(:mdm_task_session, :task => task, :session => session)
+      task_session2 = FactoryBot.build(:mdm_task_session, :task => task, :session => session)
       expect(task_session2).not_to be_valid
     end
   end

data/spec/app/models/mdm/task_spec.rb

@@ -3,7 +3,7 @@ RSpec.describe Mdm::Task, type: :model do
 
   context 'factory' do
     it 'should be valid' do
-      task = FactoryGirl.build(:mdm_task)
+      task = FactoryBot.build(:mdm_task)
       expect(task).to be_valid
     end
   end
@@ -34,7 +34,7 @@ RSpec.describe Mdm::Task, type: :model do
 
   context '#destroy' do
     it 'should successfully destroy the object' do
-      task = FactoryGirl.create(:mdm_task)
+      task = FactoryBot.create(:mdm_task)
       expect {
         task.destroy
       }.to_not raise_error
@@ -60,7 +60,7 @@ RSpec.describe Mdm::Task, type: :model do
   context 'callbacks' do
     context 'before_destroy' do
       it 'should call #delete_file' do
-        task = FactoryGirl.create(:mdm_task)
+        task = FactoryBot.create(:mdm_task)
         expect(task).to receive(:delete_file)
         task.destroy
       end

data/spec/app/models/mdm/user_spec.rb

@@ -30,14 +30,14 @@ RSpec.describe Mdm::User, type: :model do
 
   context 'factory' do
     it 'should be valid' do
-      user = FactoryGirl.build(:mdm_user)
+      user = FactoryBot.build(:mdm_user)
       expect(user).to be_valid
     end
   end
 
   context '#destroy' do
     it 'should successfully destroy the object' do
-      user = FactoryGirl.create(:mdm_user)
+      user = FactoryBot.create(:mdm_user)
       expect {
         user.destroy
       }.to_not raise_error
@@ -47,4 +47,4 @@ RSpec.describe Mdm::User, type: :model do
     end
   end
 
-end
+end

data/spec/app/models/mdm/vuln_attempt_spec.rb

@@ -25,7 +25,7 @@ RSpec.describe Mdm::VulnAttempt, type: :model do
 
   context 'validations' do
     it 'should require a vuln_id' do
-      orphan_detail = FactoryGirl.build(:mdm_vuln_detail, :vuln => nil)
+      orphan_detail = FactoryBot.build(:mdm_vuln_detail, :vuln => nil)
       expect(orphan_detail).not_to be_valid
       expect(orphan_detail.errors[:vuln_id]).to include("can't be blank")
     end
@@ -33,14 +33,14 @@ RSpec.describe Mdm::VulnAttempt, type: :model do
 
   context 'factory' do
     it 'should be valid' do
-      vuln_attempt = FactoryGirl.build(:mdm_vuln_attempt)
+      vuln_attempt = FactoryBot.build(:mdm_vuln_attempt)
       expect(vuln_attempt).to be_valid
     end
   end
 
   context '#destroy' do
     it 'should successfully destroy the object' do
-      vuln_attempt = FactoryGirl.create(:mdm_vuln_attempt)
+      vuln_attempt = FactoryBot.create(:mdm_vuln_attempt)
       expect {
         vuln_attempt.destroy
       }.to_not raise_error
@@ -50,4 +50,4 @@ RSpec.describe Mdm::VulnAttempt, type: :model do
     end
   end
 
-end
+end

data/spec/app/models/mdm/vuln_detail_spec.rb

@@ -37,7 +37,7 @@ RSpec.describe Mdm::VulnDetail, type: :model do
 
   context 'validations' do
     it 'should require a vuln_id' do
-      orphan_detail = FactoryGirl.build(:mdm_vuln_detail, :vuln => nil)
+      orphan_detail = FactoryBot.build(:mdm_vuln_detail, :vuln => nil)
       expect(orphan_detail).not_to be_valid
       expect(orphan_detail.errors[:vuln_id]).to include("can't be blank")
     end
@@ -45,14 +45,14 @@ RSpec.describe Mdm::VulnDetail, type: :model do
 
   context 'factory' do
     it 'should be valid' do
-      vuln_detail = FactoryGirl.build(:mdm_vuln_detail)
+      vuln_detail = FactoryBot.build(:mdm_vuln_detail)
       expect(vuln_detail).to be_valid
     end
   end
 
   context '#destroy' do
     it 'should successfully destroy the object' do
-      vuln_detail = FactoryGirl.create(:mdm_vuln_detail)
+      vuln_detail = FactoryBot.create(:mdm_vuln_detail)
       expect {
         vuln_detail.destroy
       }.to_not raise_error

data/spec/app/models/mdm/vuln_ref_spec.rb

@@ -4,7 +4,7 @@ RSpec.describe Mdm::VulnRef, type: :model do
   context 'factories' do
     context 'mdm_vuln_ref' do
       subject(:mdm_vuln_ref) do
-        FactoryGirl.build(:mdm_vuln_ref)
+        FactoryBot.build(:mdm_vuln_ref)
       end
 
       it { is_expected.to be_valid }
@@ -26,14 +26,14 @@ RSpec.describe Mdm::VulnRef, type: :model do
 
   context 'factory' do
     it 'should be valid' do
-      vuln_ref = FactoryGirl.build(:mdm_vuln_ref)
+      vuln_ref = FactoryBot.build(:mdm_vuln_ref)
       expect(vuln_ref).to be_valid
     end
   end
 
   context '#destroy' do
     it 'should successfully destroy the object' do
-      vuln_ref = FactoryGirl.create(:mdm_vuln_ref)
+      vuln_ref = FactoryBot.create(:mdm_vuln_ref)
       expect {
         vuln_ref.destroy
       }.to_not raise_error
@@ -43,4 +43,4 @@ RSpec.describe Mdm::VulnRef, type: :model do
     end
   end
   
-end
+end