wip: testit kuntoon

Opetushallitus · Jan 8, 2025 · 7dadab2 · 7dadab2
1 parent 23762b3
commit 7dadab2
Show file tree

Hide file tree

Showing 22 changed files with 1,818 additions and 267 deletions.
diff --git a/web/app/components-v2/controls/Checkbox.tsx b/web/app/components-v2/controls/Checkbox.tsx
@@ -25,7 +25,7 @@ export const Checkbox: React.FC<CheckboxProps> = (props) => {
         onChange={(event) => props.onChange(event.target.checked)}
         data-testid={testId}
       />
-      <label htmlFor={id}>
+      <label htmlFor={id} onClick={() => props.onChange(!props.checked)}>
         <span className="Checkbox__label">{t(props.label)}</span>
       </label>
     </div>

diff --git a/web/app/components-v2/opiskeluoikeus/KoodistoField.tsx b/web/app/components-v2/opiskeluoikeus/KoodistoField.tsx
@@ -3,7 +3,7 @@ import { t } from '../../i18n/i18n'
 import { Koodistokoodiviite } from '../../types/fi/oph/koski/schema/Koodistokoodiviite'
 import { CommonProps, common } from '../CommonProps'
 import { FieldEditorProps, FieldViewerProps } from '../forms/FormField'
-import { TestIdLayer } from '../../appstate/useTestId'
+import { TestIdLayer, useTestId } from '../../appstate/useTestId'
 import {
   Select,
   SelectOption,
@@ -24,11 +24,14 @@ export type KoodistoViewProps<T extends string> = CommonProps<
   >
 >
 
-export const KoodistoView = <T extends string>(props: KoodistoViewProps<T>) => (
-  <div {...common(props)} data-testid={props.testId}>
-    {t(props.value?.nimi) || '–'}
-  </div>
-)
+export const KoodistoView = <T extends string>(props: KoodistoViewProps<T>) => {
+  const testId = useTestId(props.testId)
+  return (
+    <div {...common(props)} data-testid={testId}>
+      {t(props.value?.nimi) || '–'}
+    </div>
+  )
+}
 
 export type KoodistoEditProps<T extends string> = CommonProps<
   FieldEditorProps<

diff --git a/web/app/components-v2/opiskeluoikeus/OpiskeluoikeudenTila.tsx b/web/app/components-v2/opiskeluoikeus/OpiskeluoikeudenTila.tsx
@@ -83,13 +83,16 @@ export const OpiskeluoikeudenTilaView = <T extends OpiskeluoikeudenTila>(
                     </TestIdText>
                   </time>,
                   <React.Fragment key={`jakso_tila_lisatiedot_${index}`}>
-                    <TestIdText id="tila">
-                      <span>{t(jakso.tila.nimi)}</span>
-                      <span>
-                        {isRahoituksellinenOpiskeluoikeusjakso(jakso) &&
-                          ` (${t(jakso.opintojenRahoitus?.nimi)})`}
-                      </span>
-                    </TestIdText>
+                    <TestIdText id="tila">{t(jakso.tila.nimi)}</TestIdText>
+                    {isRahoituksellinenOpiskeluoikeusjakso(jakso) && (
+                      <>
+                        {' ('}
+                        <TestIdText id="rahoitus">
+                          {t(jakso.opintojenRahoitus?.nimi)}
+                        </TestIdText>
+                        {')'}
+                      </>
+                    )}
                   </React.Fragment>
                 ]}
               </KeyColumnedValuesRow>