Skip to content

Commit

Permalink
[INTERNAL] Don't pass workspace to Resolver constructor / fix tests
Browse files Browse the repository at this point in the history
  • Loading branch information
matz3 committed Mar 10, 2023
1 parent dfb837e commit 4254fb7
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
2 changes: 1 addition & 1 deletion lib/graph/helpers/ui5Framework.js
Original file line number Diff line number Diff line change
Expand Up @@ -354,7 +354,7 @@ export default {

log.info(`Using ${frameworkName} version: ${version}`);

const resolver = new Resolver({cwd: rootProject.getRootPath(), version, workspace});
const resolver = new Resolver({cwd: rootProject.getRootPath(), version});

let startTime;
if (log.isLevelEnabled("verbose")) {
Expand Down
3 changes: 2 additions & 1 deletion test/lib/graph/helpers/ui5Framework.js
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,8 @@ test.serial("enrichProjectGraph", async (t) => {

t.is(t.context.Sapui5ResolverInstallStub.callCount, 1, "Sapui5Resolver#install should be called once");
t.deepEqual(t.context.Sapui5ResolverInstallStub.getCall(0).args, [
referencedLibraries
referencedLibraries,
undefined
], "Sapui5Resolver#install should be called with expected args");

t.is(ProjectProcessorStub.callCount, 1, "ProjectProcessor#constructor should be called once");
Expand Down

0 comments on commit 4254fb7

Please sign in to comment.