Simplify 'create_test_default_tiles' by removing explicit call to tile_id constructor

[richelbilderbeek]

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

Currently, create_test_default_tiles looks like this:

std::vector<tile> create_test_default_tiles() noexcept //!OCLINT indeed a function that is too long
{
  // ...
    tile t(0, 0, 0, 90, 0, tile_type::grassland, tile_id());
  // ...
}

The explicit call to tile_id can be removed, as it is called by default in the tile constructor.

Describe the solution you'd like

Remove the tile_id() argument in a tile constructor.

For example:

    tile t(0, 0, 0, 90, 0, tile_type::grassland, tile_id());

Becomes:

    tile t(0, 0, 0, 90, 0, tile_type::grassland);

All tests should keep passing.

Describe alternatives you've considered

None.

Additional context

None.