Docs: Add ecosystem examples to CoR, add Factory tests #466

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Open

SanjanaG-01 wants to merge 1 commit into faif:master from SanjanaG-01:master

+35 −1

SanjanaG-01 commented Feb 9, 2026

Description

This PR improves documentation, adds test coverage, and fixes a bug in the Creational patterns.

Changes

Chain of Responsibility: Added an "Examples in Python ecosystem" section (referencing Django Middleware) to patterns/behavioral/chain_of_responsibility.py to align with contribution guidelines.
Factory Pattern:
- Created tests/creational/test_factory.py with unit tests for the Factory pattern (previously untested).
- Fixed an IndentationError in patterns/creational/factory.py that was preventing execution.

Motivation

To improve documentation quality and compliance.
To increase test coverage and ensure the reliability of the Factory pattern implementation.
To fix a syntax error in the existing codebase that was blocking usage.

Verification

Ran python -m unittest tests/creational/test_factory.py and confirmed all tests passed.
Verified docstring rendering locally.


          Docs: Add ecosystem examples to CoR, add Factory tests

fae387a

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet