Fix default loading of identity

Author: tupui

src/soroban/invoke.py

@@ -34,11 +34,11 @@ def invoke(
     )
 
     soroban_server = stellar_sdk.SorobanServer(network.rpc_url)
-    source_account = soroban_server.load_account(identity.public_key)
+    source_account_ = soroban_server.load_account(identity.public_key)
 
     tx = (
         stellar_sdk.TransactionBuilder(
-            source_account, network.network_passphrase, base_fee=network.base_fee
+            source_account_, network.network_passphrase, base_fee=network.base_fee
         )
         .add_time_bounds(0, 0)
         .append_invoke_contract_function_op(

src/soroban/models.py

@@ -36,7 +36,10 @@ class Identity(BaseSettings):
     @model_validator(mode="after")
     def load_keys(self) -> "Identity":
         if self.keypair is None and self.secret_key is None:
-            raise ValueError("Either provide a secret key or a Keypair object.")
+            raise ValueError(
+                "Either provide a secret key or a Keypair object. Also look"
+                "in 'identity.toml'"
+            )
         if self.keypair is not None:
             self.secret_key = self.keypair.secret
         else:
@@ -50,7 +53,9 @@ def from_source_account(
     ) -> "Identity":
         if account is None:
             identity = Identity()
-        if isinstance(account, str) and account.startswith("S") and len(account) == 56:
+        elif (
+            isinstance(account, str) and account.startswith("S") and len(account) == 56
+        ):
             identity = Identity(secret_key=account)
         elif isinstance(account, (str, pathlib.Path)):
             fname = _load_configuration(account, "identity")

tests/identity.toml

@@ -0,0 +1 @@
+secret_key = "SDTZV5HD6WBU2LZKVUGFCUUR3JTOJHDHEXO2TSYVITMSH6RCY433HEYB"

tests/test_models.py

@@ -1,4 +1,5 @@
 import json
+import os
 import pathlib
 
 import pytest
@@ -17,6 +18,9 @@ def test_from_pk(self):
         soroban.Identity(keypair=keypair)
 
     def test_from_source_account(self):
+        # identity.toml exists
+        os.chdir(pathlib.Path(__file__).parent)
+        soroban.Identity.from_source_account()
         alice_fname = pathlib.Path(__file__).parent / "alice.toml"
         soroban.Identity.from_source_account(account=alice_fname)
 
@@ -26,6 +30,7 @@ def test_from_source_account(self):
         soroban.Identity.from_source_account(account=keypair.secret)
 
     def test_raises(self):
+        os.chdir(pathlib.Path(__file__).parent.parent)
         with pytest.raises(ValueError, match="provide a secret key or a Keypair"):
             soroban.Identity()