Skip to content

Commit b00aa0e

Browse files
committed
docs: some review comments
1 parent 9de486d commit b00aa0e

File tree

3 files changed

+12
-11
lines changed

3 files changed

+12
-11
lines changed

examples/kafka/src/kafka_consumer.py

+6-5
Original file line numberDiff line numberDiff line change
@@ -36,12 +36,13 @@ def message_decoder(obj):
3636
raise CustomError('type unknown')
3737

3838

39-
def send_dog_event_foo(foo, payload):
40-
# ignoring foo here
41-
return send_dog_event(payload)
39+
def receive_dog_event_foo(foo, payload):
40+
# ignoring foo here - a more complicated interface. An example of
41+
# not using the pact interface and testing events
42+
return receive_dog_event(payload)
4243

4344

44-
def send_dog_event(payload):
45+
def receive_dog_event(payload):
4546
dog = json.loads(payload, object_hook=message_decoder)
4647
print(dog)
4748
return dog
@@ -56,7 +57,7 @@ def start_consumer(kafka_consumer, topic):
5657
if msg is None:
5758
continue
5859
elif not msg.error():
59-
send_dog_event(msg.value())
60+
receive_dog_event(msg.value())
6061

6162
elif msg.error().code() == KafkaError._PARTITION_EOF:
6263
print('End of partition reached {0}/{1}'

examples/kafka/src/kafka_producer.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ class Dog():
1919
def __init__(self):
2020
i = random.randint(0, len(NAMES)-1)
2121
j = random.randint(0, len(BREEDS)-1)
22-
self.__type__ = 'dog'
22+
self.type = 'dog'
2323
self.name = NAMES[i]
2424
self.breed = BREEDS[j]
2525

examples/kafka/tests/test_consumer.py

+5-5
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
import json
66

77
from pact import MessageConsumer, Provider
8-
from src.kafka_consumer import send_dog_event, send_dog_event_foo, CustomError
8+
from src.kafka_consumer import receive_dog_event, receive_dog_event_foo, CustomError
99

1010
log = logging.getLogger(__name__)
1111
logging.basicConfig(level=logging.INFO)
@@ -45,7 +45,7 @@ def test_assert_verify_message(pact):
4545
.with_metadata({
4646
'Content-Type': 'application/json'
4747
})
48-
.verify(send_dog_event)
48+
.verify(receive_dog_event)
4949
)
5050

5151

@@ -59,7 +59,7 @@ def test_throw_exception_handler(pact):
5959
.with_metadata({
6060
'Content-Type': 'application/json'
6161
})
62-
.verify(send_dog_event))
62+
.verify(receive_dog_event))
6363

6464

6565
def test_assert_dog_returned(pact):
@@ -71,7 +71,7 @@ def test_assert_dog_returned(pact):
7171
.with_metadata({
7272
'Content-Type': 'application/json'
7373
})
74-
.verify(send_dog_event)
74+
.verify(receive_dog_event)
7575
)
7676

7777
with pact:
@@ -92,6 +92,6 @@ def test_assert_calling_dog(pact):
9292
)
9393

9494
with pact:
95-
dog = send_dog_event_foo({}, str(json.dumps(EXPECTED_DOG)))
95+
dog = receive_dog_event_foo({}, str(json.dumps(EXPECTED_DOG)))
9696
assert dog.name == 'spot'
9797
assert dog.breed == 'poodle'

0 commit comments

Comments
 (0)