Merge pull request #209 from LiskHQ/208-add-edge-case-scenario-handli…

…ng-in-getLogLevel

Add edge-case scenario handling in getLogLevel

README.md

@@ -81,7 +81,7 @@ $ npm install -g lisk-migrator
 $ lisk-migrator COMMAND
 running command...
 $ lisk-migrator (-v|--version|version)
-lisk-migrator/2.0.0 darwin-arm64 node-v18.16.1
+lisk-migrator/2.0.1 darwin-arm64 node-v18.16.1
 $ lisk-migrator --help [COMMAND]
 USAGE
   $ lisk-migrator COMMAND
@@ -108,9 +108,9 @@ Lisk Migrator has an extensive set of unit tests. To run the tests, please insta
 $ npm test
 ```
 
-## Migrating from Lisk Core v3.1.0 to v4.0.0
+## Migrating from Lisk Core v3.1.0 to v4.0.1
 
-The [migration guide](./docs/migration.md) explains the transition process from Lisk Core v3.1.0 (or later) to Lisk Core v4.0.0 using Lisk Migrator v2.
+The [migration guide](./docs/migration.md) explains the transition process from Lisk Core v3.1.0 (or later) to Lisk Core v4.0.1 using Lisk Migrator v2.
 
 ## Get Involved
 

docs/migration.md

@@ -1,8 +1,8 @@
 # Migration Guide
 
-This section explains how to migrate a Lisk Core v3.1.0 node to Lisk Core v4.0.0 using the Lisk Migrator.
+This section explains how to migrate a Lisk Core v3.1.0 node to Lisk Core v4.0.1 using the Lisk Migrator.
 
-The Lisk Migrator CLI tool will generate a new genesis (snapshot) block for Lisk Core v4.0.0.
+The Lisk Migrator CLI tool will generate a new genesis (snapshot) block for Lisk Core v4.0.1.
 The new genesis block is created based on a snapshot of the existing blockchain (running on Lisk Core v3.1.0) at a pre-determined height.
 
 Lisk Migrator automatically exports the node's Forging Status information to the file named `forgingStatus.json` under the output directory.

package.json

@@ -1,6 +1,6 @@
 {
 	"name": "lisk-migrator",
-	"version": "2.0.0",
+	"version": "2.0.1",
 	"description": "A command-line tool for migrating the blockchain state to the latest protocol after a hard fork",
 	"author": "Lisk Foundation <[email protected]>, lightcurve GmbH <[email protected]>",
 	"license": "Apache-2.0",

src/utils/config.ts

@@ -50,8 +50,10 @@ export const getNetworkByNetworkID = (_networkID: string): string | Error => {
 
 export const getLogLevel = (loggerConfig: LoggerConfig): string => {
 	const highestLogPriority = Math.max(
-		LOG_LEVEL_PRIORITY[String(loggerConfig.fileLogLevel).toUpperCase()],
-		LOG_LEVEL_PRIORITY[String(loggerConfig.consoleLogLevel).toUpperCase()],
+		LOG_LEVEL_PRIORITY[String(loggerConfig.fileLogLevel || '').toUpperCase()] ??
+			LOG_LEVEL_PRIORITY.INFO,
+		LOG_LEVEL_PRIORITY[String(loggerConfig.consoleLogLevel || '').toUpperCase()] ??
+			LOG_LEVEL_PRIORITY.INFO,
 	);
 
 	try {

src/utils/node.ts

@@ -36,7 +36,7 @@ import {
 } from '../constants';
 import { MigratorException } from './exception';
 
-const INSTALL_LISK_CORE_COMMAND = 'npm i -g lisk-core@^4.0.0-rc.1';
+const INSTALL_LISK_CORE_COMMAND = 'npm i -g lisk-core@^4.0.1';
 const INSTALL_PM2_COMMAND = 'npm i -g pm2';
 const PM2_FILE_NAME = 'pm2.migrator.config.json';
 
@@ -158,6 +158,9 @@ const resolveLiskCoreStartCommand = async (_this: Command, network: string, conf
 	let customStartCommand = baseStartCommand;
 
 	_this.log('Customizing Lisk Core start command');
+	_this.log(
+		`Kindly do not forget to include '--config ${configPath}' in your custom start command, if you still want to use this config.`,
+	);
 	let userInput = await cli.prompt(
 		"Please provide the Lisk Core start command flags (e.g. --api-ws), except the '--network (-n)' flag:",
 	);

test/unit/utils/config.spec.ts

@@ -21,14 +21,15 @@ import { configV3, configV4 } from '../fixtures/config';
 import { NETWORK_CONSTANT } from '../../../src/constants';
 import {
 	getNetworkByNetworkID,
+	getLogLevel,
 	migrateUserConfig,
 	validateConfig,
 	writeConfig,
 	resolveConfigPathByNetworkID,
 	createBackup,
 	getConfig,
 } from '../../../src/utils/config';
-import { ApplicationConfigV3 } from '../../../src/types';
+import { ApplicationConfigV3, LoggerConfig } from '../../../src/types';
 
 const migratedConfigFilePath = join(__dirname, 'test/config');
 const expectedBackupPath = join(__dirname, '../../..', 'backup');
@@ -38,6 +39,41 @@ const mockCommand = {
 	error,
 };
 
+describe('Test getLogLevel method', () => {
+	it('should return highest priority logLevel provided by user', async () => {
+		const loggerConfig = {
+			fileLogLevel: 'trace',
+			consoleLogLevel: 'error',
+		} as LoggerConfig;
+		const logLevel = getLogLevel(loggerConfig);
+		expect(logLevel).toBe('trace');
+	});
+
+	it('should return info when logger config is not available', async () => {
+		const loggerConfig = {} as LoggerConfig;
+		const logLevel = getLogLevel(loggerConfig);
+		expect(logLevel).toBe('info');
+	});
+
+	it('should return the highest priority log level when one of the specified logLevel is incorrect', async () => {
+		const loggerConfig = {
+			fileLogLevel: 'trace',
+			consoleLogLevel: 'err',
+		} as LoggerConfig;
+		const logLevel = getLogLevel(loggerConfig);
+		expect(logLevel).toBe('trace');
+	});
+
+	it('should return info if the specified valid log level is of lower priority than info and other is incorrect', async () => {
+		const loggerConfig = {
+			fileLogLevel: 'fatal',
+			consoleLogLevel: 'err',
+		} as LoggerConfig;
+		const logLevel = getLogLevel(loggerConfig);
+		expect(logLevel).toBe('info');
+	});
+});
+
 describe('Migrate user configuration', () => {
 	afterAll(() => {
 		fs.removeSync(migratedConfigFilePath);