Sort terms alphabetically in ARBs

convert/poe2arb/converter.go

@@ -7,6 +7,7 @@ import (
 	"fmt"
 	"io"
 	"regexp"
+	"sort"
 	"strings"
 
 	"github.com/leancodepl/poe2arb/convert"
@@ -57,6 +58,15 @@ func (c *Converter) Convert(output io.Writer) error {
 	prefixedRegexp := regexp.MustCompile("(?:([a-zA-Z]+):)?(.*)")
 	var errs []error
 
+	// Sort terms by key alphabetically
+	sort.SliceStable(jsonContents, func(i, j int) bool {
+		a, b := jsonContents[i], jsonContents[j]
+		aKey := prefixedRegexp.FindStringSubmatch(a.Term)[2]
+		bKey := prefixedRegexp.FindStringSubmatch(b.Term)[2]
+
+		return aKey < bKey
+	})
+
 	for _, term := range jsonContents {
 		// Filter by term prefix
 		matches := prefixedRegexp.FindStringSubmatch(term.Term)

convert/poe2arb/converter_test.go

@@ -17,6 +17,9 @@ func TestConverterConvert(t *testing.T) {
 	if err != nil {
 		t.Fatal(err)
 	}
+	if len(paths) == 0 {
+		t.Fatal("no test files found")
+	}
 
 	for _, path := range paths {
 		_, filename := filepath.Split(path)
@@ -44,7 +47,7 @@ func TestConverterConvert(t *testing.T) {
 			expect := string(golden)
 
 			// Actual test
-			actual, err := convert(t, string(source), template, requireResourceAttributes, termPrefix)
+			actual, err := convert(string(source), template, requireResourceAttributes, termPrefix)
 
 			assert.NoError(t, err)
 			assert.Equal(t, expect, actual)
@@ -70,15 +73,15 @@ func TestConverterConvert(t *testing.T) {
 `
 
 	t.Run("issue 41 template", func(t *testing.T) {
-		actual, err := convert(t, issue41Source, true, false, "")
+		actual, err := convert(issue41Source, true, false, "")
 
 		assert.Error(t, err)
 		assert.EqualError(t, err, `decoding term "testPlural" failed: missing "other" plural category`)
 		assert.Equal(t, "", actual)
 	})
 
 	t.Run("issue 41 non-template", func(t *testing.T) {
-		actual, err := convert(t, issue41Source, false, false, "")
+		actual, err := convert(issue41Source, false, false, "")
 
 		assert.NoError(t, err)
 		assert.Equal(t, "{\n    \"@@locale\": \"en\"\n}\n", actual)
@@ -94,7 +97,6 @@ func flutterMustParseLocale(lang string) flutter.Locale {
 }
 
 func convert(
-	t *testing.T,
 	input string,
 	template bool,
 	requireResourceAttributes bool,

convert/testdata/attributes-req-attrs.golden → convert/poe2arb/testdata/attributes-req-attrs.golden

@@ -1,13 +1,13 @@
 {
     "@@locale": "en",
-    "noAttrs": "Foobar",
-    "@noAttrs": {},
     "attrs": "Foo {bar}",
     "@attrs": {
         "placeholders": {
             "bar": {
                 "type": "String"
             }
         }
-    }
+    },
+    "noAttrs": "Foobar",
+    "@noAttrs": {}
 }

convert/testdata/attributes.golden → convert/poe2arb/testdata/attributes.golden

@@ -1,12 +1,12 @@
 {
     "@@locale": "en",
-    "noAttrs": "Foobar",
     "attrs": "Foo {bar}",
     "@attrs": {
         "placeholders": {
             "bar": {
                 "type": "String"
             }
         }
-    }
+    },
+    "noAttrs": "Foobar"
 }