Skip to content

Commit a9d5d99

Browse files
devleadgep13
authored andcommitted
(GH-4150,GH-4197) Codegen styling & nullable param
* fixes #4150 - Fix styling warnings * fixes #4197 - Fix parameter nullability error
1 parent eda04f6 commit a9d5d99

File tree

57 files changed

+197
-393
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

57 files changed

+197
-393
lines changed

src/Cake.Core.Tests/Data/MethodAliasGeneratorData.cs

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -246,5 +246,21 @@ public static dynamic NonGeneric_ExtensionMethodWithDynamicReturnValue(this ICak
246246
{
247247
throw new NotImplementedException();
248248
}
249+
250+
[CakeMethodAlias]
251+
#nullable enable
252+
public static void NonGeneric_ExtensionMethodWithNullableParameter(this ICakeContext context, string? parameter)
253+
#nullable disable
254+
{
255+
throw new NotImplementedException();
256+
}
257+
258+
[CakeMethodAlias]
259+
#nullable enable
260+
public static string? NonGeneric_ExtensionMethodWithNullableReturnValue(this ICakeContext context)
261+
#nullable disable
262+
{
263+
throw new NotImplementedException();
264+
}
249265
}
250266
}

src/Cake.Core.Tests/Data/PropertyAliasGeneratorData.cs

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -123,5 +123,13 @@ public static int Cached_Obsolete_ExplicitError_WithMessage(this ICakeContext co
123123
{
124124
throw new NotImplementedException();
125125
}
126+
127+
[CakePropertyAlias(Cache = true)]
128+
#nullable enable
129+
public static string? Cached_Nullable_Type(this ICakeContext context)
130+
#nullable disable
131+
{
132+
throw new NotImplementedException();
133+
}
126134
}
127135
}
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,3 @@
11
[System.Diagnostics.DebuggerStepThrough]
22
public void Generic_ExtensionMethod<TTest>()
3-
{
4-
Cake.Core.Tests.Data.MethodAliasGeneratorData.Generic_ExtensionMethod<TTest>(Context);
5-
}
3+
=> Cake.Core.Tests.Data.MethodAliasGeneratorData.Generic_ExtensionMethod<TTest>(Context);
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,3 @@
11
[System.Diagnostics.DebuggerStepThrough]
22
public TTest Generic_ExtensionMethodWithGenericReturnValue<TTest>(TTest value)
3-
{
4-
return Cake.Core.Tests.Data.MethodAliasGeneratorData.Generic_ExtensionMethodWithGenericReturnValue<TTest>(Context, value);
5-
}
3+
=> Cake.Core.Tests.Data.MethodAliasGeneratorData.Generic_ExtensionMethodWithGenericReturnValue<TTest>(Context, value);

src/Cake.Core.Tests/Expectations/MethodAliasGeneratorTests.TheGeneratorMethod.Should_Return_Correct_Generated_Code_For_Generic_Methods_name=Generic_ExtensionMethodWithGenericReturnValueAndTypeParamConstraints.verified.cake

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,4 @@
22
public TOut Generic_ExtensionMethodWithGenericReturnValueAndTypeParamConstraints<TIn, TOut>(TIn arg)
33
where TIn : class, new()
44
where TOut : System.Collections.ArrayList, System.IDisposable
5-
{
6-
return Cake.Core.Tests.Data.MethodAliasGeneratorData.Generic_ExtensionMethodWithGenericReturnValueAndTypeParamConstraints<TIn, TOut>(Context, arg);
7-
}
5+
=> Cake.Core.Tests.Data.MethodAliasGeneratorData.Generic_ExtensionMethodWithGenericReturnValueAndTypeParamConstraints<TIn, TOut>(Context, arg);
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,3 @@
11
[System.Diagnostics.DebuggerStepThrough]
22
public void Generic_ExtensionMethodWithParameter<TTest>(TTest value)
3-
{
4-
Cake.Core.Tests.Data.MethodAliasGeneratorData.Generic_ExtensionMethodWithParameter<TTest>(Context, value);
5-
}
3+
=> Cake.Core.Tests.Data.MethodAliasGeneratorData.Generic_ExtensionMethodWithParameter<TTest>(Context, value);
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,3 @@
11
[System.Diagnostics.DebuggerStepThrough]
22
public dynamic NonGeneric_ExtensionMethodWithDynamicReturnValue()
3-
{
4-
return Cake.Core.Tests.Data.MethodAliasGeneratorData.NonGeneric_ExtensionMethodWithDynamicReturnValue(Context);
5-
}
3+
=> Cake.Core.Tests.Data.MethodAliasGeneratorData.NonGeneric_ExtensionMethodWithDynamicReturnValue(Context);
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,3 @@
11
[System.Diagnostics.DebuggerStepThrough]
22
public void NonGeneric_ExtensionMethodWithGenericCollectionOfNestedType(System.Collections.Generic.ICollection<Cake.Core.Tests.Data.MethodAliasGeneratorData.TestNestedEnum> items)
3-
{
4-
Cake.Core.Tests.Data.MethodAliasGeneratorData.NonGeneric_ExtensionMethodWithGenericCollectionOfNestedType(Context, items);
5-
}
3+
=> Cake.Core.Tests.Data.MethodAliasGeneratorData.NonGeneric_ExtensionMethodWithGenericCollectionOfNestedType(Context, items);
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,3 @@
11
[System.Diagnostics.DebuggerStepThrough]
22
public void NonGeneric_ExtensionMethodWithGenericExpressionArrayParameter(System.Linq.Expressions.Expression<System.Func<System.String, System.String>>[] expression)
3-
{
4-
Cake.Core.Tests.Data.MethodAliasGeneratorData.NonGeneric_ExtensionMethodWithGenericExpressionArrayParameter(Context, expression);
5-
}
3+
=> Cake.Core.Tests.Data.MethodAliasGeneratorData.NonGeneric_ExtensionMethodWithGenericExpressionArrayParameter(Context, expression);
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,3 @@
11
[System.Diagnostics.DebuggerStepThrough]
22
public void NonGeneric_ExtensionMethodWithGenericExpressionParameter(System.Linq.Expressions.Expression<System.Func<System.String, System.String>> expression)
3-
{
4-
Cake.Core.Tests.Data.MethodAliasGeneratorData.NonGeneric_ExtensionMethodWithGenericExpressionParameter(Context, expression);
5-
}
3+
=> Cake.Core.Tests.Data.MethodAliasGeneratorData.NonGeneric_ExtensionMethodWithGenericExpressionParameter(Context, expression);

0 commit comments

Comments
 (0)