Skip to content

Commit c6b58ff

Browse files
Modify test_lib.go to test the changes
1 parent 1d1245a commit c6b58ff

File tree

1 file changed

+8
-5
lines changed

1 file changed

+8
-5
lines changed

Diff for: internal/integrationtest/lib/lib_test.go

+8-5
Original file line numberDiff line numberDiff line change
@@ -416,7 +416,7 @@ func TestListWithFqbn(t *testing.T) {
416416
require.NoError(t, err)
417417

418418
// Install core
419-
_, _, err = cli.Run("core", "install", "arduino:avr")
419+
_, _, err = cli.Run("core", "install", "arduino:avr@1.8.6")
420420
require.NoError(t, err)
421421

422422
// Look at the plain text output
@@ -430,7 +430,7 @@ func TestListWithFqbn(t *testing.T) {
430430
require.NoError(t, err)
431431
require.Empty(t, stderr)
432432
lines := strings.Split(strings.TrimSpace(string(stdout)), "\n")
433-
require.Len(t, lines, 2)
433+
require.Len(t, lines, 7)
434434

435435
// Verifies library is compatible
436436
lines[1] = strings.Join(strings.Fields(lines[1]), " ")
@@ -442,11 +442,14 @@ func TestListWithFqbn(t *testing.T) {
442442
stdout, stderr, err = cli.Run("lib", "list", "-b", "arduino:avr:uno", "--format", "json")
443443
require.NoError(t, err)
444444
require.Empty(t, stderr)
445-
requirejson.Len(t, stdout, 1)
445+
requirejson.Len(t, stdout, 6)
446446

447447
// Verifies library is compatible
448-
requirejson.Query(t, stdout, `.[0] | .library | .name`, `"ArduinoJson"`)
449-
requirejson.Query(t, stdout, `.[0] | .library | .compatible_with | ."arduino:avr:uno"`, `true`)
448+
requirejson.Query(t, stdout, `sort_by(.library | .name) | .[0] | .library | .name`, `"ArduinoJson"`)
449+
requirejson.Query(t, stdout, `sort_by(.library | .name) | .[0] | .library | .compatible_with | ."arduino:avr:uno"`, `true`)
450+
451+
// Verifies bundled libs are shown if -b flag is used
452+
requirejson.Parse(t, stdout).Query(`.[] | .library | select(.container_platform=="arduino:[email protected]")`).MustNotBeEmpty()
450453
}
451454

452455
func TestListProvidesIncludesFallback(t *testing.T) {

0 commit comments

Comments
 (0)