gh-112: Turn Game, GameData to interfaces

backend/repositories/GameRepository.test.ts

@@ -14,6 +14,8 @@ describe('GameRepository', () => {
   })
   describe('addGame', () => {
     it.each([
+      ['has not a game data', undefined, undefined],
+      ['has nulls', null, null],
       ['has empty data', '', ''],
       ['has no description', FOOSBALL_DATA.name, ''],
       ['has no name', '', FOOSBALL_DATA.description],

backend/repositories/GameRepository.ts

@@ -1,19 +1,22 @@
-import { GameData } from '../types/Game'
+import { GameData, isGameData } from '../types/Game'
 import { InputError } from '../errors/InputError'
 import * as storage from '../storage/Storage'
 
-const isValidGame = (gameData: GameData): boolean => {
-  return gameData.name &&
-    gameData.description &&
+const isValidGameData = (gameData: GameData): boolean => {
+  return gameData.name !== '' &&
+    gameData.description !== '' &&
     gameData.name.trim() == gameData.name &&
     gameData.name.toLowerCase() == gameData.name &&
     !gameData.name.includes(' ') &&
     gameData.description.trim() == gameData.description
 }
 
-export const addGame = async (gameData: GameData): Promise<void> => {
-  if (!isValidGame(gameData)) {
+export const addGame = async (data: unknown): Promise<void> => {
+  if (!isGameData(data)) {
+    throw new InputError('Data is not valid!')
+  }
+  if (!isValidGameData(data)) {
     throw new InputError('Game data is not valid!')
   }
-  storage.insertGame(gameData)
+  storage.insertGame(data)
 }

backend/storage/db/db-transformations.ts

@@ -44,8 +44,8 @@ export const createMatchFromDbRow = (matchRow): MatchWithId => {
   }
 }
 
-export const createGameFromDbRow = (gameRow): Game => new Game(
-  Number(gameRow.Id),
-  gameRow.Name,
-  gameRow.Description
-)
+export const createGameFromDbRow = (gameRow): Game => ({
+  id: Number(gameRow.Id),
+  name: gameRow.Name,
+  description: gameRow.Description,
+})

backend/types/Game.ts

@@ -1,14 +1,14 @@
-export class Game {
-  constructor(
-    readonly id: number,
-    readonly name: string,
-    readonly description: string
-  ) {}
+export interface GameData {
+  readonly name: string;
+  readonly description: string;
 }
 
-export class GameData {
-  constructor(
-    readonly name: string,
-    readonly description: string
-  ) {}
+export const isGameData = (data: unknown): data is GameData => {
+  const test = data as GameData
+  return test.name !== undefined && test.name !== null &&
+  test.description !== null && test.description !== null
+}
+
+export interface Game extends GameData {
+  readonly id: number;
 }