test(coderd/x/chatd): fold taskTestFixture into workerTestFixture - #30924

taskTestFixture in tasks_test.go repeated workerTestFixture field for field, with the same seed rows, and carried its own copies of five chat setup helpers, publishChatUpdate and two message builders. This PR deletes the fixture and the copies and moves the 39 task, hook and turn-trace tests that built it onto workerTestFixture and the shared helpers. It also drops the fixture's API key row and the apiKeyID parameter of userTextMessage, which nothing read.

Why it is safe

The copies differed from the shared helpers in two ways, and both are kept:

• Clearing the recorder. The copies cleared the recording pubsub after setup, and interruptChat cleared it before interrupting, so assertions on published events see only what the code under test published. workerTestFixture.pubsub is now that recorder (from the layer below), and the shared helpers clear it at the same points. No worker test reads the fixture recorder's events, so the extra clears do not change them.
• The deadline argument. The task copy of forceExecutionState took an explicit requires-action deadline. forceExecutionStateWithDeadline takes it as a pointer (nil keeps the current one) for setRequiresActionDeadline and the one call that cleared an expired deadline. The other three callers passed an empty deadline for chats that never had one, so they call forceExecutionState.

The rest of the move keeps the old behavior:

• startRealTaskWorker now starts from testOptions, which holds the same values. It unsets the recording task starter so the worker builds its real one, as before.
• No chatd code reads the fixture's API key. The synthetic gateway key is looked up by its own token name.

After the change, rg 'taskTestFixture|taskUserTextMessage|taskAssistantToolCallMessage|f\.apiKey' coderd/ enterprise/ finds nothing.

Review focus

The clear() placement in helpers_test.go. Setup helpers clear when they finish, and interruptChat clears before it runs.

Validation

Same commands as the layer below: the 142 top-level tests in the ten files listed there, run from go test -c ./coderd/x/chatd/ binaries (plain and -race) with -test.count=1 -test.parallel 4.

The pre-commit hook passed.

Stack

The stack folds the task test fixture into the shared worker test fixture.

1. test(coderd/x/chatd): reuse recordingPubsub in the task test fixture #30923 reuse recordingPubsub in the task test fixture
2. test(coderd/x/chatd): fold taskTestFixture into workerTestFixture #30924 fold taskTestFixture into workerTestFixture (this PR)

Xum, an AI agent, opened this PR for @ibetitsmike and wrote this description.