hendrik/lens
1
0
Fork 0
mirror of https://github.com/lensapp/lens.git synced 2025-05-20 05:10:56 +00:00
Code Issues Projects Releases Wiki Activity

Fix linting

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

Signed-off-by: Iku-turso <mikko.aspiala@gmail.com>
This commit is contained in:
Iku-turso 2023-03-03 09:20:16 +02:00
parent 9414626c66
commit 92ddb33467
2 changed files with 37 additions and 38 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
packages/core/src/features/telemetry/emit-telemetry-from-specific-function-calls.test.ts
View File

@ -63,14 +63,14 @@ describe("emit-telemetry-from-specific-function-calls", () => {
di.register(
whiteListedInjectable,
whiteListedInjectableWithArgument,
nonWhiteListedInjectable
nonWhiteListedInjectable,
);
});
injectedWhiteListedFunction = di.inject(whiteListedInjectable);
injectedWhiteListedFunctionWithArgument = di.inject(
whiteListedInjectableWithArgument
whiteListedInjectableWithArgument,
);
injectedNonWhiteListedFunction = di.inject(nonWhiteListedInjectable);
@ -135,7 +135,7 @@ describe("emit-telemetry-from-specific-function-calls", () => {
injectedWhiteListedFunctionWithArgument(
"irrelevant-argument",
someObservable
someObservable,
);
});

69
packages/core/src/features/telemetry/renderer/telemetry-decorator.injectable.ts
View File

@ -14,9 +14,8 @@ import assert from "assert";
import { isFunction } from "lodash/fp";
import emitTelemetryInjectable from "./emit-telemetry.injectable";
import telemetryWhiteListForFunctionsInjectable, {
WhiteListItem,
} from "./telemetry-white-list-for-functions.injectable";
import type { WhiteListItem } from "./telemetry-white-list-for-functions.injectable";
import telemetryWhiteListForFunctionsInjectable from "./telemetry-white-list-for-functions.injectable";
const telemetryDecoratorInjectable = getInjectable({
id: "telemetry-decorator",
@ -25,7 +24,7 @@ const telemetryDecoratorInjectable = getInjectable({
const emitTelemetry = diForDecorator.inject(emitTelemetryInjectable);
const whiteList = diForDecorator.inject(
telemetryWhiteListForFunctionsInjectable
telemetryWhiteListForFunctionsInjectable,
);
const whitleListMap = getWhitleListMap(whiteList);
@ -33,32 +32,32 @@ const telemetryDecoratorInjectable = getInjectable({
return {
decorate:
(instantiateToBeDecorated: any) =>
(di: DiContainerForInjection, instantiationParameter: any) => {
const instance = instantiateToBeDecorated(di, instantiationParameter);
(di: DiContainerForInjection, instantiationParameter: any) => {
const instance = instantiateToBeDecorated(di, instantiationParameter);
if (isFunction(instance)) {
return (...args: any[]) => {
const currentContext = di.context.at(-1);
if (isFunction(instance)) {
return (...args: any[]) => {
const currentContext = di.context.at(-1);
assert(currentContext);
assert(currentContext);
const whiteListed = whitleListMap.get(
currentContext.injectable.id
);
const whiteListed = whitleListMap.get(
currentContext.injectable.id,
);
if (whiteListed) {
emitTelemetry({
action: currentContext.injectable.id,
params: whiteListed.getParams(...args),
});
}
if (whiteListed) {
emitTelemetry({
action: currentContext.injectable.id,
params: whiteListed.getParams(...args),
});
}
return instance(...args);
};
}
return instance(...args);
};
}
return instance;
},
return instance;
},
};
},
@ -73,18 +72,18 @@ const getWhitleListMap = (whiteList: WhiteListItem[]) =>
whiteList.map((item) =>
typeof item === "string"
? [
item,
{
getParams: (...args: any[]) => undefined,
},
]
item,
{
getParams: () => undefined,
},
]
: [
item.id,
{
getParams: item.getParams,
},
]
)
item.id,
{
getParams: item.getParams,
},
],
),
);
export default telemetryDecoratorInjectable;