Skip to content

Commit

Permalink
Detail: adjust names of derived record types
Browse files Browse the repository at this point in the history
(The type name `DerivedEmptyGenericRecord` suggests that it is generic,
which is not the case; `DerivedFromEmptyGenericRecord` is more accurate.
For consistency, let's apply the same name change to `EmptyRecord` and
its subtype.)
  • Loading branch information
stakx committed Dec 29, 2022
1 parent 2d944e4 commit 06884f4
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@

namespace Castle.DynamicProxy.Tests.Records
{
public record DerivedEmptyGenericRecord : EmptyGenericRecord<object>
public record DerivedFromEmptyGenericRecord : EmptyGenericRecord<object>
{
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@

namespace Castle.DynamicProxy.Tests.Records
{
public record DerivedEmptyRecord : EmptyRecord
public record DerivedFromEmptyRecord : EmptyRecord
{
}
}
8 changes: 4 additions & 4 deletions src/Castle.Core.Tests/DynamicProxy.Tests/RecordsTestCase.cs
Original file line number Diff line number Diff line change
Expand Up @@ -28,9 +28,9 @@ public void Can_proxy_empty_record()
}

[Test]
public void Can_proxy_derived_empty_record()
public void Can_proxy_record_derived_from_empty_record()
{
_ = generator.CreateClassProxy<DerivedEmptyRecord>(new StandardInterceptor());
_ = generator.CreateClassProxy<DerivedFromEmptyRecord>(new StandardInterceptor());
}

[Test]
Expand All @@ -40,9 +40,9 @@ public void Can_proxy_empty_generic_record()
}

[Test]
public void Can_proxy_derived_empty_generic_record()
public void Can_proxy_record_derived_from_empty_generic_record()
{
_ = generator.CreateClassProxy<DerivedEmptyGenericRecord>(new StandardInterceptor());
_ = generator.CreateClassProxy<DerivedFromEmptyGenericRecord>(new StandardInterceptor());
}
}
}

0 comments on commit 06884f4

Please sign in to comment.