after REVIEW a3128843f

fix shouldInitPlayer test
change emplace_back back to push_back because of pointer usage in vector