Remove duplicate namespace filter validation

What does this MR do and why?

AuditEvents::Group::ExternalStreamingDestination includes the AuditEvents::ExternallyStreamable concern, then re-declares three things the concern already provides word for word: the MAXIMUM_NAMESPACE_FILTER_COUNT constant, the validate :no_more_than_5_namespace_filters? registration, and the method behind it. This MR deletes all three.

The sibling AuditEvents::Instance::ExternalStreamingDestination relies on the concern alone, so the two classes now have the same shape.

Behaviour is unchanged. Nothing outside the class referenced the shadowed constant, and Ruby resolves Klass::CONST through included modules, so such a reference would keep working. The group spec already asserts a single error message for the over-limit case and runs the concern shared example.

References

None.

MR acceptance checklist

Evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Merge request reports

Loading
Loading