Skip to content

Commit

Permalink
Remove temporally workflow and add test admin controller
Browse files Browse the repository at this point in the history
  • Loading branch information
KapStorm committed Sep 4, 2024
1 parent 8f7b8ce commit ead3da5
Show file tree
Hide file tree
Showing 3 changed files with 18 additions and 12 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/codepreview.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,9 @@ name: Create preview environment

on:
pull_request:
branches: [ master ]
branches: [ ]
push:
branches: [ master, scala3 ]
branches: [ ]

concurrency:
# The preview script can't handle concurrent deploys
Expand Down
25 changes: 15 additions & 10 deletions .github/workflows/pull_request.yml
Original file line number Diff line number Diff line change
Expand Up @@ -38,17 +38,22 @@ jobs:
key: compiled-code-cache-${{ hashFiles('**/build.sbt') }}
restore-keys: compiled-code-cache-

- name: Check code format
run: sbt scalafmtCheckAll
- name: Run test admin controller
run: CI=true sbt testOnly controllers.AdminControllerSpec

- name: Compile
run: CI=true sbt compile
# TODO uncomment when tests are added
# - name: Check code format
# run: sbt scalafmtCheckAll

- name: Run tests
run: CI=true sbt test
# - name: Compile
# run: CI=true sbt compile

- name: Prepare web build
run: sbt web/build
# - name: Run tests
# run: CI=true sbt test

- name: Prepare server build
run: sbt server/dist
#
# - name: Prepare web build
# run: sbt web/build
#
# - name: Prepare server build
# run: sbt server/dist
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,7 @@ class AdminControllerSpec extends PlayPostgresSpec with LoginUtils with MockitoS

"GET /admin/users" should {
"get every user" in withApiClient { implicit client =>
throw new NotImplementedError("Kappa error")
val expected = 3
(1 to expected).foreach { _ =>
createVerifyLoginUser(
Expand Down

0 comments on commit ead3da5

Please sign in to comment.