Pull log text if ConsoleLogCard starts expanded. (#355)

jenkinsci · Mar 8, 2024 · 6c0d11f · 6c0d11f
1 parent 2afb476
commit 6c0d11f
Show file tree

Hide file tree

Showing 2 changed files with 26 additions and 0 deletions.
diff --git a/src/main/frontend/pipeline-console-view/pipeline-console/main/ConsoleLogCard.spec.tsx b/src/main/frontend/pipeline-console-view/pipeline-console/main/ConsoleLogCard.spec.tsx
@@ -64,4 +64,20 @@ describe("ConsoleLogCard", () => {
     expect(getByText(/This is a step/));
     expect(findByText(/Hello, world!/));
   });
+
+  it("calls handleMoreConsoleClick on load was card isExpanded set", async () => {
+    console.log = jest.fn();
+    render(<ConsoleLogCard {...DefaultTestProps} isExpanded={true} />);
+    expect(console.log).toHaveBeenCalledWith(
+      "handleMoreConsoleClick triggered"
+    );
+  });
+
+  it("does not call handleMoreConsoleClick on load was card isExpanded set", async () => {
+    console.log = jest.fn();
+    render(<ConsoleLogCard {...DefaultTestProps} />);
+    expect(console.log).not.toHaveBeenCalledWith(
+      "handleMoreConsoleClick triggered"
+    );
+  });
 });
diff --git a/src/main/frontend/pipeline-console-view/pipeline-console/main/ConsoleLogCard.tsx b/src/main/frontend/pipeline-console-view/pipeline-console/main/ConsoleLogCard.tsx
@@ -80,6 +80,16 @@ export class ConsoleLogCard extends React.Component<
     this.props.handleStepToggle(event, this.props.step.id);
   }
 
+  componentDidMount(): void {
+    if (this.props.isExpanded) {
+      // If we start expanded then request logs.
+      this.props.handleMoreConsoleClick(
+        this.props.step.id,
+        this.props.stepBuffer.startByte
+      );
+    }
+  }
+
   getTruncatedLogWarning() {
     if (this.props.stepBuffer.lines && this.props.stepBuffer.startByte > 0) {
       return (