Fix bug in syntax highlighting caused by incorrect function call

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();
  });
 });