Skip to content

Cleanup#45

Merged
tmeiczin merged 5 commits into
mainfrom
cleanup
Feb 27, 2026
Merged

Cleanup#45
tmeiczin merged 5 commits into
mainfrom
cleanup

Conversation

@tmeiczin

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR performs a comprehensive cleanup of the Reliqua REST API framework, adding extensive test coverage, fixing multiple bugs, improving documentation, and enhancing code quality.

Changes:

  • Adds comprehensive test suites for 8 modules covering status codes, OpenAPI generation, authentication, middleware, parsers, and application utilities
  • Fixes critical bugs in type checking, parameter validation, authentication logic, and binary request handling
  • Updates example code to use proper context managers and correct configuration
  • Significantly expands README.md with complete API documentation, usage examples, and feature descriptions

Reviewed changes

Copilot reviewed 18 out of 19 changed files in this pull request and generated no comments.

Show a summary per file
File Description
tests/test_status_codes.py Tests for HTTP status code dictionary and helper functions
tests/test_sphinx_parser.py Tests for docstring parsing, parameter extraction, and OpenAPI metadata generation
tests/test_openapi.py Tests for OpenAPI schema generation including parameters, operations, and content types
tests/test_middleware.py Tests for parameter conversion, validation, and type coercion middleware
tests/test_media_handlers.py Tests for JSON, YAML, and text serialization/deserialization handlers
tests/test_database.py Tests for base64 validation helper used in database operations
tests/test_auth.py Tests for authentication and authorization mechanisms including Basic auth and access control
tests/test_app.py Tests for application configuration loading and dictionary merging utilities
src/reliqua/status_codes.py Fixes MESSAGES dict creation from set comprehension to dict comprehension
src/reliqua/openapi.py Fixes binary body handling and request body type checking for arrays
src/reliqua/middleware.py Fixes python_type to use builtins module, adds return statement to as_array, improves required parameter checking
src/reliqua/database.py Fixes is_b64 to handle string encoding before base64 validation
src/reliqua/auth.py Fixes AccessList logic, adds null checks in AccessMap, improves case-insensitive method matching in AccessResource
example/resources/media.py Adds context manager for file handling to prevent resource leaks
example/main.py Fixes IP address typo, removes incorrect action parameter, renames gunicorn_options to gunicorn
pyproject.toml Adds pylint and ruff configuration to skip docstring checks in tests
README.md Complete rewrite with comprehensive documentation of all features and usage examples
.cspell/dictionary.txt Adds new words for spell checking

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@tmeiczin
tmeiczin merged commit b3fe69d into main Feb 27, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants