Why isn't this script working, it brings players to spawn points?

function BringPlayers()
   	local Spawns = game.Workspace.Map.MapSpawners:GetChildren()
    	local Players = game.Players:GetPlayers()
  		for i = 1, #Players do
       		local SelectedSpawn = math.random(#Spawns)
       		local Spawn = Spawns[SelectedSpawn]
        	if Players[i].Character then --Make sure they are spawned in
           		Players[i].Character:SetPrimaryPartCFrame(Spawn.Spawn.CFrame)
			table.remove(Spawns,SelectedSpawn)
       		end
 	end
end

Don't set a variable to table.remove as that will never work right as you just removed the value you want. I also changed setting the torso CFrame(Your code would only work for R15) and using SetPrimaryPartCFrame on a model is generally better.

I also removed the 2and BringPlayers() as this prevents executing the code at all

the remove code is also inside the if statement now so that if there is no character to move, the spawn can still be used for a different player.

Hope this helped