refactor and remove tests/scripts/test_all_pip.py

[brainwane]

Per developer docs, the single file in tests/scripts/(test_all_pip.py):

will probably die in future in a refactor -- scripts for running all of the tests, but we use pytest now. Someone could make a PR to remove this! Good first issue!

Is test_all_pip.py actually used anywhere? I think it's not but I'm not sure, and reviewing and refactoring it looks too complicated to be a good first issue.

Looks like the necessary steps are:

• verify that test_all_pip is not actually used anywhere in our testing or in our downstreams' tests
• git rm the file and the scripts/ directory

Is that correct? cc @pradyunsg

[Bhavam]

Hi , so I checked out the code base and found no use of the script test_all_pip.py , I'll create a PR for version with the folder tests/scripts removed.
I hope that's agreeable if not please suggest any more changes required for the issue #7659 .

[pradyunsg]

Please make sure you mention the PR number here, once you file the PR. Use the #nnnn format, which would result in links to-and-from the relevant issue. :)

[pradyunsg]

verify that test_all_pip is not actually used anywhere in our testing

It is not; verified.

in our downstreams' tests

I don't think so, but it shouldn't matter much to us -- IMO if they're still using this, they should probably update to invoke via pytest; or they can maintain a patch to restore this file.