chore: more testing

ilikejames · Feb 25, 2023 · e05abe3 · e05abe3
1 parent 74df20b
commit e05abe3
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 6 deletions.
diff --git a/packages/leader/src/ElectionSvc/ElectionSvc.ts b/packages/leader/src/ElectionSvc/ElectionSvc.ts
@@ -153,11 +153,13 @@ export class ElectionSvc {
     }
 
     private countVotes() {
+        this.logger?.debug(loggerName, 'counting votes', JSON.stringify(this.votes))
+
         const result = voteCounter(this.instanceId, this.votes)
         this.electionTimerId = null
         this.hasVoted = false
         this.votes.clear()
-        this.logger?.debug(loggerName, 'Election results', result)
+        this.logger?.debug(loggerName, 'Election results', JSON.stringify(result))
         // Do we need to ensure all agree on the outcome?
         this.events.next({ type: ElectionSvcEventTypes.Complete, payload: result })
     }

diff --git a/packages/test-e2e/src/leadership.spec.ts b/packages/test-e2e/src/leadership.spec.ts
@@ -141,7 +141,7 @@ test.describe.parallel('leadership', () => {
         instances.forEach((instance, i) => {
             logs.set(tabNames[i], [])
             instance.on('console', msg => {
-                log.debug('console', tabNames[i], msg.text())
+                log.debug('console-' + tabNames[i], msg.text())
                 logs.get(tabNames[i])!.push(msg.text())
             })
         })
@@ -173,10 +173,10 @@ test.describe.parallel('leadership', () => {
             expect(followerTabs).toHaveLength(afterInstances.length - 1)
             followerTabs.every(f => expect(f.leader).toEqual(newLeaderTab.iam))
         } catch (e) {
-            log.debug('ERROR')
-            logs.forEach((logs, name) => {
-                log.debug(name, JSON.stringify(logs))
-            })
+            // log.debug('ERROR')
+            // logs.forEach((logs, name) => {
+            //     log.debug(name, JSON.stringify(logs))
+            // })
             throw e
         }
     })