Use API key from environment if available

[jo-sm]

This PR passes the API key via context, and gets it from the environment if available, rather than hardcoding a test API key.

Summary by CodeRabbit

Release Notes

• Tests
  • Updated integration test infrastructure for improved API key configuration handling in test scenarios.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: cde3f338-c8cb-4f49-bd18-ff20ac5e59cf

📥 Commits

Reviewing files that changed from the base of the PR and between 6636aa9 and 46c8981.

📒 Files selected for processing (2)

• tests/integration/features/environment.py
• tests/integration/features/steps/cli_steps.py

---

📝 Walkthrough

Walkthrough

API key configuration for integration tests has been refactored to source from the TOWER_API_KEY environment variable with a fallback default value, replacing hardcoded values in test steps. The setup is initialized in the scenario hook and consumed during CLI command execution.

Changes

Cohort / File(s)	Summary
Test Configuration Refactoring tests/integration/features/environment.py, tests/integration/features/steps/cli_steps.py	API key configuration moved from hardcoded values to environment-based sourcing with fallback. The before_scenario hook now initializes context.api_key from TOWER_API_KEY env variable (defaulting to "sk-test-api-key"), and CLI steps now reference this context attribute instead of hardcoded values.

Poem

🐰 A key from the air, no longer a cage,
From environment whispers, a flexible page,
The test setup dances with configurable grace,
No hardcoded secrets, just context in place! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately reflects the main change: using API key from environment instead of hardcoded values in tests.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch use-api-key-from-env

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}