From e035b3f986df1fbd38bad7d7e570d5421eccf66e Mon Sep 17 00:00:00 2001
From: Emopusta <emreduman225@hotmail.com>
Date: Wed, 2 Oct 2024 00:40:16 +0300
Subject: [PATCH] Add open behavior type must implement IPipelineBehavior
 interface rule and some refactor with summaries.

---
 src/MediatR/Entities/OpenBehavior.cs | 41 +++++++++++++++++++++++++---
 1 file changed, 37 insertions(+), 4 deletions(-)

diff --git a/src/MediatR/Entities/OpenBehavior.cs b/src/MediatR/Entities/OpenBehavior.cs
index e7201da3..71088415 100644
--- a/src/MediatR/Entities/OpenBehavior.cs
+++ b/src/MediatR/Entities/OpenBehavior.cs
@@ -1,20 +1,53 @@
-using System;
 using Microsoft.Extensions.DependencyInjection;
+using System;
+using System.Linq;
 
 namespace MediatR.Entities;
 /// <summary>
-/// Creates open behavior entity.
+/// Represents a registration entity for pipeline behaviors with a specified service lifetime.
 /// </summary>
 public class OpenBehavior
 {
+    /// <summary>
+    /// Initializes a new instance of the <see cref="OpenBehavior"/> class.
+    /// </summary>
+    /// <param name="openBehaviorType">The type of the pipeline behavior to register.</param>
+    /// <param name="serviceLifetime">The lifetime of the registered service. Defaults to Transient.</param>
+    /// <exception cref="InvalidOperationException">Thrown if the specified type does not implement IPipelineBehavior.</exception>
+    /// <exception cref="ArgumentNullException">Thrown if <paramref name="openBehaviorType"/> is null.</exception>
     public OpenBehavior(Type openBehaviorType, ServiceLifetime serviceLifetime = ServiceLifetime.Transient)
     {
+        ValidatePipelineBehaviorType(openBehaviorType);
         OpenBehaviorType = openBehaviorType;
         ServiceLifetime = serviceLifetime;
     }
 
-    public Type? OpenBehaviorType { get; } 
+    /// <summary>
+    /// The type of the open behavior.
+    /// </summary>
+    public Type OpenBehaviorType { get; }
+
+    /// <summary>
+    /// The service lifetime of the open behavior.
+    /// </summary>
     public ServiceLifetime ServiceLifetime { get; }
 
-    
+    /// <summary>
+    /// Validates whether the specified type implements the <see cref="IPipelineBehavior{TRequest, TResponse}"/> interface.
+    /// </summary>
+    /// <param name="openBehaviorType">The type to validate.</param>
+    /// <exception cref="InvalidOperationException">Thrown if the type does not implement <see cref="IPipelineBehavior{TRequest, TResponse}"/>.</exception>
+    /// <exception cref="ArgumentNullException">Thrown if <paramref name="openBehaviorType"/> is null.</exception>
+    private static void ValidatePipelineBehaviorType(Type openBehaviorType)
+    {
+        if (openBehaviorType == null) throw new ArgumentNullException(nameof(openBehaviorType));
+
+        bool isPipelineBehavior = openBehaviorType.GetInterfaces()
+            .Any(i => i.IsGenericType && i.GetGenericTypeDefinition() == typeof(IPipelineBehavior<,>));
+
+        if (!isPipelineBehavior)
+        {
+            throw new InvalidOperationException($"The type \"{openBehaviorType.Name}\" must implement IPipelineBehavior<,> interface.");
+        }
+    }
 }
\ No newline at end of file