test: fix flaky test-grpc-continuous-profile

[santigimeno]

A couple of changes:

• Extend time between enabling/disabling cont profiles.
• Fix last test to make sure it start with cont profile disabled.

Summary by CodeRabbit

• Tests
  • Improved CPU profiling test timing and validation to enhance test stability
  • Enhanced test assertions to verify profile data integrity during continuous profiling operations

[coderabbitai]

Walkthrough

A test file for gRPC continuous profiling was updated with increased timing delays, added console logging for profile counts, duplicated environment variable cleanup, and enhanced return value destructuring with assertions to verify profile data and request IDs.

Changes

Cohort / File(s)	Summary
Test Timing & Assertions test/agents/test-grpc-continuous-profile.mjs	Increased several timing waits from 250ms to 500ms, added 500ms delays for asset toggle operations, added console.log statements to report profile counts, duplicated NSOLID_CONT_CPU_PROFILE deletion in env setup, enhanced grpcServer.cpuProfile return value destructuring to capture data and requestId with validation assertions.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 Whiskers twitched with timing delight,
500ms waits now burn so bright,
Data and requestId dance in place,
Assertions verify each trace,
Test stability wins the race! ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'test: fix flaky test-grpc-continuous-profile' directly and clearly describes the main change: fixing flakiness in a specific test file.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch santi/deflake_test_grpc_cont

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[santigimeno]

Landed in f4360e5