14145379140

Committed 29 Mar 2025 11:58AM UTC coverage: 84.27% (-9.7%) from 93.985%

Build # 14145379140

Build Type

Pull #2808

github

Committed by

nimmolo

Commit Message

Handle rss_log content filters with subqueries

Pull Request Pull Request #2808: Convert Query to AR scopes

Coverage Stats

477 of 2306 new or added lines in 78 files covered. (20.69%)

1381 existing lines in 27 files now uncovered.

26562 of 31520 relevant lines covered (84.27%)

536.24 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

61.54

/app/classes/query/filter/string_filter.rb

# frozen_string_literal: true

class Query::Filter
  class StringFilter < Query::Filter
    def type
      [:string]
    end

    def on?(val)
      val.present?
    end

    def sql_conditions(query, model, vals)
      vals.map { |val| sql_condition(query, model, val) }.join(" OR ")
    end

    def scope_conditions(query, model, vals)
      return unless vals.length.positive?

      # Start without `or`, chain subsequent conditions
      conditions = sql_condition(query, model, vals.shift)
      vals.each do |val|
        conditions = conditions.or(sql_condition(query, model, val))
      end
    end
  end
end

1	# frozen_string_literal: true
2
3	class Query::Filter	1✔
4	class StringFilter < Query::Filter	1✔
5	def type	1✔
6	[:string]	8,501✔
7	end
8
9	def on?(val)	1✔
10	val.present?	16✔
11	end
12
13	def sql_conditions(query, model, vals)	1✔
UNCOV 14	vals.map { \|val\| sql_condition(query, model, val) }.join(" OR ")	×
15	end
16
17	def scope_conditions(query, model, vals)	1✔
NEW 18	return unless vals.length.positive?	×
19
20	# Start without `or`, chain subsequent conditions
NEW 21	conditions = sql_condition(query, model, vals.shift)	×
NEW 22	vals.each do \|val\|	×
NEW 23	conditions = conditions.or(sql_condition(query, model, val))	×
24	end
25	end
26	end
27	end

MushroomObserver / mushroom-observer / 14145379140

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous