Only trim leading empty lines, not tabs, when submitting #%% cells to…

… the interactive window (#7483)

news/2 Fixes/7303.md

@@ -0,0 +1 @@
+Preserve leading tab characters on code lines for #%% cells submitted to interactive window.

src/client/datascience/interactive-window/interactiveWindow.ts

@@ -59,7 +59,7 @@ import {
     WebViewViewChangeEventArgs
 } from '../types';
 import { createInteractiveIdentity, getInteractiveWindowTitle } from './identity';
-import { generateMarkdownFromCodeLines } from '../../../datascience-ui/common';
+import { generateMarkdownFromCodeLines, removeLinesFromFrontAndBack } from '../../../datascience-ui/common';
 import { chainWithPendingUpdates } from '../notebook/helpers/notebookUpdater';
 import { LineQueryRegex, linkCommandAllowList } from '../interactive-common/linkProvider';
 import { INativeInteractiveWindow } from './types';
@@ -627,7 +627,7 @@ export class InteractiveWindow implements IInteractiveWindowLoadable {
         const isMarkdown = cellMatcher.getCellType(code) === MARKDOWN_LANGUAGE;
         const strippedCode = isMarkdown
             ? generateMarkdownFromCodeLines(code.splitLines()).join('')
-            : cellMatcher.stripFirstMarker(code).trim();
+            : removeLinesFromFrontAndBack(cellMatcher.stripFirstMarker(code));
         const interactiveWindowCellMarker = cellMatcher.getFirstMarker(code);
 
         // Insert cell into NotebookDocument

src/test/datascience/interactiveWindow.vscode.test.ts

@@ -242,6 +242,28 @@ for i in range(10):
         );
     });
 
+    test('Leading and trailing empty lines in #%% cell are trimmed', async () => {
+        const actualCode = `    print('foo')
+
+
+
+    print('bar')`;
+        const codeWithWhitespace = `    # %%
+
+
+
+${actualCode}
+
+
+
+
+`;
+        const { activeInteractiveWindow: interactiveWindow } = await submitFromPythonFile(codeWithWhitespace);
+        const lastCell = await waitForLastCellToComplete(interactiveWindow);
+        const actualCellText = lastCell.document.getText();
+        assert.equal(actualCellText, actualCode);
+    });
+
     // todo@joyceerhl
     // test('Verify CWD', () => { });
     // test('Multiple executes go to last active window', async () => { });