Browse Source

Add a default ordering for schemes

pull/1030/head
Kat 4 years ago
parent
commit
749a8d0bd6
  1. 2
      app/controllers/schemes_controller.rb
  2. 1
      app/models/scheme.rb
  3. 15
      spec/models/scheme_spec.rb

2
app/controllers/schemes_controller.rb

@ -9,7 +9,7 @@ class SchemesController < ApplicationController
def index def index
redirect_to schemes_organisation_path(current_user.organisation) unless current_user.support? redirect_to schemes_organisation_path(current_user.organisation) unless current_user.support?
all_schemes = Scheme.all.order("service_name ASC") all_schemes = Scheme.all
@pagy, @schemes = pagy(filtered_collection(all_schemes, search_term)) @pagy, @schemes = pagy(filtered_collection(all_schemes, search_term))
@searched = search_term.presence @searched = search_term.presence

1
app/models/scheme.rb

@ -21,6 +21,7 @@ class Scheme < ApplicationRecord
validate :validate_confirmed validate :validate_confirmed
auto_strip_attributes :service_name auto_strip_attributes :service_name
default_scope { order(confirmed: :asc, service_name: :asc) }
SENSITIVE = { SENSITIVE = {
No: 0, No: 0,

15
spec/models/scheme_spec.rb

@ -175,4 +175,19 @@ RSpec.describe Scheme, type: :model do
end end
end end
end end
describe "all schemes" do
before do
FactoryBot.create_list(:scheme, 4)
FactoryBot.create_list(:scheme, 3, confirmed: false)
end
it "sorts the schemes by status" do
all_schemes = described_class.all
expect(all_schemes.count).to eq(7)
expect(all_schemes[0].status).to eq(:incomplete)
expect(all_schemes[1].status).to eq(:incomplete)
expect(all_schemes[2].status).to eq(:incomplete)
end
end
end end

Loading…
Cancel
Save