Disallow game names with spaces (#474)

* WIP trying to fix #459 by replacing spaces with dashes

* Revert changes in socketio.js and api.ts

* Throw error if a game name has spaces

Author: jasonharrison

src/core/game.js

@@ -67,6 +67,10 @@ export function Game(game) {
   if (game.playerView === undefined) game.playerView = G => G;
   if (game.plugins === undefined) game.plugins = [];
 
+  if (game.name.includes(' ')) {
+    throw new Error(game.name + ': Game name must not include spaces');
+  }
+
   const flow = Flow(game);
 
   return {

src/core/game.test.js

@@ -211,3 +211,10 @@ test('serpentine setup phases', () => {
   expect(client.getState().ctx.currentPlayer).toBe('1');
   expect(client.getState().ctx.phase).toBe('main phase');
 });
+
+test('game name with spaces should raise Error', () => {
+  const game = () => {
+    Game({ name: 'tic tac toe' });
+  };
+  expect(game).toThrow();
+});