Increase the retry limit to 20 times and the interval to 200ms (#5134)

The original settings has less tolerance and would fail on some
environments.

diff --git a/models/test_fixtures.go b/models/test_fixtures.go
index 2a70bc98160be2cbed10aed4c93d5e01138dbf94..8ef9e1af907d1525bc75573f6c3c480650314e9c 100644 (file)
--- a/models/test_fixtures.go
+++ b/models/test_fixtures.go
@@ -5,6 +5,9 @@
 package models
 
 import (
+       "fmt"
+       "time"
+
        "gopkg.in/testfixtures.v2"
 )
 
@@ -21,12 +24,16 @@ func InitFixtures(helper testfixtures.Helper, dir string) (err error) {
 func LoadFixtures() error {
        var err error
        // Database transaction conflicts could occur and result in ROLLBACK
-       // As a simple workaround, we just retry 5 times.
-       for i := 0; i < 5; i++ {
+       // As a simple workaround, we just retry 20 times.
+       for i := 0; i < 20; i++ {
                err = fixtures.Load()
                if err == nil {
                        break
                }
+               time.Sleep(200 * time.Millisecond)
+       }
+       if err != nil {
+               fmt.Printf("LoadFixtures failed after retries: %v\n", err)
        }
        return err
 }