Skip to content

Commit 2eaf48f

Browse files
committed
increase code testing coverage
1 parent 622099f commit 2eaf48f

16 files changed

+1672
-512
lines changed

.github/codecov.yml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,11 @@ comment:
5353
ignore:
5454
- core/src/examples
5555
- core/src/tests
56+
- packages/client/dist
57+
- packages/client/scripts
5658
- packages/client/tests
59+
- packages/client/src/*_pb.js
60+
- packages/client/src/client_version.ts
5761
- server/
5862
- "**/node_modules"
5963
- "**/*.test.*"

.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -64,6 +64,7 @@ project
6464
!server/scala/project
6565

6666
## typescript
67+
packages/client/coverage/
6768
**/*.js.map
6869
**/out/
6970
**/dist/

packages/client/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@
4242
"package": "yarn prepare && yarn pack -f omega-edit-node-client-v${npm_package_version}.tgz",
4343
"test:client": "mocha --timeout 100000 --slow 50000 --require ts-node/register --require tests/fixtures.ts --exclude ./tests/specs/server.spec.ts ./tests/specs/*.spec.ts --exit",
4444
"test:lifecycle": "mocha --timeout 50000 --slow 35000 --require ts-node/register ./tests/specs/server.spec.ts --exit",
45-
"test:coverage": "c8 --reporter=lcov --reporter=text --reports-dir=coverage mocha --timeout 100000 --slow 50000 --require ts-node/register --require tests/fixtures.ts --exclude ./tests/specs/server.spec.ts ./tests/specs/*.spec.ts --exit",
45+
"test:coverage": "c8 --all --src src --exclude-after-remap --exclude=src/**/*.d.ts --exclude=src/client_version.ts --exclude=src/*_pb.js --exclude=dist/** --exclude=tests/** --exclude=scripts/** --reporter=lcov --reporter=text --reports-dir=coverage node scripts/run-coverage-tests.js",
4646
"test:client:uds": "OMEGA_EDIT_TEST_TRANSPORT=uds mocha --timeout 100000 --slow 50000 --require ts-node/register --require tests/fixtures.ts --exclude ./tests/specs/server.spec.ts ./tests/specs/*.spec.ts --exit",
4747
"test:lifecycle:uds": "OMEGA_EDIT_TEST_TRANSPORT=uds mocha --timeout 50000 --slow 35000 --require ts-node/register ./tests/specs/server.spec.ts --exit",
4848
"test:uds": "(yarn test:lifecycle:uds && yarn test:client:uds) || (exit 1)",
Lines changed: 68 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,68 @@
1+
const path = require('path')
2+
const fs = require('fs')
3+
const Mocha = require('mocha')
4+
5+
require('ts-node/register')
6+
7+
const { resetClient } = require(
8+
path.join(__dirname, '..', 'dist', 'cjs', 'index.js')
9+
)
10+
11+
async function runSuite(files, options = {}) {
12+
const mocha = new Mocha(options)
13+
for (const file of files) {
14+
mocha.addFile(file)
15+
}
16+
mocha.loadFiles()
17+
return new Promise((resolve) => {
18+
mocha.run((failures) => resolve(failures))
19+
})
20+
}
21+
22+
function getClientSpecFiles() {
23+
const specsDir = path.join(__dirname, '..', 'tests', 'specs')
24+
return fs
25+
.readdirSync(specsDir)
26+
.filter((file) => file.endsWith('.spec.ts') && file !== 'server.spec.ts')
27+
.map((file) => path.join(specsDir, file))
28+
}
29+
30+
async function main() {
31+
const lifecycleSpec = path.join(
32+
__dirname,
33+
'..',
34+
'tests',
35+
'specs',
36+
'server.spec.ts'
37+
)
38+
const lifecycleFailures = await runSuite([lifecycleSpec], {
39+
timeout: 50000,
40+
slow: 35000,
41+
})
42+
43+
if (lifecycleFailures !== 0) {
44+
process.exit(lifecycleFailures)
45+
}
46+
47+
resetClient()
48+
49+
const fixtures = require(path.join(__dirname, '..', 'tests', 'fixtures.ts'))
50+
await fixtures.mochaGlobalSetup()
51+
52+
let clientFailures = 1
53+
try {
54+
clientFailures = await runSuite(getClientSpecFiles(), {
55+
timeout: 100000,
56+
slow: 50000,
57+
})
58+
} finally {
59+
await fixtures.mochaGlobalTeardown()
60+
}
61+
62+
process.exit(clientFailures)
63+
}
64+
65+
main().catch((err) => {
66+
console.error(err)
67+
process.exit(1)
68+
})

0 commit comments

Comments
 (0)