Siivottu cljs pois, pakettipäivityksiä, ci testit uutta fronttia vasten

Opetushallitus · Feb 25, 2025 · dca3ebf · dca3ebf
1 parent 50d37fd
commit dca3ebf
Show file tree

Hide file tree

Showing 93 changed files with 3,496 additions and 12,065 deletions.
diff --git a/.eslintrc.js b/.eslintrc.js
@@ -28,14 +28,8 @@ module.exports = {
         argsIgnorePattern: '^_',
       },
     ],
-    '@typescript-eslint/ban-types': [
+    '@typescript-eslint/no-empty-object-type': [
       'error',
-      {
-        types: {
-          object: false,
-        },
-        extendDefaults: true,
-      },
     ],
     '@typescript-eslint/array-type': [
       'error',

diff --git a/.gitignore b/.gitignore
@@ -29,5 +29,6 @@
 /maksut.ipr
 .clj-kondo/
 .open-next
-/test-results/
-*.swp
+**/test-results/
+*.swp
+**/.npmrc
diff --git a/Makefile b/Makefile
@@ -1,11 +1,6 @@
-NODE_MODULES = node_modules
 PM2 = PM2_HOME=.pm2 npx pm2
 DOCKER_COMPOSE=COMPOSE_PARALLEL_LIMIT=8 $(if $(DOCKER_SUDO),sudo )docker compose
 
-$(NODE_MODULES): package.json package-lock.json
-	npm ci
-	touch $(NODE_MODULES)
-
 start-docker:
 	@$(DOCKER_COMPOSE) up -d maksut-nginx-local
 
@@ -18,8 +13,7 @@ start-docker-local:
 	@$(DOCKER_COMPOSE) up -d maksut-mailcatcher-local
 	@$(DOCKER_COMPOSE) up -d maksut-wiremock-local
 
-start-docker-test:
-	@$(DOCKER_COMPOSE) up -d maksut-nginx-local
+start-docker-test: start-docker-local
 	@$(DOCKER_COMPOSE) up -d maksut-e2e-db-local
 
 kill-docker:
@@ -28,27 +22,24 @@ kill-docker:
 kill-docker-test:
 	@$(DOCKER_COMPOSE) kill maksut-e2e-db-local
 
-start: $(NODE_MODULES) start-docker
-	@$(PM2) start pm2.config.js --only maksut-frontend
+start: start-docker
 	@$(PM2) start pm2.config.js --only maksut-backend
 
-start-local: $(NODE_MODULES) start-docker-local
-	@$(PM2) start pm2.config.js --only maksut-frontend
+start-local: start-docker-local
 	@$(PM2) start pm2.config.js --only maksut-backend
 
-log: $(NODE_MODULES)
+log:
 	@$(PM2) logs --timestamp
 
 logs: log
 
-status: $(NODE_MODULES)
+status:
 	@$(PM2) status
 
-kill: $(NODE_MODULES) kill-docker
+kill: kill-docker
 	@$(PM2) kill
 
 start-test: start-docker-test
-	@$(PM2) start pm2.config.js --only maksut-frontend
 	@$(PM2) start pm2.config.js --only maksut-backend-test
 
 kill-test: kill-docker-test
@@ -60,5 +51,4 @@ restart-test: kill-test start-test
 
 reload:
 	@$(PM2) kill
-	@$(PM2) start pm2.config.js --only maksut-frontend
 	@$(PM2) start pm2.config.js --only maksut-backend
diff --git a/README.md b/README.md
@@ -13,7 +13,7 @@
   * [Swagger](#swagger)
 * [Testien ajaminen](#testien-ajaminen)
   * [Lint](#lint)
-    * [Clojure(Script) -tiedostojen lint](#clojurescript--tiedostojen-lint)
+    * [Clojure -tiedostojen lint](#clojure--tiedostojen-lint)
 * [REPL-yhteys palvelimeen ja selaimeen](#repl-yhteys-palvelimeen-ja-selaimeen)
 * [Palvelun paikalliset osoitteet](#palvelun-paikalliset-osoitteet)
 * [Tuotantokäyttö](#tuotantokäyttö)
@@ -82,13 +82,6 @@ Paytrail-flowta voi kehittää/testata lokaalilla kannalla seuraavilla askelilla
 
 7. Tapahtuman tuloksena lähetetyt mailit voi katsoa Mailcatcherista osoitteesta: http://localhost:1080/.
 
-
-8. Käyttöliittymätestit voi ajaa lokaalia kehitysympäristöä ja Paytrailia vasten komennolla:
-
-   ```bash
-    WITH_PAYTRAIL=TRUE npx playwright test
-    ```
-
 ## Palvelun ajaminen paikallisesti testiympäristöä vasten
 
 Kloonaa ja valmistele omien ohjeiden mukaan käyttökuntoon [local-environment](https://github.com/Opetushallitus/local-environment) -ympäristö.
@@ -239,7 +232,7 @@ Swagger JSON löytyy polusta `/maksut/swagger.json`
 
 ### Lint
 
-#### Clojure(Script) -tiedostojen lint
+#### Clojure -tiedostojen lint
 
 ```sh
 npm run lint:clj
@@ -259,6 +252,27 @@ Aja sen jälkeen lein testit käyttäen lokaalia konfiguraatiotiedostoa:
 CONFIG=oph-configuration/config.test.local-environment.edn lein test
 ```
 
+### Playwrigth käyttöliittymätestit
+
+Käynnistä backend kuten backend-testejä varten, sekä käyttöliittymä
+
+```bash
+cd src/maksut-ui
+npm run test
+```
+
+Käynnistä testit (avaa käyttöliittymä lisäämällä --ui flag)
+
+```bash
+npx playwright test
+```
+
+Oikeaa Paytrailia vasten komennolla:
+
+```bash
+WITH_PAYTRAIL=TRUE npx playwright test
+```
+
 ## REPL-yhteys palvelimeen ja selaimeen
 
 REPL-yhteys palvelimelle avautuu sanomalla komentorivillä
@@ -267,16 +281,9 @@ REPL-yhteys palvelimelle avautuu sanomalla komentorivillä
 lein repl :connect localhost:9034
 ```
 
-REPL-yhteys selaimeen avautuu sanomalla em. REPL-yhteyden sisällä. Muistathan ensin avata selaimellasi palvelun (ks. osoite alta).
-
-```clj
-(shadow.cljs.devtools.api/nrepl-select :maksut)
-```
-
 ## Palvelun paikalliset osoitteet
 
 * Palvelun osoite: (http://localhost:9099/maksut)
-* Palvelun Shadow CLJS -palvelimen osoite (http://localhost:9630)
 
 ## Tuotantokäyttö
 

diff --git a/bin/cibuild.sh b/bin/cibuild.sh
@@ -14,10 +14,17 @@ test-lein() {
     lein test
 }
 
+test-e2e() {
+  cd src/maksut-ui
+  npm run test-ci
+  cd -
+}
+
 run-all-tests() {
   lint \
     && run-mocked-maksut \
-    && test-lein
+    && test-lein \
+    && test-e2e
 }
 
 create-uberjar() {
@@ -34,10 +41,8 @@ run-mocked-maksut() {
 }
 
 run-all-tests-and-create-uberjar() {
-  lint \
-    && create-uberjar \
-    && run-mocked-maksut \
-    && test-lein
+  run-all-tests \
+  && create-uberjar
 }
 
 COMMAND=$1

diff --git a/oph-configuration/clj-kondo.config.edn b/oph-configuration/clj-kondo.config.edn
@@ -1,17 +1,3 @@
-{:lint-as {day8.re-frame.tracing/defn-traced clojure.core/defn
-           day8.re-frame.tracing/fn-traced   clojure.core/fn}
- :linters {:unresolved-symbol   {:exclude [(compojure.api.sweet/context)
+{:linters {:unresolved-symbol   {:exclude [(compojure.api.sweet/context)
                                            (clojure.java.jdbc/with-db-transaction)
-                                           goog.DEBUG]}
-           :unused-namespace    {:exclude [cljs.repl]}
-           :unused-referred-var {:exclude {cljs.repl [Error->map
-                                                      apropos
-                                                      dir
-                                                      doc
-                                                      error->str
-                                                      ex-str
-                                                      ex-triage
-                                                      find-doc
-                                                      print-doc
-                                                      pst
-                                                      source]}}}}
+                                           goog.DEBUG]}}}
diff --git a/oph-configuration/config.dev.edn b/oph-configuration/config.dev.edn
@@ -39,5 +39,4 @@
                         :oppija-baseurl            "http://localhost:3000/maksut-ui"}
  :oph-organisaatio-oid "1.2.246.562.10.00000000001"
  :public-config        {:environment   :development
-                        :default-panel :panel/tutu-maksut
                         :caller-id     "1.2.246.562.10.00000000001.maksut.frontend"}}
diff --git a/oph-configuration/config.edn.template b/oph-configuration/config.edn.template
@@ -35,5 +35,4 @@
                         :oppija-baseurl     "{{ maksut_oppija_base_url }}"}
  :oph-organisaatio-oid "1.2.246.562.10.00000000001"
  :public-config        {:environment   {{ maksut_environment | default(':development') }}
-                        :default-panel :panel/tutu-maksut
                         :caller-id     "{{ maksut_frontend_caller_id | default('1.2.246.562.10.00000000001.maksut.frontend')}}"}}
diff --git a/oph-configuration/config.test.github.edn b/oph-configuration/config.test.github.edn
@@ -32,8 +32,7 @@
                                                      :session-cookie-name  "JSESSIONID"}}}
  :urls                 {:virkailija-baseurl        "http://localhost"
                         :maksut-url                "http://localhost:19033/maksut"
-                        :oppija-baseurl            "http://localhost:3000/maksut-ui"}
+                        :oppija-baseurl            "http://localhost:3003/maksut-ui"}
  :oph-organisaatio-oid "1.2.246.562.10.00000000001"
  :public-config        {:environment   :it
-                        :default-panel :panel/tutu-maksut
                         :caller-id     "1.2.246.562.10.00000000001.maksut.frontend"}}
diff --git a/oph-configuration/config.test.local-environment.edn b/oph-configuration/config.test.local-environment.edn
@@ -32,8 +32,7 @@
                                                      :session-cookie-name  "JSESSIONID"}}}
  :urls                 {:virkailija-baseurl        "http://localhost"
                         :maksut-url                "http://localhost:19033/maksut"
-                        :oppija-baseurl            "http://localhost:3000/maksut-ui"}
+                        :oppija-baseurl            "http://localhost:3003/maksut-ui"}
  :oph-organisaatio-oid "1.2.246.562.10.00000000001"
  :public-config        {:environment   :it
-                        :default-panel :panel/tutu-maksut
                         :caller-id     "1.2.246.562.10.00000000001.maksut.frontend"}}