fix: Regression on project name sorting for materials collections

rails/app/controllers/materials_collections_controller.rb

@@ -93,7 +93,7 @@ def find_and_authorize_material_collection
   end
 
   def load_projects
-    @projects = policy_scope(Admin::Project)
+    @projects = policy_scope(Admin::Project).order(:name)
   end
 
 end

rails/spec/controllers/materials_collections_controller_spec.rb

@@ -27,8 +27,9 @@
       login_admin
     end
 
-    let(:materials_collection) { FactoryBot.create(:materials_collection) }
-    let(:project) { FactoryBot.create(:project) }
+    let(:project) { FactoryBot.create(:project, name: "z project") }
+    let(:project2) { FactoryBot.create(:project, name: "a project") }
+    let(:materials_collection) { FactoryBot.create(:materials_collection, project_id: project.id) }
     let(:valid_attributes)     { { name: "Some name", description: "Some description", project_id: project.id } }
 
     describe "GET index" do
@@ -37,6 +38,20 @@
         get :index
         expect(assigns(:materials_collections).to_a).to eq([materials_collection])
       end
+
+      it "assigns all sorted projects as @projects" do
+        project # create "z project" first
+        project2 # create "a project" second
+        materials_collection
+        get :index
+        # make sure project was created before project2
+        expect(project.id).to be < project2.id
+        # make sure the projects are sorted by name
+        ids = assigns(:projects).map {|p| p.id}
+        names = assigns(:projects).map {|p| p.name}
+        expect(ids).to eq([project2.id, project.id])
+        expect(names).to eq(["a project", "z project"])
+      end
     end
 
     describe "GET show" do