diff --git a/arduino/errors.go b/arduino/errors.go index ec60b268ea9..0f30a85dd73 100644 --- a/arduino/errors.go +++ b/arduino/errors.go @@ -837,11 +837,8 @@ type MultiplePlatformsError struct { } func (e *MultiplePlatformsError) Error() string { - return tr("Found %d platform for reference \"%s\":\n%s", - len(e.Platforms), - e.UserPlatform, - strings.Join(e.Platforms, "\n"), - ) + return tr("Found %d platforms matching \"%s\": %s", + len(e.Platforms), e.UserPlatform, strings.Join(e.Platforms, ", ")) } // ToRPCStatus converts the error into a *status.Status diff --git a/internal/cli/arguments/reference.go b/internal/cli/arguments/reference.go index 696c963cc2b..a870a0616a2 100644 --- a/internal/cli/arguments/reference.go +++ b/internal/cli/arguments/reference.go @@ -116,11 +116,13 @@ func ParseReference(arg string) (*Reference, error) { } // replace the returned Reference only if only one occurrence is found, // otherwise return an error to the user because we don't know on which platform operate - if len(foundPlatforms) == 1 { - ret.PackageName = toks[0] - ret.Architecture = toks[1] - } else { + if len(foundPlatforms) == 0 { + return nil, &arduino.PlatformNotFoundError{Platform: arg} + } + if len(foundPlatforms) > 1 { return nil, &arduino.MultiplePlatformsError{Platforms: foundPlatforms, UserPlatform: arg} } + ret.PackageName = toks[0] + ret.Architecture = toks[1] return ret, nil } diff --git a/internal/integrationtest/core/core_test.go b/internal/integrationtest/core/core_test.go index a6a2fe17dea..40742676fa5 100644 --- a/internal/integrationtest/core/core_test.go +++ b/internal/integrationtest/core/core_test.go @@ -883,7 +883,7 @@ func TestCoreDownloadMultiplePlatforms(t *testing.T) { // The cli should not allow it since optimizing the casing results in finding two cores _, stderr, err := cli.Run("core", "upgrade", "Packager:Arch") require.Error(t, err) - require.Contains(t, string(stderr), "Invalid argument passed: Found 2 platform for reference") + require.Contains(t, string(stderr), "Invalid argument passed: Found 2 platforms matching") } func TestCoreWithMissingCustomBoardOptionsIsLoaded(t *testing.T) {