Move remaining skip-related comments outside test functions for consistency

Co-authored-by: minhanh-phan <[email protected]>

Author: Copilot

sdk/identity/identity/test/public/node/deviceCodeCredential.spec.ts

@@ -92,12 +92,11 @@ describe("DeviceCodeCredential", function () {
     },
   );
 
+  // Because of the user interaction, this test works inconsistently in our live test pipelines.
+  // On playback we can't quite control the time needed to trigger this error.
   it.skipIf(isLiveMode() || isPlaybackMode())(
     "allows cancelling the authentication",
     async function () {
-      // Because of the user interaction, this test works inconsistently in our live test pipelines.
-      // On playback we can't quite control the time needed to trigger this error.
-
       const credential = new DeviceCodeCredential(
         recorder.configureClientOptions({
           tenantId: env.AZURE_TENANT_ID,

sdk/identity/identity/test/public/node/environmentCredential.spec.ts

@@ -84,10 +84,10 @@ describe.skipIf(shouldRunSPTest())("EnvironmentCredential", function () {
     },
   );
 
+  // Live test run not supported on CI at the moment. Locally should work though.
   it.skipIf(isLiveMode())(
     "authenticates with a client certificate and password on the environment variables",
     async function () {
-      // Live test run not supported on CI at the moment. Locally should work though.
       // The following environment variables must be set for this to work.
       // On TEST_MODE="playback", the recorder automatically fills them with stubbed values.
       process.env.AZURE_TENANT_ID = cachedValues.AZURE_TENANT_ID;