🐛 model dump시 json mode 적용

[mingi3314]

---

Generated summary (powered by Graphite)

TL;DR

This pull request modifies the to_toml method in the Account class of the account.py file. The method now uses the model_dump function with the "json" mode.

What changed

The to_toml method in the Account class was previously using the model_dump function without any arguments and then updating the model_dict with the brokerage value. This has been changed to use the model_dump function with the "json" mode directly.

-        model_dict = self.model_dump()
-        model_dict.update({"brokerage": self.brokerage.value})
+        model_dict = self.model_dump(mode="json")

How to test

To test this change, you can create an instance of the Account class and call the to_toml method. Ensure that the output is as expected and that the brokerage value is included in the output.

Why make this change

This change simplifies the to_toml method by removing an unnecessary update to the model_dict. By using the "json" mode in the model_dump function, we can ensure that all necessary values, including brokerage, are included in the output. This makes the code cleaner and more efficient.

[mingi3314]

Current dependencies on/for this PR:

• 🔖 release version 0.9.1 #91
• ✅ 계좌 생성, 등록 api 테스트코드 추가 #90
• ♻️ 공통 fixture 분리 #89
• ➕ add httpx dev dependency #88
• 🐛 model dump시 json mode 적용 #87 👈
• ♻️ refactor test structure #86
• ✨ 계좌 등록 API 추가 #85
• main

This stack of pull requests is managed by Graphite.

[mingi3314]

Merge activity

• Feb 24, 9:32 AM EST: @mingi3314 started a stack merge that includes this pull request via Graphite.
• Feb 24, 9:35 AM EST: Graphite rebased this pull request as part of a merge.
• Feb 24, 9:36 AM EST: @mingi3314 merged this pull request with Graphite.