medplum / medplum / 26187698230

92%

push

github

refactor(server): combine `systemLogger` + `globalLogger` (#9186)

* refactor(server): combine `systemLogger` + `globalLogger`

Signed-off-by: Derrick Farris <derrick@medplum.com>

* fix(server): use `writeLineToStdout` in `context` logger

Signed-off-by: Derrick Farris <derrick@medplum.com>

* fix(server): mock `globalLogger` methods instead of `console.log`

Signed-off-by: Derrick Farris <derrick@medplum.com>

* test(server): fix lint

Signed-off-by: Derrick Farris <derrick@medplum.com>

* [autofix.ci] apply automated fixes

* test: refactor to mock `Logger.log` instead of `Logger.{level}`

Signed-off-by: Derrick Farris <derrick@medplum.com>

* test(server): make `Logger.write` instead of `Logger.log`

Signed-off-by: Derrick Farris <derrick@medplum.com>

* [autofix.ci] apply automated fixes

* fix(server): convert all usage of `console.log` -> `globalLogger.write`

Signed-off-by: Derrick Farris <derrick@medplum.com>

* [autofix.ci] apply automated fixes

* fix(config): rm `logRequests` from default conf

Signed-off-by: Derrick Farris <derrick@medplum.com>

* test(logger): await drain on shutdown, don't wait for drain while logging

Signed-off-by: Derrick Farris <derrick@medplum.com>

* test(server): update seed tests to use `globalLogger`

Signed-off-by: Derrick Farris <derrick@medplum.com>

* test(server): use unaliased `console.log`

Signed-off-by: Derrick Farris <derrick@medplum.com>

* fix(server): only await drain if necessary

Signed-off-by: Derrick Farris <derrick@medplum.com>

* fix(server): use `once` promisified helper

Signed-off-by: Derrick Farris <derrick@medplum.com>

* fix(server): make sure we drain before calling process.exit

Signed-off-by: Derrick Farris <derrick@medplum.com>

* test(server): fix logger tests, add tests for `drainStdout`

Signed-off-by: Derrick Farris <derrick@medplum.com>

* test(server): test main case for `index.ts`, make sure log before exit

Signed-off-by: Derrick Farris <derrick@medplum.com>

* [autofix.ci] ... (continued)

21167 of 23871 branches covered (88.67%)

Branch coverage included in aggregate %.

20 of 25 new or added lines in 10 files covered. (80.0%)

34923 of 37097 relevant lines covered (94.14%)

23756.03 hits per line