Skip op critical path benchmark test on FRS (all R11s actually)

[markfields]

Description

Fixes AB#46428

This test intentionally ignores server interactions (specifically it doesn't measure the op roundtrip time), so it shouldn't matter which service it runs against. We have seen some flakiness against FRS so drop that variant.

[Copilot]

Pull Request Overview

This PR fixes flakiness issues with the op critical path benchmark test by excluding FRS (Fluid Relay Service) and R11s variants. Since the test intentionally ignores server interactions and doesn't measure op roundtrip time, the specific service used shouldn't affect the benchmark results.

• Adds conditional skipping for R11s and routerlicious driver types
• Changes arrow function to regular function to access this.skip()

[Copilot]

The provider variable is not defined at this point in the test lifecycle. The provider is assigned in the setup() function which is called in the before hook, but this beforeEach runs before setup(). This will cause a runtime error when accessing provider.driver.type.

Suggested change

	beforeEach("conditionalSkip", async function () {
	if (provider.driver.type === "r11s" || provider.driver.type === "routerlicious") {
	this.skip(); // This test is not reliable with FRS for some reason
	}
	});
	const setup = async () => {
	testId++;
	provider = getTestObjectProvider();
	// Removed beforeEach conditionalSkip; logic moved to setup()
	const setup = async function () {
	testId++;
	provider = getTestObjectProvider();
	if (provider.driver.type === "r11s" || provider.driver.type === "routerlicious") {
	// This test is not reliable with FRS for some reason
	this.skip?.();
	return;
	}