Addressed sonar scanner code smells

+Rule of 5 for Kafka consumer and producer
+ Remove virtual destructor for CARMAStreetsPlugin

src/tmx/TmxUtils/src/kafka/kafka_consumer_worker.h

@@ -73,6 +73,11 @@ namespace tmx::utils {
              * 
              */
             ~kafka_consumer_worker();
+
+            kafka_consumer_worker(kafka_consumer_worker& other) = delete;
+            kafka_consumer_worker& operator=(const kafka_consumer_worker& other) = delete;
+            kafka_consumer_worker(kafka_consumer_worker &&consumer) = delete;
+            kafka_consumer_worker const & operator=(kafka_consumer_worker &&consumer) = delete;
             /**
              * @brief Initialize kafka_consumer_worker
              * 
@@ -94,7 +99,7 @@ namespace tmx::utils {
             /**
              * @brief Stop running kafka consumer.
              */
-            virtual void stop();
+            void stop();
             /**
              * @brief Print current configurations.
              */

src/tmx/TmxUtils/src/kafka/kafka_producer_worker.h

@@ -62,6 +62,11 @@ namespace tmx::utils
             explicit kafka_producer_worker(const std::string &brokers);
 
             virtual ~kafka_producer_worker();
+
+            kafka_producer_worker(kafka_producer_worker& other) = delete;
+            kafka_producer_worker& operator=(const kafka_producer_worker& other) = delete;
+            kafka_producer_worker(kafka_producer_worker &&producer) = delete;
+            kafka_producer_worker const & operator=(kafka_producer_worker &&producer) = delete;
             /**
              * @brief Initialize kafka_producer_worker. This method must be called before send!
              * 
@@ -102,7 +107,7 @@ namespace tmx::utils
             /**
              * @brief Stop running kafka producer.
              */
-            virtual void stop();
+            void stop();
             /**
              * @brief Print current configurations.
              */

src/tmx/TmxUtils/src/kafka/mock_kafka_consumer_worker.h

@@ -31,7 +31,6 @@ namespace tmx::utils {
             MOCK_METHOD(bool, init,(),(override));
             MOCK_METHOD(const char*, consume, (int timeout_ms), (override));
             MOCK_METHOD(void, subscribe, (), (override));
-            MOCK_METHOD(void, stop, (), (override));
             MOCK_METHOD(void, printCurrConf, (), (override));
             MOCK_METHOD(bool, is_running, (), (const override));
     };

src/tmx/TmxUtils/src/kafka/mock_kafka_producer_worker.h

@@ -26,7 +26,6 @@ namespace tmx::utils {
             MOCK_METHOD(bool, init,(),(override));
             MOCK_METHOD(void, send, (const std::string &msg), (override));
             MOCK_METHOD(bool, is_running, (), (const, override));
-            MOCK_METHOD(void, stop, (), (override));
             MOCK_METHOD(void, printCurrConf, (), (override));
     };
 }

src/tmx/TmxUtils/test/KafkaTestEnvironment.cpp

@@ -23,6 +23,5 @@ class KafkaTestEnvironment : public ::testing::Environment {
             else {
                 std::cout << "All Objects are destroyed!" << std::endl;
             }
-
         }
 };

src/v2i-hub/CARMAStreetsPlugin/src/CARMAStreetsPlugin.h

@@ -36,7 +36,6 @@ namespace CARMAStreetsPlugin {
 class CARMAStreetsPlugin: public PluginClientClockAware {
 public:
 	CARMAStreetsPlugin(std::string);
-	virtual ~CARMAStreetsPlugin() = default ;
 	int Main();
 protected: