fix(jsbattle-server): fix winner selection

Author: jamro

packages/jsbattle-server/app/services/LeagueScheduler.service.js

@@ -70,11 +70,13 @@ class LeagueScheduler extends Service {
       }
     });
 
-    let winner = teamList.reduce((winner, current) => {
-      if(current.score > winner.score) {
+    this.logger.info('Battle result: ' + teamList.map((t) => `${t.name} (${t.battleScore})`).join(' vs '))
+
+    let winner = teamList.reduce((best, current) => {
+      if(current.battleScore > best.battleScore) {
         return current;
       } else {
-        return winner;
+        return best;
       }
     }, teamList[0]);
 

packages/jsbattle-server/test/unit/services/LeagueScheduler.spec.js

@@ -188,10 +188,10 @@ describe("Test 'League' service", () => {
 
 		expect(entityId1).toBe('987243');
 		expect(entityId2).toBe('50872');
-		expect(entityWon1).toBe(true);
-		expect(entityWon2).toBe(false);
-	});
+		expect(entityWon1).toBe(false);
+		expect(entityWon2).toBe(true);
 
+	});
 
 	it('should not process battle errors',  async () => {
 		leagueUpdate.mockReset();