fix(detectors): Better wording for single check fail / recovery

[evanpurkhiser]

No description provided.

[cursor]

Bug: Inconsistent Pluralization in Check Messages

The pluralization for "failed check" and "successful check" messages is inconsistent. Singular forms now omit "consecutive" while plural forms retain it, leading to confusing messaging about check thresholds.

 

[codecov]

❌ 1 Tests Failed:

Tests completed	Failed	Passed	Skipped
12154	1	12153	10

View the top 1 failed test(s) by shortest run time

DetectorDetails uptime detectors displays correct detector details

Stack Traces | 0.265s run time

TestingLibraryElementError: Unable to find an element with the text: 1 consecutive successful check.. This could be because the text is broken up by multiple elements. In this case, you can provide a function for your text matcher to make your matcher more flexible.

Ignored nodes: comments, script, style
...
    at Object.getElementError (.../sentry/node_modules/.pnpm/@testing-library+dom@10.4.0/node_modules/@.../dom/dist/config.js:37:19)
    at .../sentry/node_modules/.pnpm/@testing-library+dom@10.4.0/node_modules/@.../dom/dist/query-helpers.js:76:38
    at .../sentry/node_modules/.pnpm/@testing-library+dom@10.4.0/node_modules/@.../dom/dist/query-helpers.js:52:17
    at .../sentry/node_modules/.pnpm/@testing-library+dom@10.4.0/node_modules/@.../dom/dist/query-helpers.js:95:19
    at Object.getByText (.../views/detectors/detail.spec.tsx:276:21)
    at runNextTicks (node:internal/process/task_queues:65:5)
    at listOnTimeout (node:internal/timers:549:9)
    at processTimers (node:internal/timers:523:7)

To view more test analytics, go to the Test Analytics Dashboard
_{📋 Got 3 mins? Take this short survey to help us improve Test Analytics.}