Raymond Hettinger added the comment:
The test code now includes this:
def test_shuffle_random_argument(self):
# Test random argument to shuffle.
shuffle = self.gen.shuffle
mock_random = unittest.mock.Mock(return_value=0.5)
seq = bytearray(b'abcdefghijk')
Charles Nodell added the comment:
Sounds fine! I look forward to seeing how to do this properly!
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24533
___
New submission from Charles Nodell:
This patch just extends test coverage on the random library by a few lines.
--
components: Tests
files: mywork.patch
keywords: patch
messages: 245956
nosy: Charles Nodell
priority: normal
severity: normal
status: open
title: Increased Test Coverage
Changes by Serhiy Storchaka storch...@gmail.com:
--
nosy: +mark.dickinson, rhettinger
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24533
___
___
Raymond Hettinger added the comment:
I don't mind adding a test but would like it to written somewhat differently.
I'll work something up when I get a chance. Marking this as a low priority
because the code is already working fine and this is just a nice addition to
improve coverage.
