Skip to content

Commit 3c8f54d

Browse files
committed
TST: fix mixed up assertion calls.
Use assertEqual instead of assertTrue.
1 parent 174a8c1 commit 3c8f54d

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

src/pyobjcryst/tests/testmolecule.py

+8-8
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,7 @@ def testAtoms(self):
8383
This tests GetAtom.
8484
8585
"""
86-
self.assertTrue(60, self.m.GetNbAtoms())
86+
self.assertEqual(60, self.m.GetNbAtoms())
8787
for i in range(60):
8888
a1 = self.m.GetAtom(i)
8989
self.assertEqual(a1.GetName(), "C%i"%i)
@@ -92,11 +92,11 @@ def testAtoms(self):
9292
x = a.X
9393
sp = a.GetScatteringPower()
9494

95-
self.assertTrue(60, self.m.GetNbAtoms())
95+
self.assertEqual(60, self.m.GetNbAtoms())
9696

9797
self.m.RemoveAtom(a)
9898

99-
self.assertTrue(59, self.m.GetNbAtoms())
99+
self.assertEqual(59, self.m.GetNbAtoms())
100100

101101
# Make sure the atom is still valid. We don't want RemoveAtom deleting
102102
# the memory for an object we still have access to.
@@ -431,9 +431,9 @@ def testAccessors(self):
431431
a = self.a
432432

433433
# Test name Get/Set
434-
self.assertTrue(a.GetName(), "C0")
434+
self.assertEqual("C0", a.GetName())
435435
a.SetName("test")
436-
self.assertTrue(a.GetName(), "test")
436+
self.assertEqual("test", a.GetName())
437437

438438
# Test xyz & occ Get/Set
439439
self.assertAlmostEquals(3.451266498, a.X, numplaces)
@@ -508,7 +508,7 @@ def testAccessors(self):
508508
a2 = self.a2
509509

510510
# Check the name
511-
self.assertTrue("C0-C1", b.GetName())
511+
self.assertEqual("C0-C1", b.GetName())
512512

513513
# Get the atoms
514514
at1 = b.GetAtom1()
@@ -571,7 +571,7 @@ def testAccessors(self):
571571
a3 = self.a3
572572

573573
# Check the name
574-
self.assertTrue("C0-C1-C2", ba.GetName())
574+
self.assertEqual("C0-C1-C2", ba.GetName())
575575

576576
# Get the atoms
577577
at1 = ba.GetAtom1()
@@ -637,7 +637,7 @@ def testAccessors(self):
637637
a4 = self.a4
638638

639639
# Check the name
640-
self.assertTrue("C0-C1-C2-C3", da.GetName())
640+
self.assertEqual("C0-C1-C2-C3", da.GetName())
641641

642642
# Get the atoms
643643
at1 = da.GetAtom1()

0 commit comments

Comments
 (0)