[tests] Rename FakeCanDriver to CanDriver

and move it to modm_test::platform namespace

Author: strongly-typed

test/modm/communication/xpcc/backend/can/can_connector_test.cpp

@@ -69,7 +69,7 @@ CanConnectorTest::createMessage(modm::can::Message& message,
 void
 CanConnectorTest::setUp()
 {
-	this->driver = new FakeCanDriver();
+	this->driver = new modm_test::platform::CanDriver();
 	this->connector = new TestingCanConnector(this->driver);
 }
 

test/modm/communication/xpcc/backend/can/can_connector_test.hpp

@@ -61,7 +61,7 @@ class CanConnectorTest : public unittest::TestSuite
 
 private:
 	TestingCanConnector *connector;
-	FakeCanDriver *driver;
+	modm_test::platform::CanDriver *driver;
 
 	xpcc::Header xpccHeader;
 	uint32_t normalIdentifier;

test/modm/communication/xpcc/backend/can/fake_can_driver.cpp renamed to test/modm/communication/xpcc/backend/can/can_driver.cpp

@@ -1,7 +1,7 @@
 /*
  * Copyright (c) 2009-2010, Fabian Greif
  * Copyright (c) 2012-2014, Niklas Hauser
- * Copyright (c) 2015, Sascha Schade
+ * Copyright (c) 2015-2020, Sascha Schade
  *
  * This file is part of the modm project.
  *
@@ -11,21 +11,21 @@
  */
 // ----------------------------------------------------------------------------
 
-#include "fake_can_driver.hpp"
+#include "can_driver.hpp"
 
-FakeCanDriver::FakeCanDriver() :
+modm_test::platform::CanDriver::CanDriver() :
 	sendSlots(0)
 {
 }
 
 bool
-FakeCanDriver::isMessageAvailable()
+modm_test::platform::CanDriver::isMessageAvailable()
 {
 	return (not receiveList.isEmpty());
 }
 
 bool
-FakeCanDriver::getMessage(modm::can::Message& message)
+modm_test::platform::CanDriver::getMessage(modm::can::Message& message)
 {
 	if (isMessageAvailable())
 	{
@@ -40,13 +40,13 @@ FakeCanDriver::getMessage(modm::can::Message& message)
 }
 
 bool
-FakeCanDriver::isReadyToSend()
+modm_test::platform::CanDriver::isReadyToSend()
 {
 	return (this->sendSlots > 0);
 }
 
 bool
-FakeCanDriver::sendMessage(const modm::can::Message& message)
+modm_test::platform::CanDriver::sendMessage(const modm::can::Message& message)
 {
 	if (this->isReadyToSend())
 	{
@@ -60,19 +60,19 @@ FakeCanDriver::sendMessage(const modm::can::Message& message)
 }
 
 uint8_t
-FakeCanDriver::getReceiveErrorCounter()
+modm_test::platform::CanDriver::getReceiveErrorCounter()
 {
 	return 0;
 }
 
 uint8_t
-FakeCanDriver::getTransmitErrorCounter()
+modm_test::platform::CanDriver::getTransmitErrorCounter()
 {
 	return 0;
 }
 
 modm::Can::BusState
-FakeCanDriver::getBusState()
+modm_test::platform::CanDriver::getBusState()
 {
 	return modm::Can::BusState::Connected;
 }

test/modm/communication/xpcc/backend/can/fake_can_driver.hpp renamed to test/modm/communication/xpcc/backend/can/can_driver.hpp

@@ -2,6 +2,7 @@
  * Copyright (c) 2009-2010, Martin Rosekeit
  * Copyright (c) 2009-2011, Fabian Greif
  * Copyright (c) 2012-2014, Niklas Hauser
+ * Copyright (c) 2020, Sascha Schade
  *
  * This file is part of the modm project.
  *
@@ -11,17 +12,27 @@
  */
 // ----------------------------------------------------------------------------
 
-#ifndef FAKE_CAN_DRIVER_HPP
-#define FAKE_CAN_DRIVER_HPP
+#ifndef MODM_TEST_MOCK_CAN_DRIVER_HPP
+#define MODM_TEST_MOCK_CAN_DRIVER_HPP
 
-#include <modm/container/linked_list.hpp>
 #include <modm/architecture/interface/can.hpp>
+#include <modm/container/linked_list.hpp>
+
+namespace modm_test
+{
+
+namespace platform
+{
 
-/// @ingroup modm_test_test_communication
-class FakeCanDriver : public modm::Can
+/**
+ * Mock CAN peripheral interface for unittests.
+ *
+ * @ingroup modm_test_mock_can
+ */
+class CanDriver : public modm::Can
 {
 public:
-	FakeCanDriver();
+	CanDriver();
 
 	bool
 	isMessageAvailable();
@@ -55,4 +66,8 @@ class FakeCanDriver : public modm::Can
 	uint8_t sendSlots;
 };
 
-#endif	// FAKE_CAN_DRIVER_HPP
+} // namespace platform
+
+} // namespace modm_test
+
+#endif	// MODM_TEST_MOCK_CAN_DRIVER_HPP

test/modm/communication/xpcc/backend/can/testing_can_connector.cpp

@@ -14,7 +14,7 @@
 
 #include "testing_can_connector.hpp"
 
-TestingCanConnector::TestingCanConnector(FakeCanDriver *driver) :
-	xpcc::CanConnector<FakeCanDriver>(driver)
+TestingCanConnector::TestingCanConnector(modm_test::platform::CanDriver *driver) :
+	xpcc::CanConnector<modm_test::platform::CanDriver>(driver)
 {
 }

test/modm/communication/xpcc/backend/can/testing_can_connector.hpp

@@ -17,16 +17,16 @@
 
 #include <modm/communication/xpcc/backend/can/connector.hpp>
 
-#include "fake_can_driver.hpp"
+#include "can_driver.hpp"
 
 /// @ingroup modm_test_test_communication
-class TestingCanConnector : public xpcc::CanConnector<FakeCanDriver>
+class TestingCanConnector : public xpcc::CanConnector<modm_test::platform::CanDriver>
 {
 public:
-	TestingCanConnector(FakeCanDriver *driver);
+	TestingCanConnector(modm_test::platform::CanDriver *driver);
 
 	// expose the internal variable for testing
-	using xpcc::CanConnector<FakeCanDriver>::messageCounter;
+	using xpcc::CanConnector<modm_test::platform::CanDriver>::messageCounter;
 };
 
 #endif	// TESTING_CAN_CONNECTOR_HPP