feat: search module improvements

[hpohekar]

closes #3496, #3498, #3499

Improve synset handling

1. Removed empty synset lists.
2. Removed duplicate values from synset list.
3. Store synset names.

Remove duplicate data and dependency on api_tree_version.pickle file

1. Removed need of the api_tree_<version>.pickle file.

Refactor flobject.py for search function

1. Refactored _search.
2. Added module level variable to return or print results as per the requirement.
3. Refactored flobject.py
4. Refactored test_search.py and removed tests related to legacy search function.
5. Refactored settings_extrenal.py

Reorder search function

1. Changed the order of search based on the arguments preferences

Add more tests

1. Added test from test: Example of unittests for search-related functions #3393

[mkundu1]

@hpohekar We don't support search_root in the new search function. Feel free to clean up the function _get_version_path_prefix_from_obj and relevant tests (in same or different PR).

[mkundu1]

@hpohekar we'll check if any change in required for PyConsole after merging this PR

[mkundu1]

@hpohekar Can you please add this test?

api_tree_data = {
    "api_objects": [
        "<solver_session>.first_last (Object)",
        "<solver_session>.none (Object)",
    ],
    ...
}
assert pyfluent.search("first", match_whole_word=True) == ["<solver_session>.first_last (Object)"]
assert pyfluent.search("last", match_whole_word=True) == ["<solver_session>.first_last (Object)"]
# It is OK if we don't support the following - the above 2 are more important
assert pyfluent.search("first_last", match_whole_word=True) == ["<solver_session>.first_last (Object)"]

[hpohekar]

@hpohekar we'll check if any change in required for PyConsole after merging this PR

@mkundu1 Sure.

[hpohekar]

@hpohekar Can you please add this test?

api_tree_data = {
    "api_objects": [
        "<solver_session>.first_last (Object)",
        "<solver_session>.none (Object)",
    ],
    ...
}
assert pyfluent.search("first", match_whole_word=True) == ["<solver_session>.first_last (Object)"]
assert pyfluent.search("last", match_whole_word=True) == ["<solver_session>.first_last (Object)"]
# It is OK if we don't support the following - the above 2 are more important
assert pyfluent.search("first_last", match_whole_word=True) == ["<solver_session>.first_last (Object)"]

@mkundu1

We are supporting all 3 tests.

Done. Thank you.

[hpohekar]

@hpohekar We don't support search_root in the new search function. Feel free to clean up the function _get_version_path_prefix_from_obj and relevant tests (in same or different PR).

@mkundu1 Removed. Thank you.

[hpohekar]

@seanpearsonuk @mkundu1 We have implemented all suggestions and added more tests. Let me know if you want to add any specific test cases. Thank you.

[mkundu1]

The present changes look good to me, thanks. There is one issue related to the presentationof the results, will file a separate defect.

[hpohekar]

The present changes look good to me, thanks. There is one issue related to the presentationof the results, will file a separate defect.

Sure. Please go ahead. Thank you.