ran tests before PRs to check functionality

[bishoy-at-pieces]

No description provided.

[tsavo-at-pieces]

Approved - will proceed to comprehensive review on main!

[Copilot]

Pull Request Overview

This PR enhances test infrastructure before PRs by adding comprehensive shell completion tests and fixing platform-specific file handling issues. The changes primarily focus on testing completions for multiple shells and improving the robustness of existing tests.

Key Changes:

• Added comprehensive shell completion test suite for Fish, Bash, Zsh, and PowerShell
• Fixed Windows file locking issues by properly closing file handlers in logger tests
• Added GitHub Actions workflow for automated testing across platforms
• Improved MCP gateway test reliability and skipped problematic tests

Reviewed Changes

Copilot reviewed 25 out of 26 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
tests/test_logger.py	Added file handler cleanup to prevent Windows file locking issues
tests/completion_tests/	Complete new test suite for shell completions with platform-specific testers
tests/mcps/mcp_gateway_test.py	Improved connection checking method
tests/mcps/mcp_gateway_e2e_test.py	Simplified JSON formatting and skipped problematic test
.github/workflows/run_tests.yaml	New CI workflow for cross-platform testing
tests/conftest.py	Added global headless mode fixture
pyproject.toml	Added pytest configuration

Comments suppressed due to low confidence (7)

tests/completion_tests/test_powershell.py:253

• The file ends without a newline character, which violates common coding standards. Add a newline at the end of the file.

            pytest.skip("PowerShell completion not working on Windows")

tests/completion_tests/test_fish.py:186

• The file ends without a newline character, which violates common coding standards. Add a newline at the end of the file.

        assert len(completions) > 0, "No completions returned"

tests/completion_tests/test_bash.py:167

• The file ends without a newline character, which violates common coding standards. Add a newline at the end of the file.

        assert len(completions) > 0, "No completions returned on Unix system"

tests/completion_tests/powershell_tester.py:131

• The file ends without a newline character, which violates common coding standards. Add a newline at the end of the file.

                pass

.github/workflows/run_tests.yaml:79

• The file ends without a newline character, which violates common coding standards. Add a newline at the end of the file.

          poetry run pytest tests/ --maxfail=1 --exitfirst -v

[tsavo-at-pieces]

Do we not need windows shell deps?