[community]: [remove arbitrary return in google results]

[cuiqui]

PR Description

This PR fixes the implementation to match the documented behavior of this method.

Method's documentation clearly states that:

        Returns:
            A list of dictionaries with the following keys:
                snippet - The description of the result.
                title - The title of the result.
                link - The link to the result.

Much to my demise, there's a strange early return

        if len(results) == 0:
            return [{"Result": "No good Google Search Result was found"}]

if no results were found.

There's no need to represent the "emptiness" of results, an empty list takes care of that. In any case, if we need to represent the absence, this is not a sensible null pattern implementation, since the returned object is not even polymorphic wit h what the documentation says.

This PR removes that early return and considers that an empty list correctly represents the absence of results

Relevant issues

Type

🧹 Refactoring
📖 Documentation

Changes(optional)

Testing(optional)

Note(optional)