Skip to content

Release/v0.4.5#112

Merged
lchoquel merged 4 commits intomainfrom
release/v0.4.5
Jun 23, 2025
Merged

Release/v0.4.5#112
lchoquel merged 4 commits intomainfrom
release/v0.4.5

Conversation

@lchoquel
Copy link
Copy Markdown
Member

Changed

  • Test structure overhaul: Reorganized test directory structure for better organization:
    • Tests now separated into unit/, integration/, and e2e/ directories
    • Created tests/cases/ package for pure test data and constants
    • Created tests/helpers/ package for test utilities
    • Cleaned up test imports and removed empty __init__.py files
  • Class registry refactoring: Updated kajson from 0.1.6 to 0.2.0, adapted to changes in Kajson's class registry with new ClassRegistryUtils (better separation of concerns)
  • Dependency updates:
    • Added pytest-mock to dev dependencies for improved unit testing

Added

  • Coverage commands: New Makefile targets for test coverage analysis:
    • make cov: Run tests with coverage report
    • make cov-missing (or make cm): Show coverage with missing lines
  • Test configuration: Set xfail_strict = true in pytest config for stricter test failure handling
  • Pydantic validation errors: Enhanced error formatting to properly handle model_type errors

Fixed

  • External links: Removed broken Markdown target="_blank" syntax from MANIFESTO.md links
  • Variable naming consistency: Fixed redundant naming in OpenAI config (openai_openai_config → openai_config)
  • Makefile optimization: Removed parallel test execution (-n auto) from codex-tests, works better now

Tests

  • Unit tests added: New comprehensive unit tests for:
    • ClassRegistryUtils
    • FuncRegistry
    • ModuleInspector
    • File finding utilities

lchoquel and others added 3 commits June 20, 2025 13:01
* Fix manifesto links
### 📝 Description

- ClassRegistryUtils implements features removed from dependency `kajson` 

### 🔄 Type of Change
- [ ] 🐛 Bug fix
- [ ] ✨ New feature
- [ ] 💥 Breaking change
- [ ] 📚 Documentation update
- [X] 🧹 Code refactor
- [ ] ⚡ Performance improvement
- [X] ✅ Test update

### 🧪 Tests

- test pyramid: unit/ and integration/ tests, e2e/ empty for now
@lchoquel lchoquel requested a review from thomashebrard June 23, 2025 16:04
@lchoquel lchoquel merged commit f81886a into main Jun 23, 2025
16 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 23, 2025
@lchoquel lchoquel deleted the release/v0.4.5 branch June 23, 2025 21:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants