Remove unneeded disabling of an unused callback

hitobito · Dec 17, 2024 · 2de86cc · 2de86cc
1 parent 86f0b18
commit 2de86cc
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 12 deletions.
diff --git a/app/models/group/root.rb b/app/models/group/root.rb
@@ -13,10 +13,6 @@ class Admin < ::Role
     self.permissions = [:layer_and_below_full, :admin]
     self.two_factor_authentication_enforced = true
 
-    def skip_alumnus_callback
-      true
-    end
-
     def alumnus_manager
       @alumnus_manager ||= Jubla::Role::NullAlumnusManager.new
     end

diff --git a/app/models/nejb_role.rb b/app/models/nejb_role.rb
@@ -7,10 +7,6 @@
 
 class NejbRole < Role
   module AlumnusFreeRole
-    def skip_alumnus_callback
-      true
-    end
-
     def alumnus_manager
       @alumnus_manager ||= Jubla::Role::NullAlumnusManager.new
     end

diff --git a/spec/models/nejb_role_spec.rb b/spec/models/nejb_role_spec.rb
@@ -14,10 +14,6 @@
     expect(subject).to be_a Jubla::Role
   end
 
-  it "disables the alumnus-callback" do
-    expect(subject.skip_alumnus_callback).to be_truthy
-  end
-
   it "uses the NullAlumnusManager" do
     expect(subject.alumnus_manager).to be_a Jubla::Role::NullAlumnusManager
   end