Board identification: always check vid.1/pid.1

Even if vid.0/pid.0 is missing.

Fix #456

Author: cmaglie

arduino/cores/packagemanager/identify.go

@@ -55,7 +55,7 @@ func (pm *PackageManager) IdentifyBoard(idProps *properties.Map) []*cores.Board
 			if found {
 				foundBoards = append(foundBoards, board)
 			}
-			if !again {
+			if !again && id > 0 { // Always check id 0 and 1 (https://github.com/arduino/arduino-cli/issues/456)
 				break
 			}
 			id++

arduino/cores/packagemanager/package_manager_test.go

@@ -18,6 +18,7 @@
 package packagemanager_test
 
 import (
+	"fmt"
 	"net/url"
 	"testing"
 
@@ -295,3 +296,23 @@ func TestFindToolsRequiredForBoard(t *testing.T) {
 	}
 	require.Equal(t, bossac18.InstallDir.String(), uploadProperties.Get("runtime.tools.bossac.path"))
 }
+
+func TestIdentifyBoard(t *testing.T) {
+	pm := packagemanager.NewPackageManager(customHardware, customHardware, customHardware, customHardware)
+	pm.LoadHardwareFromDirectory(customHardware)
+
+	identify := func(vid, pid string) []*cores.Board {
+		return pm.IdentifyBoard(properties.NewFromHashmap(map[string]string{
+			"vid": vid, "pid": pid,
+		}))
+	}
+	require.Equal(t, "[arduino:avr:uno]", fmt.Sprintf("%v", identify("0x2341", "0x0001")))
+
+	// Check indexed vid/pid format (vid.0/pid.0)
+	require.Equal(t, "[test:avr:a]", fmt.Sprintf("%v", identify("0x9999", "0x0001")))
+	require.Equal(t, "[test:avr:b]", fmt.Sprintf("%v", identify("0x9999", "0x0002")))
+	require.Equal(t, "[test:avr:c]", fmt.Sprintf("%v", identify("0x9999", "0x0003")))
+	require.Equal(t, "[test:avr:c]", fmt.Sprintf("%v", identify("0x9999", "0x0004")))
+	// https://github.com/arduino/arduino-cli/issues/456
+	require.Equal(t, "[test:avr:d]", fmt.Sprintf("%v", identify("0x9999", "0x0005")))
+}

arduino/cores/packagemanager/testdata/custom_hardware/test/avr/boards.txt

@@ -0,0 +1,17 @@
+a.name=Board A
+a.vid=0x9999
+a.pid=0x0001
+
+b.name=Board B
+b.vid.0=0x9999
+b.pid.0=0x0002
+
+c.name=Board C
+c.vid.0=0x9999
+c.pid.0=0x0003
+c.vid.1=0x9999
+c.pid.1=0x0004
+
+d.name=Board D
+d.vid.1=0x9999
+d.pid.1=0x0005