Update SchoolClass::Delete to mark as deleted

Instead of calling `destroy!` on SchoolClass, we now call
`mark_as_deleted!` instead.

Author: fspeirs

lib/concepts/school_class/operations/delete.rb

@@ -5,7 +5,7 @@ class Delete
     class << self
       def call(school:, school_class_id:)
         response = OperationResponse.new
-        delete_school_class(school, school_class_id)
+        mark_school_class_as_deleted(school, school_class_id)
         response
       rescue StandardError => e
         Sentry.capture_exception(e)
@@ -15,9 +15,9 @@ def call(school:, school_class_id:)
 
       private
 
-      def delete_school_class(school, school_class_id)
+      def mark_school_class_as_deleted(school, school_class_id)
         school_class = school.classes.find(school_class_id)
-        school_class.destroy!
+        school_class.mark_as_deleted!
       end
     end
   end

spec/concepts/school_class/delete_spec.rb

@@ -18,16 +18,21 @@
     expect(response.success?).to be(true)
   end
 
-  it 'deletes a school class' do
-    expect { described_class.call(school:, school_class_id:) }.to change(SchoolClass, :count).by(-1)
+  it 'marks the school class as deleted' do
+    described_class.call(school:, school_class_id:)
+    expect(school_class.reload.deleted?).to be true
   end
 
-  it 'deletes class students in the school class' do
-    expect { described_class.call(school:, school_class_id:) }.to change(ClassStudent, :count).by(-1)
+  it 'does not delete the school class record' do
+    expect { described_class.call(school:, school_class_id:) }.not_to change(SchoolClass, :count)
   end
 
-  it 'deletes class teachers in the school class' do
-    expect { described_class.call(school:, school_class_id:) }.to change(ClassTeacher, :count).by(-1)
+  it 'does not delete class students in the school class' do
+    expect { described_class.call(school:, school_class_id:) }.not_to change(ClassStudent, :count)
+  end
+
+  it 'does not delete class teachers in the school class' do
+    expect { described_class.call(school:, school_class_id:) }.not_to change(ClassTeacher, :count)
   end
 
   context 'when deletion fails' do