Skip to content

Commit e41beb1

Browse files
committed
fix: Fix template body parameter
1 parent 9de1ce2 commit e41beb1

2 files changed

Lines changed: 6 additions & 6 deletions

File tree

mailjet_rest/builders.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -244,7 +244,7 @@ def set_content(self, text: str | None = None, html: str | None = None, mjml: st
244244
if html:
245245
self._data["HTMLPart"] = html
246246
if mjml:
247-
self._data["MJMLPart"] = mjml
247+
self._data["MJMLContent"] = mjml
248248
return self
249249

250250
def set_headers(self, headers: dict[str, str]) -> Self:
@@ -268,7 +268,7 @@ def build(self) -> dict[str, Any]:
268268
Raises:
269269
ValueError: If no valid text, html or mjml boundary tokens are passed.
270270
"""
271-
if not any(k in self._data for k in ("TextPart", "HTMLPart", "MJMLPart")):
271+
if not any(k in self._data for k in ("TextPart", "HTMLPart", "MJMLContent")):
272272
msg = "Template validation failed: At least one of text, html, or mjml content is required."
273273
raise ValueError(msg)
274274

tests/unit/test_builders.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ def test_template_content_builder_mapping() -> None:
5555
# Check for correct hyphenated keys required by the Template API
5656
assert payload.get("TextPart") == "Plain text"
5757
assert payload.get("HTMLPart") == "<h1>Hello</h1>"
58-
assert payload.get("MJMLPart") == "<mjml></mjml>"
58+
assert payload.get("MJMLContent") == "<mjml></mjml>"
5959
assert payload.get("Headers") == {"Reply-To": "support@example.com"}
6060

6161

@@ -66,7 +66,7 @@ def test_template_content_builder_partial_data() -> None:
6666

6767
assert "TextPart" in payload
6868
assert "HTMLPart" not in payload
69-
assert "MJMLPart" not in payload
69+
assert "MJMLContent" not in payload
7070

7171

7272
def test_message_builder_validation_fails() -> None:
@@ -168,7 +168,7 @@ def test_template_content_builder_exhaustive() -> None:
168168
res = builder.build()
169169
assert res.get("TextPart") == "Text"
170170
assert res.get("HTMLPart") == "HTML"
171-
assert res.get("MJMLPart") == "MJML"
171+
assert res.get("MJMLContent") == "MJML"
172172
assert res.get("Headers", {}).get("Key") == "Val"
173173

174174
def test_message_builder_attachments_branches() -> None:
@@ -263,4 +263,4 @@ def test_template_content_builder_partial_content() -> None:
263263
assert "HTMLPart" in builder2.build()
264264

265265
builder3 = TemplateContentBuilder().set_content(mjml="mjml")
266-
assert "MJMLPart" in builder3.build()
266+
assert "MJMLContent" in builder3.build()

0 commit comments

Comments
 (0)