refactor(frontend): hide planning preview component when plan content is empty (#12879)

This commit is contained in:
Hiep Le
2026-02-16 18:35:20 +07:00
committed by GitHub
parent 34547ba947
commit 8b7112abe8
2 changed files with 11 additions and 9 deletions

View File

@@ -92,19 +92,21 @@ describe("PlanPreview", () => {
});
it("should render nothing when planContent is null", () => {
renderPlanPreview(<PlanPreview planContent={null} />);
// Arrange & Act
const { container } = renderPlanPreview(<PlanPreview planContent={null} />);
const contentDiv = screen.getByTestId("plan-preview-content");
expect(contentDiv).toBeInTheDocument();
expect(contentDiv.textContent?.trim() || "").toBe("");
// Assert
expect(container.firstChild).toBeNull();
});
it("should render nothing when planContent is undefined", () => {
renderPlanPreview(<PlanPreview planContent={undefined} />);
// Arrange & Act
const { container } = renderPlanPreview(
<PlanPreview planContent={undefined} />,
);
const contentDiv = screen.getByTestId("plan-preview-content");
expect(contentDiv).toBeInTheDocument();
expect(contentDiv.textContent?.trim() || "").toBe("");
// Assert
expect(container.firstChild).toBeNull();
});
it("should render markdown content when planContent is provided", () => {

View File

@@ -65,7 +65,7 @@ export function PlanPreview({
return `${planContent.slice(0, MAX_CONTENT_LENGTH)}...`;
}, [planContent]);
if (!shouldUsePlanningAgent) {
if (!shouldUsePlanningAgent || !planContent) {
return null;
}