Skip to content

Commit e30db1d

Browse files
Copilotjaviercn
andcommitted
Clean up debug output and finalize PropertyGetter fix
Co-authored-by: javiercn <[email protected]>
1 parent ff39b4c commit e30db1d

File tree

2 files changed

+2
-22
lines changed

2 files changed

+2
-22
lines changed

src/Components/Components/src/Reflection/PropertyGetter.cs

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -76,21 +76,15 @@ public PropertyGetter(Type targetType, PropertyInfo property)
7676
object target)
7777
where TTarget : notnull
7878
{
79-
Console.WriteLine($"CallPropertyGetter called: TTarget={typeof(TTarget)}, TValue={typeof(TValue)}, target={target}");
80-
var result = Getter((TTarget)target);
81-
Console.WriteLine($"CallPropertyGetter result: {result}");
82-
return result;
79+
return Getter((TTarget)target);
8380
}
8481

8582
private static object? CallPropertyGetterByReference<TTarget, TValue>(
8683
ByRefFunc<TTarget, TValue> Getter,
8784
object target)
8885
where TTarget : notnull
8986
{
90-
Console.WriteLine($"CallPropertyGetterByReference called: TTarget={typeof(TTarget)}, TValue={typeof(TValue)}, target={target}");
9187
var unboxed = (TTarget)target;
92-
var result = Getter(ref unboxed);
93-
Console.WriteLine($"CallPropertyGetterByReference result: {result}");
94-
return result;
88+
return Getter(ref unboxed);
9589
}
9690
}

src/Components/Components/test/SupplyParameterFromPersistentComponentStateValueProviderTests.cs

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -514,15 +514,8 @@ public async Task PersistAsync_CanPersistValueTypes_TupleProperty()
514514

515515
var renderer = new TestRenderer();
516516
var component = new ValueTypeTestComponent { TupleValue = ("test", 456) };
517-
518-
// Debug: Verify the property value is set correctly
519-
Console.WriteLine($"Component TupleValue before state creation: {component.TupleValue}");
520-
521517
var componentStates = CreateComponentState(renderer, [(component, null)], null);
522518
var componentState = componentStates.First();
523-
524-
// Debug: Verify the component in the state has the right value
525-
Console.WriteLine($"Component TupleValue after state creation: {((ValueTypeTestComponent)componentState.Component).TupleValue}");
526519

527520
// Create the provider and subscribe the component
528521
var provider = new SupplyParameterFromPersistentComponentStateValueProvider(persistenceManager.State);
@@ -540,13 +533,6 @@ public async Task PersistAsync_CanPersistValueTypes_TupleProperty()
540533
newState.InitializeExistingState(store.State);
541534

542535
var key = SupplyParameterFromPersistentComponentStateValueProvider.ComputeKey(componentState, cascadingParameterInfo.PropertyName);
543-
544-
// Debug: Check what's actually stored
545-
Assert.True(store.State.ContainsKey(key), $"Key {key} not found in store. Available keys: {string.Join(", ", store.State.Keys)}");
546-
var rawValue = store.State[key];
547-
var stringValue = System.Text.Encoding.UTF8.GetString(rawValue);
548-
Console.WriteLine($"Raw stored value: {stringValue}");
549-
550536
Assert.True(newState.TryTakeFromJson<(string, int)>(key, out var retrievedValue));
551537
Assert.Equal(("test", 456), retrievedValue);
552538
}

0 commit comments

Comments
 (0)