fix: replace hardcoded key with env in tests

[rongquan1]

Summary by CodeRabbit

• Tests
  • Updated test configuration to use environment-based API key management.

[coderabbitai]

Walkthrough

A test file was updated to replace a hard-coded API key with an imported configuration constant, eliminating magic values and updating corresponding test assertions to reference the constant.

Changes

Cohort / File(s)	Change Summary
Test configuration refactoring src/common/utils.test.ts	Imported INFURA_API_KEY from config module and replaced hard-coded API key in generateProvider test with the constant; updated assertion to validate against the imported constant.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

• Simple constant extraction and assertion update in test file
• No logic changes, purely a refactoring to remove magic values

Poem

🐰 A magic key was loose and free,
Now nestled safe in config tree,
The tests align, the truth runs clear,
No secrets hiding far or near! ✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	No pull request description was provided, but the required template sections (Summary, Changes, Issues) are completely missing.	Add a pull request description following the template: include a Summary section explaining the background, a Changes section listing what was modified, and an Issues section referencing related issues.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and specifically describes the main change: replacing hardcoded API keys with environment variables in tests.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/refactor

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 91562e6 and b498001.

📒 Files selected for processing (1)

• src/common/utils.test.ts (2 hunks)

🧰 Additional context used

🧬 Code graph analysis (1)

src/common/utils.test.ts (2)

src/config.ts (1)

• INFURA_API_KEY (1-1)

src/common/utils.ts (1)

• generateProvider (65-103)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)

• GitHub Check: Tests / Run Tests (18.x)
• GitHub Check: Tests / Run Tests (20.x)

🔇 Additional comments (2)

src/common/utils.test.ts (2)

1-1: LGTM! Import correctly added.

The import statement properly brings in the INFURA_API_KEY constant from the config module, eliminating the magic value in the test.

---

281-292: LGTM! Test properly uses the configuration constant.

The test correctly validates that when INFURA_API_KEY is passed as an option, it's properly used by the provider. This change eliminates magic values and aligns the test with the production configuration.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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

[github-actions]

🎉 This PR is included in version 9.6.2 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀