Extract minimal abstraction for specifically logging error instead of also warn, info, etc.

Co-authored-by: Janne Savolainen <janne.savolainen@live.fi>

Signed-off-by: Iku-turso <mikko.aspiala@gmail.com>
This commit is contained in:
Iku-tursoandJanne Savolainen committed 2022-06-22 13:54:34 +03:00
1 parent 2f4a409a1c
commit f56b48fea7
2 files changed
+17 -4

No files matched your search

+13
View File
@@ -0,0 +1,13 @@
/**
* Copyright (c) OpenLens Authors. All rights reserved.
* Licensed under MIT License. See LICENSE in root directory for more information.
*/
import { getInjectable } from "@ogre-tools/injectable";
import loggerInjectable from "./logger.injectable";
const logErrorInjectable = getInjectable({
id: "log-error",
instantiate: (di) => di.inject(loggerInjectable).error,
});
export default logErrorInjectable;
@@ -3,8 +3,8 @@
* Licensed under MIT License. See LICENSE in root directory for more information.
*/
import { getInjectable } from "@ogre-tools/injectable";
import loggerInjectable from "../../logger.injectable";
import { isPromise } from "../is-promise/is-promise";
import logErrorInjectable from "../../log-error.injectable";
export type WithErrorLoggingFor = (
getErrorMessage: (error: unknown) => string
@@ -16,7 +16,7 @@ const withErrorLoggingInjectable = getInjectable({
id: "with-error-logging",
instantiate: (di): WithErrorLoggingFor => {
const logger = di.inject(loggerInjectable);
const logError = di.inject(logErrorInjectable);
return (getErrorMessage) =>
(toBeDecorated) =>
@@ -28,7 +28,7 @@ const withErrorLoggingInjectable = getInjectable({
returnValue.catch((e) => {
const errorMessage = getErrorMessage(e);
logger.error(errorMessage, e);
logError(errorMessage, e);
});
}
@@ -36,7 +36,7 @@ const withErrorLoggingInjectable = getInjectable({
} catch (e) {
const errorMessage = getErrorMessage(e);
logger.error(errorMessage, e);
logError(errorMessage, e);
throw e;
}