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

refactor(composer): use autoReplace

parent 69fedcfc
No related merge requests found
import { extractPackageFile } from './extract';
import { updateDependency } from '../npm/update';
import { updateArtifacts } from './artifacts';
import { getRangeStrategy } from './range';
import { LANGUAGE_PHP } from '../../constants/languages';
......@@ -7,14 +6,9 @@ import * as composerVersioning from '../../versioning/composer';
const language = LANGUAGE_PHP;
export const supportsLockFileMaintenance = true;
export const autoReplace = true;
export {
extractPackageFile,
updateArtifacts,
language,
updateDependency,
getRangeStrategy,
};
export { extractPackageFile, updateArtifacts, language, getRangeStrategy };
export const defaultConfig = {
fileMatch: ['(^|/)([\\w-]*)composer.json$'],
......
......@@ -82,7 +82,7 @@ Object {
"stderr": "some error",
},
],
"parentBranch": undefined,
"parentBranch": "some-branch",
"updatedArtifacts": Array [],
"updatedPackageFiles": Array [
Object {
......@@ -96,7 +96,7 @@ Object {
exports[`workers/branch/get-updated getUpdatedPackageFiles() handles lock files 1`] = `
Object {
"artifactErrors": Array [],
"parentBranch": undefined,
"parentBranch": "some-branch",
"updatedArtifacts": Array [
Object {
"contents": "some contents",
......
......@@ -77,7 +77,7 @@ describe('workers/branch/get-updated', () => {
config.upgrades.push({
manager: 'composer',
});
composer.updateDependency.mockReturnValue('some new content');
autoReplace.doAutoReplace.mockResolvedValueOnce('some new content');
composer.updateArtifacts.mockResolvedValueOnce([
{
file: {
......@@ -86,6 +86,9 @@ describe('workers/branch/get-updated', () => {
},
},
]);
config.upgrades.forEach(upgrade => {
upgrade.autoReplace = true; // eslint-disable-line no-param-reassign
});
const res = await getUpdatedPackageFiles(config);
expect(res).toMatchSnapshot();
});
......@@ -128,7 +131,7 @@ describe('workers/branch/get-updated', () => {
config.upgrades.push({
manager: 'composer',
});
composer.updateDependency.mockReturnValue('some new content');
autoReplace.doAutoReplace.mockResolvedValueOnce('some new content');
composer.updateArtifacts.mockResolvedValueOnce([
{
artifactError: {
......@@ -137,6 +140,9 @@ describe('workers/branch/get-updated', () => {
},
},
]);
config.upgrades.forEach(upgrade => {
upgrade.autoReplace = true; // eslint-disable-line no-param-reassign
});
const res = await getUpdatedPackageFiles(config);
expect(res).toMatchSnapshot();
});
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment