Merge pull request #773 from isucon/feat/check-static-files-first

feat: 静的ファイルのチェックを初期化処理の前に行う
isucon · Dec 7, 2024 · 3ecc7ad · 3ecc7ad
2 parents 1ea6e2b + c255b09
commit 3ecc7ad
Show file tree

Hide file tree

Showing 2 changed files with 28 additions and 24 deletions.
diff --git a/bench/benchmarker/scenario/prevalidation.go b/bench/benchmarker/scenario/prevalidation.go
@@ -15,33 +15,12 @@ import (
 	"github.com/isucon/isucon14/bench/benchrun"
 )
 
-// 実装の検証を行う
-func (s *Scenario) prevalidation(ctx context.Context, client *webapp.Client) error {
-	clientConfig := webapp.ClientConfig{
+func (s *Scenario) validateFrontendFiles(ctx context.Context) error {
+	client, err := webapp.NewClient(webapp.ClientConfig{
 		TargetBaseURL:         s.target,
 		TargetAddr:            s.addr,
 		ClientIdleConnTimeout: 10 * time.Second,
-	}
-
-	if s.skipStaticFileSanityCheck {
-		s.contestantLogger.Info("静的ファイルのチェックをスキップします")
-	} else {
-		if err := validateFrontendFiles(ctx, clientConfig); err != nil {
-			s.contestantLogger.Error("静的ファイルのチェックに失敗しました", slog.String("error", err.Error()))
-			return err
-		}
-	}
-
-	if err := validateInitialData(ctx, clientConfig); err != nil {
-		s.contestantLogger.Error("初期データのチェックに失敗しました", slog.String("error", err.Error()))
-		return err
-	}
-
-	return nil
-}
-
-func validateFrontendFiles(ctx context.Context, clientConfig webapp.ClientConfig) error {
-	client, err := webapp.NewClient(clientConfig)
+	})
 	if err != nil {
 		return err
 	}
@@ -88,6 +67,22 @@ func validateFrontendFiles(ctx context.Context, clientConfig webapp.ClientConfig
 	return nil
 }
 
+// 実装の検証を行う
+func (s *Scenario) prevalidation(ctx context.Context, client *webapp.Client) error {
+	clientConfig := webapp.ClientConfig{
+		TargetBaseURL:         s.target,
+		TargetAddr:            s.addr,
+		ClientIdleConnTimeout: 10 * time.Second,
+	}
+
+	if err := validateInitialData(ctx, clientConfig); err != nil {
+		s.contestantLogger.Error("初期データのチェックに失敗しました", slog.String("error", err.Error()))
+		return err
+	}
+
+	return nil
+}
+
 func validateInitialData(ctx context.Context, clientConfig webapp.ClientConfig) error {
 	validationData := LoadData()
 

diff --git a/bench/benchmarker/scenario/scenario.go b/bench/benchmarker/scenario/scenario.go
@@ -179,6 +179,15 @@ func NewScenario(target, addr, paymentURL string, logger *slog.Logger, reporter
 
 // Prepare はシナリオの初期化処理を行う
 func (s *Scenario) Prepare(ctx context.Context, step *isucandar.BenchmarkStep) error {
+	if s.skipStaticFileSanityCheck {
+		s.contestantLogger.Info("静的ファイルのチェックをスキップします")
+	} else {
+		if err := s.validateFrontendFiles(ctx); err != nil {
+			s.contestantLogger.Error("静的ファイルのチェックに失敗しました", slog.String("error", err.Error()))
+			return err
+		}
+	}
+
 	client, err := webapp.NewClient(webapp.ClientConfig{
 		TargetBaseURL:         s.target,
 		TargetAddr:            s.addr,