IQSS / dataverse-frontend / 25388126577

98%

develop: 97%

push

github

Fix lint errors and the component-test failures from CI on PR #898

CI was red on three jobs; fixing everything that's deterministic locally:

Stylelint (23 errors -> 0):
- Modern color-function notation (rgba 4-arg -> rgb / %).
- Alpha-as-percent (0.06 -> 6%).
- Empty-line-before-comment in FilesTree.module.scss.
- Selector ordering on .icon-btn :disabled / :focus-visible / :hover
  (was descending specificity).
- Add /dist-uploader to .prettierignore so production bundle artifacts
  don't fail the prettier --check step.

Component tests (35 failures -> expected 0 once CI re-runs):
- entry.fullPath.startsWith(...) was throwing 'Cannot read properties
  of undefined' on test fixtures that don't carry a fullPath. Made the
  guard null-safe in useFileUploadOperations.ts. Fixes 29 failing
  FileUploader specs, 2 failing ReplaceFile specs, 1 failing
  UploadDatasetFiles spec — all hit the same code path.
- FilesTree.spec.tsx had module-scope cy.stub() calls and an
  AccessRepositoryProvider wrap that's no longer needed since the
  streaming-zip flow bypasses the access API. Removed both. (Cypress
  15 fails on cy.stub() outside a running test.)
- useStreamingZipDownload.spec.tsx happy-path and pause-on-fail tests
  were racing on cy.stub(win, 'fetch') ordering; mount now runs first,
  fetch is replaced by direct assignment afterwards (sidesteps Sinon
  defineProperty failures on non-configurable getters).

1578 of 1634 branches covered (96.57%)

Branch coverage included in aggregate %.

3789 of 3839 relevant lines covered (98.7%)

10902.77 hits per line