Auto update and expand _lights class variable in AdaptiveSwitch on access

[th3w1zard1]

It seemed arbitrary or unnecessarily picky when switch.py would used _expand_light_groups() before using _lights, now it just auto-expands on each access.

Code also uses switch._lights more sparingly and only once per method.

I also added # pylint: disable=protected-access to any function that accessed a class method outside of the class as that seemed to be what we've been going with.

[basnijholt]

what about creating a property on the switch:

@property
def all_lights(self):
    return self._expand_light_groups()

then use switch.all_lights everywhere?

Or maybe just lights (I not already in use in HA somewhere).

[th3w1zard1]

brilliant.

[th3w1zard1]

We'd have to modify all the references with this method.

[basnijholt]

I see your reasoning :)

However, a custom __getattribute__ might be hard to overlook when debugging because it is not super obvious what happens from just looking at the code.

[th3w1zard1]

Done

[th3w1zard1]

I'm not looking forward to modifying the other PRs. Perhaps we merge this one last?