Skip to content

Commit 42113b7

Browse files
committed
Draw pen lines in SpriteModel::onMoved()
1 parent ede2fa0 commit 42113b7

File tree

2 files changed

+19
-0
lines changed

2 files changed

+19
-0
lines changed

src/spritemodel.cpp

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -62,6 +62,8 @@ void SpriteModel::onYChanged(double y)
6262

6363
void SpriteModel::onMoved(double oldX, double oldY, double newX, double newY)
6464
{
65+
if (m_penDown && m_penLayer)
66+
m_penLayer->drawLine(m_penAttributes, oldX, oldY, newX, newY);
6567
}
6668

6769
void SpriteModel::onSizeChanged(double size)

test/target_models/spritemodel_test.cpp

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -135,6 +135,23 @@ TEST(SpriteModelTest, OnYChanged)
135135
model.onYChanged(-46.1);
136136
}
137137

138+
TEST(SpriteModelTest, OnMoved)
139+
{
140+
SpriteModel model;
141+
142+
PenLayerMock penLayer;
143+
model.setPenLayer(&penLayer);
144+
145+
EXPECT_CALL(penLayer, drawLine).Times(0);
146+
model.onMoved(-15.6, 54.9, 159.04, -2.5);
147+
148+
model.setPenDown(true);
149+
PenAttributes &attr = model.penAttributes();
150+
151+
EXPECT_CALL(penLayer, drawLine(_, -15.6, 54.9, 159.04, -2.5)).WillOnce(WithArgs<0>(Invoke([&attr](const PenAttributes &attrArg) { ASSERT_EQ(&attr, &attrArg); })));
152+
model.onMoved(-15.6, 54.9, 159.04, -2.5);
153+
}
154+
138155
TEST(SpriteModelTest, OnSizeChanged)
139156
{
140157
SpriteModel model;

0 commit comments

Comments
 (0)