-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: Add jest unit testing #67
Changes from 38 commits
9a54d5f
b75a9e4
9baaa28
000743b
d1411ec
c5ff7fe
ddf7f95
4eff0e6
29271ce
eefce76
91ae46e
a7d8b1c
27ea02f
2f3fdbe
004afcf
409a841
45fb166
b75845f
7274414
2ec79e6
d89bc0a
3b81807
9c4579b
cc00f12
f1cadf1
6aeff3c
0d22c3c
9374cc2
7893dd0
8244aaa
f684433
8dbb400
4fa7b05
1c10573
694101c
1764998
1963564
9c2dd99
63a1b5a
250f37f
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,7 +1,6 @@ | ||
on: | ||
push: | ||
branches: | ||
- main | ||
release: | ||
types: [published] | ||
Comment on lines
+2
to
+3
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. this is in preparation for changing the release cadence. stop pushing every PR to staging |
||
|
||
name: Deploy main to staging | ||
jobs: | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,27 @@ | ||
name: Jest unit tests | ||
on: push | ||
|
||
jobs: | ||
jest-unit-tests: | ||
runs-on: [ARM64, self-hosted, Linux] | ||
steps: | ||
- name: Check out code | ||
uses: actions/checkout@v3 | ||
- name: Install node | ||
uses: actions/setup-node@v4 | ||
with: | ||
node-version-file: '.node-version' | ||
- name: Install system dependencies | ||
run: | | ||
sudo apt-get update | ||
sudo apt-get install -y libcurl4-openssl-dev | ||
- name: Build Docker image | ||
run: | | ||
make build | ||
- name: Create Docker network | ||
run: | | ||
docker swarm init | ||
docker network create czidnet --attachable --driver overlay | ||
- name: Run tests in Docker container | ||
run: | | ||
make test |
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,6 @@ | ||
query ZipLinkQuery($workflowRunId: String) { | ||
ZipLink(workflowRunId: $workflowRunId) { | ||
url | ||
error | ||
} | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,5 @@ | ||
/** @type {import('ts-jest').JestConfigWithTsJest} */ | ||
module.exports = { | ||
preset: "ts-jest", | ||
testEnvironment: "node", | ||
}; |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,7 +1,7 @@ | ||
{ | ||
"type": "object", | ||
"properties": { | ||
"quality_metrics": { | ||
"metric_amr": { | ||
"type": "object", | ||
"properties": { | ||
"total_reads": { | ||
|
@@ -33,7 +33,7 @@ | |
} | ||
} | ||
}, | ||
"report_table_data": { | ||
"amr_hit": { | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Align to schema returned by resolver. This was preventing the tests from running, because the schema could not be generated |
||
"type": "array", | ||
"items": { | ||
"type": "object", | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -104,7 +104,7 @@ | |
"type": "object", | ||
"properties": { | ||
"tax_id": { | ||
"type": "integer" | ||
"type": "string" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Align to schema returned by resolver. This was preventing the tests from running, because the schema could not be generated |
||
}, | ||
"name": { | ||
"type": "string" | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,4 @@ | ||
apply: Query.ZipLink | ||
store: | ||
type: ZipLink | ||
key: "{args.workflowRunId}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since #61 was closed in favor of this PR, I am pulling this change in that was there.