docs: fixed typos in pip install "web3[test]"

[LuizDMM]

What was wrong?

I tried to install the project with "tester", following the current documentation in a project and the dependencies for testing (eth_tester) were not installed.

When checking the current setup.py I found that the correct command to do would be pip install web3[test] and not pip installl web3[tester].

How was it fixed?

I changed the documentation in every reference I could find from pip install web3[tester] to pip install web3[test].

Todo:

• Clean up commit history
• Add or update documentation related to these changes
• Add entry to the release notes

Cute Animal Picture

[kclowes]

Thank you for this PR! I think what we actually want to do is add the tester optional dependency back. It should include py-evm and eth-tester. It looks like it got inadvertently taken out. If you're up for changing this PR, that would be great, otherwise I'll get to it over the next few days.

Edit: Oops, should include py-geth and eth-tester. py-evm will be installed via eth-tester[py-evm]

[LuizDMM]

Hey @kclowes, sure! I can handle that myself. I'll change the PR later tomorrow when I'm done with some work.

[kclowes]

Sounds great! Thank you!

[LuizDMM]

Hey @kclowes, just pushed a new commit replacing my previous commit (clean commit history 😉). Please check it out when you can and let me know if I forgot anything, I'm available to adjust any point.

Edit: I forgot to mention, I added both py-geth and eth-tester in the current versions used in the test optional dependencies.

[kclowes]

Looking good! We'll want to add extras_require["tester"] to the dev dependencies on line 47, and we can remove eth-tester and py-geth from the test requirements. Thanks!

[LuizDMM]

Ok, just a min.

[LuizDMM]

Done! Just replaced the commit again to keep the history clean.

[LuizDMM]

Looking good! We'll want to add extras_require["tester"] to the dev dependencies on line 47, and we can remove eth-tester and py-geth from the test requirements. Thanks!

Taking a look in the checks that failed here, I think that maybe we shouldn't remove the dependencies from test? 🤔 Just guessing.

[kclowes]

Looks great! Those were just flaky tests. I'll add a newsfragment and merge. Thank you!