Merge pull request #1442 from nagelm/fix/machine-independent-date-tests

test(frontend): make date assertions machine-independent

CHANGELOG.md

@@ -44,6 +44,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Fixed
 
+- **Frontend date/time unit tests no longer fail outside UTC / en-US.** Three tests encoded the machine timezone or locale into their expectations (`dateTimeUtils.isSame`, `RecordingUtils.toDateString`, `SeriesModalUtils.getEpisodeAirdate`), so the suite failed on boxes east of UTC or non-US locales. Fixtures now use local calendar datetimes and locale-computed expectations so the suite passes in every environment. — Thanks [@nagelm](https://github.com/nagelm)
 - **API error toasts no longer dump raw HTML error pages.** Failed responses that return Django or nginx HTML (for example 500/502/504 when the backend is down) were interpolated into the toast body as markup. Errors are now formatted for display: JSON bodies prefer `detail` / `error` / field messages, HTML and empty bodies collapse to a short status line, and long plain-text bodies are truncated. — Thanks [@nagelm](https://github.com/nagelm)
 - **Swagger/OpenAPI schema generation no longer fails under concurrent gevent requests.** Concurrent hits to `/api/schema/` (for example Swagger UI loading) could race on DRF's shared `AutoSchema` state and raise `AssertionError: Schema generation REQUIRES a view instance`, leaving Swagger empty. Cold builds could also hang the gevent uWSGI worker. Schema introspection now runs in a real OS thread, builds are single-flight per process, and the result is cached in Django's cache (Redis) so all workers share it.
 - **`/proxy/ts/change_stream` now persists `stream_id` and waits for the real switch outcome in multi-worker deployments.** When the request landed on a worker that didn't own the channel, the owning worker's `STREAM_SWITCH` pubsub handler performed the switch but wrote only `url`/`user_agent` back to the channel's Redis metadata, so `GET /proxy/ts/status` kept reporting the old `stream_id` indefinitely; the handler now persists the full switch metadata (`stream_id`, `m3u_profile`, stream name) via the same `_update_channel_metadata()` helper the direct owner path uses. `stream_name` from the initial `get_stream_info_for_switch()` lookup is now forwarded through the pubsub payload (and from `change_stream` / `next_stream` on the direct owner path) so the owner no longer re-queries the database when writing metadata. The endpoint also no longer returns an optimistic 200 on the non-owner path: the requesting worker waits (up to 15s) for the owner to confirm via the `switch_status` Redis key and answers 502 if the owner reports failure or 504 if no confirmation arrives (e.g. owner worker gone); `next_stream` gets the same treatment. Requesting a switch to the URL the channel is already playing is now treated as success (with a metadata refresh) instead of a silent no-op, and the `switch_status` key now expires (60s TTL) instead of persisting forever. (Fixes #1412)

frontend/src/utils/__tests__/dateTimeUtils.test.js

@@ -132,14 +132,14 @@ describe('dateTimeUtils', () => {
 
   describe('isSame', () => {
     it('should return true when dates are same day', () => {
-      const date1 = '2024-01-15T10:00:00Z';
-      const date2 = '2024-01-15T11:00:00Z';
+      const date1 = '2024-01-15 10:00:00';
+      const date2 = '2024-01-15 11:00:00';
       expect(dateTimeUtils.isSame(date1, date2)).toBe(true);
     });
 
     it('should return false when dates are different days', () => {
-      const date1 = '2024-01-15T10:00:00Z';
-      const date2 = '2024-01-16T10:00:00Z';
+      const date1 = '2024-01-15 10:00:00';
+      const date2 = '2024-01-16 10:00:00';
       expect(dateTimeUtils.isSame(date1, date2)).toBe(false);
     });
 

frontend/src/utils/components/__tests__/SeriesModalUtils.test.js

@@ -353,7 +353,7 @@ describe('SeriesModalUtils', () => {
     it('should format valid air date', () => {
       const episode = { air_date: '2024-01-15' };
       const formatted = getEpisodeAirdate(episode);
-      expect(formatted).toMatch(/1\/1[4|5]\/2024/);
+      expect(formatted).toBe(new Date('2024-01-15').toLocaleDateString());
     });
 
     it('should return N/A for missing air date', () => {

frontend/src/utils/forms/__tests__/RecordingUtils.test.js

@@ -111,7 +111,7 @@ describe('RecordingUtils', () => {
 
   describe('toDateString', () => {
     it('formats a Date to YYYY-MM-DD', () => {
-      const d = new Date('2024-06-15T12:00:00Z');
+      const d = new Date(2024, 5, 15, 12, 0, 0);
       const result = toDateString(d);
       expect(result).toBe('2024-06-15');
     });