Order return of collections.list_all() alphabetically by key

[saireddythfc]

1. Modified the _collection_configs_from_json and _collection_configs_simple_from_json functions in weaviate/collections/classes/config_methods.py to sort the dictionaries by key before returning them.

2. Added unit tests in test/collection/test_collections_sorting.py to verify the sorting functionality.

3. Added integration tests in integration/test_collections_sorting.py to verify the sorting in a real environment.

[orca-security-eu]

Orca Security Scan Summary

Status	Check	Issues by priority
Passed	Infrastructure as Code	0   0   0   0	View in Orca
Passed	SAST	0   0   0   0	View in Orca
Passed	Secrets	0   0   0   0	View in Orca
Passed	Vulnerabilities	0   0   0   0	View in Orca

[saireddythfc]

1. Modified the _collection_configs_from_json and _collection_configs_simple_from_json functions in weaviate/collections/classes/config_methods.py to sort the dictionaries by key before returning them.

2. Added unit tests in test/collection/test_collections_sorting.py to verify the sorting functionality.

3. Added integration tests in integration/test_collections_sorting.py to verify the sorting in a real environment.

[weaviate-git-bot]

To avoid any confusion in the future about your contribution to Weaviate, we work with a Contributor License Agreement. If you agree, you can simply add a comment to this PR that you agree with the CLA so that we can merge.

beep boop - the Weaviate bot 👋🤖

PS:
Are you already a member of the Weaviate Slack channel?

[tsmith023]

Hi @saireddythfc, thanks for the contribution! Looks like the linter needs to be ran on your code, we use black and flake8, plus the test you've introduced is not scoped for pytest appropriately. Otherwise, the changes look good! Once you fix the tests, I'll give the PR a formal review 😁

[saireddythfc]

Update:

1. Fixed name collision occurring in integration test (client -> sorting_test_client).
2. Changed pytest fixture from module to function.
3. Manually ran black linter before pre-commit did its task on commit.

[saireddythfc]

Update:

1. Fixed unused pytest import

@tsmith023 any chance of a formal review?

[tsmith023]

Implementation LGTM, just the test needs fixing imo!

[tsmith023]

LGTM now 😁 The only thing left is the linter complaining about a docstring!

integration/test_collections_sorting.py:15:1: D202 No blank lines allowed after function docstring

Once fixed, I'll merge

[saireddythfc]

I agree to the CLA shown by @weaviate-git-bot