* Use f-strings in integrations starting with F
* Use f-strings in tests for integrations starting with F
* Use f-strings in integrations starting with G
* Use f-strings in tests for integrations starting with G
* Fix pylint error
* Fix broken test
* Fixed volume control for Frontier Silicon media players.
* Removed unnecessary else which caused pylint test to fail.
* Removed whitespace on empty line.
* Tweaks from on springstan's suggestions and other fixes
* Apply suggestions from code review
Co-Authored-By: springstan <46536646+springstan@users.noreply.github.com>
* Comment to explain why 1 is subtracted from volume_Steps
Also reformatted using black after code review changes.
* Split up the set volume functions into steps rather than all in-line as suggested.
* Renamed _volume_steps to _max_volume.
* Prevent asnyc_update from failing if we can't get the volume steps
Co-authored-by: springstan <46536646+springstan@users.noreply.github.com>
* Added the option to specify the name of the device in confirguration.yaml
* Adding missing default name parameter in auto-discovery FSAPIDevice constructor.
* Fixed Black formatting.
* Removed DEFAULT_NAME constant.
* Apply suggestions from code review
Co-Authored-By: springstan <46536646+springstan@users.noreply.github.com>
Co-authored-by: springstan <46536646+springstan@users.noreply.github.com>
The player would report itself as ‘off’ when in certain modes (e.g ‘Music player’ or ‘Spotify’) which meant HA would lose all control (it can’t change input or set volume etc. as it thinks it’s off). Now reports STATE_IDLE in these cases and only STATE_OFF if it is actually off.
This fixes issue #20728.
* Consolidate
* Fix tests
* Update imports
* Fix import
* Use importlib because integration and package share name
* Fix more tests
* Update .coveragerc and CODEOWNERS
Erik Montnemery