We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
proto_var
These test scripts all contain a finally: block that attempts to print proto_ver which does not exist:
finally:
proto_ver
test/broker/02-subpub-qos1-message-expiry.py
test/broker/02-subpub-qos1-message-expiry-retain.py
test/broker/02-subpub-qos1-message-expiry-will.py
test/broker/02-subpub-qos2-pubrec-error.py
print("proto_ver=%d" % (proto_ver))
This mistake appears to be a copy-and-paste error in commit b2a9daf ("02 broker subpub tests with v5 support").
As a fix, I suggest:
s/proto_ver=5/proto_ver=proto_ver/g
s/do_test():/do_test(proto_ver):/g
s/do_test()/do_test(proto_ver=5)/g
I'd submit a patch, but the CLA process is too much for what is a single sed invocation.
sed
sed -i \ -e 's/proto_ver=5/proto_ver=proto_ver/g' \ -e 's/do_test():/do_test(proto_ver):/g' \ -e 's/do_test()/do_test(proto_ver=5)/g' \ test/broker/02-subpub-qos1-message-expiry.py \ test/broker/02-subpub-qos1-message-expiry-retain.py \ test/broker/02-subpub-qos1-message-expiry-will.py \ test/broker/02-subpub-qos2-pubrec-error.py
The text was updated successfully, but these errors were encountered:
02-subpub-qos1-message-expiry.py
No branches or pull requests
These test scripts all contain a
finally:
block that attempts to printproto_ver
which does not exist:test/broker/02-subpub-qos1-message-expiry.py
test/broker/02-subpub-qos1-message-expiry-retain.py
test/broker/02-subpub-qos1-message-expiry-will.py
test/broker/02-subpub-qos2-pubrec-error.py
This mistake appears to be a copy-and-paste error in commit b2a9daf ("02 broker subpub tests with v5 support").
As a fix, I suggest:
s/proto_ver=5/proto_ver=proto_ver/g
s/do_test():/do_test(proto_ver):/g
s/do_test()/do_test(proto_ver=5)/g
I'd submit a patch, but the CLA process is too much for what is a single
sed
invocation.The text was updated successfully, but these errors were encountered: