Skip to content

Commit

Permalink
Merge pull request #6501 from serverless/remove-enterprise-from-logline
Browse files Browse the repository at this point in the history
Remove Enterprise and Platform from log info
  • Loading branch information
pmuens committed Aug 5, 2019
2 parents a1976eb + 8f86fb2 commit 9159899
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion lib/classes/CLI.js
Original file line number Diff line number Diff line change
Expand Up @@ -330,7 +330,7 @@ class CLI {

getVersionNumber() {
this.consoleLog(
`${version} (Enterprise Plugin: ${enterpriseVersion}, Platform SDK: ${sdkVersion})`
`Framework Core: ${version}\nPlugin: ${enterpriseVersion}\nSDK: ${sdkVersion}\n`
);
}

Expand Down
6 changes: 3 additions & 3 deletions lib/classes/CLI.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -448,9 +448,9 @@ describe('CLI', () => {
cli.getVersionNumber();

expect(consoleLogSpy).to.have.been.calledOnce;
expect(consoleLogSpy.args[0][0]).to.match(/[0-9]+\.[0-9]+\.[0-9]+ .+/);
expect(consoleLogSpy.args[0][0]).to.include('Enterprise Plugin');
expect(consoleLogSpy.args[0][0]).to.include('Platform SDK');
expect(consoleLogSpy.args[0][0]).to.include('Framework Core');
expect(consoleLogSpy.args[0][0]).to.include('Plugin');
expect(consoleLogSpy.args[0][0]).to.include('SDK');
});
});

Expand Down

0 comments on commit 9159899

Please sign in to comment.