fix: prevent users from passing on_complete in transaction params on app_client calls

[aorumbayev]

Proposed Changes

• PR fix: avoid ignoring on_complete if provided explicitly via transaction parameters #79 re released as a breaking change

[github-actions]

Coverage Report

File	Stmts	Miss	Cover	Missing
src/algokit_utils
_debugging.py	140	7	95%	20, 41, 76, 80, 89, 129, 157
_ensure_funded.py	69	1	99%	99
_transfer.py	67	3	96%	13, 76–77
account.py	85	13	85%	14–17, 61–65, 96, 109, 136, 139, 183
application_client.py	533	77	86%	59–60, 166, 171, 200, 312, 317–318, 320, 322, 787, 802, 820–823, 913, 953, 965, 978, 1020, 1080–1086, 1090–1095, 1097, 1133, 1140, 1253, 1283, 1297, 1342–1344, 1346, 1356–1413, 1424–1429, 1449–1452
application_specification.py	97	11	89%	92, 94, 193–202, 206
asset.py	79	5	94%	9, 27–30
common.py	13	1	92%	13
config.py	51	18	65%	38–39, 50, 55, 60, 64–69, 100–109
deploy.py	459	23	95%	31–34, 169, 173–174, 191, 247, 403, 414–422, 439–442, 452, 460, 653–654, 678
dispenser_api.py	82	12	85%	112–113, 117–120, 155–157, 176–178
logic_error.py	38	2	95%	6, 29
models.py	227	6	97%	45, 50–52, 61–62
network_clients.py	74	3	96%	106–107, 138
src/algokit_utils/beta
account_manager.py	55	14	75%	39–40, 64, 123–130, 183–187, 198–200
algorand_client.py	101	15	85%	111–112, 121–122, 143–145, 154–155, 224, 259, 274, 290, 303, 319
client_manager.py	37	10	73%	40, 61–63, 68–70, 75–78
composer.py	320	71	78%	335–336, 339–340, 343–344, 347–348, 355–356, 359–360, 389, 391, 393, 396, 399, 404, 407, 411, 414, 456–489, 494–505, 510–516, 521–529, 549–562, 566, 590, 593–610, 618, 643, 659–660, 662–663, 665–666, 668–669, 671–672, 678–682
TOTAL	2541	292	89%

Tests	Skipped	Failures	Errors	Time
205	0 💤	0 ❌	0 🔥	1m 14s ⏱️