Skip to content

Commit 9fe9746

Browse files
committed
Remove old injection examples, update readme
1 parent 5777369 commit 9fe9746

9 files changed

Lines changed: 5 additions & 300 deletions

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ __Design for Testability Patterns__:
5757

5858
| Pattern | Description |
5959
|:-------:| ----------- |
60-
| [setter_injection](patterns/dft/setter_injection.py) | the client provides the depended-on object to the SUT via the setter injection (implementation variant of dependency injection) |
60+
| [dependency_injection](patterns/dependency_injection.py) | 3 variants of dependency injection |
6161

6262
__Fundamental Patterns__:
6363

patterns/dependency_injection.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -90,8 +90,10 @@ def main():
9090
9191
>>> time_with_si = SetterInjection()
9292
93-
# >>> time_with_si.get_current_time_as_html_fragment()
94-
# AttributeError: 'SetterInjection' object has no attribute 'time_provider'
93+
>>> time_with_si.get_current_time_as_html_fragment()
94+
Traceback (most recent call last):
95+
...
96+
AttributeError: 'SetterInjection' object has no attribute 'time_provider'
9597
9698
>>> time_with_si.set_time_provider(midnight_time_provider)
9799
>>> time_with_si.get_current_time_as_html_fragment()

patterns/dft/__init__.py

Whitespace-only changes.

patterns/dft/constructor_injection.py

Lines changed: 0 additions & 53 deletions
This file was deleted.

patterns/dft/parameter_injection.py

Lines changed: 0 additions & 54 deletions
This file was deleted.

patterns/dft/setter_injection.py

Lines changed: 0 additions & 57 deletions
This file was deleted.

tests/dft/test_constructor_injection.py

Lines changed: 0 additions & 41 deletions
This file was deleted.

tests/dft/test_parameter_injection.py

Lines changed: 0 additions & 46 deletions
This file was deleted.

tests/dft/test_setter_injection.py

Lines changed: 0 additions & 46 deletions
This file was deleted.

0 commit comments

Comments
 (0)