diff --git a/internal/entity/location_fixtures.go b/internal/entity/location_fixtures.go index fa2c2a869..85603b0f8 100644 --- a/internal/entity/location_fixtures.go +++ b/internal/entity/location_fixtures.go @@ -26,7 +26,7 @@ func (m LocationMap) Pointer(name string) *Location { var LocationFixtures = LocationMap{ "mexico": { - ID: s2.TokenPrefix+"85d1ea7d382c", + ID: s2.TokenPrefix + "85d1ea7d382c", PlaceID: PlaceFixtures.Get("mexico").ID, LocName: "Adosada Platform", LocCategory: "botanical garden", @@ -36,10 +36,10 @@ var LocationFixtures = LocationMap{ UpdatedAt: time.Now(), }, "caravan park": { - ID: s2.TokenPrefix+"1ef75a71a36c", - PlaceID: s2.TokenPrefix+"1ef75a71a36c", + ID: s2.TokenPrefix + "1ef75a71a36c", + PlaceID: s2.TokenPrefix + "1ef75a71a36c", Place: &Place{ - ID: "x1ef75a71a36", + ID: s2.TokenPrefix + "1ef75a71a36", LocLabel: "Mandeni, KwaZulu-Natal, South Africa", LocCity: "Mandeni", LocState: "KwaZulu-Natal", @@ -54,7 +54,7 @@ var LocationFixtures = LocationMap{ UpdatedAt: time.Now(), }, "zinkwazi": { - ID: s2.TokenPrefix+"1ef744d1e28c", + ID: s2.TokenPrefix + "1ef744d1e28c", PlaceID: PlaceFixtures.Get("zinkwazi").ID, Place: PlaceFixtures.Pointer("zinkwazi"), LocName: "Zinkwazi Beach", @@ -64,7 +64,7 @@ var LocationFixtures = LocationMap{ UpdatedAt: time.Now(), }, "hassloch": { - ID: s2.TokenPrefix+"1ef744d1e280", + ID: s2.TokenPrefix + "1ef744d1e280", PlaceID: PlaceFixtures.Get("holidaypark").ID, Place: PlaceFixtures.Pointer("holidaypark"), LocName: "Holiday Park", @@ -74,7 +74,7 @@ var LocationFixtures = LocationMap{ UpdatedAt: time.Now(), }, "emptyNameLongCity": { - ID: s2.TokenPrefix+"1ef744d1e281", + ID: s2.TokenPrefix + "1ef744d1e281", PlaceID: PlaceFixtures.Get("emptyNameLongCity").ID, Place: PlaceFixtures.Pointer("emptyNameLongCity"), LocName: "", @@ -84,7 +84,7 @@ var LocationFixtures = LocationMap{ UpdatedAt: time.Now(), }, "emptyNameShortCity": { - ID: s2.TokenPrefix+"1ef744d1e282", + ID: s2.TokenPrefix + "1ef744d1e282", PlaceID: PlaceFixtures.Get("emptyNameShortCity").ID, Place: PlaceFixtures.Pointer("emptyNameShortCity"), LocName: "", @@ -94,7 +94,7 @@ var LocationFixtures = LocationMap{ UpdatedAt: time.Now(), }, "veryLongLocName": { - ID: s2.TokenPrefix+"1ef744d1e283", + ID: s2.TokenPrefix + "1ef744d1e283", PlaceID: PlaceFixtures.Get("veryLongLocName").ID, Place: PlaceFixtures.Pointer("veryLongLocName"), LocName: "longlonglonglonglonglonglonglonglonglonglonglonglongName", @@ -104,7 +104,7 @@ var LocationFixtures = LocationMap{ UpdatedAt: time.Now(), }, "mediumLongLocName": { - ID: s2.TokenPrefix+"1ef744d1e283", + ID: s2.TokenPrefix + "1ef744d1e283", PlaceID: PlaceFixtures.Get("mediumLongLocName").ID, Place: PlaceFixtures.Pointer("mediumLongLocName"), LocName: "longlonglonglonglonglongName", diff --git a/internal/entity/place_fixtures.go b/internal/entity/place_fixtures.go index de05b522e..0ecc846f3 100644 --- a/internal/entity/place_fixtures.go +++ b/internal/entity/place_fixtures.go @@ -26,7 +26,7 @@ func (m PlacesMap) Pointer(name string) *Place { var PlaceFixtures = PlacesMap{ "mexico": { - ID: s2.TokenPrefix+"85d1ea7d3278", + ID: s2.TokenPrefix + "85d1ea7d3278", LocLabel: "Teotihuacán, Mexico, Mexico", LocCity: "Teotihuacán", LocState: "State of Mexico", @@ -39,7 +39,7 @@ var PlaceFixtures = PlacesMap{ UpdatedAt: time.Now(), }, "zinkwazi": { - ID: s2.TokenPrefix+"1ef744d1e279", + ID: s2.TokenPrefix + "1ef744d1e279", LocLabel: "KwaDukuza, KwaZulu-Natal, South Africa", LocCity: "KwaDukuza", LocState: "KwaZulu-Natal", @@ -52,7 +52,7 @@ var PlaceFixtures = PlacesMap{ UpdatedAt: time.Now(), }, "holidaypark": { - ID: s2.TokenPrefix+"1ef744d1e280", + ID: s2.TokenPrefix + "1ef744d1e280", LocLabel: "Holiday Park, Amusement", LocCity: "", LocState: "Rheinland-Pfalz", @@ -65,7 +65,7 @@ var PlaceFixtures = PlacesMap{ UpdatedAt: time.Now(), }, "emptyNameLongCity": { - ID: s2.TokenPrefix+"1ef744d1e281", + ID: s2.TokenPrefix + "1ef744d1e281", LocLabel: "labelEmptyNameLongCity", LocCity: "longlonglonglonglongcity", LocState: "Rheinland-Pfalz", @@ -78,7 +78,7 @@ var PlaceFixtures = PlacesMap{ UpdatedAt: time.Now(), }, "emptyNameShortCity": { - ID: s2.TokenPrefix+"1ef744d1e282", + ID: s2.TokenPrefix + "1ef744d1e282", LocLabel: "labelEmptyNameShortCity", LocCity: "shortcity", LocState: "Rheinland-Pfalz", @@ -91,7 +91,7 @@ var PlaceFixtures = PlacesMap{ UpdatedAt: time.Now(), }, "veryLongLocName": { - ID: s2.TokenPrefix+"1ef744d1e283", + ID: s2.TokenPrefix + "1ef744d1e283", LocLabel: "labelVeryLongLocName", LocCity: "Mainz", LocState: "Rheinland-Pfalz", @@ -104,7 +104,7 @@ var PlaceFixtures = PlacesMap{ UpdatedAt: time.Now(), }, "mediumLongLocName": { - ID: s2.TokenPrefix+"1ef744d1e284", + ID: s2.TokenPrefix + "1ef744d1e284", LocLabel: "labelMediumLongLocName", LocCity: "New york", LocState: "New york", diff --git a/internal/entity/place_test.go b/internal/entity/place_test.go index 2a4bf272c..b6291cd2a 100644 --- a/internal/entity/place_test.go +++ b/internal/entity/place_test.go @@ -58,7 +58,7 @@ func TestPlace_Find(t *testing.T) { }) t.Run("record does not exist", func(t *testing.T) { place := &Place{ - ID: s2.TokenPrefix+"1110", + ID: s2.TokenPrefix + "1110", LocLabel: "test", LocCity: "testCity", LocState: "", diff --git a/pkg/s2/prefix.go b/pkg/s2/prefix.go index c115fb4b9..719e98531 100644 --- a/pkg/s2/prefix.go +++ b/pkg/s2/prefix.go @@ -28,7 +28,7 @@ func Prefix(token string) string { return token } - return TokenPrefix+token + return TokenPrefix + token } // PrefixedToken returns the prefixed S2 cell token for coordinates using the default level. diff --git a/pkg/s2/prefix_test.go b/pkg/s2/prefix_test.go index 252e0b909..d30a04bf2 100644 --- a/pkg/s2/prefix_test.go +++ b/pkg/s2/prefix_test.go @@ -9,7 +9,7 @@ import ( func TestNormalizeToken(t *testing.T) { t.Run(TokenPrefix+"1242342bac", func(t *testing.T) { - input := TokenPrefix+"1242342bac" + input := TokenPrefix + "1242342bac" output := NormalizeToken(input) @@ -29,7 +29,7 @@ func TestNormalizeToken(t *testing.T) { func TestPrefix(t *testing.T) { t.Run(TokenPrefix+"1242342bac", func(t *testing.T) { - input := TokenPrefix+"1242342bac" + input := TokenPrefix + "1242342bac" output := Prefix(input) @@ -57,7 +57,7 @@ func TestPrefix(t *testing.T) { func TestPrefixedToken(t *testing.T) { t.Run("germany", func(t *testing.T) { token := PrefixedToken(48.56344833333333, 8.996878333333333) - expected := TokenPrefix+"4799e370" + expected := TokenPrefix + "4799e370" assert.True(t, strings.HasPrefix(token, expected)) })