VA-447: Remove toiminto from codebase

Opetushallitus · Jan 14, 2025 · adc1bee · adc1bee
1 parent b6fa088
commit adc1bee
Show file tree

Hide file tree

Showing 42 changed files with 78 additions and 165 deletions.
diff --git a/playwright/multipleProjectCodesTest.ts b/playwright/multipleProjectCodesTest.ts
@@ -10,7 +10,6 @@ export const multipleProjectCodesTest = muutoshakemusTest.extend({
     const codes = await koodienhallintaPage.createCodeValues({
       operationalUnit: uniqueCode(),
       project: [uniqueCode(), uniqueCode(), uniqueCode()],
-      operation: uniqueCode(),
     })
     await use(codes)
   },

diff --git a/playwright/pages/virkailija/hakujen-hallinta/HaunTiedotPage.ts b/playwright/pages/virkailija/hakujen-hallinta/HaunTiedotPage.ts
@@ -115,10 +115,7 @@ export const HaunTiedotPage = (page: Page) => {
     await setEndDate(`1.1.${previousYear} 0.00`)
   }
 
-  async function selectCode(
-    codeType: 'operational-unit' | 'project' | 'operation',
-    code: string
-  ): Promise<void> {
+  async function selectCode(codeType: 'operational-unit' | 'project', code: string): Promise<void> {
     const selectLocators = createReactSelectLocators(
       locators.dropdownForCode(codeType),
       `code-value-dropdown-${codeType}-id`
@@ -158,8 +155,6 @@ export const HaunTiedotPage = (page: Page) => {
         await page.getByTestId(projectCode).click()
       }
     }
-
-    await selectCode('operation', codes.operation)
   }
 
   async function addValmistelija(name: string, waitForSave = true) {
@@ -197,7 +192,7 @@ export const HaunTiedotPage = (page: Page) => {
   }
 
   async function getInputPlaceholderCodeStyles(
-    codeType: 'operational-unit' | 'project' | 'operation'
+    codeType: 'operational-unit' | 'project'
   ): Promise<CSSStyleDeclaration> {
     const locator = page.getByTestId(`singlevalue-${codeType}`)
     await expect(locator).toBeVisible()

diff --git a/playwright/pages/virkailija/hakujen-hallinta/maksatuksetPage.ts b/playwright/pages/virkailija/hakujen-hallinta/maksatuksetPage.ts
@@ -79,7 +79,6 @@ export function MaksatuksetPage(page: Page) {
 
   function getExpectedPaymentXML({
     projekti,
-    toiminto,
     toimintayksikko,
     pitkaviite,
     invoiceNumber,
@@ -88,7 +87,6 @@ export function MaksatuksetPage(page: Page) {
     talousarviotili,
   }: {
     projekti: string
-    toiminto: string
     toimintayksikko: string
     pitkaviite: string
     invoiceNumber: string
@@ -97,7 +95,7 @@ export function MaksatuksetPage(page: Page) {
     talousarviotili: string
   }): string {
     const today = moment(new Date()).format('YYYY-MM-DD')
-    return `<?xml version="1.0" encoding="UTF-8"?><objects><object><header><toEdiID>${ovt}</toEdiID><invoiceType>INVOICE</invoiceType><vendorName>Akaan kaupunki</vendorName><addressFields><addressField1></addressField1><addressField2></addressField2><addressField5></addressField5></addressFields><vendorRegistrationId>2050864-5</vendorRegistrationId><bic>OKOYFIHH</bic><bankAccount>FI95 6682 9530 0087 65</bankAccount><invoiceNumber>${invoiceNumber}</invoiceNumber><longReference>${pitkaviite}</longReference><documentDate>${today}</documentDate><dueDate>${dueDate}</dueDate><paymentTerm>Z001</paymentTerm><currencyCode>EUR</currencyCode><grossAmount>99999</grossAmount><netamount>99999</netamount><vatamount>0</vatamount><voucherSeries>XE</voucherSeries><postingDate>${today}</postingDate><ownBankShortKeyCode></ownBankShortKeyCode><handler><verifierName>[email protected]</verifierName><verifierEmail>[email protected]</verifierEmail><approverName>[email protected]</approverName><approverEmail>[email protected]</approverEmail><verifyDate>${today}</verifyDate><approvedDate>${today}</approvedDate></handler><otsData><otsBankCountryKeyCode></otsBankCountryKeyCode></otsData><invoicesource>VA</invoicesource></header><postings><postingRows><postingRow><rowId>1</rowId><generalLedgerAccount>82010000</generalLedgerAccount><postingAmount>99999</postingAmount><accountingObject01>${toimintayksikko}</accountingObject01><accountingObject02>${talousarviotili}</accountingObject02><accountingObject04>${projekti}</accountingObject04><accountingObject05>${toiminto}</accountingObject05><accountingObject08></accountingObject08></postingRow></postingRows></postings></object></objects>`
+    return `<?xml version="1.0" encoding="UTF-8"?><objects><object><header><toEdiID>${ovt}</toEdiID><invoiceType>INVOICE</invoiceType><vendorName>Akaan kaupunki</vendorName><addressFields><addressField1></addressField1><addressField2></addressField2><addressField5></addressField5></addressFields><vendorRegistrationId>2050864-5</vendorRegistrationId><bic>OKOYFIHH</bic><bankAccount>FI95 6682 9530 0087 65</bankAccount><invoiceNumber>${invoiceNumber}</invoiceNumber><longReference>${pitkaviite}</longReference><documentDate>${today}</documentDate><dueDate>${dueDate}</dueDate><paymentTerm>Z001</paymentTerm><currencyCode>EUR</currencyCode><grossAmount>99999</grossAmount><netamount>99999</netamount><vatamount>0</vatamount><voucherSeries>XE</voucherSeries><postingDate>${today}</postingDate><ownBankShortKeyCode></ownBankShortKeyCode><handler><verifierName>[email protected]</verifierName><verifierEmail>[email protected]</verifierEmail><approverName>[email protected]</approverName><approverEmail>[email protected]</approverEmail><verifyDate>${today}</verifyDate><approvedDate>${today}</approvedDate></handler><otsData><otsBankCountryKeyCode></otsBankCountryKeyCode></otsData><invoicesource>VA</invoicesource></header><postings><postingRows><postingRow><rowId>1</rowId><generalLedgerAccount>82010000</generalLedgerAccount><postingAmount>99999</postingAmount><accountingObject01>${toimintayksikko}</accountingObject01><accountingObject02>${talousarviotili}</accountingObject02><accountingObject04>${projekti}</accountingObject04><accountingObject08></accountingObject08></postingRow></postingRows></postings></object></objects>`
   }
 
   async function sendMaksatukset(): Promise<void> {

diff --git a/playwright/pages/virkailija/koodienHallintaPage.ts b/playwright/pages/virkailija/koodienHallintaPage.ts
@@ -4,7 +4,7 @@ import { randomString } from '../../utils/random'
 import { NoProjectCodeProvided, VaCodeValues } from '../../utils/types'
 import { HakujenHallintaPage } from './hakujen-hallinta/hakujenHallintaPage'
 
-type KoodienhallintaTab = 'operational-unit' | 'project' | 'operation'
+type KoodienhallintaTab = 'operational-unit' | 'project'
 
 export const KoodienhallintaPage = (page: Page) => {
   const locators = {
@@ -75,8 +75,6 @@ export const KoodienhallintaPage = (page: Page) => {
       }
     }
 
-    await clickKoodienhallintaTab('operation')
-    await createCode('Toiminto', codeValues.operation)
     return codeValues
   }
   return {
@@ -115,7 +113,6 @@ export const KoodienhallintaPage = (page: Page) => {
       const codeValues = {
         operationalUnit: uniqueCode(),
         project: [uniqueCode()],
-        operation: uniqueCode(),
       }
       return createCodeValues(codeValues)
     },

diff --git a/playwright/tests/maksatukset/maksatukset.test.ts b/playwright/tests/maksatukset/maksatukset.test.ts
@@ -26,7 +26,6 @@ const correctOVTTest = test.extend({
   codes: async ({ page }, use) => {
     const codes = {
       operationalUnit: '6600105300',
-      operation: '3425324634',
       project: [NoProjectCodeProvided.code, '523452346'],
     }
     const koodienHallintaPage = KoodienhallintaPage(page)
@@ -39,7 +38,6 @@ const showProjectCodeTest = test.extend({
   codes: async ({ page }, use) => {
     const codes = {
       operationalUnit: '6600105300',
-      operation: '3425324634',
       project: [
         NoProjectCodeProvided.code,
         randomString().substring(0, 13),
@@ -229,7 +227,6 @@ correctOVTTest(
     expect(maksatukset).toContainEqual(
       maksatuksetPage.getExpectedPaymentXML({
         projekti: codeValues.project[1],
-        toiminto: codeValues.operation,
         toimintayksikko: codeValues.operationalUnit,
         pitkaviite,
         invoiceNumber: `${registerNumber}_1`,
@@ -286,7 +283,7 @@ showProjectCodeTest(
     acceptedHakemus,
     avustushakuName,
     talousarviotili,
-    codes: { operation, operationalUnit },
+    codes: { operationalUnit },
     projektikoodi,
   }) => {
     expectToBeDefined(acceptedHakemus)
@@ -320,7 +317,6 @@ showProjectCodeTest(
 
     const expectedXML = maksatusPage.getExpectedPaymentXML({
       projekti: projektikoodi,
-      toiminto: operation,
       toimintayksikko: operationalUnit,
       pitkaviite,
       invoiceNumber: `${registerNumber}_1`,
@@ -366,9 +362,6 @@ test('sending maksatukset disables changing code values for haku', async ({
   await expect(
     hakujenHallintaPage.page.locator('.code-value-dropdown-operational-unit-id--is-disabled')
   ).toBeVisible()
-  await expect(
-    hakujenHallintaPage.page.locator('.code-value-dropdown-operation-id--is-disabled')
-  ).toBeVisible()
 
   const projects = await hakujenHallintaPage.page.locator(
     '.code-value-dropdown-project-id--is-disabled'

diff --git a/playwright/tests/maksatukset/pitkaviite.test.ts b/playwright/tests/maksatukset/pitkaviite.test.ts
@@ -81,7 +81,7 @@ test.describe('maksatukset pitkaviite', async () => {
     page,
     avustushakuName,
     acceptedHakemus: { hakemusID },
-    codes: { operation, operationalUnit },
+    codes: { operationalUnit },
     projektikoodi,
     talousarviotili,
   }) => {
@@ -128,7 +128,6 @@ test.describe('maksatukset pitkaviite', async () => {
     expect(maksatukset).toContainEqual(
       maksatuksetPage.getExpectedPaymentXML({
         projekti: projektikoodi,
-        toiminto: operation,
         toimintayksikko: operationalUnit,
         pitkaviite,
         invoiceNumber: `${registerNumber}_1`,

diff --git a/playwright/tests/useamman-projektikoodin-tuki/multiple-projects-util.ts b/playwright/tests/useamman-projektikoodin-tuki/multiple-projects-util.ts
@@ -20,7 +20,6 @@ export async function alustaAvustushaunTaytto(page: Page, hakuProps: HakuProps)
 
   if (hakuProps.vaCodes) {
     await haunTiedotPage.selectCode('operational-unit', hakuProps.vaCodes.operationalUnit)
-    await haunTiedotPage.selectCode('operation', hakuProps.vaCodes.operation)
   }
   return haunTiedotPage
 }
diff --git a/playwright/utils/types.ts b/playwright/utils/types.ts
@@ -35,7 +35,6 @@ export interface VaCodeValues {
   operationalUnit: string
   operationalUnitName?: string
   project: string[]
-  operation: string
 }
 
 export const NoProjectCodeProvided = {

diff --git a/run_isolated_system_tests.sh b/run_isolated_system_tests.sh
@@ -6,7 +6,7 @@ source "$( cd "$( dirname "${BASH_SOURCE[0]}" )" && pwd )/scripts/common-functio
 function run_tests {
   npm run prettier-check-project "$@"
   run_playwright_tests "$@"
-  ./lein with-profile test-legacy,user spec -f d
+  ./lein with-profile test-legacy,user spec -f d "$@"
 }
 
 function run_playwright_tests {

diff --git a/server/resources/help-texts.tsv b/server/resources/help-texts.tsv
@@ -12,7 +12,6 @@ Hakujen hallinta -> Haun tiedot -välilehti	Välilehden infopallo välilehtirivi
 	Raportin määräaika	Päivämäärä, jolloin raportti tulee viimeistään toimittaa raportointivelvoitteen asettajalle.	hakujen_hallinta__haun_tiedot___raportin_määräaika
 	Asha-tunnus	Raportointivelvoitteet kuvaavan ohjaavan asiakirjan (esim. määrärahan asettamiskirje) asiatunnus Asha-asianhallintajärjestelmässä.	hakujen_hallinta__haun_tiedot___asha-tunnus
 	Lisätiedot	Mahdolliset lisätiedot, kuten raportoinnin laajuus, vastuuhenkilöt tai poikkeavuudet aikataulussa.	hakujen_hallinta__haun_tiedot___raportointi_lisätiedot
-	Toiminto	Toiminto kuvaa tekemistä. Yleensä VA-järjestelmän kautta maksettaessa käytetään koodia valtionavustusprosessi (6600151502) ja KV-ohjelmissa käytetään koodia Kv-ohjelmien toimeenpano (6600212160).	hakujen_hallinta__haun_tiedot___toiminto
 	Tila	Poistettu = Virheellisille hauille valitaan tämä tila, jonka jälkeen ne siirtyvät arkistoon ja poistuvat hakulistasta. Luonnos = Haku pidetään tässä tilassa niin kauan kuin sitä muokataan ja valmistellaan, Julkaistu = Haku muutetaan tähän tilaan kun se halutaan tuoda www.oph.fi -palveluun. Julkaisua seuraavana yönä haun tiedot siirtyvät automaattisesti VA-järjestelmästä Drupaliin ja www.oph.fi -palveluun aukeaa oma rahoitussivu julkaistulle haulle. Hakulomake aukeaa kuitenkin vasta ajankohtana, joka on kirjattu alla olevaan Hakuaika -kenttään. Haku pidetään Julkaistu-tilassa haku- ja arviointiaikana. Ratkaistu = haku muutetaan tähän tilaan, kun arvioinnit on tehty valmiiksi ja halutaan lähettää avustuspäätökset sekä toteuttaa maksatukset. Tieto haun ratkaisusta siirtyy seuraavan yön aikana www.oph.fi -verkkopalveluun muuttaen haun statuksen myös siellä muotoon Ratkaistu.	hakujen_hallinta__haun_tiedot___tila
 	Hakuaika	Valitse aika, jolloin haluat haun olevan avoinna hakijoille. Hakulomake avautuu ja sulkeutuu www.oph.fi -palvelussa sekä VA-järjestelmässä automaattisesti tähän asetettujen ajankohtien mukaan edellyttäen, että haku on VA-järjestelmässä tilassa Julkaistu. Suositeltavaa on, että hakuaika päättyy klo 16.15 ja haun avaamista/sulkemista ei ajoiteta perjantaille mahdollisten teknisten ongelmien vuoksi. Hakuajan suositellaan kestävän aina vähintään kuusi viikkoa.	hakujen_hallinta__haun_tiedot___hakuaika
 	Hakutyyppi	Erityisavustuksia ovat esimerkiksi hankeavustukset kokeilu-, käynnistämis-, tutkimus- ja kehittämistoimintaan. Yleisavustuksia ovat avustukset, jotka suunnataan avustuksen saajan yleiseen perustoimintaan. OPH:n avustuksista enemmistö on erityisavustuksia. Saat tarvittaessa tiedon hakutyypistä oman yksikön assistentilta tai taloushallinnosta.	hakujen_hallinta__haun_tiedot___hakutyyppi

diff --git a/server/resources/sql/hakija/avustushaku/archive.sql b/server/resources/sql/hakija/avustushaku/archive.sql
@@ -11,7 +11,6 @@ insert into archived_avustushaut (
     operational_unit_id,
     hankkeen_alkamispaiva,
     hankkeen_paattymispaiva,
-    operation_id,
     muutoshakukelpoinen)
   select
     id,
@@ -26,7 +25,6 @@ insert into archived_avustushaut (
     operational_unit_id,
     hankkeen_alkamispaiva,
     hankkeen_paattymispaiva,
-    operation_id,
     muutoshakukelpoinen
   from avustushaut
   where id = :id
diff --git a/server/resources/sql/hakija/avustushaku/create.sql b/server/resources/sql/hakija/avustushaku/create.sql
@@ -4,7 +4,6 @@ insert into avustushaut (
     register_number,
     decision,
     haku_type,
-    operation_id,
     operational_unit_id,
     muutoshakukelpoinen,
     form_loppuselvitys,
@@ -16,7 +15,6 @@ insert into avustushaut (
     :register_number,
     :decision,
     :haku_type,
-    :operation_id,
     :operational_unit_id,
     :muutoshakukelpoinen,
     :form_loppuselvitys,

diff --git a/server/resources/sql/hakija/avustushaku/update.sql b/server/resources/sql/hakija/avustushaku/update.sql
@@ -11,7 +11,6 @@ set content = :content,
     hankkeen_paattymispaiva = :hankkeen_paattymispaiva,
     loppuselvitysdate = :loppuselvitysdate,
     valiselvitysdate = :valiselvitysdate,
-    operation_id = :operation_id,
     operational_unit_id = :operational_unit_id,
     muutoshakukelpoinen = :muutoshakukelpoinen,
     allow_visibility_in_external_system = :allow_visibility_in_external_system,

diff --git a/server/resources/sql/virkailija/external/get-grants-for-year.sql b/server/resources/sql/virkailija/external/get-grants-for-year.sql
@@ -3,7 +3,7 @@ SELECT
   h.valiselvitysdate, h.loppuselvitysdate, h.form_loppuselvitys,
   h.form_valiselvitys, h.is_academysize, h.haku_type, h.operational_unit_id,
   h.hankkeen_alkamispaiva, h.hankkeen_paattymispaiva,
-  null as project_id, h.operation_id, (h.decision->>'valmistelija')::jsonb as valmistelija
+  null as project_id, (h.decision->>'valmistelija')::jsonb as valmistelija
 FROM
   hakija.avustushaut h
 WHERE

diff --git a/server/resources/sql/virkailija/grants/get-grant.sql b/server/resources/sql/virkailija/grants/get-grant.sql
@@ -2,7 +2,7 @@ SELECT
   h.id, h.created_at, h.form, h.content, h.status, h.register_number,
   h.valiselvitysdate, h.loppuselvitysdate, h.form_loppuselvitys,
   h.form_valiselvitys, h.is_academysize, h.haku_type, h.operational_unit_id,
-  h.operation_id, h.allow_visibility_in_external_system, h.arvioitu_maksupaiva
+  h.allow_visibility_in_external_system, h.arvioitu_maksupaiva
 FROM
   hakija.avustushaut h
 WHERE

diff --git a/server/resources/sql/virkailija/grants/get-grants.sql b/server/resources/sql/virkailija/grants/get-grants.sql
@@ -1,7 +1,7 @@
 SELECT
   h.id, h.created_at, h.form, h.status, h.register_number, h.valiselvitysdate,
   h.loppuselvitysdate, h.form_loppuselvitys, h.form_valiselvitys,
-  h.is_academysize, h.haku_type, operational_unit_id, operation_id,
+  h.is_academysize, h.haku_type, operational_unit_id,
   h.allow_visibility_in_external_system, h.arvioitu_maksupaiva
 FROM
   hakija.avustushaut h

diff --git a/server/resources/sql/virkailija/grants/get-resolved-grants-with-content.sql b/server/resources/sql/virkailija/grants/get-resolved-grants-with-content.sql
@@ -2,7 +2,7 @@ SELECT
   h.id, h.created_at, h.form, h.content, h.status, h.register_number,
   h.valiselvitysdate, h.loppuselvitysdate, h.form_loppuselvitys,
   h.form_valiselvitys, h.is_academysize, h.haku_type, h.operational_unit_id,
-  h.operation_id, h.allow_visibility_in_external_system, h.arvioitu_maksupaiva
+  h.allow_visibility_in_external_system, h.arvioitu_maksupaiva
 FROM
   hakija.avustushaut h
 WHERE