Skip to content

Commit 9f18350

Browse files
committed
Remove doctest from pre-commit, too slow
Signed-off-by: Mihai Criveti <[email protected]>
1 parent 1155c8c commit 9f18350

File tree

2 files changed

+101
-10
lines changed

2 files changed

+101
-10
lines changed

.pre-commit-config.yaml

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -515,13 +515,13 @@ repos:
515515
# -----------------------------------------------------------------------------
516516
# 🧪 DOCTEST VALIDATION
517517
# -----------------------------------------------------------------------------
518-
- repo: local
519-
hooks:
520-
- id: doctest
521-
name: 🧪 Doctest - Validate Documentation Examples
522-
description: Runs doctest on all Python modules to ensure documentation examples work.
523-
entry: python3 -m pytest --doctest-modules mcpgateway/ --tb=short
524-
language: system
525-
pass_filenames: false
526-
always_run: true
527-
types: [python]
518+
# - repo: local
519+
# hooks:
520+
# - id: doctest
521+
# name: 🧪 Doctest - Validate Documentation Examples
522+
# description: Runs doctest on all Python modules to ensure documentation examples work.
523+
# entry: python3 -m pytest --doctest-modules mcpgateway/ --tb=short
524+
# language: system
525+
# pass_filenames: false
526+
# always_run: true
527+
# types: [python]

docs/docs/testing/acceptance.md

Lines changed: 91 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -391,3 +391,94 @@ MCPGATEWAY_ADMIN_API_ENABLED=true
391391
- **MCP Servers**: Start each MCP server on its designated port before registering as gateway
392392
- **VS Code**: Reload window after changing MCP configuration in settings.json
393393
- **AI Agents**: Ensure you have API keys configured for LLM providers when testing agents
394+
395+
396+
## MCP Gateway Test Report Template
397+
398+
**Test Environment**: MCP Gateway v___
399+
**Test Date**: ___________
400+
**Tester**: ___________
401+
**Gateway URL**: ___________
402+
403+
## Status Legend
404+
405+
-**Passed** - All tests in section completed successfully
406+
- ⚠️ **Partial** - Some tests passed, some failed or incomplete
407+
-**Failed** - Critical failures preventing section completion
408+
- 🚧 **In Progress** - Testing currently underway
409+
- 🔒 **Blocked** - Cannot proceed due to dependencies or external factors
410+
411+
### Executive Summary
412+
413+
Overall test completion: **___%** | Total Tests: **___** | Passed: **___** | Failed: **___** | Blocked: **___**
414+
415+
### Test Results by Section
416+
417+
| Section | Test Area | Status | Tests | Passed | Failed | Blocked | Comments |
418+
|---------|-----------|--------|-------|--------|--------|---------|----------|
419+
| **Setup** | Test Environment Setup | | | | | | |
420+
| **1** | MCP Servers Setup | | | | | | |
421+
| **2** | Core Protocol Operations | | | | | | |
422+
| **3** | Gateway Federation | | | | | | |
423+
| **4** | Tool Discovery & Execution | | | | | | |
424+
| **5** | Virtual Server Management | | | | | | |
425+
| **6** | Resource Management | | | | | | |
426+
| **7** | Prompt Management | | | | | | |
427+
| **8** | REST Tool Creation | | | | | | |
428+
| **9** | MCP Wrapper Testing | | | | | | |
429+
| **10.1** | VS Code Integration (Wrapper) | | | | | | |
430+
| **10.2** | VS Code Integration (Direct) | | | | | | |
431+
| **11.1** | LangChain Integration | | | | | | |
432+
| **11.2** | CrewAI Integration | | | | | | |
433+
| **12** | Open Source MCP Servers | | | | | | |
434+
| **13** | Health & Monitoring | | | | | | |
435+
| **14** | Admin Interface | | | | | | |
436+
| **15** | Input Validation | | | | | | |
437+
| **16** | Error Handling | | | | | | |
438+
| **17** | System Summary | | | | | | |
439+
| **Cleanup** | Test Cleanup | | | | | | |
440+
441+
### Issue Summary
442+
443+
#### 🔴 Critical Issues
444+
1.
445+
2.
446+
3.
447+
448+
#### 🟡 Major Issues
449+
1.
450+
2.
451+
3.
452+
453+
#### 🟢 Minor Issues
454+
1.
455+
2.
456+
3.
457+
458+
### Recommendations
459+
460+
1. **Immediate Actions**:
461+
-
462+
-
463+
-
464+
465+
2. **Short-term**:
466+
-
467+
-
468+
-
469+
470+
3. **Long-term**:
471+
-
472+
-
473+
-
474+
475+
### Test Environment Notes
476+
477+
- **Performance**:
478+
- **Stability**:
479+
- **Federation**:
480+
- **Compatibility**:
481+
482+
---
483+
484+
**Next Test Cycle**:

0 commit comments

Comments
 (0)