review comments addressed

Author: prathameshzarkar9

src/spec-common/injectHeadless.ts

@@ -535,7 +535,9 @@ async function runLifecycleCommand({ lifecycleHook }: ResolverParameters, contai
 			if (err && (err.code === 130 || err.signal === 2)) { // SIGINT seen on darwin as code === 130, would also make sense as signal === 2.
 				infoOutput.raw(`\r\n\x1b[1m${lifecycleHookName} interrupted.\x1b[0m\r\n\r\n`);
 			} else {
-				infoOutput.write(toErrorText(`${err.cmdOutput}`));
+				if (err.cmdOutput) {
+					infoOutput.write(toErrorText(`${err.cmdOutput}`));
+				}
 				if (err?.code) {
 					infoOutput.write(toErrorText(`${lifecycleHookName} failed with exit code ${err.code}. Skipping any further user-provided commands.`));
 				}

src/test/cli.up.test.ts

@@ -25,13 +25,6 @@ describe('Dev Containers CLI', function () {
 	});
 
 	describe('Command up', () => {
-		it('should succeed (dockerfile) with supported --error for poetry', async () => {
-			const testFolder = `${__dirname}/configs/dockerfile-with-target`;
-			const res = await shellExec(`${cli} up --workspace-folder ${testFolder}`);
-			const response = JSON.parse(res.stdout);
-			assert.equal(response.outcome, 'success');
-		});
-
 		it('should execute successfully with valid config', async () => {
 			const res = await shellExec(`${cli} up --workspace-folder ${__dirname}/configs/image`);
 			const response = JSON.parse(res.stdout);