[test-utils-recorder] beforeEach recording even when the test is skipped

[willmtemple]

If a test is skipped, ideally it would generate no recording. When we were using recorder.ts in the various packages, the recording would be cancelled as soon as it would go to look for a recording and find that it was configured to be skipped, so that was the case.

In the common recorder, however, if a test suite contains setup logic in beforeEach that makes a request, then that request will be recorded (and the recording will be required for playback) even if the test is subsequently skipped, meaning that even skipped tests will require a recording (of only the beforeEach steps).

Example:

describe("A Test Suite", () => {
  beforeEach(async () => {
    recorder = record(this);
    shareName = recorder.getUniqueName("share");
    shareClient = serviceClient.getShareClient(shareName);
    await shareClient.create();
  });

  it("this test will still generate a recording", async () => {
    recorder.skip(undefined, "some reason");
    assert.ok(true);
  });
});

[HarshaNalluru]

Documented here #4336

[HarshaNalluru]

fine with this tradeoff now, nothing left to do here.