medplum / medplum / 26187698230 / 1

92%

main: 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)

22012 of 24847 branches covered (88.59%)

Branch coverage included in aggregate %.

44631 of 47404 relevant lines covered (94.15%)

18590.78 hits per line