Add Philips RDM004 (HUE Wall Switch rev 004) to quirks

[Tijgerd]

Proposed change

Added the RDM004 to the quirks.
Credits to Regenbui (Wesley Westerveld): https://community.home-assistant.io/t/how-to-configure-the-philips-hue-wall-module-to-use-push-button-momentary-type-wall-switches-zha/451125/27

Additional information

The RDM004 was not usable in ZHA, the fix as proposed by "Regenbui" was working as expected!
Also fixed a minor

Checklist

• The changes are tested and work correctly
• pre-commit checks pass / the code has been formatted using Black
• Tests have been added to verify that the new code works

[codecov]

Codecov Report

Attention: Patch coverage is 76.31579% with 9 lines in your changes missing coverage. Please review.

Project coverage is 88.22%. Comparing base (8317dc6) to head (3a5a470).
Report is 93 commits behind head on dev.

Files with missing lines	Patch %	Lines
zhaquirks/philips/rdm004.py	75.67%	9 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##              dev    #3237      +/-   ##
==========================================
+ Coverage   88.18%   88.22%   +0.03%     
==========================================
  Files         301      303       +2     
  Lines        9412     9484      +72     
==========================================
+ Hits         8300     8367      +67     
- Misses       1112     1117       +5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

---

🚨 Try these New Features:

• Flaky Tests Detection - Detect and resolve failed and flaky tests

[MattWestb]

Some spaces is after the comma that the tests dont like.
Put in 0x0034: ("mode", t.enum8, True), and it shall working OK

[MattWestb]

Looks nice but one new error is detected:

zhaquirks/philips/rdm004.py:3:1: UP035 `typing.List` is deprecated, use `list` instead
Found 1 error.

Error: Process completed with exit code 1.

I dont knowing if its only tinging the command or its needs more work then im not one code warier.

[Tijgerd]

Give me a second :D will fix this!
But it was working in home assistant like a charm

[MattWestb]

All green !!!!
Only function code test implantation failing.

[Tijgerd]

I see this is also failing on the RDM001 (and will possibly also fail on the other quirks in the Philips folder.
Looking into this but I am not able to properly debug this myself at the moment..

[Tijgerd]

This I am unable to get the tests to work...
I am running the custom quirk for over a week now without any problem, it is just the test that I am unable to integrate..

[MattWestb]

Its one space after the ) that making the test complaining.

[MattWestb]

The other:

+++ b/tests/test_philips.py
@@ -1,11 +1,7 @@
 """Tests for Philips RDM001 and RDM004 quirks."""
-import pytest
 
-from tests.common import ClusterListener
 
 import zhaquirks
-from zhaquirks.philips.rdm001 import PhilipsRDM001
-from zhaquirks.philips.rdm004 import PhilipsRDM004
 
 zhaquirks.setup()

Its looks the test i not liking importing this function but i dont knowing if its needed for the code or not then im not one code warier.
I think Puddly or TheJullian is knowing it bettter.

[Tijgerd]

nope, the space after the ) was not the problem apparently

[MattWestb]

The first error is now gone only the import section and the the strange last ones . . . so one step forewords and . . .

[TheJulianJES]

This should be implemented with the following PRs:

• Add Philips RDM002 Tap Dial switch support, refactor Hue remotes #2340
• Add Philips Hue wall switch RDM004 variant #3075
• Add Philips Hue wall switch RDM004 variant #3543

Everything should be available in Home Assistant Core 2024.12. Some of it is already available in 2024.11.
@Tijgerd Still, thanks for the PR!