Skip to content

Upgrade react router #3727

Upgrade react router

Upgrade react router #3727

Triggered via pull request January 22, 2025 22:02
Status Failure
Total duration 2m 13s
Artifacts 1

ci.yml

on: pull_request
Matrix: linting
Create 'production' build
1m 52s
Create 'production' build
Run Javascript tests
2m 2s
Run Javascript tests
Build (and push) Docker image
1m 50s
Build (and push) Docker image
Publish the NPM package
0s
Publish the NPM package
update-docker-readme  /  Generate and update Docker Hub description
update-docker-readme / Generate and update Docker Hub description
Fit to window
Zoom out
Zoom in

Annotations

5 errors and 5 warnings
src/sdk.spec.jsx > OpenForm > should accept a DOM node as languageSelectorTarget: src/sdk.spec.jsx#L43
Error: expect(element).not.toBeEmptyDOMElement() Received: "" ❯ src/sdk.spec.jsx:43:24
src/sdk.spec.jsx > OpenForm > should accept a target selector string as languageSelectorTarget: src/sdk.spec.jsx#L67
Error: expect(element).not.toBeEmptyDOMElement() Received: "" ❯ src/sdk.spec.jsx:67:24
src/sdk.spec.jsx > OpenForm > should re-fetch the form to get new literals after language change: node_modules/react-router/dist/development/index.js#L327
TestingLibraryElementError: Unable to find an element with the text: Nederlandse versie. 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 <div> <h2> Unexpected Application Error! </h2> <h3 style="font-style: italic;" > useLocation() may be used only in the context of a &lt;Router&gt; component. </h3> <pre style="padding: 0.5rem; background-color: rgba(200, 200, 200, 0.5);" > Error: useLocation() may be used only in the context of a &lt;Router&gt; component. at invariant (/home/runner/work/open-forms-sdk/open-forms-sdk/node_modules/react-router/dist/development/index.js:327:11) at useLocation (/home/runner/work/open-forms-sdk/open-forms-sdk/node_modules/react-router/dist/development/index.js:4229:3) at useSearchParams (/home/runner/work/open-forms-sdk/open-forms-sdk/node_modules/react-router/dist/development/index.js:7606:18) at App (/home/runner/work/open-forms-sdk/open-forms-sdk/src/components/App.jsx:17:18) at renderWithHooks (/home/runner/work/open-forms-sdk/open-forms-sdk/node_modules/react-dom/cjs/react-dom.development.js:16305:18) at mountIndeterminateComponent (/home/runner/work/open-forms-sdk/open-forms-sdk/node_modules/react-dom/cjs/react-dom.development.js:20074:13) at beginWork (/home/runner/work/open-forms-sdk/open-forms-sdk/node_modules/react-dom/cjs/react-dom.development.js:21587:16) at beginWork$1 (/home/runner/work/open-forms-sdk/open-forms-sdk/node_modules/react-dom/cjs/react-dom.development.js:27426:14) at performUnitOfWork (/home/runner/work/open-forms-sdk/open-forms-sdk/node_modules/react-dom/cjs/react-dom.development.js:26560:12) at workLoopSync (/home/runner/work/open-forms-sdk/open-forms-sdk/node_modules/react-dom/cjs/react-dom.development.js:26466:5) </pre> <p> 💿 Hey developer 👋 </p> <p> You can provide a way better UX than this when your app throws errors by providing your own <code style="padding: 2px 4px; background-color: rgba(200, 200, 200, 0.5);" > ErrorBoundary </code> or <code style="padding: 2px 4px; background-color: rgba(200, 200, 200, 0.5);" > errorElement </code> prop on your route. </p> </div> ❯ invariant node_modules/react-router/dist/development/index.js:327:11 ❯ useLocation node_modules/react-router/dist/development/index.js:4229:3 ❯ useSearchParams node_modules/react-router/dist/development/index.js:7606:18 ❯ App src/components/App.jsx:17:18 ❯ renderWithHooks node_modules/react-dom/cjs/react-dom.development.js:16305:18 ❯ mountIndeterminateComponent node_modules/react-dom/cjs/react-dom.development.js:20074:13 ❯ beginWork node_modules/react-dom/cjs/react-dom.development.js:21587:16 ❯ beginWork$1 node_modules/react-dom/cjs/react-dom.development.js:27426:14 ❯ performUnitOfWork node_modules/react-dom/cjs/react-dom.development.js:26560:12 ❯ Object.getElementError node_modules/@testing-library/dom/dist/config.js:37:19 ❯ node_modules/@testing-library/dom/dist/query-helpers.js:76:38 ❯ getAllByText node_modules/@testing-library/dom/dist/query-helpers.js:109:15 ❯ src/sdk.spec.jsx:113:29
src/sdk.spec.jsx > OpenForm > should call the onLanguageChange callback on language change: src/sdk.spec.jsx#L142
Error: expect(element).not.toBeEmptyDOMElement() Received: "" ❯ src/sdk.spec.jsx:142:24
Run Javascript tests
Process completed with exit code 1.
Lint code (Prettier)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Lint code (ESLint)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Build (and push) Docker image
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Create 'production' build
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Run Javascript tests
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636

Artifacts

Produced during runtime
Name Size
sdk-build Expired
9.68 MB