Update algorithm for parsing smart filters

[Dr-Blank]

Description

algorithm for parsing is updated to cover multilevel filter groups parsing

Fixes

• SmartFilterMixin._parseFilters algorithm fails when multi-level filters are to be parsed #1274

Type of change

• Bug fix (non-breaking change which fixes an issue)

Checklist:

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have added or updated the docstring for new or existing methods
• I have added tests when applicable

[JonnyWong16]

Thanks. Could you add some tests with some more complex filters? Right now the tests only check that searching with the parsed filters returns the same items.

python-plexapi/tests/test_collection.py

Lines 278 to 279 in e44939c

	filters = collection.filters()
	assert movies.search(**filters) == collection.items()

python-plexapi/tests/test_playlist.py

Lines 198 to 200 in e44939c

	filters = playlist.filters()
	filters['libtype'] = tvshows.METADATA_TYPE # Override libtype to check playlist items
	assert tvshows.search(**filters) == playlist.items()

The new tests should actually check that the filter and parsed filter dictionaries are equal.

parsedFilters = collection.filters()
assert filters == parsedFilters

parsedFilters = playlist.filters()
assert filters == parsedFilters

[Dr-Blank]

Thanks. Could you add some tests with some more complex filters? Right now the tests only check that searching with the parsed filters returns the same items.

Could you please guide me to a document where I can learn how to setup plex environment for testing?

Currently running test locally fails all of them except a few. I have to use GH Actions every time, and it would be great to have it run locally.

I know it sets up pms using docker but if there was a script for this to do automatically?

[JonnyWong16]

See Discord.

[Dr-Blank]

@JonnyWong16

I have added 2 tests which will fail for the old algorithm that are passing for this PR.

[JonnyWong16]

The logic looks good. Thanks!

Just a few stylistic changes.