diff options
| author | Jon Dufresne <jon.dufresne@gmail.com> | 2020-04-13 11:49:39 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2020-04-13 11:49:39 -0700 |
| commit | ae173f0f5126dcd8db9f43d6fdb37439873e9233 (patch) | |
| tree | d9a57bdbb7d1d254abd1dc409ea403b6b0dae9fd /redis/utils.py | |
| parent | 1f857f0053606c23cb3f1abd794e3efbf6981e09 (diff) | |
| download | redis-py-ae173f0f5126dcd8db9f43d6fdb37439873e9233.tar.gz | |
Enable BytesWarning during test and fix discovered case (#1322)
The Python command line argument -b causes Python to emit a warning when
bytes and str usage is mixed. This is generally considered bad practice
as either one or the other is required. Enabling this feature during
tests helps catch them before reaching production.
The warning appeared as:
tests/test_scripting.py::TestScripting::test_eval_msgpack_pipeline_error_in_lua
.../redis-py/redis/client.py:3967: BytesWarning: str() on a bytes instance
cmd = ' '.join(imap(safe_unicode, command))
Diffstat (limited to 'redis/utils.py')
0 files changed, 0 insertions, 0 deletions