Affiche les sources d'aide par ordre de categories

betagouv · Jan 29, 2024 · 3590738 · 3590738
1 parent bdb98d8
commit 3590738
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 10 deletions.
diff --git a/app/models/source_aide.rb b/app/models/source_aide.rb
@@ -11,7 +11,7 @@ def display_name
   end
 
   def self.sources_par_categorie
-    all.each_with_object({}) do |source, result|
+    order(:categorie).each_with_object({}) do |source, result|
       result[source.categorie] ||= []
       result[source.categorie] << source
     end

diff --git a/spec/factories/source_aide.rb b/spec/factories/source_aide.rb
@@ -3,7 +3,7 @@
 FactoryBot.define do
   factory :source_aide do
     titre { 'titre' }
-    categorie { 'prise_en_main' }
+    categorie { :prise_en_main }
     description { 'description' }
     url { 'https://mon.domaine.net' }
     type_document { 'pdf' }

diff --git a/spec/models/source_aide_spec.rb b/spec/models/source_aide_spec.rb
@@ -10,16 +10,16 @@
   it { is_expected.to validate_presence_of(:type_document) }
 
   describe 'retourne les sources par catégories' do
-    let!(:source1) { create :source_aide, categorie: 'prise_en_main' }
-    let!(:source2) { create :source_aide, categorie: 'prise_en_main' }
-    let!(:source3) { create :source_aide, categorie: 'animer_restituer' }
+    let!(:source3) { create :source_aide, categorie: :animer_restituer }
+    let!(:source1) { create :source_aide, categorie: :prise_en_main }
+    let!(:source2) { create :source_aide, categorie: :prise_en_main }
 
     it do
-      expect(SourceAide.sources_par_categorie).to eql(
-        {
-          'prise_en_main' => [source1, source2],
-          'animer_restituer' => [source3]
-        }
+      expect(SourceAide.sources_par_categorie.to_a).to eql(
+        [
+          ['prise_en_main', [source1, source2]],
+          ['animer_restituer', [source3]]
+        ]
       )
     end
   end