diff --git a/lib/gitlab/database/schema_validation/schema_inconsistency.rb b/lib/gitlab/database/schema_validation/schema_inconsistency.rb deleted file mode 100644 index 9f39db5b4c0f6b90315571efe9202c69c35d23ab..0000000000000000000000000000000000000000 --- a/lib/gitlab/database/schema_validation/schema_inconsistency.rb +++ /dev/null @@ -1,17 +0,0 @@ -# frozen_string_literal: true - -module Gitlab - module Database - module SchemaValidation - class SchemaInconsistency < ApplicationRecord - self.table_name = :schema_inconsistencies - - belongs_to :issue - - validates :object_name, :valitador_name, :table_name, :diff, presence: true - - scope :with_open_issues, -> { joins(:issue).where('issue.state_id': Issue.available_states[:opened]) } - end - end - end -end diff --git a/spec/factories/gitlab/database/background_migration/schema_inconsistencies.rb b/spec/factories/gitlab/database/background_migration/schema_inconsistencies.rb deleted file mode 100644 index 1d2c460144d4e5285690d36cc946b0434bf1fc10..0000000000000000000000000000000000000000 --- a/spec/factories/gitlab/database/background_migration/schema_inconsistencies.rb +++ /dev/null @@ -1,12 +0,0 @@ -# frozen_string_literal: true - -FactoryBot.define do - factory :schema_inconsistency, class: '::Gitlab::Database::SchemaValidation::SchemaInconsistency' do - issue factory: :issue - - object_name { 'name' } - table_name { 'table' } - valitador_name { 'validator' } - diff { 'diff' } - end -end diff --git a/spec/lib/gitlab/database/schema_validation/schema_inconsistency_spec.rb b/spec/lib/gitlab/database/schema_validation/schema_inconsistency_spec.rb deleted file mode 100644 index fbaf8474f22020759914e4e48a2eb000a54b69d9..0000000000000000000000000000000000000000 --- a/spec/lib/gitlab/database/schema_validation/schema_inconsistency_spec.rb +++ /dev/null @@ -1,41 +0,0 @@ -# frozen_string_literal: true - -require 'spec_helper' - -RSpec.describe Gitlab::Database::SchemaValidation::SchemaInconsistency, type: :model, feature_category: :database do - it { is_expected.to be_a ApplicationRecord } - - describe 'associations' do - it { is_expected.to belong_to(:issue) } - end - - describe "Validations" do - it { is_expected.to validate_presence_of(:object_name) } - it { is_expected.to validate_presence_of(:valitador_name) } - it { is_expected.to validate_presence_of(:table_name) } - it { is_expected.to validate_presence_of(:diff) } - end - - describe 'scopes' do - describe '.with_open_issues' do - subject(:inconsistencies) { described_class.with_open_issues } - - let(:closed_issue) { create(:issue, :closed) } - let(:open_issue) { create(:issue, :opened) } - - let!(:schema_inconsistency_with_issue_closed) do - create(:schema_inconsistency, object_name: 'index_name', table_name: 'achievements', - valitador_name: 'different_definition_indexes', issue: closed_issue) - end - - let!(:schema_inconsistency_with_issue_opened) do - create(:schema_inconsistency, object_name: 'index_name', table_name: 'achievements', - valitador_name: 'different_definition_indexes', issue: open_issue) - end - - it 'returns only schema inconsistencies with GitLab issues open' do - expect(inconsistencies).to eq([schema_inconsistency_with_issue_opened]) - end - end - end -end