scarver2 / ehr-portal / 23445133996
77%

DEFAULT BRANCH: main
Ran
Jobs 1
Files 44
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 98.713% (+37.8%) from 60.915%
23445133996

push

github

web-flow 
feat(photos): batch resize actor photos and update seed references (#39)

* fix(tests): add global localStorage mock for all portal tests

- Add localStorage implementation to vitest.setup.ts
- Provides getItem, setItem, removeItem, and clear methods
- Clears localStorage after each test to prevent test pollution
- Update useInsuranceVerificationStream tests to use localStorage API
- Replace Storage.prototype spies with direct localStorage calls
- All 156 portal tests now pass ✅

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

* fix: remove unused vi import from vitest.setup.ts

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

* refactor(i18n): move photo_url validation message to locale file

- Add photo_url_format error message to config/locales/en.yml
- Update Patient model validation to use I18n key :photo_url_format
- Update Provider model validation to use I18n key :photo_url_format
- Follows Rails best practices for internationalization
- Fixes RuboCop Rails/I18nLocaleTexts violations

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

* fix: use \A instead of ^ in photo_url regex validation

- Replace multiline anchor ^ with \A for string start
- Prevents regex injection security vulnerability
- Matches Rails security best practices for regex validators
- Fixes ArgumentError about multiline anchors

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

* fix(admin): remove global before_action that caused login redirect loop

- Remove config.before_action :require_admin_role from ActiveAdmin config
- This filter was being applied to all ActiveAdmin routes, including the login page
- Caused "too many redirects" error when visiting /admin/login
- Devise's authenticate_admin_user! already handles authentication
- Add note explaining that authorization should be per-resource instead

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

* fix(smoke): add error trap and verbose debug mode

- Add ERR trap to catch u... (continued)

844 of 855 relevant lines covered (98.71%)

3.63 hits per line

Jobs
ID Job ID Ran Files Coverage
1 23445133996.1 44
98.71
 GitHub Action Run
Source Files on build 23445133996