Замечания по паттерну Адаптер

[alexey-malov]

• LineTo соединяет начальную точку с указанной и перемещает начальную точку в указанную. В вашем случае адаптеры этого не делают.

• проверить в тестах

[alexey-malov]

• внутри классов, унаследованных от класса из другого namespace можно не указывать название namespace и имя класса, а просто вызывать метод. Имя класса нужно указывать лишь при вызове метода базового класса, который совпадает с именем наследника

[alexey-malov]

class ModernGraphiscsClassAdapter : 
	public graphics_lib::ICanvas,
	private modern_graphics_lib::CModernGraphicsRenderer
{
public:
	ModernGraphiscsClassAdapter(std::ostream& output = std::cout);
	using CModernGraphicsRenderer::BeginDraw;
	using CModernGraphicsRenderer::EndDraw;
	//void BeginDraw();
	//void EndDraw();
	void MoveTo(int x, int y) override;
	void LineTo(int x, int y) override;
private:
	modern_graphics_lib::CPoint m_lineBegin;
};

• заменить метод или конструктор, вызывающий родительский одноименный метод/конструктор с теми же параметрам можно на using BaseClassName::MethodName;

[alexey-malov]

struct classAdapterFixture

• Использовать UpperCamelCase

[alexey-malov]

ModernGraphiscsClassAdapter
ModernGraphiscsObjectAdapter

• опечатка в имени класса Graphiscs

возможно еще где-то

[alexey-malov]

shape_drawing_lib::CTriangle::CTriangle(const Point & p1, const Point & p2, const Point & p3)
	:m_vertices({p1, p2, p3})
{
}

• чтобы не писать имя namespace в .cpp реализации, надо поместить внутрь namespace {}:

namespace shape_drawing_lib
{
CTriangle::CTriangle(const Point & p1, const Point & p2, const Point & p3)
	:m_vertices({p1, p2, p3})
{
}
.... // прочие методы
}

[alexey-malov]

• Добавить использование нового API (с цветом) в основном приложении