From ac3a51d013dd2e4b7dcaa8a51493cbd852dcec3c Mon Sep 17 00:00:00 2001 From: Bruno Cadonna Date: Mon, 27 Jul 2020 18:38:14 +0200 Subject: [PATCH] MINOR: Remove staticmethod tag to be able to use logger of instance (#9086) A system test failed with the following error: global name 'self' is not defined The reason was that `self` was accessed to log a message in a static method. This commit makes the method an instance method. Reviewer: Matthias J. Sax --- tests/kafkatest/tests/streams/base_streams_test.py | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/tests/kafkatest/tests/streams/base_streams_test.py b/tests/kafkatest/tests/streams/base_streams_test.py index 256693ca44c..2cfe27bd738 100644 --- a/tests/kafkatest/tests/streams/base_streams_test.py +++ b/tests/kafkatest/tests/streams/base_streams_test.py @@ -95,8 +95,7 @@ class BaseStreamsTest(KafkaTest): timeout_sec=60, err_msg="Did expect to read '%s' from %s" % (message, processor.node.account)) - @staticmethod - def verify_from_file(processor, message, file): + def verify_from_file(self, processor, message, file): result = processor.node.account.ssh_output("grep -E '%s' %s | wc -l" % (message, file), allow_fail=False) try: return int(result)