fix: add type assertion safety in tournament result parsing

Author: vfusco

internal/prt/itournament_adapter.go

@@ -45,8 +45,10 @@ func NewITournamentAdapter(
 func (a *ITournamentAdapterImpl) Result(opts *bind.CallOpts) (bool, [32]byte, [32]byte, error) {
 	result, err := a.tournament.ArbitrationResult(opts)
 	// ArbitrationResult reverts when it has finished with no winners
-	if info, ok := ExtractJsonErrorInfo(err); ok && info.HasData && info.Data == TournamentFailedNoWinner {
-		return true, [32]byte{}, [32]byte{}, nil
+	if info, ok := ExtractJsonErrorInfo(err); ok && info.HasData {
+		if dataStr, ok := info.Data.(string); ok && dataStr == TournamentFailedNoWinner {
+			return true, [32]byte{}, [32]byte{}, nil
+		}
 	}
 	return result.Finished, result.WinnerCommitment, result.FinalState, err
 }

internal/prt/prt.go

@@ -502,14 +502,14 @@ func (s *Service) fetchTournamentData(
 		t, err = s.createTournament(ctx, app, epoch, level,
 			parentMatchIDHash, parentTournamentAddress, tournamentAddress)
 		if err != nil {
-			s.Logger.Error("failed to create new tournament", "application", "level", level, app.Name,
+			s.Logger.Error("failed to create new tournament", "level", level, "application", app.Name,
 				"epoch", epoch.Index, "tournament_address", tournamentAddress.String(), "error", err)
 			return err
 		}
 	} else if t.FinishedAtBlock == 0 {
 		err = s.updateTournamentIfFinished(ctx, app, epoch, level, adapter, t, mostRecentBlock)
 		if err != nil {
-			s.Logger.Error("failed to check if tournament was finished", "application", "level", level, app.Name,
+			s.Logger.Error("failed to check if tournament was finished", "level", level, "application", app.Name,
 				"epoch", epoch.Index, "tournament_address", tournamentAddress.String(), "error", err)
 			return err
 		}