Skip to content

Commit

Permalink
add test case for coercible boolean and small fix to no compression test
Browse files Browse the repository at this point in the history
  • Loading branch information
davidlm committed Jul 25, 2023
1 parent ccd81fa commit 82e6bed
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 1 deletion.
6 changes: 6 additions & 0 deletions tests/unit/test_args.py
Original file line number Diff line number Diff line change
Expand Up @@ -572,6 +572,12 @@ def test_coercible_value_request_min_compression_size_bytes(self):
config = client_args['client_config']
self.assertEqual(config.request_min_compression_size_bytes, 100)

def test_coercible_value_disable_request_compression(self):
config = Config(disable_request_compression='true')
client_args = self.call_get_client_args(client_config=config)
config = client_args['client_config']
self.assertTrue(config.disable_request_compression)

def test_bad_type_request_min_compression_size_bytes(self):
with self.assertRaises(exceptions.InvalidConfigError):
config = Config(request_min_compression_size_bytes='foo')
Expand Down
2 changes: 1 addition & 1 deletion tests/unit/test_compress.py
Original file line number Diff line number Diff line change
Expand Up @@ -274,7 +274,7 @@ def test_no_compression(config, request_dict, operation_model):
original_body = request_dict['body']
maybe_compress_request(config, request_dict, operation_model)
assert request_dict['body'] is original_body
assert ce_header == request_dict['headers'].get('Content-Encoding')
assert ce_header is request_dict['headers'].get('Content-Encoding')


def test_dict_no_compression():
Expand Down

0 comments on commit 82e6bed

Please sign in to comment.