Skip to content
This repository has been archived by the owner on May 7, 2020. It is now read-only.

MQTT: Add MQTT receive tests for all channel types #6664

Merged
merged 1 commit into from Dec 12, 2018

Conversation

davidgraeff
Copy link
Contributor

The TypeParser commit a few days ago allowed to send command strings
like INCREASE/DECREASE. Unfortunately it broke a few channels.

  • This commit adds tests for all channel types.
  • The RGB color channel was broken (even before the TypeParser commit): Fixed.

Added features:

  • Add location, datetime and image channel types including
    tests and readme sections.

Refactor:

  • The internal.values.* classes are a lot slimmer. Value itself
    is not an interface but an abstract class now.
  • Move the xml thing/channel configurations to ESH-INF/config.
  • Rename two methods for clarity, so that value.getValue().getValue()
    is not a think anymore.

Signed-off-by: David Gräff david.graeff@web.de

The TypeParser commit a few days ago allowed to send command strings
like INCREASE/DECREASE. Unfortunately it broke a few channels.

* This commit adds tests for all channel types.
* The RGB color channel was broken (even before the TypeParser commit): Fixed.

Added features:
* Add location, datetime and image channel types including
  tests and readme sections.

Refactor:
* The internal.values.* classes are a lot slimmer. Value itself
  is not an interface but an abstract class now.
* Move the xml thing/channel configurations to ESH-INF/config.
* Rename two methods for clarity, so that value.getValue().getValue()
  is not a think anymore.

Signed-off-by: David Gräff <david.graeff@web.de>
@kaikreuzer kaikreuzer merged commit 804aaee into eclipse-archived:master Dec 12, 2018
@davidgraeff davidgraeff deleted the mqttvalues branch December 12, 2018 10:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants