[BugFix] Fix Currency Search

[deeleeramone]

1. Why?:

   • This command has 3 different providers but then it doesn’t have any standard argument and all the available provider arguments belong to polygon only.

2. What?:

   • There is now one parameter, "query", and it is valid for all 3 providers.

3. Impact:

   • This endpoint is pretty unlikely to be used, is for generating lists of valid symbols for a source, and a basic text search through every field returned is all its capable of.

4. Testing Done:

   • Updated integration tests and Polygon unit test.

obb.currency.search("euro", provider="polygon")

OBBject

id: 0663c5e6-9a47-7b78-8000-84fc8542c7a8
results: [{'symbol': 'AEDEUR', 'name': 'Euro', 'currency_symbol': 'EUR', 'base_curr...
provider: polygon
warnings: None
chart: None
extra: {'metadata': {'arguments': {'provider_choices': {'provider': 'polygon'}, 'st...

obb.currency.search("europe", provider="polygon")

Detail -> No results were found with the query supplied. -> europe

obb.currency.search("europe", provider="fmp")

OpenBBError: 
Type -> EmptyDataError

Detail -> No results were found with the query supplied. -> europe Hint: Names and descriptions are not searchable from FMP, try 3-letter symbols.

[hjoaquim]

Works well for all providers. I believe having the "query" facilitates understanding how the command works, nice!

[hjoaquim]

Why removing this Polygon query parameters?

[deeleeramone]

Because we can do everything post-request with no real impact on performance while making it operate like everything else and simplifying it dramatically for the end-user.