mafredri changed the title test(cli): remove unnecessary IncludeProvisionerDaemon from task snapshot tests fix(dbfake): set StartedAt when completing jobs to prevent acquisition

mafredri changed the title fix(dbfake): set StartedAt when completing jobs to prevent acquisition test(cli/task): skip flaky SnapshotWithLogs_Table test

mafredri changed the title test(cli/task): skip flaky SnapshotWithLogs_Table test test(cli): remove unnecessary IncludeProvisionerDaemon from task snapshot tests

…shot tests

These tests use dbfake to set up database state directly and don't
need a provisioner daemon. Removing it fixes a flaky failure on
Windows where the provisioner daemon acquired a job that dbfake had
already "completed", causing the task status to be "error" instead
of "paused".

Fixes coder/internal#1322
Refs coder/internal#1323