diff --git a/cypress/integration/E2E.spec.ts b/cypress/integration/E2E.spec.ts index c2912a243..5687ac30a 100644 --- a/cypress/integration/E2E.spec.ts +++ b/cypress/integration/E2E.spec.ts @@ -9,7 +9,6 @@ import { HankeDataDraft, HankeIndexData, HANKE_INDEX_TYPE, - HANKE_SUUNNITTELUVAIHE, } from '../../src/domain/types/hanke'; import { createHankeFromUI } from '../utils/formFiller'; import { validateIndexes } from '../utils/indexValidator'; @@ -68,7 +67,6 @@ const hankeMock: HankeDataDraft = { ], tyomaaKatuosoite: 'Mannerheimintie 14', vaihe: HANKE_VAIHE.SUUNNITTELU, - suunnitteluVaihe: HANKE_SUUNNITTELUVAIHE.KATUSUUNNITTELU_TAI_ALUEVARAUS, omistajat: [ { id: null, // not used but types require it diff --git a/cypress/utils/formFiller.ts b/cypress/utils/formFiller.ts index 10ebd3ff7..49b68006d 100644 --- a/cypress/utils/formFiller.ts +++ b/cypress/utils/formFiller.ts @@ -13,15 +13,10 @@ import { HANKE_TARINAHAITTA_KEY, HANKE_VAIHE, HankeDataDraft, - HANKE_SUUNNITTELUVAIHE, HANKE_VAIHE_KEY, - HANKE_SUUNNITTELUVAIHE_KEY, } from '../../src/domain/types/hanke'; -export const selectHankeVaihe = ( - vaihe: HANKE_VAIHE_KEY, - suunnitteluVaihe?: HANKE_SUUNNITTELUVAIHE_KEY -) => { +export const selectHankeVaihe = (vaihe: HANKE_VAIHE_KEY) => { cy.get('#vaihe-toggle-button').click(); switch (vaihe) { case HANKE_VAIHE.OHJELMOINTI: @@ -29,27 +24,6 @@ export const selectHankeVaihe = ( break; case HANKE_VAIHE.SUUNNITTELU: cy.get('#vaihe-item-1').click(); - if (suunnitteluVaihe) { - cy.get('#suunnitteluVaihe-toggle-button').click(); - switch (suunnitteluVaihe) { - case HANKE_SUUNNITTELUVAIHE.YLEIS_TAI_HANKE: - cy.get('#suunnitteluVaihe-item-0').click(); - break; - case HANKE_SUUNNITTELUVAIHE.KATUSUUNNITTELU_TAI_ALUEVARAUS: - cy.get('#suunnitteluVaihe-item-1').click(); - break; - case HANKE_SUUNNITTELUVAIHE.RAKENNUS_TAI_TOTEUTUS: - cy.get('#suunnitteluVaihe-item-2').click(); - break; - case HANKE_SUUNNITTELUVAIHE.TYOMAAN_TAI_HANKKEEN_AIKAINEN: - cy.get('#suunnitteluVaihe-item-3').click(); - break; - default: - break; - } - } else { - throw new Error('Tämä testin vaihe tarvitsee suunnitteluvaiheen'); - } break; case HANKE_VAIHE.RAKENTAMINEN: cy.get('#vaihe-item-2').click(); @@ -69,10 +43,7 @@ export const fillForm0 = (hankeData: HankeDataDraft) => { } cy.get('input[data-testid=nimi]').click(); - selectHankeVaihe( - hankeData.vaihe, - hankeData.suunnitteluVaihe ? hankeData.suunnitteluVaihe : undefined - ); + selectHankeVaihe(hankeData.vaihe); if (hankeData.onYKTHanke) { cy.get('input[data-testid=onYKTHanke]').click(); diff --git a/package.json b/package.json index 938727973..564eaa83e 100644 --- a/package.json +++ b/package.json @@ -13,7 +13,7 @@ "build-and-serve": "yarn run build && yarn run serve", "update-runtime-env": "ts-node -P ./scripts/tsconfig.json --files scripts/update-runtime-env.ts", "test": "TEST=true yarn run update-runtime-env && DEBUG_PRINT_LIMIT=50000 react-scripts test", - "testCI": "CI=true react-scripts test", + "testCI": "TEST=true yarn run update-runtime-env && CI=TRUE DEBUG_PRINT_LIMIT=50000 react-scripts test", "eject": "react-scripts eject", "type-check": "tsc --pretty --noEmit", "format": "prettier --write", diff --git a/src/domain/hanke/edit/HankeForm.test.tsx b/src/domain/hanke/edit/HankeForm.test.tsx index 2ff0299ce..3348e53b8 100644 --- a/src/domain/hanke/edit/HankeForm.test.tsx +++ b/src/domain/hanke/edit/HankeForm.test.tsx @@ -21,7 +21,6 @@ const hankeData: HankeDataDraft = { alkuPvm: '24.03.2025', loppuPvm: '25.03.2025', vaihe: 'OHJELMOINTI', - suunnitteluVaihe: 'KATUSUUNNITTELU_TAI_ALUEVARAUS', omistajat: [ { id: null, @@ -88,34 +87,6 @@ async function setupYhteystiedotPage(jsx: JSX.Element) { } describe('HankeForm', () => { - test('suunnitteluVaihde should be required when vaihe is suunnittelu', async () => { - const handleIsDirtyChange = jest.fn(); - const handleFormClose = jest.fn(); - - const { user } = render( - - child - , - ); - - fireEvent.change(screen.getByTestId(FORMFIELD.NIMI), { target: { value: nimi } }); - fireEvent.change(screen.getByTestId(FORMFIELD.KUVAUS), { target: { value: hankkeenKuvaus } }); - fireEvent.change(screen.getByTestId(FORMFIELD.KATUOSOITE), { - target: { value: hankkeenOsoite }, - }); - - await user.click(screen.getByRole('radio', { name: 'Suunnittelu' })); - - await user.click(screen.getByRole('checkbox', { name: 'Hanke on YKT-hanke' })); - - await user.click(screen.getByText('Hankkeen suunnitteluvaihe')); - await user.click(screen.getByText('Yleis- tai hankesuunnittelu')); - }); - test('Form should be populated correctly ', () => { render( > = ({ formData, }) => { const { t } = useTranslation(); - const { watch, setValue } = useFormContext(); + const { watch } = useFormContext(); const { JOHTOSELVITYSHAKEMUS } = useLocalizedRoutes(); useFormPage(); - // Subscribe to vaihe changes + // Subscribe to vaihe changes in order to update the selected radio button const hankeVaiheField = watch(FORMFIELD.VAIHE); - useEffect(() => { - if (hankeVaiheField !== HANKE_VAIHE.SUUNNITTELU) { - setValue(FORMFIELD.SUUNNITTELUVAIHE, null); - } - }, [hankeVaiheField, setValue]); - return (
@@ -98,21 +91,6 @@ const HankeFormPerustiedot: React.FC> = ({ })}
-
- ({ - value, - label: t(`hanke:suunnitteluVaihe:${value}`), - }))} - defaultValue={formData[FORMFIELD.SUUNNITTELUVAIHE] || null} - label={t(`hankeForm:labels:${FORMFIELD.SUUNNITTELUVAIHE}`)} - invalid={!!errors[FORMFIELD.SUUNNITTELUVAIHE]} - required={hankeVaiheField === HANKE_VAIHE.SUUNNITTELU} - disabled={hankeVaiheField !== HANKE_VAIHE.SUUNNITTELU} - /> -