Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ Versioning].
- Improve performance by reducing the AES plaintext size. Note: this changes the
ordering of integers for existing seeds, which may be a breaking change depending
on the use case.
- Add support for negative indexes.

## [1.1.0] - 2022-12-29

Expand Down
2 changes: 2 additions & 0 deletions src/shuffled/core.py
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,8 @@ def __getitem__(self, index: slice) -> Sequence[int]: ...
def __getitem__(self, index: Union[int, slice]) -> Union[int, Sequence[int]]:
if isinstance(index, slice):
raise NotImplementedError
if index < 0:
index += self._size
if index < 0 or index >= self._size:
raise IndexError
return self._encryptor.encrypt(index)
Expand Down
12 changes: 12 additions & 0 deletions tests/test_shuffled.py
Original file line number Diff line number Diff line change
Expand Up @@ -31,3 +31,15 @@ def test_normal(self, range_size):
)
def test_seed(self, seed, expected):
assert list(Shuffled(20, seed)) == expected

def test_negative_index(self):
s = Shuffled(10, seed=b"\x00")

assert s[-1] == s[9]
assert s[-10] == s[0]

def test_negative_index_out_of_bounds(self):
s = Shuffled(10, seed=b"\x00")

with pytest.raises(IndexError):
s[-11]