Skip to content

Commit

Permalink
update note about adding material check to integration test
Browse files Browse the repository at this point in the history
Signed-off-by: Ashton Larkin <[email protected]>
  • Loading branch information
adlarkin committed Feb 18, 2021
1 parent 18b0194 commit 2a6549e
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions test/integration/particle_emitter.cc
Original file line number Diff line number Diff line change
Expand Up @@ -103,8 +103,7 @@ TEST_F(ParticleEmitterTest, SDFLoad)
EXPECT_EQ(math::Vector3d(3, 3, 3),
msgs::Convert(_emitter->Data().particle_size()));
EXPECT_DOUBLE_EQ(2.0, _emitter->Data().lifetime());
// TODO(anyone) add material check once other particle PRs
// have been merged
// TODO(anyone) add material check here
EXPECT_DOUBLE_EQ(10.0, _emitter->Data().min_velocity());
EXPECT_DOUBLE_EQ(20.0, _emitter->Data().max_velocity());
EXPECT_EQ(math::Color::Blue,
Expand Down Expand Up @@ -140,8 +139,7 @@ TEST_F(ParticleEmitterTest, SDFLoad)
EXPECT_EQ(math::Vector3d(1, 1, 1),
msgs::Convert(_emitter->Data().particle_size()));
EXPECT_DOUBLE_EQ(5.0, _emitter->Data().lifetime());
// TODO(anyone) add material check once other particle PRs
// have been merged
// TODO(anyone) add material check here
EXPECT_DOUBLE_EQ(1.0, _emitter->Data().min_velocity());
EXPECT_DOUBLE_EQ(1.0, _emitter->Data().max_velocity());
EXPECT_EQ(math::Color::White,
Expand Down

0 comments on commit 2a6549e

Please sign in to comment.