diff mbox series

[Concept,07/22] buildman: Fix singleton comparisons in test.py

Message ID 20260106142834.2511220-8-sjg@u-boot.org
State New
Headers
Series buildman: Clean up test.py for pylint compliance |

Commit Message

Simon Glass Jan. 6, 2026, 2:28 p.m. UTC 
  From: Simon Glass <simon.glass@canonical.com>

Use 'is not None' instead of '!= None' for proper singleton
comparison.

Co-developed-by: Claude <noreply@anthropic.com>
Signed-off-by: Simon Glass <simon.glass@canonical.com>
---

 tools/buildman/test.py | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
diff mbox series

Patch

diff --git a/tools/buildman/test.py b/tools/buildman/test.py
index 70dcf082755..430e9ee118c 100644
--- a/tools/buildman/test.py
+++ b/tools/buildman/test.py
@@ -616,7 +616,7 @@  class TestBuild(unittest.TestCase):
 
     def test_toolchain_aliases(self):
         """Test that toolchain aliases are handled correctly"""
-        self.assertTrue(self.toolchains.select('arm') != None)
+        self.assertTrue(self.toolchains.select('arm') is not None)
         with self.assertRaises(ValueError):
             self.toolchains.select('no-arch')
         with self.assertRaises(ValueError):
@@ -624,11 +624,11 @@  class TestBuild(unittest.TestCase):
 
         self.toolchains = toolchain.Toolchains()
         self.toolchains.add('x86_64-linux-gcc', test=False)
-        self.assertTrue(self.toolchains.select('x86') != None)
+        self.assertTrue(self.toolchains.select('x86') is not None)
 
         self.toolchains = toolchain.Toolchains()
         self.toolchains.add('i386-linux-gcc', test=False)
-        self.assertTrue(self.toolchains.select('x86') != None)
+        self.assertTrue(self.toolchains.select('x86') is not None)
 
     def test_toolchain_download(self):
         """Test that we can download toolchains"""