From 1d74d3a2181124bf8d148d17529a0896b1e63804 Mon Sep 17 00:00:00 2001 From: Michael Fritch Date: Thu, 28 Jan 2021 11:24:57 -0700 Subject: [PATCH] mgr/cephadm: use ==/!= to compare constant literals (F632) Signed-off-by: Michael Fritch (cherry picked from commit b40d28ce900fc4eb88be2480ae584c095120a51a) --- src/pybind/mgr/cephadm/tests/test_cephadm.py | 4 ++-- src/pybind/mgr/tox.ini | 1 - 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/src/pybind/mgr/cephadm/tests/test_cephadm.py b/src/pybind/mgr/cephadm/tests/test_cephadm.py index 6aa72b2ac67eb..4979045ae7679 100644 --- a/src/pybind/mgr/cephadm/tests/test_cephadm.py +++ b/src/pybind/mgr/cephadm/tests/test_cephadm.py @@ -851,13 +851,13 @@ class TestCephadm(object): with with_host(cephadm_module, 'test', refresh_hosts=False): code, out, err = cephadm_module.check_host('test') # First should succeed. - assert err is '' + assert err == '' # On second it should attempt to reuse the connection, where the # connection is "down" so will recreate the connection. The old # code will blow up here triggering the BOOM! code, out, err = cephadm_module.check_host('test') - assert err is '' + assert err == '' @mock.patch("cephadm.module.CephadmOrchestrator._get_connection") @mock.patch("remoto.process.check") diff --git a/src/pybind/mgr/tox.ini b/src/pybind/mgr/tox.ini index 972f8e82ecbeb..4a277ce266f65 100644 --- a/src/pybind/mgr/tox.ini +++ b/src/pybind/mgr/tox.ini @@ -13,7 +13,6 @@ max-line-length = 100 ignore = E501, F401, - F632, F811, F821, F841, -- 2.39.5