Resolve coverity issues

* Added Rule of Five compliance to the reported classes
* Added explicit default destructors and assignment operators
* Corrected comparison statement

Tested:
UTs passed

Change-Id: Ifcfc1055e2e65d9f3ce10e3878ff3f1dd5d84929
Signed-off-by: Piotr Sulewski <piotrx.sulewski@intel.com>
diff --git a/src/on_change_threshold.hpp b/src/on_change_threshold.hpp
index 91a595f..dd193f2 100644
--- a/src/on_change_threshold.hpp
+++ b/src/on_change_threshold.hpp
@@ -23,8 +23,11 @@
         const std::string& triggerId, Sensors sensors,
         std::vector<std::unique_ptr<interfaces::TriggerAction>> actions,
         std::unique_ptr<interfaces::Clock> clock);
+    ~OnChangeThreshold() = default;
     OnChangeThreshold(const OnChangeThreshold&) = delete;
+    OnChangeThreshold& operator=(const OnChangeThreshold&) = delete;
     OnChangeThreshold(OnChangeThreshold&&) = delete;
+    OnChangeThreshold& operator=(OnChangeThreshold&&) = delete;
 
     void initialize() override;
     void sensorUpdated(interfaces::Sensor&, Milliseconds, double) override;