fix: always disconnect the session after a test run

[nirinchev]

Proposed changes

Checklist

• I have signed the MongoDB CLA

[Copilot]

Pull Request Overview

This PR modifies the test cleanup logic to always disconnect the session after each test run, removing a conditional check that previously only disconnected when no Atlas cluster was connected.

• Simplified the session disconnect logic in the afterEach hook to always disconnect when an MCP server exists
• Removed the conditional check for connectedAtlasCluster status before disconnection

[Copilot]

Consider adding error handling around the disconnect operation to prevent test failures if disconnection throws an exception. The current implementation could cause afterEach to fail and potentially leave resources in an inconsistent state.

Suggested change

	await mcpServer.session.disconnect();
	try {
	await mcpServer.session.disconnect();
	} catch (error) {
	console.error("Error during mcpServer session disconnection:", error);
	}

[coveralls]

Pull Request Test Coverage Report for Build 16219386259

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

• For more information on this, see Tracking coverage changes with pull request builds.
• To avoid this issue with future PRs, see these Recommended CI Configurations.
• For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

• 0 of 0 changed or added relevant lines in 0 files are covered.
• 2 unchanged lines in 1 file lost coverage.
• Overall coverage decreased (-0.06%) to 75.27%

---

Files with Coverage Reduction	New Missed Lines	%
src/session.ts	2	77.5%

Totals
Change from base Build 16217682500:	-0.06%
Covered Lines:	886
Relevant Lines:	1087

---

💛 - Coveralls