Commit 661ec80
committed
feat(tests): Comprehensive test coverage improvements across 6 MCP servers
Improve test coverage to >90% for pandas, jarvis, parquet, ndp, parallel-sort, and hdf5 MCP servers.
## Coverage Improvements
- **Pandas MCP**: 61% → 92% implementation coverage
- Added 166 new tests across 6 files
- Fixed numpy array handling bug in output_formatter
- Tests: data_cleaning, data_io, filtering, transformations, profiling, statistics
- **Jarvis MCP**: 62% → 99% coverage
- Added 51 tests in test_server_direct.py
- Direct testing of tool wrappers using FastMCP .fn attribute
- **Parquet MCP**: 60% → 94% coverage
- Added 106 tests across 5 files
- Tests: server, aggregate_column, filter_operations, estimate_slice_size, edge_cases
- **NDP MCP**: 72% → 99% coverage
- Added 46 tests across 3 files
- Tests: package init, main entry point, all MCP tool handlers
- **Parallel-sort MCP**: 78% → 93% implementation coverage
- Added 86 tests across 2 new files + expanded 2 existing
- Tests: advanced filtering (12 operators), edge cases, all 13 handlers
- **HDF5 MCP**: 27% → 98% implementation coverage
- Fixed test hanging issue (removed autouse fixture importing server)
- Added 209 tests across 3 comprehensive modules
- Tests: config (71), utils (52), resources (68)
- Coverage: config 100%, resources 96%, utils 98%
## Bug Fixes
- pandas/output_formatter.py: Fixed numpy array handling causing "ambiguous truth value" errors
- hdf5/conftest.py: Removed autouse fixture that caused test hangs due to server module initialization
## Total New Tests: 722
All tests passing with comprehensive coverage of success paths, error paths, and edge cases.1 parent 5126d48 commit 661ec80
File tree
26 files changed
+8521
-15
lines changed- agent-toolkit-mcp-servers
- hdf5
- tests
- jarvis/tests
- ndp/tests
- pandas/tests
- parallel-sort
- tests
- parquet
- tests
26 files changed
+8521
-15
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
29 | 29 | | |
30 | 30 | | |
31 | 31 | | |
| 32 | + | |
32 | 33 | | |
33 | 34 | | |
34 | 35 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
130 | 130 | | |
131 | 131 | | |
132 | 132 | | |
133 | | - | |
134 | | - | |
135 | | - | |
136 | | - | |
137 | | - | |
138 | | - | |
139 | | - | |
140 | | - | |
141 | | - | |
142 | | - | |
143 | | - | |
144 | | - | |
145 | | - | |
146 | | - | |
| 133 | + | |
| 134 | + | |
147 | 135 | | |
148 | 136 | | |
149 | 137 | | |
| |||
0 commit comments