Skip to content

Commit d146d4e

Browse files
committed
refactor: Remove TupleReflectionConverter and related tests to streamline serialization logic
1 parent cac3706 commit d146d4e

3 files changed

Lines changed: 1 addition & 80 deletions

File tree

ReflectorNet.Tests/src/ReflectorTests/ValueTuplePropertiesTests.cs

Lines changed: 1 addition & 36 deletions
Original file line numberDiff line numberDiff line change
@@ -134,48 +134,13 @@ public void FilterIndexerProperties()
134134
}
135135
}
136136

137-
[Fact]
138-
public void TupleReflectionConverter_IsSelectedForValueTuple()
139-
{
140-
// Arrange
141-
var reflector = new Reflector();
142-
var tupleType = typeof((int, bool));
143-
144-
// Act
145-
var converter = reflector.Converters.GetConverter(tupleType);
146-
147-
// Assert
148-
Assert.NotNull(converter);
149-
Assert.IsType<TupleReflectionConverter>(converter);
150-
151-
_output.WriteLine($"Converter type: {converter.GetType().Name}");
152-
_output.WriteLine($"Converter priority for {tupleType.Name}: {converter.SerializationPriority(tupleType)}");
153-
}
154-
155-
[Fact]
156-
public void TupleReflectionConverter_IsSelectedForTuple3()
157-
{
158-
// Arrange
159-
var reflector = new Reflector();
160-
var tupleType = typeof((int, string, bool));
161-
162-
// Act
163-
var converter = reflector.Converters.GetConverter(tupleType);
164-
165-
// Assert
166-
Assert.NotNull(converter);
167-
Assert.IsType<TupleReflectionConverter>(converter);
168-
169-
_output.WriteLine($"Converter type: {converter.GetType().Name}");
170-
}
171-
172137
[Fact]
173138
public void TupleReflectionConverter_FiltersIndexerProperties()
174139
{
175140
// Arrange
176141
var reflector = new Reflector();
177142
var tupleType = typeof((int, bool));
178-
var converter = reflector.Converters.GetConverter(tupleType) as TupleReflectionConverter;
143+
var converter = reflector.Converters.GetConverter(tupleType);
179144
var flags = BindingFlags.Instance | BindingFlags.Public | BindingFlags.NonPublic;
180145

181146
Assert.NotNull(converter);

ReflectorNet/src/Converter/Reflection/TupleReflectionConverter.cs

Lines changed: 0 additions & 41 deletions
This file was deleted.

ReflectorNet/src/Reflector/Reflector.Registry.cs

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -60,9 +60,6 @@ public Registry()
6060
// Specialized converters for read-only system types
6161
Add(new TypeReflectionConverter());
6262
Add(new AssemblyReflectionConverter());
63-
64-
// Specialized converter for ITuple types (ValueTuple, Tuple)
65-
Add(new TupleReflectionConverter());
6663
}
6764

6865
/// <summary>

0 commit comments

Comments
 (0)