Skip to content

Commit

Permalink
OPHJOD-129: Näytetään yhteystiedot myös oppilaitokselle, jota ei ole …
Browse files Browse the repository at this point in the history
…vielä tallennettu
  • Loading branch information
heidilm committed Nov 29, 2023
1 parent 3b4a2be commit 8c7e471
Show file tree
Hide file tree
Showing 8 changed files with 158 additions and 26 deletions.
47 changes: 24 additions & 23 deletions src/main/app/playwright/fixtures/oppilaitos.ts
Original file line number Diff line number Diff line change
Expand Up @@ -14,29 +14,6 @@ export default () => {
kampuksia: 3,
korkeakouluja: 5,
opiskelijoita: 100,
yhteystiedot: [
{
nimi: {
fi: 'Yhteystiedon nimi',
sv: 'Yhteystiedon nimi sv',
},
postiosoite: {
osoite: {
fi: 'Fi osoite',
sv: 'Sv osoite',
},
postinumeroKoodiUri: 'posti_00350#2',
},
puhelinnumero: {
fi: '1234',
sv: '5678',
},
sahkoposti: {
fi: 'fi@sahkoposti.fi',
sv: 'sv@sahkoposti.sv',
},
},
],
wwwSivu: {
url: {
fi: 'http://www.verkkosivu.fi',
Expand All @@ -60,5 +37,29 @@ export default () => {
toimipisteita: 9,
yksikoita: 2,
},
_enrichedData: {
muokkaajanNimi: 'Anonymisoitu Virkailija',
organisaationYhteystiedot: {
nimi: {
fi: 'Organisaation nimi',
sv: 'Organisaation nimi sv',
},
postiosoite: {
osoite: {
fi: 'Fi osoite',
sv: 'Sv osoite',
},
postinumeroKoodiUri: 'posti_00350#2',
},
puhelinnumero: {
fi: '1234',
sv: '5678',
},
sahkoposti: {
fi: 'fi@sahkoposti.fi',
sv: 'sv@sahkoposti.sv',
},
},
},
};
};
35 changes: 35 additions & 0 deletions src/main/app/playwright/fixtures/oppilaitosWithOnlyYhteystiedot.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,35 @@
export default () => {
return {
oid: '1.2.246.562.13.00000000000000000072',
_enrichedData: {
organisaationYhteystiedot: {
nimi: {
fi: 'Organisaatio',
sv: 'Organisaatio sv',
},
postiosoite: {
osoite: {
fi: 'Horonpohjantie 279, 40101 Jyväskylä',
sv: 'Sv osoite',
},
postinumeroKoodiUri: 'posti_040101',
},
kayntiosoite: {
osoite: {
fi: 'Verhonkulmala 220, 40720 Jyväskylä',
sv: 'Sv osoite',
},
postinumeroKoodiUri: 'posti_040101',
},
puhelinnumero: {
fi: '050 28144921',
sv: '050 28144922',
},
sahkoposti: {
fi: 'hakija-31832505@oph.fi',
sv: 'sv@sahkoposti.sv',
},
},
},
};
};
Original file line number Diff line number Diff line change
@@ -1,6 +1,36 @@
{
"organisaatioOid": "1.1.1.1.1.1",
"oid": "1.1.1.1.1.1",
"_enrichedData": {
"organisaationYhteystiedot": {
"nimi": {
"fi": "Organisaatio",
"sv": "Organisaatio sv"
},
"postiosoite": {
"osoite": {
"fi": "Horonpohjantie 279, 40101 Jyväskylä",
"sv": "Sv osoite"
},
"postinumeroKoodiUri": "posti_040101"
},
"kayntiosoite": {
"osoite": {
"fi": "Verhonkulmala 220, 40720 Jyväskylä",
"sv": "Sv osoite"
},
"postinumeroKoodiUri": "posti_040101"
},
"puhelinnumero": {
"fi": "050 28144921",
"sv": "050 28144922"
},
"sahkoposti": {
"fi": "hakija-31832505@oph.fi",
"sv": "sv@sahkoposti.sv"
}
}
},
"tila": "julkaistu",
"muokkaaja": "1.2.246.562.24.62301161440",
"kielivalinta": [
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,36 @@
{
"organisaatioOid": "1.1.1.1.1.1",
"oid": "1.1.1.1.1.1",
"_enrichedData": {
"organisaationYhteystiedot": {
"nimi": {
"fi": "Organisaatio",
"sv": "Organisaatio sv"
},
"postiosoite": {
"osoite": {
"fi": "Horonpohjantie 279, 40101 Jyväskylä",
"sv": "Sv osoite"
},
"postinumeroKoodiUri": "posti_040101"
},
"kayntiosoite": {
"osoite": {
"fi": "Verhonkulmala 220, 40720 Jyväskylä",
"sv": "Sv osoite"
},
"postinumeroKoodiUri": "posti_040101"
},
"puhelinnumero": {
"fi": "050 28144921",
"sv": "050 28144922"
},
"sahkoposti": {
"fi": "hakija-31832505@oph.fi",
"sv": "sv@sahkoposti.sv"
}
}
},
"tila": "julkaistu",
"muokkaaja": "1.2.246.562.24.62301161440",
"kielivalinta": [
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,30 @@
}
}
},
"_enrichedData": {
"muokkaajanNimi": "Anonymisoitu Virkailija",
"organisaationYhteystiedot": {
"nimi": {
"fi": "Organisaation nimi",
"sv": "Organisaation nimi sv"
},
"postiosoite": {
"osoite": {
"fi": "Fi osoite",
"sv": "Sv osoite"
},
"postinumeroKoodiUri": "posti_00350#2"
},
"puhelinnumero": {
"fi": "1234",
"sv": "5678"
},
"sahkoposti": {
"fi": "fi@sahkoposti.fi",
"sv": "sv@sahkoposti.sv"
}
}
},
"muokkaaja": "1.2.246.562.24.62301161440",
"esikatselu": false
}
9 changes: 9 additions & 0 deletions src/main/app/playwright/tests/createOppilaitos.spec.ts
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
import { Page, test, expect } from '@playwright/test';

import oppilaitos from '#/playwright/fixtures/oppilaitosWithOnlyYhteystiedot';
import {
fillAsyncSelect,
fillKieliversiotSection,
Expand All @@ -9,6 +10,7 @@ import {
withinSection,
typeToEditor,
} from '#/playwright/playwright-helpers';
import { fixtureJSON } from '#/playwright/playwright-mock-utils';
import { stubOppilaitosRoutes } from '#/playwright/stubOppilaitosRoutes';
import { ENTITY } from '#/src/constants';

Expand Down Expand Up @@ -87,6 +89,13 @@ const checkYhteystiedotSection = (page: Page) =>
test.describe('Create oppilaitos', () => {
test.beforeEach(async ({ page }) => {
await stubOppilaitosRoutes(page, organisaatioOid);
await page.route(
`**/oppilaitos/${organisaatioOid}`,
fixtureJSON({
...oppilaitos(),
oid: organisaatioOid,
})
);
await page.goto(`/kouta/organisaatio/${organisaatioOid}/oppilaitos`);
});

Expand Down
2 changes: 1 addition & 1 deletion src/main/app/src/pages/OppilaitosPage/OppilaitosPage.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,7 @@ export const OppilaitosPage = () => {
[formMode, oppilaitos, contactInfo]
);

const stepsEnabled = !oppilaitos;
const stepsEnabled = !oppilaitos?.lastModified;

return isFetching ? (
<FullSpin />
Expand Down
7 changes: 5 additions & 2 deletions src/main/app/src/types/domainTypes.ts
Original file line number Diff line number Diff line change
Expand Up @@ -12,9 +12,12 @@ export type ValintaperusteModel = components['schemas']['Valintaperuste'];

export type SoraKuvausModel = components['schemas']['Sorakuvaus'];

export type OppilaitosModel = components['schemas']['Oppilaitos'];
export type OppilaitosModel = components['schemas']['Oppilaitos'] & {
lastModified: string | null;
};

export type OppilaitoksenOsaModel = components['schemas']['OppilaitoksenOsa'];
export type OppilaitoksenOsaModel =
components['schemas']['OppilaitoksenOsa'] & { lastModified: string | null };

export type Valintakokeet = Array<components['schemas']['Valintakoe']>;

Expand Down

0 comments on commit 8c7e471

Please sign in to comment.