Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(Chore) Improve tsconfig #1830

Merged
merged 1 commit into from
Oct 28, 2021
Merged

(Chore) Improve tsconfig #1830

merged 1 commit into from
Oct 28, 2021

Conversation

janjaap
Copy link
Contributor

@janjaap janjaap commented Oct 28, 2021

Makes sure that only TS files are included in the list of files to parse by the compiler

Makes sure that only TS files are included in the list of files to parse by the compiler
@cypress
Copy link

cypress bot commented Oct 28, 2021



Test summary

202 26 1 0Flakiness 0


Run details

Project Signals-frontend
Status Failed
Commit a20c728 ℹ️
Started Oct 28, 2021 7:44 AM
Ended Oct 28, 2021 7:51 AM
Duration 06:41 💡
OS Linux Ubuntu - 20.04
Browser Electron 91

View run in Cypress Dashboard ➡️


Failures

cypress/integration/handleSignals/statusTransitionsOfSignal.spec.ts 10 Failed
1 All status transitions of a signal > Set 01 status transitions > Should change status from 'Gemeld' to 'In afwachting van behandeling'
2 All status transitions of a signal > Set 01 status transitions > Should change status from 'In afwachting van behandeling' to 'In behandeling'
3 All status transitions of a signal > Set 01 status transitions > Should change status from 'In behandeling' to 'Gemeld'
4 All status transitions of a signal > Set 01 status transitions > Should change status from 'Gemeld' to 'Ingepland'
5 All status transitions of a signal > Set 01 status transitions > Should change status from 'Ingepland' to 'Extern: verzoek tot afhandeling'
6 All status transitions of a signal > Set 01 status transitions > Should change status from 'Extern: verzoek tot afhandeling' to 'Afgehandeld'
7 All status transitions of a signal > Set 01 status transitions > Should change status from 'Afgehandeld' to 'Heropend'
8 All status transitions of a signal > Set 01 status transitions > Should change status from 'Heropend' to 'In behandeling'
9 All status transitions of a signal > Set 01 status transitions > Should change status from 'In behandeling' to 'Ingepland'
10 All status transitions of a signal > Set 01 status transitions > Should change status from 'Ingepland' to 'In behandeling'
This comment includes only the first 10 test failures. See all 26 failures in the Cypress Dashboard.

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

@janjaap janjaap merged commit fa14a7e into develop Oct 28, 2021
@janjaap janjaap deleted the chore-improve-ts-config branch October 28, 2021 07:54
This was referenced Nov 2, 2021
janjaap pushed a commit that referenced this pull request Nov 3, 2021
* Prevent breaking app when browser (cookie/local/session) storage is disabled

* Prevent error when incident location is null

* Adds array flatMap polyfill

* Improves incident-list type

* Change core-js polyfill to local polyfill

* Notify userwhen login fails

* (Chore) Empty ref fix

* (Chore) AbortController sub

* [SIG-4035] Convert categories model to TS

* Converts category model selectors to TS

* Fixes tests

* Corrects mock implementation for incident detail

* Changes type

* Corrects type errors

* Removes debugger statement

* Fixes another round of types

* Adds App mode ADR

* (Chore) cleanup same-area spec

* Removes flakiness from test; disables unreliable e2e test

* Removes commented line

* Fixes incident type to prevent type errors

* Removes faker package

* [SIG-3824] End status styling

* Adds global css class for incident status styling

* Fixes type error

* Fixes test

* Creates separate Status component

* Removes outdated test cases

* - adds data-testid attributes to elements for better QA targeting
- removes style-specific integration tests; not the responsibility of the e2e-tests

* Adds api url where msw request handler is used

* Fixes unit test errors

* (Chore) Improve tsconfig (#1830)

Makes sure that only TS files are included in the list of files to parse by the compiler

* [SIG-3281] Show number of chars for textarea (#1829)

* Adds separate AddNote components
Corrects types
Moves components to different scope

* Fixes types for register ref

* Corrects Select prop type

* Adds complete coverage for AddNote component

* Adds SelectLoader component

* Prefers defaultValue over value when instantiating TextArea component

* Allows for configurable field name in getError for note.

* Deletes temp files

* Replaces register prop with onChange handler

* Replaces register prop with onChange handler for IncidentSplitRadioInput component

* Refactors addNote error function so that it is no longer a higher order function

* Renders IncidentSplitFormIncident tests in context provider

* Makes sure incident detail AddNote component calls error function correctly

* Upgrades react-hook-form to latest stable

* Adjusts IncidentReplyContainer components to work with the new version of react-hook-form

* Removes generic AddNote component to top level components folder

* Converts StatusForm related imports to TS and corrects types accordingly

* Corrects types for Status and context

* Increases default test timeout because of slow tests in FilterForm

* Removes memo from internal FilterForm components

* Makes sure value updates are reflected by the textarea element

* Uses form context provider for incident split form

* Corrects condition in category selectors

* Ensures correct types for IncidentDetail components

* Ensures correct type usage

* Moves default texts styled components into their own file

* Removes empty test case

* Corrects Coordinates type

* Ensures cleanup after each test case for useFetchReporter hook

* Removes unused file

* (Chore) Improve page rendering speed (#1831)

* (Chore) Improve page rendering speed
Sets the `contain` CSS property on several elements to improve the in-page rendering. See https://css-tricks.com/almanac/properties/c/contain/ for reference.

* Adds one more contain rule

* Update PDOK WMTS service endpoint (#1832)

* [FE#92] Asset selector (#1820)

* Copy container selector

* Make the asset select usable

* Make the language configurable

* Make the wfs endpoint more flexible

* Remove obsolete context

* Add fallback icon config

* Update unit tests

* Use single curly brace in endpoint

* Remove memo

* Fix typing

* Sync/master develop (#1838)

* Release/v1.63.0 (#1828)

* Prevent breaking app when browser (cookie/local/session) storage is disabled

* Prevent error when incident location is null

* Adds array flatMap polyfill

* Improves incident-list type

* Change core-js polyfill to local polyfill

* Notify userwhen login fails

* (Chore) Empty ref fix

* (Chore) AbortController sub

* [SIG-4035] Convert categories model to TS

* Converts category model selectors to TS

* Fixes tests

* Corrects mock implementation for incident detail

* Changes type

* Corrects type errors

* Removes debugger statement

* Fixes another round of types

* Adds App mode ADR

* (Chore) cleanup same-area spec

* Removes flakiness from test; disables unreliable e2e test

* Removes commented line

* Fixes incident type to prevent type errors

* Removes faker package

* [SIG-3824] End status styling

* Adds global css class for incident status styling

* Fixes type error

* Fixes test

* Creates separate Status component

* Removes outdated test cases

* - adds data-testid attributes to elements for better QA targeting
- removes style-specific integration tests; not the responsibility of the e2e-tests

* Adds api url where msw request handler is used

* Fixes unit test errors

Co-authored-by: Maurits Ridder <maurits.ridder@amsterdam.nl>
Co-authored-by: Maurits <10672773+MauritsR@users.noreply.github.com>

* Update PDOK WMTS service endpoint (#1832)

* Reverts conversion to object

* Increments node version

* Reverts packages update

Co-authored-by: Maurits <10672773+MauritsR@users.noreply.github.com>
Co-authored-by: Maurits Ridder <maurits.ridder@amsterdam.nl>
Co-authored-by: Boris Arkenaar <boris@arkenaar.com>

* Reverts package lock changes

Co-authored-by: Maurits Ridder <maurits.ridder@amsterdam.nl>
Co-authored-by: Maurits <10672773+MauritsR@users.noreply.github.com>
Co-authored-by: Boris Arkenaar <boris@arkenaar.com>
@janjaap janjaap mentioned this pull request Nov 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant