Title: Refactoring: replacing some assertTrue by assertIn
Type: enhancement Stage: resolved
Components: Tests Versions:
Status: closed Resolution: duplicate
Dependencies: Superseder: Using appropriate checks in tests
View: 16510
Assigned To: Nosy List: sblondon, serhiy.storchaka
Priority: normal Keywords:

Created on 2018-03-29 20:07 by sblondon, last changed 2018-03-29 20:29 by serhiy.storchaka. This issue is now closed.

Messages (2)
msg314670 - (view) Author: St├ęphane Blondon (sblondon) * Date: 2018-03-29 20:07
In several cases, tests use ```self.assertTrue(a in b)```. Using ```self.assertIn(a, b)``` seems to be better.
For examples:
   self.assertTrue('(po, pk' in repr(sig))
   self.assertTrue('that_value' in cf['Spacey Bar'])
   self.assertTrue(elem in c)

There are some cases where ```self.assertTrue(a not in b)``` could be replaced by  ```self.assertNotIn(a, b)```

   self.assertTrue('.' not in value)
   self.assertTrue(not ('a' in d))
   self.assertTrue('a' not in d)

$ find . -name "*.py" | xargs grep -r "assertTrue.* in "
finds 131 occurences but there are some false positives inside the output.

I can write a patch if you are interested.
msg314671 - (view) Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) Date: 2018-03-29 20:29
This is a duplicate of issue16510.
Date User Action Args
2018-03-29 20:29:02serhiy.storchakasetstatus: open -> closed

superseder: Using appropriate checks in tests

nosy: + serhiy.storchaka
messages: + msg314671
resolution: duplicate
stage: resolved
2018-03-29 20:07:40sblondoncreate