Skip to content

Commit fea3e9a

Browse files
joshsmithxrmclaude
andcommitted
fix(tui): update service caching tests to match Lazy<T> singleton behavior
Tests previously asserted transient (NotSame) behavior. Now that services use Lazy<T> for thread safety, they correctly return the same instance. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 78c06b1 commit fea3e9a

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

tests/PPDS.Cli.Tests/Tui/InteractiveSessionTests.cs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -84,13 +84,13 @@ public void GetProfileService_ReturnsNonNullService()
8484
}
8585

8686
[Fact]
87-
public void GetProfileService_MultipleCallsReturnNewInstances()
87+
public void GetProfileService_MultipleCallsReturnSameInstance()
8888
{
8989
var service1 = _session.GetProfileService();
9090
var service2 = _session.GetProfileService();
9191

92-
// Each call creates a new instance (transient)
93-
Assert.NotSame(service1, service2);
92+
// Lazy<T> ensures thread-safe singleton per session
93+
Assert.Same(service1, service2);
9494
}
9595

9696
[Fact]
@@ -103,13 +103,13 @@ public void GetEnvironmentService_ReturnsNonNullService()
103103
}
104104

105105
[Fact]
106-
public void GetEnvironmentService_MultipleCallsReturnNewInstances()
106+
public void GetEnvironmentService_MultipleCallsReturnSameInstance()
107107
{
108108
var service1 = _session.GetEnvironmentService();
109109
var service2 = _session.GetEnvironmentService();
110110

111-
// Each call creates a new instance (transient)
112-
Assert.NotSame(service1, service2);
111+
// Lazy<T> ensures thread-safe singleton per session
112+
Assert.Same(service1, service2);
113113
}
114114

115115
[Fact]

0 commit comments

Comments
 (0)