Remove redundant t.Chdir()

[amacneil]

Remove redundant t.Chdir(cwd) call and add comments to clarify directory cleanup in TestLoadEnvFiles.

Identified by #709 (comment) although that bugbot comment was incorrect.

The t.Chdir function automatically registers a cleanup to restore the original directory, making the explicit t.Chdir(cwd) call in the t.Cleanup block redundant. This change simplifies the test setup and clarifies that t.Chdir handles its own directory restoration.

---

 

---

Note

Simplifies TestLoadEnvFiles setup by relying on t.Chdir's built-in directory restoration and tightening environment handling.

• Removes manual cwd tracking and redundant t.Chdir(cwd) from t.Cleanup
• Keeps os.Clearenv() after t.Chdir (since it sets PWD) and restores env vars in t.Cleanup
• Adds clarifying comments about directory and environment cleanup

^{Written by Cursor Bugbot for commit 159749d. This will update automatically on new commits. Configure here.}

[cursor]

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
_{Learn more about Cursor Agents}