[Bug] Broken ETF functionalities

[soggyomelette]

Description

Tries to solve #1972 by changing the code in stockanalysis_model.py to ensure it extracts the data from stockanalysis.com correctly

Also modifies the code in etf_controller.py to handle comparison between ETFs when spaces are used instead of commas (the parser isn't able to interpret arguments with spaces correctly)

Before:

After:

• 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.
• 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/....

[DidierRLopes]

Thank you!

Can you fix the tests as well please? @Chavithra can help you here if needed

[JerBouma]

Fixed it for you, @soggyomelette

[soggyomelette]

Fixed it for you, @soggyomelette

Thanks a lot! @Chavithra showed me how to do it but am travelling atm so hadn't done it yet

[JerBouma]

Fixed it for you, @soggyomelette

Thanks a lot! @Chavithra showed me how to do it but am travelling atm so hadn't done it yet

No problem! Feel free to merge :)

[DidierRLopes]

Nice one @soggyomelette 🚀