test(chart-engine): cover deferred template-copy replacement path

[tmustier]

Summary

Add one more module-boundary smoke test for chart-engine builder behavior.

Changes

• tests/test_chart_engine_smoke.py
  • added test_build_chart_defers_chart_template_copy_replacement

This test exercises:

• build_chart(..., save=False, defer_template_copy=True)
• bar spec with chart_template + chart_template_copy: true

And validates that:

• exactly one ChartTemplateReplacement is queued
• queued replacement points at the configured template path
• replacement targets a chart part (ppt/charts/chart...)

This covers the deferred template-copy path at the chart-engine boundary without depending on CLI wiring.

Validation

• .venv/bin/pyright
• .venv/bin/pytest -q
• .venv/bin/pre-commit run --all-files

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: c32d34fd25

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[chatgpt-codex-connector]

Cover defer_template_copy branch by saving presentation

This test sets save=False, but build_chart only uses defer_template_copy inside the if save: block (clean_slides/chart_engine/builder.py), so the result here is identical whether defer_template_copy is True or False. In practice, this means the new test does not validate the deferred replacement behavior it claims to cover and would still pass if the defer logic regressed.

Useful? React with 👍 / 👎.