fix(staff): admin > Users table never refreshes after a CSV import

[dylancleal]

Summary

STAFF-12. The admin Users table never refreshed after importing a
users.csv — the server created the accounts, but the table kept
showing the pre-import list until a full page reload.

Root cause

onUserUploadSuccess (users.component.ts:111) called
this.userService.query(); with nothing subscribing it, so no
request was ever made. Subscribing it alone wouldn't have fixed
this either: ngOnInit already runs this.userService.query().subscribe()
on load, and CachedEntityService.query() returns the cached
observer once a query key has been used, with a 24h cache expiry.
So the same query key would just replay the pre-import cache.

Fix

Replaced it with the cache-bypassing fetch:

this.userService.fetchAll().subscribe({error: (e) => this.alerts.error(e)});

fetchAll() is what ngx-entity-service documents as the call that
forces bypassing the cache, and it writes into userService.cache,
which is what the table's dataSource is bound to.

Built against

• doubtfire-web fix/staff-12 off origin/11.0.x @ 1671005
• doubtfire-api 11.0.x @ 04b2c8fd99cb98f8ab5a0722b0a8e3228c598fac
• doubtfire-deploy 11.0.x @ c2bdf9d0be6fcf446238f4a58b1c71e9e0ee76da

Tests

Added users.component.spec.ts with 2 cases:

• refetches the user list from the server after a CSV upload succeeds
• surfaces an alert if the post-upload refetch fails, instead of failing silently

Both verified to fail against the pre-fix line and pass after it.

npx ng test --include='src/app/admin/states/users/users.component.spec.ts' --watch=false
-> Test Files  1 passed (1)
-> Tests  2 passed (2)

Files touched

• src/app/admin/states/users/users.component.ts
• src/app/admin/states/users/users.component.spec.ts (new)

Not covered by the test

The tests exercise onUserUploadSuccess directly with a mocked
UserService/AlertService, not a real CSV upload through the
file-uploader component and HTTP layer end-to-end.