• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

pulibrary / orangelight / 0e37073d-109c-440c-949f-49d2aa86311b

18 Aug 2025 09:05PM UTC coverage: 0.482% (-94.9%) from 95.343%
0e37073d-109c-440c-949f-49d2aa86311b

push

circleci

web-flow
Replace per_page_options_for_select with custom component to avoid deprecation issue (#5186)

* Start creating new component to address deprecaton warning

* Replace per_page_options_for_select with custom component to avoid deprecation issue

Co-authored-by: Jane Sandberg <sandbergja@users.noreply.github.com>

---------

Co-authored-by: Ryan Jensen <rj1044@princeton.edu>
Co-authored-by: Jane Sandberg <sandbergja@users.noreply.github.com>

0 of 33 new or added lines in 1 file covered. (0.0%)

9374 existing lines in 213 files now uncovered.

47 of 9753 relevant lines covered (0.48%)

0.01 hits per line

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

0.0
/app/models/requests/holding_group.rb
1
# frozen_string_literal: true
2

3
# This class represents a group of physical holdings that
4
# are grouped together for display
UNCOV
5
class Requests::HoldingGroup
×
UNCOV
6
  include Comparable
×
7

UNCOV
8
  def initialize(group_name:, holdings:)
×
UNCOV
9
    @group_name = group_name
×
UNCOV
10
    @holdings = holdings
×
UNCOV
11
  end
×
12

13
  # rubocop:disable Lint/DuplicateBranch
14
  # :reek:DuplicateMethodCall
UNCOV
15
  def <=>(other)
×
UNCOV
16
    if firestone? && !other.firestone?
×
UNCOV
17
      -1 # Firestone should go first
×
UNCOV
18
    elsif !firestone? && other.firestone?
×
UNCOV
19
      1
×
UNCOV
20
    elsif off_site? && !other.off_site?
×
UNCOV
21
      1 # Off site locations should go last
×
UNCOV
22
    elsif !off_site? && other.off_site?
×
UNCOV
23
      -1
×
UNCOV
24
    else
×
UNCOV
25
      group_name <=> other.group_name
×
UNCOV
26
    end
×
UNCOV
27
  end
×
28
  # rubocop:enable Lint/DuplicateBranch
29

UNCOV
30
  def firestone?
×
UNCOV
31
    group_name.start_with? 'Firestone'
×
UNCOV
32
  end
×
33

UNCOV
34
  def off_site?
×
UNCOV
35
    group_name.start_with?('Annex', 'Forrestal') || group_name.include?('Remote Storage') || group_name.downcase.include?('(off-site)')
×
UNCOV
36
  end
×
37

UNCOV
38
  attr_reader :group_name, :holdings
×
UNCOV
39
end
×
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2025 Coveralls, Inc