Add error based response #68
Conversation
soroshsabz
added
enhancement
| @@ -0,0 +1,30 @@ | |||
| **/.classpath | |||
There was a problem hiding this comment.
سلام؛
آقا ما از پروژه های این مخزن داکر ایمیج میسازیم ؟ فکر کنم به ازای هر Dockerfile باید یدونه از اینا داشته باشیم.
There was a problem hiding this comment.
برای پروژه های تستی نیاز هست که DockerImage داشته باشیم ؟
| /// Represents a single response of a command/query service with additional informations about invalid items. | ||
| /// </summary> | ||
| /// <typeparam name="ValidationResultType"></typeparam> | ||
| public interface IResponse<ValidationResultType> : IResponse |
There was a problem hiding this comment.
بهتر نیست اسم این واسط بشود
IValidatedResponse
| public IList<ValidationResult> InvalidItems { get; set; } | ||
| [Obsolete("Due to incompatability with Grpc this response type is only used for backward compatibility.")] | ||
| [DataContract] | ||
| public class ErrorResponseBase : ResponseBase |
There was a problem hiding this comment.
البته منم موافقم اینطور بشود.
یه کدهایی در PR خانوم سلطانزاده دیدم بنظرم مفید است.
در سرویس ها لازم نیست همه Return Typeها از جنس Response<> باشد. البته با حفظ سلسله مراتب ارث بری
در تولید ExampleResponse خیلی بچه ها را کمک میکند
| @@ -22,4 +22,4 @@ public class InvalidItem | |||
| [DataMember(Order = 2)] | |||
| @@ -53,36 +52,21 @@ public class Response: IResponse<InvalidItem> | |||
| /// namely the 'StatusCode' property of the 'ResponseBase' class which should keep it's default numeral value when being converted. | |||
| /// </remark> | |||
| [DataContract] | |||
| public class Response<T> where T : class | |||
| // TODO: [ProtoImplement(typeof(Response))] | |||
| public class Response<T> : Response where T : class | |||
There was a problem hiding this comment.
پیاده سازی این یه باگی دارد فکر کنم باید به ازای تمام T ها اون بساط ProtoImplement را اضافه کنیم
| { | ||
| internal class GrpcTestContext<TStartup> : IDisposable where TStartup : class | ||
| { | ||
| private readonly Stopwatch _stopwatch; |
There was a problem hiding this comment.
اینها به انتهای فایل منتقل بشوند 🌹
| { | ||
| internal class ForwardingLoggerProvider : ILoggerProvider | ||
| { | ||
| private readonly LogMessage _logAction; |
There was a problem hiding this comment.
اینها به انتهای فایل منتقل بشوند 🌹
| { | ||
| public class IntegrationTestBase<TStartup> : IDisposable where TStartup : class | ||
| { | ||
| private GrpcChannel? _channel; |
| @@ -0,0 +1 @@ | |||
| global using NUnit.Framework; | |||
There was a problem hiding this comment.
فکر کنم قرار بود از اینها استفاده نکنیم
| @@ -0,0 +1,35 @@ | |||
| using BSN.Commons.GrpcIntegrationTest.Sample.AppService.Contract; | |||
There was a problem hiding this comment.
یک کسری از این ها استفاده نشده است
ITNOA
This PR resolve #67