trivial: assert SendtoCalls consistently

Code Issues Proposed changes

Use same assertion pattern as introduced in the Related-Change and
include the host and port in the assertion.
Change-Id: I7cccf3b8cdfe157c27fb8e0b1f432a649970a865
Related-Change: I9a016de1010dd17e91bc85877413b34d2c3287c7

This commit is contained in:

Alistair Coles

2024年06月25日 16:29:19 +01:00

parent a21d078009

commit a7b69628d5

1 changed files with 2 additions and 2 deletions

Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL

Download Patch File Download Diff File Expand all files Collapse all files

4
test/unit/common/middleware/s3api/test_s3api.py

View File

@@ -238,8 +238,8 @@ class TestS3ApiMiddleware(S3ApiTestCase):

self.assertEqual('s3api.', s3api.logger.logger.statsd_client._prefix)

client = s3api.logger.logger.statsd_client

self.assertEqual({'test-metric': 1}, client.get_increment_counts())

self.assertEqual(1, len(client.sendto_calls))

self.assertEqual(b's3api.test-metric:1|c', client.sendto_calls[0][0])

self.assertEqual([(b's3api.test-metric:1|c', ('1.2.3.4', 8125))],

client.sendto_calls)

def test_non_s3_request_passthrough(self):

req = Request.blank('/something')

trivial: assert SendtoCalls consistently

4 test/unit/common/middleware/s3api/test_s3api.py Unescape Escape View File

4
test/unit/common/middleware/s3api/test_s3api.py

View File