fix(core): Allow license:clear command to be used for licenses that f…

…ailed renewal (#10665)

Co-authored-by: कारतोफ्फेलस्क्रिप्ट™ <[email protected]>

packages/cli/src/commands/license/clear.ts

@@ -12,10 +12,14 @@ export class ClearLicenseCommand extends BaseCommand {
 	async run() {
 		this.logger.info('Clearing license from database.');
 
-		// Invoke shutdown() to force any floating entitlements to be released
+		// Attempt to invoke shutdown() to force any floating entitlements to be released
 		const license = Container.get(License);
 		await license.init();
-		await license.shutdown();
+		try {
+			await license.shutdown();
+		} catch {
+			this.logger.info('License shutdown failed. Continuing with clearing license from database.');
+		}
 
 		await Container.get(SettingsRepository).delete({
 			key: SETTINGS_LICENSE_CERT_KEY,

packages/cli/test/integration/commands/license.cmd.test.ts

@@ -1,10 +1,14 @@
-import { License } from '@/license';
-import { LoadNodesAndCredentials } from '@/load-nodes-and-credentials';
-import { ClearLicenseCommand } from '@/commands/license/clear';
+import { Container } from 'typedi';
 
 import { setupTestCommand } from '@test-integration/utils/test-command';
 import { mockInstance } from '../../shared/mocking';
 
+import { ClearLicenseCommand } from '@/commands/license/clear';
+import { SETTINGS_LICENSE_CERT_KEY } from '@/constants';
+import { SettingsRepository } from '@/databases/repositories/settings.repository';
+import { License } from '@/license';
+import { LoadNodesAndCredentials } from '@/load-nodes-and-credentials';
+
 mockInstance(LoadNodesAndCredentials);
 const license = mockInstance(License);
 const command = setupTestCommand(ClearLicenseCommand);
@@ -15,3 +19,17 @@ test('license:clear invokes shutdown() to release any floating entitlements', as
 	expect(license.init).toHaveBeenCalledTimes(1);
 	expect(license.shutdown).toHaveBeenCalledTimes(1);
 });
+
+test('license:clear deletes the license from the DB even if shutdown() fails', async () => {
+	license.shutdown.mockRejectedValueOnce(new Error('shutdown failed'));
+
+	const settingsRepository = Container.get(SettingsRepository);
+
+	settingsRepository.delete = jest.fn();
+
+	await command.run();
+
+	expect(settingsRepository.delete).toHaveBeenCalledWith({
+		key: SETTINGS_LICENSE_CERT_KEY,
+	});
+});