Improve extend method in LimitedHistory to enforce memory depth #1458
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request fixes #1451 , specifically to the
history.py
andtest_history.py
files. The most important changes include the addition of theextend
method, and the corresponding unit tests for the new method.Changes to
axelrod/history.py
:__init__
method of theLimitedHistory
class to occur after setting thememory_depth
attribute.extend
method to theLimitedHistory
class, which extends theplays
andcoplays
lists while respecting thememory_depth
limit.Changes to
axelrod/tests/unit/test_history.py
:test_extend
to verify the functionality of theextend
method in theLimitedHistory
class.memory-depth
behavior.