Skip to content
Snippets Groups Projects
Commit a1ba00d5 authored by Rhys Arkins's avatar Rhys Arkins
Browse files

refactor: extractAllPackageFiles insert depIndex

parent 7b240690
No related branches found
No related tags found
No related merge requests found
...@@ -41,22 +41,24 @@ describe('manager', () => { ...@@ -41,22 +41,24 @@ describe('manager', () => {
}); });
describe('extractAllPackageFiles()', () => { describe('extractAllPackageFiles()', () => {
it('returns null', () => { it('returns null', async () => {
manager.getManagers().set('dummy', { manager.getManagers().set('dummy', {
defaultConfig: {}, defaultConfig: {},
}); });
expect( expect(
manager.extractAllPackageFiles('unknown', {} as any, []) await manager.extractAllPackageFiles('unknown', {} as any, [])
).toBeNull();
expect(
await manager.extractAllPackageFiles('dummy', {} as any, [])
).toBeNull(); ).toBeNull();
expect(manager.extractAllPackageFiles('dummy', {} as any, [])).toBeNull();
}); });
it('returns non-null', () => { it('returns non-null', async () => {
manager.getManagers().set('dummy', { manager.getManagers().set('dummy', {
defaultConfig: {}, defaultConfig: {},
extractAllPackageFiles: () => Promise.resolve([]), extractAllPackageFiles: () => Promise.resolve([]),
}); });
expect( expect(
manager.extractAllPackageFiles('dummy', {} as any, []) await manager.extractAllPackageFiles('dummy', {} as any, [])
).not.toBeNull(); ).not.toBeNull();
}); });
afterEach(() => { afterEach(() => {
......
...@@ -49,18 +49,27 @@ export const getLanguageList = (): string[] => languageList; ...@@ -49,18 +49,27 @@ export const getLanguageList = (): string[] => languageList;
export const getManagerList = (): string[] => managerList; export const getManagerList = (): string[] => managerList;
export const getManagers = (): Map<string, ManagerApi> => managers; export const getManagers = (): Map<string, ManagerApi> => managers;
export function extractAllPackageFiles( export async function extractAllPackageFiles(
manager: string, manager: string,
config: ExtractConfig, config: ExtractConfig,
files: string[] files: string[]
): Result<PackageFile[] | null> { ): Promise<PackageFile[] | null> {
if (!managers.has(manager)) { if (!managers.has(manager)) {
return null; return null;
} }
const m = managers.get(manager); const m = managers.get(manager);
return m.extractAllPackageFiles if (m.extractAllPackageFiles) {
? m.extractAllPackageFiles(config, files) const res = await m.extractAllPackageFiles(config, files);
: null; // istanbul ignore if
if (!res) {
return null;
}
return res.map((packageFile) => ({
...packageFile,
managerPackageFileList: files,
}));
}
return null;
} }
export function getPackageUpdates( export function getPackageUpdates(
......
...@@ -8,6 +8,9 @@ Array [ ...@@ -8,6 +8,9 @@ Array [
"lernaClient": "yarn", "lernaClient": "yarn",
"lernaDir": ".", "lernaDir": ".",
"lernaPackages": undefined, "lernaPackages": undefined,
"managerPackageFileList": Array [
"package.json",
],
"npmLock": "package-lock.json", "npmLock": "package-lock.json",
"npmrc": "{}", "npmrc": "{}",
"packageFile": "package.json", "packageFile": "package.json",
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment