Skip to content

Commit 08817b3

Browse files
authored
Simplified member access (#8422)
1 parent f060d8d commit 08817b3

File tree

19 files changed

+26
-24
lines changed

19 files changed

+26
-24
lines changed

.editorconfig

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -188,6 +188,8 @@ dotnet_diagnostic.CA1805.severity = warning
188188
dotnet_diagnostic.RCS1129.severity = none
189189
# Simplify name.
190190
dotnet_diagnostic.IDE0001.severity = warning
191+
# Simplify member access.
192+
dotnet_diagnostic.IDE0002.severity = warning
191193
# Remove unnecessary cast.
192194
dotnet_diagnostic.IDE0004.severity = warning
193195
# Remove unnecessary using directives.

src/GreenDonut/test/GreenDonut.Tests/DataLoaderTests.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -161,7 +161,7 @@ public async Task LoadParamsKeysNull()
161161
public async Task LoadParamsZeroKeys()
162162
{
163163
// arrange
164-
var fetch = TestHelpers.CreateFetch<string, string>();
164+
var fetch = CreateFetch<string, string>();
165165
var batchScheduler = new ManualBatchScheduler();
166166
var loader = new DataLoader<string, string>(fetch, batchScheduler, new DataLoaderOptions());
167167
var keys = Array.Empty<string>();
@@ -179,8 +179,8 @@ public async Task LoadParamsZeroKeys()
179179
public async Task LoadParamsResult()
180180
{
181181
// arrange
182-
var fetch = TestHelpers
183-
.CreateFetch<string, string>("Bar");
182+
var fetch =
183+
CreateFetch<string, string>("Bar");
184184
var batchScheduler = new ManualBatchScheduler();
185185
var loader = new DataLoader<string, string>(fetch, batchScheduler, new DataLoaderOptions());
186186
var keys = new[] { "Foo" };

src/HotChocolate/Core/src/Types/Internal/ExtendedType.Members.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,7 @@ private static ExtendedType Rewrite(
9595
}
9696
}
9797

98-
elementType ??= ExtendedType.FromType(itemType, cache);
98+
elementType ??= FromType(itemType, cache);
9999
}
100100

101101
if (extendedType.IsArray && elementType is null)

src/HotChocolate/Core/src/Validation/Rules/ValueVisitor.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -442,7 +442,7 @@ private static bool IsInstanceOfType(
442442
{
443443
for (var i = 0; i < list.Items.Count; i++)
444444
{
445-
if (!ValueVisitor.IsInstanceOfType(context, elementType, list.Items[i]))
445+
if (!IsInstanceOfType(context, elementType, list.Items[i]))
446446
{
447447
return false;
448448
}

src/HotChocolate/Core/test/Validation.Tests/DocumentValidatorTests.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -733,7 +733,7 @@ .. document.Definitions.Skip(1),
733733
[Fact]
734734
public void Ensure_That_Merged_Fields_Are_Not_In_Violation_Of_Duplicate_Directives_Rule()
735735
{
736-
DocumentValidatorTests.ExpectValid(
736+
ExpectValid(
737737
"""
738738
query ($a: Boolean!) {
739739
dog {

src/HotChocolate/Core/test/Validation.Tests/Types/ComplexDirective.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ protected override void Configure(
1212

1313
descriptor.Name("complex");
1414

15-
descriptor.Location(HotChocolate.Types.DirectiveLocation.Field);
15+
descriptor.Location(DirectiveLocation.Field);
1616

1717
descriptor.Argument("anyArg")
1818
.Type<AnyType>();

src/HotChocolate/Data/src/Data/Filters/Visitor/FilterProvider.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ void IFilterProviderConvention.Initialize(
5959
IFilterConvention convention)
6060
{
6161
_filterConvention = convention;
62-
base.Initialize(context);
62+
Initialize(context);
6363
}
6464

6565
void IFilterProviderConvention.Complete(IConventionContext context)

src/HotChocolate/Data/src/Data/Filters/Visitor/FilterProviderExtensions.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ void IFilterProviderConvention.Initialize(
2626
IConventionContext context,
2727
IFilterConvention convention)
2828
{
29-
base.Initialize(context);
29+
Initialize(context);
3030
}
3131

3232
void IFilterProviderConvention.Complete(IConventionContext context)

src/HotChocolate/Data/src/Data/Projections/Convention/ProjectionProviderExtension.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ public ProjectionProviderExtension(Action<IProjectionProviderDescriptor> configu
2222

2323
void IProjectionProviderConvention.Initialize(IConventionContext context)
2424
{
25-
base.Initialize(context);
25+
Initialize(context);
2626
}
2727

2828
void IProjectionProviderConvention.Complete(IConventionContext context)

src/HotChocolate/Data/src/Data/Sorting/Visitor/SortProviderExtensions.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ protected SortProviderExtensions(Action<ISortProviderDescriptor<TContext>> confi
2424

2525
void ISortProviderConvention.Initialize(IConventionContext context, ISortConvention convention)
2626
{
27-
base.Initialize(context);
27+
Initialize(context);
2828
}
2929

3030
void ISortProviderConvention.Complete(IConventionContext context)

0 commit comments

Comments
 (0)