Adds a test for binance key as asked for in #2006

[soggyomelette]

Description

Adds a test for the binance key in the keys menu as there initially wasn't one present

Before:

After with incorrect key:

After with correct key:

• Summary of the change / bug fix.
• Link # issue, if applicable.
• Screenshot of the feature or the bug before/after fix, if applicable.
• Relevant motivation and context.
• List any dependencies that are required for this change.

How has this been tested?

• Please describe the tests that you ran to verify your changes: Ran with incorrect keys and with my own keys to reproduce above results
• Provide instructions so we can reproduce.
• Please also list any relevant details for your test configuration.

Checklist:

• Update our Hugo documentation following these guidelines.
• Update our tests following these guidelines.
• Make sure you are following our CONTRIBUTING guidelines.
• If a feature was added make sure to add it to the corresponding scripts file.

Others

• I have performed a self-review of my own code.
• I have commented my code, particularly in hard-to-understand areas.
• My code passes all the checks pylint, flake8, black, ... To speed up development you should run pre-commit install.
• New and existing unit tests pass locally with my changes. You can test this locally using pytest tests/....

[soggyomelette]

I also see that there are a few other keys that don't have tests written for them, is that something I could add to a TODO list?

[DidierRLopes]

I also see that there are a few other keys that don't have tests written for them, is that something I could add to a TODO list?

Hi @soggyomelette , yes please!

[DidierRLopes]

You just need to run black file_changed_path and then we can merge this 😄

[soggyomelette]

I also see that there are a few other keys that don't have tests written for them, is that something I could add to a TODO list?

Hi @soggyomelette , yes please!

Great, I'll start working on it!

[DidierRLopes]

Thank you @soggyomelette ❤️