Clumsy and untested game::confirm_tile_move

[richelbilderbeek]

Is your feature request related to a problem? Please describe.

Currently, game has this member function:

void game::confirm_tile_move(tile& t, int direction, int tile_speed){
  switch (direction)
  {
    case 1:
      t.set_dy(-tile_speed);
      return;
    case 2:
      t.set_dx(tile_speed);
      return;
    case 3:
      t.set_dy(tile_speed);
      return;
    case 4:
      t.set_dx(-tile_speed);
      return;
    default:
      return;
  }
}

It is clumsy and untested.

Describe the solution you'd like

• Test the function
• If the appears clumsy (which I predict it will), make the member function more natural, which will simplify the test

Describe alternatives you've considered

None.

Additional context

None.

[richelbilderbeek]

Well tried, but does not fix the code coverage.

Someone can try again :+1: