Migrate TestCoreListSortedResults from test_core.py to core_test.go

Author: MatteoPologruto

internal/integrationtest/core/core_test.go

@@ -640,3 +640,76 @@ func TestCoreSearchSortedResults(t *testing.T) {
 		"[.[] | .name |=ascii_downcase | .name]").String()
 	require.True(t, strings.HasSuffix(platform, strings.TrimLeft(notSortedDeprecated, "[")))
 }
+
+func TestCoreListSortedResults(t *testing.T) {
+	env, cli := integrationtest.CreateArduinoCLIWithEnvironment(t)
+	defer env.CleanUp()
+
+	// Set up the server to serve our custom index file
+	testIndex := paths.New("..", "testdata", "test_index.json")
+	url := env.HTTPServeFile(8000, testIndex)
+
+	// update custom index
+	_, _, err := cli.Run("core", "update-index", "--additional-urls="+url.String())
+	require.NoError(t, err)
+
+	// install some core for testing
+	_, _, err = cli.Run("core", "install", "test:x86", "Retrokits-RK002:arm", "Package:x86", "--additional-urls="+url.String())
+	require.NoError(t, err)
+
+	// list all with additional url specified
+	stdout, _, err := cli.Run("core", "list", "--additional-urls="+url.String())
+	require.NoError(t, err)
+
+	out := strings.Split(strings.TrimSpace(string(stdout)), "\n")
+	var lines, deprecated, notDeprecated [][]string
+	for i, v := range out {
+		if i > 0 {
+			v = strings.Join(strings.Fields(v), " ")
+			lines = append(lines, strings.SplitN(v, " ", 4))
+		}
+	}
+	require.Len(t, lines, 3)
+	for _, v := range lines {
+		if strings.HasPrefix(v[3], "[DEPRECATED]") {
+			deprecated = append(deprecated, v)
+		} else {
+			notDeprecated = append(notDeprecated, v)
+		}
+	}
+
+	// verify that results are already sorted correctly
+	require.True(t, sort.SliceIsSorted(deprecated, func(i, j int) bool {
+		return strings.ToLower(deprecated[i][3]) < strings.ToLower(deprecated[j][3])
+	}))
+	require.True(t, sort.SliceIsSorted(notDeprecated, func(i, j int) bool {
+		return strings.ToLower(notDeprecated[i][3]) < strings.ToLower(notDeprecated[j][3])
+	}))
+
+	// verify that deprecated platforms are the last ones
+	require.Equal(t, lines, append(notDeprecated, deprecated...))
+
+	// test same behaviour with json output
+	stdout, _, err = cli.Run("core", "list", "--additional-urls="+url.String(), "--format=json")
+	require.NoError(t, err)
+	requirejson.Len(t, stdout, 3)
+
+	// verify that results are already sorted correctly
+	sortedDeprecated := requirejson.Parse(t, stdout).Query(
+		"[ .[] | select(.deprecated == true) | .name |=ascii_downcase | .name ] | sort").String()
+	notSortedDeprecated := requirejson.Parse(t, stdout).Query(
+		"[.[] | select(.deprecated == true) | .name |=ascii_downcase | .name]").String()
+	require.Equal(t, sortedDeprecated, notSortedDeprecated)
+
+	sortedNotDeprecated := requirejson.Parse(t, stdout).Query(
+		"[ .[] | select(.deprecated != true) | .name |=ascii_downcase | .name ] | sort").String()
+	notSortedNotDeprecated := requirejson.Parse(t, stdout).Query(
+		"[.[] | select(.deprecated != true) | .name |=ascii_downcase | .name]").String()
+	require.Equal(t, sortedNotDeprecated, notSortedNotDeprecated)
+
+	// verify that deprecated platforms are the last ones
+	platform := requirejson.Parse(t, stdout).Query(
+		"[.[] | .name |=ascii_downcase | .name]").String()
+	require.True(t, strings.HasSuffix(platform, strings.TrimLeft(notSortedDeprecated, "[")))
+
+}

test/test_core.py

@@ -93,52 +93,6 @@ def test_core_search_update_index_delay(run_command, data_dir):
     assert "Downloading index" not in res.stdout
 
 
-def test_core_list_sorted_results(run_command, httpserver):
-    # Set up the server to serve our custom index file
-    test_index = Path(__file__).parent / "testdata" / "test_index.json"
-    httpserver.expect_request("/test_index.json").respond_with_data(test_index.read_text())
-
-    # update custom index
-    url = httpserver.url_for("/test_index.json")
-    assert run_command(["core", "update-index", f"--additional-urls={url}"])
-
-    # install some core for testing
-    assert run_command(
-        ["core", "install", "test:x86", "Retrokits-RK002:arm", "Package:x86", f"--additional-urls={url}"]
-    )
-
-    # list all with additional url specified
-    result = run_command(["core", "list", f"--additional-urls={url}"])
-    assert result.ok
-
-    lines = [l.strip().split(maxsplit=3) for l in result.stdout.strip().splitlines()][1:]
-    assert len(lines) == 3
-    not_deprecated = [l for l in lines if not l[3].startswith("[DEPRECATED]")]
-    deprecated = [l for l in lines if l[3].startswith("[DEPRECATED]")]
-
-    # verify that results are already sorted correctly
-    assert not_deprecated == sorted(not_deprecated, key=lambda tokens: tokens[3].lower())
-    assert deprecated == sorted(deprecated, key=lambda tokens: tokens[3].lower())
-
-    # verify that deprecated platforms are the last ones
-    assert lines == not_deprecated + deprecated
-
-    # test same behaviour with json output
-    result = run_command(["core", "list", f"--additional-urls={url}", "--format=json"])
-    assert result.ok
-
-    platforms = json.loads(result.stdout)
-    assert len(platforms) == 3
-    not_deprecated = [p for p in platforms if not p.get("deprecated")]
-    deprecated = [p for p in platforms if p.get("deprecated")]
-
-    # verify that results are already sorted correctly
-    assert not_deprecated == sorted(not_deprecated, key=lambda keys: keys["name"].lower())
-    assert deprecated == sorted(deprecated, key=lambda keys: keys["name"].lower())
-    # verify that deprecated platforms are the last ones
-    assert platforms == not_deprecated + deprecated
-
-
 def test_core_list_deprecated_platform_with_installed_json(run_command, httpserver, data_dir):
     # Set up the server to serve our custom index file
     test_index = Path(__file__).parent / "testdata" / "test_index.json"