From 1bae6672ece9a3b49234f67491747456025a15f0 Mon Sep 17 00:00:00 2001 From: Juho Leinonen Date: Fri, 30 Jun 2023 13:41:01 +0300 Subject: [PATCH 1/4] AE-1917: Fix test missing an assertion --- .../etp/service/valvonta_kaytto_test.clj | 272 +++++++++--------- 1 file changed, 136 insertions(+), 136 deletions(-) diff --git a/etp-backend/src/test/clj/solita/etp/service/valvonta_kaytto_test.clj b/etp-backend/src/test/clj/solita/etp/service/valvonta_kaytto_test.clj index 466a8dcf5..2f3aab687 100644 --- a/etp-backend/src/test/clj/solita/etp/service/valvonta_kaytto_test.clj +++ b/etp-backend/src/test/clj/solita/etp/service/valvonta_kaytto_test.clj @@ -10,140 +10,140 @@ (t/deftest find-toimenpidetyypit-test (let [toimenpidetyypit (valvonta-kaytto/find-toimenpidetyypit ts/*db*)] (t/testing "find-toimenpidetyypit returns correct toimenpidetypes" - (t/is toimenpidetyypit - [{:id 0 - :label-fi "Valvonnan aloitus" - :label-sv "Valvonnan aloitus (sv)" - :valid true - :manually-deliverable false - :allow-comments false} - {:id 1 - :label-fi "Tietopyyntö 2021" - :label-sv "Begäran om uppgifter 2021" - :valid false - :manually-deliverable false - :allow-comments false} - {:id 2 - :label-fi "Kehotus" - :label-sv " Uppmaning" - :valid true - :manually-deliverable false - :allow-comments false} - {:id 3 - :label-fi "Varoitus" - :label-sv "Varning" - :valid true - :manually-deliverable false - :allow-comments false} - {:id 4 - :label-fi "Käskypäätös" - :label-sv "Käskypäätös (sv)" - :valid true - :manually-deliverable false - :allow-comments false} - {:id 5 - :label-fi "Valvonnan lopetus" - :label-sv "Valvonnan lopetus (sv)" - :valid true - :manually-deliverable false - :allow-comments false} - {:id 6 - :label-fi "HaO käsittely" - :label-sv "HaO käsittely (sv)" - :valid true - :manually-deliverable false - :allow-comments false} - {:id 7 - :label-fi "Käskypäätös / kuulemiskirje" - :label-sv "Käskypäätös / kuulemiskirje (sv)" - :valid true - :manually-deliverable true - :allow-comments true} - {:id 8 - :label-fi "Käskypäätös / varsinainen päätös" - :label-sv "Käskypäätös / varsinainen päätös (sv)" - :valid true - :manually-deliverable false - :allow-comments false} - {:id 9 - :label-fi "Käskypäätös / tiedoksianto (ensimmäinen postitus)" - :label-sv "Käskypäätös / tiedoksianto (ensimmäinen postitus) (sv)" - :valid true - :manually-deliverable false - :allow-comments false} - {:id 10 - :label-fi "Käskypäätös / tiedoksianto (toinen postitus)" - :label-sv "Käskypäätös / tiedoksianto (toinen postitus) (sv)" - :valid true - :manually-deliverable false - :allow-comments false} - {:id 11 - :label-fi "Käskypäätös / tiedoksianto (Haastemies)" - :label-sv "Käskypäätös / tiedoksianto (Haastemies) (sv)" - :valid true - :manually-deliverable false - :allow-comments false} - {:id 12 - :label-fi "Käskypäätös / odotetaan valitusajan umpeutumista" - :label-sv "Käskypäätös / odotetaan valitusajan umpeutumista (sv)" - :valid true - :manually-deliverable false - :allow-comments false} - {:id 13 - :label-fi "Käskypäätös / valitusaika umpeutunut" - :label-sv "Käskypäätös / valitusaika umpeutunut (sv)" - :valid true - :manually-deliverable false - :allow-comments false} - {:id 14 - :label-fi "Sakkopäätös / kuulemiskirje" - :label-sv "Sakkopäätös / kuulemiskirje (sv)" - :valid true - :manually-deliverable false - :allow-comments false} - {:id 15 - :label-fi "Sakkopäätös / varsinainen päätös" - :label-sv "Sakkopäätös / varsinainen päätös (sv)" - :valid true - :manually-deliverable false - :allow-comments false} - {:id 16 - :label-fi "Sakkopäätös / tiedoksianto (ensimmäinen postitus)" - :label-sv "Sakkopäätös / tiedoksianto (ensimmäinen postitus) (sv)" - :valid true - :manually-deliverable false - :allow-comments false} - {:id 17 - :label-fi "Sakkopäätös / tiedoksianto (toinen postitus)" - :label-sv "Sakkopäätös / tiedoksianto (toinen postitus) (sv)" - :valid true - :manually-deliverable false - :allow-comments false} - {:id 18 - :label-fi "Sakkopäätös / tiedoksianto (Haastemies)" - :label-sv "Sakkopäätös / tiedoksianto (Haastemies) (sv)" - :valid true - :manually-deliverable false - :allow-comments false} - {:id 19 - :label-fi "Sakkopäätös / odotetaan valitusajan umpeutumista" - :label-sv "Sakkopäätös / odotetaan valitusajan umpeutumista (sv)" - :valid true - :manually-deliverable false - :allow-comments false} - {:id 20 - :label-fi "Sakkopäätös / valitusaika umpeutunut" - :label-sv "Sakkopäätös / valitusaika umpeutunut (sv)" - :valid true - :manually-deliverable false - :allow-comments false} - {:id 21 - :label-fi "Sakkoluettelon lähetys menossa" - :label-sv "Sakkoluettelon lähetys menossa (sv)" - :valid true - :manually-deliverable false - :allow-comments false}])) + (t/is (= toimenpidetyypit + [{:id 0 + :label-fi "Valvonnan aloitus" + :label-sv "Valvonnan aloitus (sv)" + :valid true + :manually-deliverable false + :allow-comments false} + {:id 1 + :label-fi "Tietopyyntö 2021" + :label-sv "Begäran om uppgifter 2021" + :valid false + :manually-deliverable false + :allow-comments false} + {:id 2 + :label-fi "Kehotus" + :label-sv " Uppmaning" + :valid true + :manually-deliverable false + :allow-comments false} + {:id 3 + :label-fi "Varoitus" + :label-sv "Varning" + :valid true + :manually-deliverable false + :allow-comments false} + {:id 4 + :label-fi "Käskypäätös" + :label-sv "Käskypäätös (sv)" + :valid true + :manually-deliverable false + :allow-comments false} + {:id 5 + :label-fi "Valvonnan lopetus" + :label-sv "Valvonnan lopetus (sv)" + :valid true + :manually-deliverable false + :allow-comments false} + {:id 6 + :label-fi "HaO käsittely" + :label-sv "HaO käsittely (sv)" + :valid true + :manually-deliverable false + :allow-comments false} + {:id 7 + :label-fi "Käskypäätös / kuulemiskirje" + :label-sv "Käskypäätös / kuulemiskirje (sv)" + :valid true + :manually-deliverable true + :allow-comments true} + {:id 8 + :label-fi "Käskypäätös / varsinainen päätös" + :label-sv "Käskypäätös / varsinainen päätös (sv)" + :valid true + :manually-deliverable false + :allow-comments false} + {:id 9 + :label-fi "Käskypäätös / tiedoksianto (ensimmäinen postitus)" + :label-sv "Käskypäätös / tiedoksianto (ensimmäinen postitus) (sv)" + :valid true + :manually-deliverable false + :allow-comments false} + {:id 10 + :label-fi "Käskypäätös / tiedoksianto (toinen postitus)" + :label-sv "Käskypäätös / tiedoksianto (toinen postitus) (sv)" + :valid true + :manually-deliverable false + :allow-comments false} + {:id 11 + :label-fi "Käskypäätös / tiedoksianto (Haastemies)" + :label-sv "Käskypäätös / tiedoksianto (Haastemies) (sv)" + :valid true + :manually-deliverable false + :allow-comments false} + {:id 12 + :label-fi "Käskypäätös / odotetaan valitusajan umpeutumista" + :label-sv "Käskypäätös / odotetaan valitusajan umpeutumista (sv)" + :valid true + :manually-deliverable false + :allow-comments false} + {:id 13 + :label-fi "Käskypäätös / valitusaika umpeutunut" + :label-sv "Käskypäätös / valitusaika umpeutunut (sv)" + :valid true + :manually-deliverable false + :allow-comments false} + {:id 14 + :label-fi "Sakkopäätös / kuulemiskirje" + :label-sv "Sakkopäätös / kuulemiskirje (sv)" + :valid true + :manually-deliverable false + :allow-comments false} + {:id 15 + :label-fi "Sakkopäätös / varsinainen päätös" + :label-sv "Sakkopäätös / varsinainen päätös (sv)" + :valid true + :manually-deliverable false + :allow-comments false} + {:id 16 + :label-fi "Sakkopäätös / tiedoksianto (ensimmäinen postitus)" + :label-sv "Sakkopäätös / tiedoksianto (ensimmäinen postitus) (sv)" + :valid true + :manually-deliverable false + :allow-comments false} + {:id 17 + :label-fi "Sakkopäätös / tiedoksianto (toinen postitus)" + :label-sv "Sakkopäätös / tiedoksianto (toinen postitus) (sv)" + :valid true + :manually-deliverable false + :allow-comments false} + {:id 18 + :label-fi "Sakkopäätös / tiedoksianto (Haastemies)" + :label-sv "Sakkopäätös / tiedoksianto (Haastemies) (sv)" + :valid true + :manually-deliverable false + :allow-comments false} + {:id 19 + :label-fi "Sakkopäätös / odotetaan valitusajan umpeutumista" + :label-sv "Sakkopäätös / odotetaan valitusajan umpeutumista (sv)" + :valid true + :manually-deliverable false + :allow-comments false} + {:id 20 + :label-fi "Sakkopäätös / valitusaika umpeutunut" + :label-sv "Sakkopäätös / valitusaika umpeutunut (sv)" + :valid true + :manually-deliverable false + :allow-comments false} + {:id 21 + :label-fi "Sakkoluettelon lähetys menossa" + :label-sv "Sakkoluettelon lähetys menossa (sv)" + :valid true + :manually-deliverable false + :allow-comments false}])) - (t/testing "Toimenpidetyypit matches the schema" - (t/is (nil? (schema/check [valvonta-kaytto-schema/Toimenpidetyypit] - toimenpidetyypit)))))) + (t/testing "Toimenpidetyypit matches the schema" + (t/is (nil? (schema/check [valvonta-kaytto-schema/Toimenpidetyypit] + toimenpidetyypit))))))) From d6e4832100341c8c1ee5f9cfd9509194535872c0 Mon Sep 17 00:00:00 2001 From: Juho Leinonen Date: Fri, 30 Jun 2023 14:16:22 +0300 Subject: [PATCH 2/4] =?UTF-8?q?AE-1917:=20Allow=20comments=20for=20hao-k?= =?UTF-8?q?=C3=A4sittely=20toimenpide,=20fix=20a=20typo=20in=20its=20name?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../test/clj/solita/etp/service/valvonta_kaytto_test.clj | 6 +++--- .../repeatable/vk_template/r-0-vk-toimenpidetype.sql | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/etp-backend/src/test/clj/solita/etp/service/valvonta_kaytto_test.clj b/etp-backend/src/test/clj/solita/etp/service/valvonta_kaytto_test.clj index 2f3aab687..8e648232c 100644 --- a/etp-backend/src/test/clj/solita/etp/service/valvonta_kaytto_test.clj +++ b/etp-backend/src/test/clj/solita/etp/service/valvonta_kaytto_test.clj @@ -48,11 +48,11 @@ :manually-deliverable false :allow-comments false} {:id 6 - :label-fi "HaO käsittely" - :label-sv "HaO käsittely (sv)" + :label-fi "HaO-käsittely" + :label-sv "HaO-käsittely (sv)" :valid true :manually-deliverable false - :allow-comments false} + :allow-comments true} {:id 7 :label-fi "Käskypäätös / kuulemiskirje" :label-sv "Käskypäätös / kuulemiskirje (sv)" diff --git a/etp-db/src/main/sql/migration/repeatable/vk_template/r-0-vk-toimenpidetype.sql b/etp-db/src/main/sql/migration/repeatable/vk_template/r-0-vk-toimenpidetype.sql index 91d72f248..6fefb159b 100644 --- a/etp-db/src/main/sql/migration/repeatable/vk_template/r-0-vk-toimenpidetype.sql +++ b/etp-db/src/main/sql/migration/repeatable/vk_template/r-0-vk-toimenpidetype.sql @@ -6,7 +6,7 @@ values (3, 'Varoitus', 'Varning', 4, true, false, false), (4, 'Käskypäätös', 'Käskypäätös (sv)', 5, true, false, false), (5, 'Valvonnan lopetus', 'Valvonnan lopetus (sv)', 6, true, false, false), -(6, 'HaO käsittely', 'HaO käsittely (sv)', 7, true, false, false), +(6, 'HaO-käsittely', 'HaO-käsittely (sv)', 7, true, false, true), (7, 'Käskypäätös / kuulemiskirje', 'Käskypäätös / kuulemiskirje (sv)', 8, true, true, true), (8, 'Käskypäätös / varsinainen päätös', 'Käskypäätös / varsinainen päätös (sv)', 9, true, false, false), (9, 'Käskypäätös / tiedoksianto (ensimmäinen postitus)', 'Käskypäätös / tiedoksianto (ensimmäinen postitus) (sv)', 10, true, false, false), From dc10899516a6e967036d189afe627548a097e346 Mon Sep 17 00:00:00 2001 From: Juho Leinonen Date: Fri, 30 Jun 2023 15:00:42 +0300 Subject: [PATCH 3/4] =?UTF-8?q?AE-1917:=20Add=20hao-k=C3=A4sittely=20toime?= =?UTF-8?q?npidetype=20to=20id=20to=20type-key=20mapping?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../main/clj/solita/etp/service/valvonta_kaytto/toimenpide.clj | 1 + 1 file changed, 1 insertion(+) diff --git a/etp-backend/src/main/clj/solita/etp/service/valvonta_kaytto/toimenpide.clj b/etp-backend/src/main/clj/solita/etp/service/valvonta_kaytto/toimenpide.clj index 2fccd7149..7bf84a31c 100644 --- a/etp-backend/src/main/clj/solita/etp/service/valvonta_kaytto/toimenpide.clj +++ b/etp-backend/src/main/clj/solita/etp/service/valvonta_kaytto/toimenpide.clj @@ -15,6 +15,7 @@ ;; valvonnan sulkeminen (case closed) 5 :closed ;; Uhkasakkoprosessi + 6 :court-hearing 7 :decision-order-hearing-letter}) (defn type-key [type-id] From 67a14d42d10e4dc644d7534f9123f8c8a4c70475 Mon Sep 17 00:00:00 2001 From: Juho Leinonen Date: Mon, 3 Jul 2023 08:34:42 +0300 Subject: [PATCH 4/4] AE-1917: Fix testing block inside a testing block when it wasn't necessary --- .../test/clj/solita/etp/service/valvonta_kaytto_test.clj | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/etp-backend/src/test/clj/solita/etp/service/valvonta_kaytto_test.clj b/etp-backend/src/test/clj/solita/etp/service/valvonta_kaytto_test.clj index 8e648232c..ca561437c 100644 --- a/etp-backend/src/test/clj/solita/etp/service/valvonta_kaytto_test.clj +++ b/etp-backend/src/test/clj/solita/etp/service/valvonta_kaytto_test.clj @@ -142,8 +142,8 @@ :label-sv "Sakkoluettelon lähetys menossa (sv)" :valid true :manually-deliverable false - :allow-comments false}])) + :allow-comments false}]))) - (t/testing "Toimenpidetyypit matches the schema" - (t/is (nil? (schema/check [valvonta-kaytto-schema/Toimenpidetyypit] - toimenpidetyypit))))))) + (t/testing "Toimenpidetyypit matches the schema" + (t/is (nil? (schema/check [valvonta-kaytto-schema/Toimenpidetyypit] + toimenpidetyypit))))))