From 4f0d6f6e392b0787d7806175ab38b8f780fd3d93 Mon Sep 17 00:00:00 2001 From: Klaus Zanders Date: Thu, 18 Dec 2025 11:51:58 +0100 Subject: [PATCH] Move specs for role into the base class --- spec/models/global_role_spec.rb | 4 ---- spec/models/role_spec.rb | 4 ++++ 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/spec/models/global_role_spec.rb b/spec/models/global_role_spec.rb index 06337879ff9..ba745764cd9 100644 --- a/spec/models/global_role_spec.rb +++ b/spec/models/global_role_spec.rb @@ -33,10 +33,6 @@ require "spec_helper" RSpec.describe GlobalRole do let!(:global_role) { create(:global_role, name: "globalrole", permissions: ["permissions"]) } - it { is_expected.to validate_presence_of :name } - it { is_expected.to validate_uniqueness_of :name } - it { is_expected.to validate_length_of(:name).is_at_most(256) } - describe "attributes" do subject(:role) { described_class.new } diff --git a/spec/models/role_spec.rb b/spec/models/role_spec.rb index 1412c5eb5fd..be0dc6d23b0 100644 --- a/spec/models/role_spec.rb +++ b/spec/models/role_spec.rb @@ -35,6 +35,10 @@ RSpec.describe Role do let(:build_role) { build(:project_role, permissions:) } let(:created_role) { create(:project_role, permissions:) } + it { is_expected.to validate_presence_of :name } + it { is_expected.to validate_uniqueness_of(:name).case_insensitive } + it { is_expected.to validate_length_of(:name).is_at_most(256) } + describe ".create" do it "is prevented for type Role" do build_role.type = described_class.name