@@ -1566,12 +1566,14 @@ func testReconnectingPTY(ctx context.Context, t *testing.T, client *codersdk.Cli
15661566}
15671567
15681568// Accessing an app should update the workspace's LastUsedAt.
1569+ // Despite our efforts with the flush channel, this is inherently racy.
1570+ // Retry a few times to ensure that the stats collection pipeline has flushed.
15691571func assertWorkspaceLastUsedAtUpdated (t testing.TB ,details * Details ) {
15701572t .Helper ()
15711573
1574+ <- time .After (testutil .IntervalMedium )
15721575details .FlushStats ()
1573- // Despite our efforts with the flush channel, this is inherently racy.
1574- // Retry a few times to ensure that the stats collection pipeline has flushed.
1576+ <- time .After (testutil .IntervalMedium )
15751577require .Eventually (t ,func ()bool {
15761578ws ,err := details .SDKClient .Workspace (context .Background (),details .Workspace .ID )
15771579assert .NoError (t ,err )
@@ -1580,13 +1582,14 @@ func assertWorkspaceLastUsedAtUpdated(t testing.TB, details *Details) {
15801582}
15811583
15821584// Except when it sometimes shouldn't (e.g. no access)
1585+ // Despite our efforts with the flush channel, this is inherently racy.
1586+ // Wait for a short time to ensure that the stats collection pipeline has flushed.
15831587func assertWorkspaceLastUsedAtNotUpdated (t testing.TB ,details * Details ) {
15841588t .Helper ()
15851589
1590+ <- time .After (testutil .IntervalMedium )// wait for connections to close
15861591details .FlushStats ()
1587- // Despite our efforts with the flush channel, this is inherently racy.
1588- // Wait for a short time to ensure that the stats collection pipeline has flushed.
1589- <- time .After (testutil .IntervalSlow )
1592+ <- time .After (testutil .IntervalMedium )
15901593ws ,err := details .SDKClient .Workspace (context .Background (),details .Workspace .ID )
15911594require .NoError (t ,err )
15921595require .Equal (t ,ws .LastUsedAt ,details .Workspace .LastUsedAt ,"workspace LastUsedAt updated when it should not have been" )