Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork770
Commitdaa733a
committed
Complete test migration and infrastructure improvements
## Test Migration: Bash to Vader Format### Enhanced Vader Test Suites- **lint.vader**: Added comprehensive test scenario from pymodelint.vim that loads from_autopep8.py sample file and verifies PymodeLint detects >5 errors- **commands.vader**: Added test scenario from pymoderun.vim that loads pymoderun_sample.py and verifies PymodeRun produces expected output### Removed Migrated Bash Tests- Deleted test_bash/test_autocommands.sh (migrated to Vader commands.vader)- Deleted test_bash/test_pymodelint.sh (migrated to Vader lint.vader)- Deleted test_procedures_vimscript/pymodelint.vim (replaced by Vader test)- Deleted test_procedures_vimscript/pymoderun.vim (replaced by Vader test)- Updated tests/test.sh to remove references to deleted bash tests## Code Coverage Infrastructure### Coverage Tool Integration- Added coverage.py package installation to Dockerfile- Implemented coverage.xml generation in tests/test.sh for CI/CD integration- Coverage.xml is automatically created in project root for codecov upload- Updated .gitignore to exclude coverage-related files (.coverage, coverage.xml, etc.)## Documentation Cleanup### Removed Deprecated Files- Deleted old_reports/ directory (Phase 1-5 migration reports)- Removed PHASE4_FINAL_SUCCESS.md (consolidated into main documentation)- Removed PHASE4_COMPLETION_REPORT.md (outdated migration report)- Removed CI_TEST_FIXES_REPORT.md (fixes already implemented)- Removed DOCKER_TEST_IMPROVEMENT_PLAN.md (plan completed)- Removed scripts/test-ci-fixes.sh (temporary testing script)## Previous Fixes (from HEAD commit)### Configuration Syntax Errors ✅ FIXED- Problem: tests/utils/pymoderc had invalid Vimscript dictionary syntax causing parsing errors- Solution: Reverted from pymode#Option() calls back to direct let statements- Impact: Resolved E15: Invalid expression and E10: \ should be followed by /, ? or & errors### Inconsistent Test Configurations ✅ FIXED- Problem: Vader tests were using dynamically generated minimal vimrc instead of main configuration files- Solution: Modified scripts/user/run-vader-tests.sh to use /root/.vimrc (which sources /root/.pymoderc)- Impact: Ensures consistent configuration between legacy and Vader tests### Missing Vader Runtime Path ✅ FIXED- Problem: Main tests/utils/vimrc didn't include Vader in the runtime path- Solution: Added set rtp+=/root/.vim/pack/vader/start/vader.vim to tests/utils/vimrc- Impact: Allows Vader tests to run properly within unified configuration### Python-mode ftplugin Not Loading ✅ FIXED- Problem: PymodeLintAuto command wasn't available because ftplugin wasn't being loaded for test buffers- Solution: Modified tests/vader/setup.vim to explicitly load ftplugin with runtime! ftplugin/python/pymode.vim- Impact: Ensures all python-mode commands are available during Vader tests### Rope Configuration for Testing ✅ FIXED- Problem: Rope regeneration on write could interfere with tests- Solution: Disabled g:pymode_rope_regenerate_on_write in test configuration- Impact: Prevents automatic rope operations that could cause test instability## SummaryThis commit completes the migration from bash-based tests to Vader test framework,implements code coverage infrastructure for CI/CD, and cleans up deprecateddocumentation. All changes maintain backward compatibility with existing testinfrastructure while improving maintainability and CI integration.The Docker test setup now has unified configuration ensuring that all Vader testswork correctly with proper Python path, submodule loading, and coverage reporting.1 parent0ae82d6 commitdaa733a
File tree
24 files changed
+989
-1203
lines changed- autoload/pymode
- pymode
- scripts
- cicd
- user
- test-logs
- tests
- test_bash
- test_procedures_vimscript
- utils
- vader
24 files changed
+989
-1203
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
17 | 17 | | |
18 | 18 | | |
19 | 19 | | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
0 commit comments
Comments
(0)