Fix flaky tests dependent on exercise.reload for files

spec/controllers/code_ocean/files_controller_spec.rb

@@ -63,20 +63,21 @@ describe CodeOcean::FilesController do
   end
 
   describe 'DELETE #destroy' do
-    let(:exercise) { create(:fibonacci) }
+    let!(:exercise) { create(:fibonacci) }
-    let(:perform_request) { proc { delete :destroy, params: {id: exercise.files.first.id} } }
+    let(:perform_request) { proc { delete :destroy, params: {id: exercise.files.reject(&:main_file?).first.id} } }
 
-    before { perform_request.call }
+    context 'with request performed' do
+      before { perform_request.call }
 
-    expect_assigns(file: CodeOcean::File)
+      expect_assigns(file: CodeOcean::File)
 
-    it 'destroys the file' do
+      it 'redirects to exercise path' do
-      create(:fibonacci)
+        expect(controller).to redirect_to(exercise)
-      expect { perform_request.call }.to change(CodeOcean::File, :count).by(-1)
+      end
     end
 
-    it 'redirects to exercise path' do
+    it 'destroys the file' do
-      expect(controller).to redirect_to(exercise)
+      expect { perform_request.call }.to change(CodeOcean::File, :count).by(-1)
     end
   end
 end

spec/models/exercise_spec.rb

@@ -144,7 +144,7 @@ describe Exercise do
       before { create(:test_file, context: exercise) }
 
       it 'returns true' do
-        expect(exercise).to be_teacher_defined_assessment
+        expect(exercise.reload).to be_teacher_defined_assessment
       end
     end
 
@@ -152,7 +152,7 @@ describe Exercise do
       before { create(:test_file, context: exercise, role: 'teacher_defined_linter') }
 
       it 'returns true' do
-        expect(exercise).to be_teacher_defined_assessment
+        expect(exercise.reload).to be_teacher_defined_assessment
       end
     end
 
@@ -163,7 +163,7 @@ describe Exercise do
       end
 
       it 'returns true' do
-        expect(exercise).to be_teacher_defined_assessment
+        expect(exercise.reload).to be_teacher_defined_assessment
       end
     end
   end

spec/policies/exercise_policy_spec.rb

@@ -188,7 +188,10 @@ describe ExercisePolicy do
 
   permissions :submit? do
     context 'when teacher-defined assessments are available' do
-      before { create(:test_file, context: exercise) }
+      before do
+        create(:test_file, context: exercise)
+        exercise.reload
+      end
 
       it 'grants access to anyone' do
         %i[admin external_user teacher].each do |factory_name|