fix: fix the case that include_failed cannot be set to False.

Author: overcat

stellar_sdk/call_builder/base_call_builder.py

@@ -126,6 +126,8 @@ def _add_query_param(self, key: str, value: Union[str, float, int, bool, None]):
             pass
         elif value is True:
             self.params[key] = "true"
+        elif value is False:
+            self.params[key] = "false"
         else:
             self.params[key] = str(value)
 

tests/call_builder/test_operations_call_builder.py

@@ -53,3 +53,15 @@ def test_include_failed(self):
             account_id=account_id
         )
         assert builder.params == {"include_failed": "true"}
+
+    def test_not_include_failed(self):
+        account_id = "GATEMHCCKCY67ZUCKTROYN24ZYT5GK4EQZ65JJLDHKHRUZI3EUEKMTCH"
+        builder = (
+            OperationsCallBuilder(horizon_url, client)
+            .for_account(account_id)
+            .include_failed(False)
+        )
+        assert builder.endpoint == "accounts/{account_id}/operations".format(
+            account_id=account_id
+        )
+        assert builder.params == {"include_failed": "false"}

tests/call_builder/test_payments_call_builder.py

@@ -46,3 +46,15 @@ def test_include_failed(self):
             account_id=account_id
         )
         assert builder.params == {"include_failed": "true"}
+
+    def test_not_include_failed(self):
+        account_id = "GATEMHCCKCY67ZUCKTROYN24ZYT5GK4EQZ65JJLDHKHRUZI3EUEKMTCH"
+        builder = (
+            PaymentsCallBuilder(horizon_url, client)
+            .for_account(account_id)
+            .include_failed(False)
+        )
+        assert builder.endpoint == "accounts/{account_id}/payments".format(
+            account_id=account_id
+        )
+        assert builder.params == {"include_failed": "false"}

tests/call_builder/test_transactions_call_builder.py

@@ -45,3 +45,15 @@ def test_include_failed(self):
             account_id=account_id
         )
         assert builder.params == {"include_failed": "true"}
+
+    def test_not_include_failed(self):
+        account_id = "GATEMHCCKCY67ZUCKTROYN24ZYT5GK4EQZ65JJLDHKHRUZI3EUEKMTCH"
+        builder = (
+            TransactionsCallBuilder(horizon_url, client)
+            .for_account(account_id)
+            .include_failed(False)
+        )
+        assert builder.endpoint == "accounts/{account_id}/transactions".format(
+            account_id=account_id
+        )
+        assert builder.params == {"include_failed": "false"}