Fix bug in syntax highlighting caused by incorrect function call (Mar…

…kUsProject#7187)

Changelog.md

@@ -8,6 +8,8 @@
 
 ### 🐛 Bug fixes
 
+- Fix bug in syntax highlighting caused by incorrect function call (#7187)
+
 ### 🔧 Internal changes
 
 - Upgrade Docker environment to use Ruby v3.3 (#7185)

app/assets/javascripts/Components/Result/text_viewer.jsx

@@ -115,7 +115,7 @@ export class TextViewer extends React.PureComponent {
       }
     }
     if (currLine.textContent.length > 0) {
-      nodeLines.appendChild(currLine);
+      nodeLines.push(currLine);
     }
     nodeLines.push(this.raw_content.current.lastChild.cloneNode(true));
     while (this.raw_content.current.firstChild) {

app/assets/javascripts/Components/__tests__/text_viewer.test.jsx

@@ -5,7 +5,9 @@ import {TextViewer} from "../Result/text_viewer";
 describe("TextViewer", () => {
   let props;
 
-  beforeEach(() => {
+  afterEach(cleanup);
+
+  it("should render its text content when the content ends with a new line", () => {
     props = {
       content: "def f(n: int) -> int:\n    return n + 1\n",
       annotations: [],
@@ -14,11 +16,20 @@ describe("TextViewer", () => {
     };
 
     render(<TextViewer {...props} />);
+
+    expect(screen.getByText("def f(n: int) -> int:")).toBeInTheDocument();
   });
 
-  afterEach(cleanup);
+  it("should render its text content when the content doesn't end with a new line", () => {
+    props = {
+      content: "def f(n: int) -> int:\n    return n + 1",
+      annotations: [],
+      focusLine: null,
+      submission_file_id: 1,
+    };
+
+    render(<TextViewer {...props} />);
 
-  it("should render its text content", () => {
     expect(screen.getByText("def f(n: int) -> int:")).toBeInTheDocument();
   });
 });