Revert "Fix Duplicate tool bug"

Author: matteosuppo

v2/pkgs/indexes.go

@@ -7,7 +7,6 @@ import (
 	"net/url"
 	"os"
 	"path/filepath"
-	"strings"
 
 	"github.com/arduino/arduino-create-agent/gen/indexes"
 	"github.com/sirupsen/logrus"
@@ -78,17 +77,13 @@ func (c *Indexes) List(context.Context) ([]string, error) {
 		return nil, err
 	}
 
-	res := []string{}
-	for _, file := range files {
-		// Select only files that begin with http
-		if !strings.HasPrefix(file.Name(), "http") {
-			continue
-		}
+	res := make([]string, len(files))
+	for i, file := range files {
 		path, err := url.PathUnescape(file.Name())
 		if err != nil {
 			c.Log.Warn(err)
 		}
-		res = append(res, path)
+		res[i] = path
 	}
 
 	return res, nil

v2/pkgs/indexes_test.go

@@ -6,7 +6,6 @@ import (
 	"net/http"
 	"net/http/httptest"
 	"os"
-	"path/filepath"
 	"strings"
 	"testing"
 
@@ -29,9 +28,6 @@ func TestIndexes(t *testing.T) {
 	}
 	defer os.RemoveAll(tmp)
 
-	// Create extraneous folder in temp folder
-	os.MkdirAll(filepath.Join(tmp, "arduino"), 0755)
-
 	service := pkgs.Indexes{
 		Folder: tmp,
 	}

v2/pkgs/tools.go

@@ -169,15 +169,8 @@ func (c *Tools) install(ctx context.Context, path, url, checksum string) (*tools
 	var buffer bytes.Buffer
 	reader := io.TeeReader(res.Body, &buffer)
 
-	// Cleanup
-	err = os.RemoveAll(filepath.Join(c.Folder, path))
-	if err != nil {
-		return nil, err
-	}
-
 	err = extract.Archive(ctx, reader, c.Folder, rename(path))
 	if err != nil {
-		os.RemoveAll(path)
 		return nil, err
 	}
 

v2/pkgs/tools_test.go

@@ -89,16 +89,6 @@ func TestTools(t *testing.T) {
 		t.Fatalf("expected %d == %d (%s)", len(installed), 1, "len(installed)")
 	}
 
-	// Install the tool again
-	_, err = service.Install(ctx, &tools.ToolPayload{
-		Packager: "arduino",
-		Name:     "avrdude",
-		Version:  "6.0.1-arduino2",
-	})
-	if err != nil {
-		t.Fatal(err)
-	}
-
 	// Remove tool
 	_, err = service.Remove(ctx, &tools.ToolPayload{
 		Packager: "arduino",