From e7fe26862bc6feafb72fa6dbcac7b652dd6cdc42 Mon Sep 17 00:00:00 2001 From: Monica Mateiu Date: Tue, 7 Jun 2022 15:15:13 +0100 Subject: [PATCH] ignore spaminess rating to safely remove database column (#17846) * ignore spaminess rating to safely remove database column * add spec for ignored columns --- app/models/article.rb | 2 ++ spec/models/article_spec.rb | 6 ++++++ 2 files changed, 8 insertions(+) diff --git a/app/models/article.rb b/app/models/article.rb index 733293735..188388df1 100644 --- a/app/models/article.rb +++ b/app/models/article.rb @@ -401,6 +401,8 @@ class Article < ApplicationRecord scope :eager_load_serialized_data, -> { includes(:user, :organization, :tags) } + self.ignored_columns = ["spaminess_rating"] + def self.seo_boostable(tag = nil, time_ago = 18.days.ago) # Time ago sometimes returns this phrase instead of a date time_ago = 5.days.ago if time_ago == "latest" diff --git a/spec/models/article_spec.rb b/spec/models/article_spec.rb index f5a9db75c..5223dc4cb 100644 --- a/spec/models/article_spec.rb +++ b/spec/models/article_spec.rb @@ -13,6 +13,12 @@ RSpec.describe Article, type: :model do include_examples "#sync_reactions_count", :article it_behaves_like "UserSubscriptionSourceable" + describe "ignored columns" do + it "ignores spaminess rating" do + expect(described_class.ignored_columns).to eq ["spaminess_rating"] + end + end + describe "validations" do it { is_expected.to belong_to(:collection).optional } it { is_expected.to belong_to(:organization).optional }