Skip to content

Commit

Permalink
OK-623 filtering for valintapiste
Browse files Browse the repository at this point in the history
  • Loading branch information
vaeinoe committed Sep 16, 2024
1 parent 2f5cc16 commit 578c026
Show file tree
Hide file tree
Showing 4 changed files with 65 additions and 5 deletions.
55 changes: 55 additions & 0 deletions spec/ataru/virkailija/virkailija_routes_spec.clj
Original file line number Diff line number Diff line change
Expand Up @@ -133,6 +133,13 @@
(update-in [:headers] assoc "cookie" (login @virkailija-routes))
((deref virkailija-routes)))))

(defn- get-valintapiste-application-query [query]
(-> (mock/request :get "/lomake-editori/api/external/valintapiste" query)
(update-in [:headers] assoc "cookie" (login @virkailija-routes "SUPERUSER"))
(mock/content-type "application/json")
((deref virkailija-routes))
(update :body (comp (fn [content] (json/parse-string content true)) slurp))))

(defn- get-tilastokeskus-application-query [query]
(-> (mock/request :get "/lomake-editori/api/external/tilastokeskus" query)
(update-in [:headers] assoc "cookie" (login @virkailija-routes "SUPERUSER"))
Expand Down Expand Up @@ -1014,4 +1021,52 @@
(should= 200 status)
(should= 0 (count applications)))))

(describe "valintapiste"
(tags :unit)

(after-all
(db/nuke-kk-payment-data))

(it "should return an application"
(let [_ (db/init-db-fixture fixtures/payment-exemption-test-form
application-fixtures/application-without-hakemusmaksu-exemption
nil)
haku-oid (:haku application-fixtures/application-without-hakemusmaksu-exemption)
resp (get-valintapiste-application-query {:hakuOid haku-oid})
status (:status resp)
applications (:body resp)]
(should= 200 status)
(should= 1 (count applications))
(println applications)))

(it "should return an application with kk payment data"
(let [person-oid "1.2.3.4.5.303"
term "kausi_s"
year 2025
haku-oid (:haku application-fixtures/application-without-hakemusmaksu-exemption)
_ (db/init-db-fixture fixtures/payment-exemption-test-form
application-fixtures/application-without-hakemusmaksu-exemption
nil)
_ (payment/set-application-fee-not-required person-oid term year nil nil)
resp (get-valintapiste-application-query {:hakuOid haku-oid})
status (:status resp)
applications (:body resp)]
(should= 200 status)
(should= 1 (count applications))))

(it "should not return an application awaiting kk payment"
(let [person-oid "1.2.3.4.5.303"
term "kausi_s"
year 2025
haku-oid (:haku application-fixtures/application-without-hakemusmaksu-exemption)
_ (db/init-db-fixture fixtures/payment-exemption-test-form
application-fixtures/application-without-hakemusmaksu-exemption
nil)
_ (payment/set-application-fee-required person-oid term year nil nil)
resp (get-valintapiste-application-query {:hakuOid haku-oid})
status (:status resp)
applications (:body resp)]
(should= 200 status)
(should= 0 (count applications)))))

(run-specs)
4 changes: 2 additions & 2 deletions src/clj/ataru/applications/application_access_control.clj
Original file line number Diff line number Diff line change
Expand Up @@ -368,14 +368,14 @@
(constantly nil)
#(tilastokeskus-service/get-application-info-for-tilastokeskus person-service tarjonta-service valintalaskentakoostepalvelu-service suoritus-service haku-oid hakukohde-oid)))

(defn get-applications-for-valintapiste [organization-service session haku-oid hakukohde-oid]
(defn get-applications-for-valintapiste [organization-service session tarjonta-service haku-oid hakukohde-oid]
(session-orgs/run-org-authorized
session
organization-service
[:view-applications :edit-applications]
(constantly nil)
(constantly nil)
#(valintapiste-service/get-application-info-for-valintapiste haku-oid hakukohde-oid)))
#(valintapiste-service/get-application-info-for-valintapiste tarjonta-service haku-oid hakukohde-oid)))

(defn get-applications-for-valintalaskenta [organization-service session hakukohde-oid application-keys]
(session-orgs/run-org-authorized
Expand Down
10 changes: 7 additions & 3 deletions src/clj/ataru/valintapiste/valintapiste_service.clj
Original file line number Diff line number Diff line change
@@ -1,6 +1,10 @@
(ns ataru.valintapiste.valintapiste-service
(:require [ataru.applications.application-store :as application-store]))
(:require [ataru.applications.application-store :as application-store]
[ataru.kk-application-payment.kk-application-payment :as kk-application-payment]))

(defn get-application-info-for-valintapiste
[haku-oid hakukohde-oid]
(application-store/get-application-info-for-valintapiste haku-oid hakukohde-oid))
[tarjonta-service haku-oid hakukohde-oid]
(kk-application-payment/filter-out-unpaid-kk-applications
tarjonta-service
(application-store/get-application-info-for-valintapiste haku-oid hakukohde-oid)
:henkilo_oid :haku_oid))
1 change: 1 addition & 0 deletions src/clj/ataru/virkailija/virkailija_routes.clj
Original file line number Diff line number Diff line change
Expand Up @@ -1672,6 +1672,7 @@
:return [ataru-schema/ValintapisteApplication]
(if-let [applications (access-controlled-application/get-applications-for-valintapiste organization-service
session
tarjonta-service
hakuOid
hakukohdeOid)]
(response/ok applications)
Expand Down

0 comments on commit 578c026

Please sign in to comment.