Fix faulty not check

[Booplicate]

Issue:

If you check how you're handling writing to multiple registers, you notice you're using explicit is None check, which allows passing "falsy" values.

However for coils, you're doing not check, which means it's impossible to write 0 as it will get converted to an empty list.

It's not a big deal and it's a good practice to explicitly wrap values into a list, but since you do support it for input registers, can make sense to support for coils as well.

Changelog:

• replaces not check with explicit is None
• added a test case for this

Note:

I'm not too familiar with pytest, but I hope I added a valid test case for this.

[janiversen]

LGTM, thanks.

Just wondering if the opposite solution would have been better (bring coil inline).

[Booplicate]

Just wondering if the opposite solution would have been better (bring coil inline).

Up to you, wasn't sure about that since it'd be a breaking change.

[janiversen]

good point, lets keep it the way you made it.