Update longest multi-profile sketch E2E test to not check specific KCL values (#8010)

We should and do capture that sort of thing elsewhere. This is another
breakout PR from #7802, towards #7222. I've confirmed it works locally
on MacOS both against `main` and the branch for #7802.

Author: franknoirot

e2e/playwright/fixtures/editorFixture.ts

@@ -22,19 +22,21 @@ export class EditorFixture {
   private diagnosticsGutterIcon!: Locator
   public codeContent!: Locator
   public activeLine!: Locator
+  public lines!: Locator
 
   constructor(page: Page) {
     this.page = page
     this.codeContent = page.locator('.cm-content[data-language="kcl"]')
     this.diagnosticsTooltip = page.locator('.cm-tooltip-lint')
     this.diagnosticsGutterIcon = page.locator('.cm-lint-marker-error')
     this.activeLine = this.page.locator('.cm-activeLine')
+    this.lines = this.page.locator('.cm-line')
   }
 
   private _expectEditorToContain =
     (not = false) =>
     async (
-      code: string,
+      code: string | RegExp,
       {
         shouldNormalise = false,
         timeout = 5_000,
@@ -49,17 +51,21 @@ export class EditorFixture {
           await this.closePane()
         }
       }
+
+      await this.scrollToBottom()
       if (!shouldNormalise) {
-        const expectStart = expect.poll(() => this.codeContent.textContent())
-        if (not) {
-          const result = await expectStart.not.toContain(code)
-          await resetPane()
-          return result
-        }
-        const result = await expectStart.toContain(code)
+        const result = not
+          ? await expect(this.codeContent).not.toContainText(code)
+          : await expect(this.codeContent).toContainText(code)
+
         await resetPane()
         return result
       }
+      if (typeof code !== 'string') {
+        throw new Error(
+          'The `shouldNormalise` option does not work with a RegExp value for `code` argument'
+        )
+      }
       const normalisedCode = code.replaceAll(/\s+/g, '').trim()
       const expectStart = expect.poll(
         async () => {
@@ -88,14 +94,24 @@ export class EditorFixture {
     },
   }
   getCurrentCode = async () => {
-    return await this.codeContent.innerText()
+    const isOpen = await this.checkIfPaneIsOpen()
+    if (isOpen) {
+      await this.scrollToBottom()
+      return await this.codeContent.innerText()
+    }
+
+    await this.openPane()
+    await this.scrollToBottom()
+    const code = await this.codeContent.innerText()
+    await this.closePane()
+    return code
   }
   snapshot = async (options?: { timeout?: number; name?: string }) => {
     const wasPaneOpen = await this.checkIfPaneIsOpen()
     if (!wasPaneOpen) {
       await this.openPane()
     }
-
+    await this.scrollToBottom()
     try {
       // Use expect.poll to implement retry logic
       await expect
@@ -180,6 +196,16 @@ export class EditorFixture {
   openPane() {
     return openPane(this.page, this.paneButtonTestId)
   }
+  async scrollToBottom() {
+    let nextLineCount = await this.lines.count()
+    let currLineCount: number
+    do {
+      currLineCount = nextLineCount
+      await this.lines.last().scrollIntoViewIfNeeded()
+      await this.page.waitForTimeout(50)
+      nextLineCount = await this.lines.count()
+    } while (nextLineCount !== currLineCount)
+  }
   scrollToText(text: string, placeCursor?: boolean) {
     return this.page.evaluate(
       (args: { text: string; placeCursor?: boolean }) => {