Add ness alarm control panel using nessclient

[nickw444]

Description:

Adds new ness D8x/D16x alarm platform using nessclient client library. Underlying library takes advantage of the Ness D8x/D16x ASCII protocol

Related issue (if applicable): fixes #

Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#7630

Example entry for configuration.yaml (if applicable):

ness_alarm:
  host: alarm.local
  port: 2401
  zones:
    - name: Garage
      id: 1
    - name: Storeroom
      id: 2
    - name: Kitchen
      id: 3
    - name: Front Entrance
      id: 4

Checklist:

• The code change is tested and works locally.
• Local tests pass with tox. Your PR cannot be merged unless tests pass
• There is no commented out code in this PR.

If user exposed functionality or configuration variables are added/changed:

• Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

• New dependencies have been added to the REQUIREMENTS variable (example).
• New dependencies are only imported inside functions that use them (example).
• New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
• New files were added to .coveragerc.

If the code does not interact with devices:

• Tests have been added to verify that the new code works.

[nickw444]

Hi @frenck this pull request is ready for review. It seems the labels and wip build status hasn’t been updated since I renamed it and finalised the code.

Cheers 🙂

[MartinHjelmare]

In general looks good. Tests should take a different approach though, and be written more like integration tests which is our standard.

[MartinHjelmare]

To remove WIP label, rebase and reword the commit messages that contain WIP.

[MartinHjelmare]

Please don't squash commits after review has started to make it easier for readers to track changes.

[MartinHjelmare]

Great!