From 952e29edf1804051fb8f596a1288c0afe8c66f6b Mon Sep 17 00:00:00 2001
From: Hamcha <hamcha@crunchy.rocks>
Date: Fri, 20 Sep 2019 15:13:15 +0200
Subject: [PATCH] Fix linting issues

---
 .eslintrc.js                       |  4 +---
 src/components/Cards/CardImage.vue |  3 +--
 src/mlpccg/draft/session.ts        | 20 ++++++++++----------
 src/mlpccg/images.ts               |  3 ++-
 src/tests/unit/.eslintrc.js        |  2 +-
 5 files changed, 15 insertions(+), 17 deletions(-)

diff --git a/.eslintrc.js b/.eslintrc.js
index fb644c6..3b46228 100644
--- a/.eslintrc.js
+++ b/.eslintrc.js
@@ -18,9 +18,7 @@ module.exports = {
 
   overrides: [
     {
-      files: [
-        '**/__tests__/*.{j,t}s?(x)'
-      ],
+      files: ["**/__tests__/*.{j,t}s?(x)"],
       env: {
         jest: true
       }
diff --git a/src/components/Cards/CardImage.vue b/src/components/Cards/CardImage.vue
index ef14af9..809aa8b 100644
--- a/src/components/Cards/CardImage.vue
+++ b/src/components/Cards/CardImage.vue
@@ -2,8 +2,7 @@
   <img :src="imageURL" />
 </template>
 
-<style lang="scss" scoped>
-</style>
+<style lang="scss" scoped></style>
 
 <script lang="ts">
 import { Component, Vue, Prop } from "vue-property-decorator";
diff --git a/src/mlpccg/draft/session.ts b/src/mlpccg/draft/session.ts
index c3e9d02..82f1aae 100644
--- a/src/mlpccg/draft/session.ts
+++ b/src/mlpccg/draft/session.ts
@@ -57,7 +57,7 @@ export class Session extends EventEmitter {
 
     // Place players in the pod
     switch (this.options.spacing) {
-      case "evenly":
+      case "evenly": {
         const playerRatio = spots / playerNum;
         let i = 0;
         for (const player of players) {
@@ -67,16 +67,15 @@ export class Session extends EventEmitter {
           i += 1;
         }
         break;
+      }
       case "randomly":
         for (const player of players) {
-          while (true) {
-            const idx = Math.floor(Math.random() * spots);
-            if (this.pod[idx].name == "") {
-              this.pod[idx].name = player;
-              assignFn(name, this.pod[idx]);
-              break;
-            }
-          }
+          const free = [...Array(spots).keys()].filter(
+            i => this.pod[i].name == ""
+          );
+          const idx = Math.floor(Math.random() * free.length);
+          const chosen = free[idx];
+          assignFn(player, this.pod[chosen]);
         }
         break;
     }
@@ -179,10 +178,11 @@ export class Session extends EventEmitter {
         const provider = DraftProvider.set(factory, options.packs);
         return new Session(options, provider);
       }
-      case "i8pcube":
+      case "i8pcube": {
         const cube = await I8PCube.fromURL(options.url);
         const provider = new DraftProvider(cube.schema());
         return new Session(options, provider);
+      }
       default:
         throw new Error("Unknown draft source");
     }
diff --git a/src/mlpccg/images.ts b/src/mlpccg/images.ts
index 66d3102..7bebabe 100644
--- a/src/mlpccg/images.ts
+++ b/src/mlpccg/images.ts
@@ -12,12 +12,13 @@ export async function cardImageURL(cardid: string): Promise<string> {
     return remoteImageURL(cardid);
   }
   switch (cardImageSource()) {
-    case "local":
+    case "local": {
       const card = await Database.images.get(`${cardid}.webp`);
       if (!card) {
         return remoteImageURL(cardid);
       }
       return URL.createObjectURL(card.image);
+    }
     //TODO
     case "remote":
       return remoteImageURL(cardid);
diff --git a/src/tests/unit/.eslintrc.js b/src/tests/unit/.eslintrc.js
index 013a195..3600451 100644
--- a/src/tests/unit/.eslintrc.js
+++ b/src/tests/unit/.eslintrc.js
@@ -2,4 +2,4 @@ module.exports = {
   env: {
     jest: true
   }
-}
\ No newline at end of file
+};