refactor: formatLicense to return undefined instead of null

This commit is contained in:
Griffithtp 2019-07-07 11:17:22 +01:00
parent cf1f47e86c
commit 13c7aa6d03
3 changed files with 6 additions and 6 deletions

View File

@ -13,9 +13,9 @@ describe('formatLicense', (): void => {
});
test('should check license field for other value', (): void => {
expect(formatLicense(null)).toBeNull();
expect(formatLicense({})).toBeNull();
expect(formatLicense([])).toBeNull();
expect(formatLicense(null)).toBeUndefined();
expect(formatLicense({})).toBeUndefined();
expect(formatLicense([])).toBeUndefined();
});
});

View File

@ -12,7 +12,7 @@ export const TIMEFORMAT = 'DD.MM.YYYY, HH:mm:ss';
*/
// License should use type License defined above, but conflicts with the unit test that provide array or empty object
/* eslint-disable @typescript-eslint/no-explicit-any */
export function formatLicense(license: any): string | null {
export function formatLicense(license: any): string | undefined {
if (isString(license)) {
return license;
}
@ -21,7 +21,7 @@ export function formatLicense(license: any): string | null {
return license.type;
}
return null;
return;
}
export interface Repository {

View File

@ -5,7 +5,7 @@ export interface PackageMetaInterface {
fileCount: number;
unpackedSize: number;
};
license?: Partial<LicenseInterface> | string | null;
license?: Partial<LicenseInterface> | string;
};
_uplinks: {};
}