Fix TestLoadTools testcase

In commit 7520e66 (Removed no more used "coan" tools), one tool was
removed, but this testcase was not updated accordingly. This commit
updates the testcase with the correct number of tools to expect.

While here, the testcase is expanded to also explicitely check the name
and version number for all tools. This makes the test more complete, but
also documents what tools and versions are expected exactly, which makes
it easier to diagnose problems like this one later.

Signed-off-by: Matthijs Kooijman <[email protected]>

Author: matthijskooijman

src/arduino.cc/builder/test/tools_loader_test.go

@@ -64,7 +64,7 @@ func TestLoadTools(t *testing.T) {
 	NoError(t, err)
 
 	tools := ctx.Tools
-	require.Equal(t, 7, len(tools))
+	require.Equal(t, 6, len(tools))
 
 	sort.Sort(ByToolIDAndVersion(tools))
 
@@ -88,6 +88,10 @@ func TestLoadTools(t *testing.T) {
 	require.Equal(t, "bossac", tools[idx].Name)
 	require.Equal(t, "1.6.1-arduino", tools[idx].Version)
 	require.Equal(t, Abs(t, "./downloaded_tools/bossac/1.6.1-arduino"), tools[idx].Folder)
+	idx++
+	require.Equal(t, "ctags", tools[idx].Name)
+	require.Equal(t, "5.8-arduino10", tools[idx].Version)
+	require.Equal(t, Abs(t, "./downloaded_tools/ctags/5.8-arduino10"), tools[idx].Folder)
 }
 
 func TestLoadToolsWithBoardManagerFolderStructure(t *testing.T) {