Fix test isolation by clearing WORKATO_* environment variables

[hovu96]

Summary

• Fixed test isolation issue where tests failed when WORKATO_API_TOKEN, WORKATO_HOST, or WORKATO_PROFILE environment variables were set in the shell
• Updated the isolate_tests fixture in tests/conftest.py to clear these environment variables before running tests
• This ensures tests are properly isolated from the developer's shell environment

Changes

• Added monkeypatch.delenv() calls in the isolate_tests fixture to clear WORKATO_API_TOKEN, WORKATO_HOST, and WORKATO_PROFILE environment variables
• Tests can still explicitly set these variables when needed for specific test scenarios

Test plan

• Verified the specific failing test now passes
• Ran all 130 tests in affected test files with environment variables set - all pass
• Ran full test suite (919 tests) with all three environment variables exported - all pass
• Verified type checker (mypy), linter (ruff), and formatter pass

🤖 Generated with Claude Code

[github-actions]

Coverage Report

File	Stmts	Miss	Cover	Missing
__init__.py	68	4	94%	10–11, 69–70
cli
__init__.py	52	3	94%	49, 51, 60
containers.py	27	0	100%
cli/commands
__init__.py	0	0	100%
api_clients.py	291	9	96%	27, 302, 448–449, 483, 493, 584, 615, 623
api_collections.py	257	3	98%	28, 183, 347
assets.py	47	2	95%	55–56
connections.py	526	5	99%	591, 593, 599, 637, 988
data_tables.py	165	5	96%	31, 253, 267, 321–322
guide.py	166	1	99%	106
init.py	90	0	100%
profiles.py	203	0	100%
properties.py	97	1	98%	21
pull.py	172	2	98%	193–194
workspace.py	39	2	94%	61, 71
cli/commands/connectors
__init__.py	0	0	100%
command.py	90	2	97%	110, 159
connector_manager.py	203	4	98%	176, 292, 300–301
cli/commands/projects
__init__.py	0	0	100%
command.py	272	10	96%	359–362, 373, 439–441, 491, 495
project_manager.py	166	7	95%	48, 66, 263–264, 276, 317, 325
cli/commands/push
__init__.py	0	0	100%
command.py	133	4	96%	109, 112, 230, 308
cli/commands/recipes
__init__.py	0	0	100%
command.py	427	9	97%	117, 133–134, 272–275, 403, 709
validator.py	706	20	97%	174, 883, 1136, 1223, 1246, 1279, 1281–1282, 1359–1361, 1457–1458, 1517–1518, 1707–1708, 1736–1738
cli/utils
__init__.py	3	0	100%
exception_handler.py	304	25	91%	134–135, 140–141, 143–144, 170–171, 177, 220, 267, 294, 321, 378, 413, 472, 474–475, 480–481, 483–487
gitignore.py	14	0	100%
ignore_patterns.py	23	0	100%
spinner.py	43	0	100%
version_checker.py	135	6	95%	24, 26, 33–34, 72, 102
cli/utils/config
__init__.py	5	0	100%
manager.py	467	19	95%	125, 136–138, 141, 154, 204–205, 357, 438–439, 478, 692, 835–836, 850–851, 912, 971
models.py	33	0	100%
profiles.py	309	10	96%	93, 189–190, 193, 228–230, 255–257
workspace.py	68	0	100%
TOTAL	5601	153	97%

[Copilot]

Pull Request Overview

This PR improves test isolation by clearing environment variables that could interfere with test execution. The change ensures that tests are not affected by environment variables set in the developer's shell, making tests more deterministic and reliable.

Key changes:

• Added cleanup of WORKATO_API_TOKEN, WORKATO_HOST, and WORKATO_PROFILE environment variables in the autouse isolate_tests fixture

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.