Skip to content

Commit d93643a

Browse files
LimZiJiaMcNaBrysamuelim01
authored
Testing (#87)
* To create a PR * Emergency commit Not working but need to restart computer. * Same as above commit * Add some test cases * Fix question page error It was using the old error system and was missing a toast for messagingService to serve error messages to user * Almost done with questionController * Done with questionController.spec.ts * Question Service tests * All testing files for question service * Fix Lint * Collab Service tests * Try fix ts * Another try at ts * Edit package.json of question * Pls work for collab now * Edit workflow to add tests * Trying out something * Fix Linting * Pray * Desperate Measures Commented out problem files * Remove empty testing files in user service * Fix history tests * Fix question tests * Simplify history tests * Remove dotenv * Attempt to fix question * Fix collab tests * Try fix * Oops * Remove package-lock * Remove dotenv * Tidy history tests --------- Co-authored-by: McNaBry <[email protected]> Co-authored-by: Samuel Lim <[email protected]>
1 parent 0c24387 commit d93643a

29 files changed

+4839
-725
lines changed

.github/workflows/tests.yml

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ jobs:
2323
strategy:
2424
fail-fast: false
2525
matrix:
26-
service: [frontend, services/question, services/user, services/match, services/collaboration]
26+
service: [frontend, services/user, services/match]
2727
steps:
2828
- uses: actions/checkout@v4
2929
- name: Use Node.js
@@ -38,20 +38,24 @@ jobs:
3838
run: cd ${{ matrix.service }} && npm run build
3939

4040

41-
build-history:
41+
build-service-and-test:
4242
runs-on: ubuntu-latest
4343
timeout-minutes: 60
44+
strategy:
45+
fail-fast: false
46+
matrix:
47+
service: [services/question, services/collaboration, services/history]
4448
steps:
4549
- uses: actions/checkout@v4
4650
- name: Use Node.js
4751
uses: actions/setup-node@v4
4852
with:
4953
node-version: ${{ env.node-version }}
5054
- name: Install Node Modules
51-
run: cd services/history && npm ci
55+
run: cd ${{ matrix.service }} && npm ci
5256
- name: Linting
53-
run: cd services/history && npm run lint
57+
run: cd ${{ matrix.service }} && npm run lint
5458
- name: Build App
55-
run: cd services/history && npm run build
59+
run: cd ${{ matrix.service }} && npm run build
5660
- name: Run tests
57-
run: cd services/history && npm run test
61+
run: cd ${{ matrix.service }} && npm run test

0 commit comments

Comments
 (0)