Upgrade react router #3727
ci.yml
on: pull_request
Matrix: linting
Create 'production' build
1m 52s
Run Javascript tests
2m 2s
Build (and push) Docker image
1m 50s
Publish the NPM package
0s
update-docker-readme
/
Generate and update Docker Hub description
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 <Router> 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 <Router> 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 |
|