polish legacy test

Author: alessio-perugini

Diff for: arduino/builder/linker.go

@@ -33,15 +33,10 @@ func (b *Builder) Link() error {
 		return nil
 	}
 
-	// TODO can we remove this multiple assignations?
-	objectFilesSketch := b.buildArtifacts.sketchObjectFiles
-	objectFilesLibraries := b.buildArtifacts.librariesObjectFiles
-	objectFilesCore := b.buildArtifacts.coreObjectsFiles
-
 	objectFiles := paths.NewPathList()
-	objectFiles.AddAll(objectFilesSketch)
-	objectFiles.AddAll(objectFilesLibraries)
-	objectFiles.AddAll(objectFilesCore)
+	objectFiles.AddAll(b.buildArtifacts.sketchObjectFiles)
+	objectFiles.AddAll(b.buildArtifacts.librariesObjectFiles)
+	objectFiles.AddAll(b.buildArtifacts.coreObjectsFiles)
 
 	coreDotARelPath, err := b.buildPath.RelTo(b.buildArtifacts.coreArchiveFilePath)
 	if err != nil {

Diff for: arduino/builder/utils/utils_test.go

@@ -16,8 +16,11 @@
 package utils
 
 import (
+	"os"
 	"testing"
+	"time"
 
+	"github.com/arduino/go-paths-helper"
 	"github.com/stretchr/testify/require"
 )
 
@@ -39,3 +42,133 @@ func TestPrintableCommand(t *testing.T) {
 	result := printableCommand(parts)
 	require.Equal(t, correct, result)
 }
+
+func tempFile(t *testing.T, prefix string) *paths.Path {
+	file, err := os.CreateTemp("", prefix)
+	file.Close()
+	require.NoError(t, err)
+	return paths.New(file.Name())
+}
+
+func TestObjFileIsUpToDateObjMissing(t *testing.T) {
+	sourceFile := tempFile(t, "source")
+	defer sourceFile.RemoveAll()
+
+	upToDate, err := ObjFileIsUpToDate(sourceFile, nil, nil)
+	require.NoError(t, err)
+	require.False(t, upToDate)
+}
+
+func TestObjFileIsUpToDateDepMissing(t *testing.T) {
+	sourceFile := tempFile(t, "source")
+	defer sourceFile.RemoveAll()
+
+	objFile := tempFile(t, "obj")
+	defer objFile.RemoveAll()
+
+	upToDate, err := ObjFileIsUpToDate(sourceFile, objFile, nil)
+	require.NoError(t, err)
+	require.False(t, upToDate)
+}
+
+func TestObjFileIsUpToDateObjOlder(t *testing.T) {
+	objFile := tempFile(t, "obj")
+	defer objFile.RemoveAll()
+	depFile := tempFile(t, "dep")
+	defer depFile.RemoveAll()
+
+	time.Sleep(time.Second)
+
+	sourceFile := tempFile(t, "source")
+	defer sourceFile.RemoveAll()
+
+	upToDate, err := ObjFileIsUpToDate(sourceFile, objFile, depFile)
+	require.NoError(t, err)
+	require.False(t, upToDate)
+}
+
+func TestObjFileIsUpToDateObjNewer(t *testing.T) {
+	sourceFile := tempFile(t, "source")
+	defer sourceFile.RemoveAll()
+
+	time.Sleep(time.Second)
+
+	objFile := tempFile(t, "obj")
+	defer objFile.RemoveAll()
+	depFile := tempFile(t, "dep")
+	defer depFile.RemoveAll()
+
+	upToDate, err := ObjFileIsUpToDate(sourceFile, objFile, depFile)
+	require.NoError(t, err)
+	require.True(t, upToDate)
+}
+
+func TestObjFileIsUpToDateDepIsNewer(t *testing.T) {
+	sourceFile := tempFile(t, "source")
+	defer sourceFile.RemoveAll()
+
+	time.Sleep(time.Second)
+
+	objFile := tempFile(t, "obj")
+	defer objFile.RemoveAll()
+	depFile := tempFile(t, "dep")
+	defer depFile.RemoveAll()
+
+	time.Sleep(time.Second)
+
+	headerFile := tempFile(t, "header")
+	defer headerFile.RemoveAll()
+
+	data := objFile.String() + ": \\\n\t" + sourceFile.String() + " \\\n\t" + headerFile.String()
+	depFile.WriteFile([]byte(data))
+
+	upToDate, err := ObjFileIsUpToDate(sourceFile, objFile, depFile)
+	require.NoError(t, err)
+	require.False(t, upToDate)
+}
+
+func TestObjFileIsUpToDateDepIsOlder(t *testing.T) {
+	sourceFile := tempFile(t, "source")
+	defer sourceFile.RemoveAll()
+
+	headerFile := tempFile(t, "header")
+	defer headerFile.RemoveAll()
+
+	time.Sleep(time.Second)
+
+	objFile := tempFile(t, "obj")
+	defer objFile.RemoveAll()
+	depFile := tempFile(t, "dep")
+	defer depFile.RemoveAll()
+
+	res := objFile.String() + ": \\\n\t" + sourceFile.String() + " \\\n\t" + headerFile.String()
+	depFile.WriteFile([]byte(res))
+
+	upToDate, err := ObjFileIsUpToDate(sourceFile, objFile, depFile)
+	require.NoError(t, err)
+	require.True(t, upToDate)
+}
+
+func TestObjFileIsUpToDateDepIsWrong(t *testing.T) {
+	sourceFile := tempFile(t, "source")
+	defer sourceFile.RemoveAll()
+
+	time.Sleep(time.Second)
+
+	objFile := tempFile(t, "obj")
+	defer objFile.RemoveAll()
+	depFile := tempFile(t, "dep")
+	defer depFile.RemoveAll()
+
+	time.Sleep(time.Second)
+
+	headerFile := tempFile(t, "header")
+	defer headerFile.RemoveAll()
+
+	res := sourceFile.String() + ": \\\n\t" + sourceFile.String() + " \\\n\t" + headerFile.String()
+	depFile.WriteFile([]byte(res))
+
+	upToDate, err := ObjFileIsUpToDate(sourceFile, objFile, depFile)
+	require.NoError(t, err)
+	require.False(t, upToDate)
+}

Diff for: internal/integrationtest/compile_4/compile_test.go

@@ -387,13 +387,15 @@ func testBuilderBridgeExample(t *testing.T, env *integrationtest.Environment, cl
 
 		// Simulate a library use in libraries build path
 		require.NoError(t, buildPath.Join("libraries", "SPI").MkdirAll())
+		require.NoError(t, buildPath.Join("libraries", "dummy_file").WriteFile([]byte{}))
 
 		// Build again...
 		_, err = tryBuild(t, env, cli, "arduino:avr:leonardo", &buildOptions{NoClean: true})
 		require.NoError(t, err)
 
 		require.False(t, buildPath.Join("libraries", "SPI").Exist())
 		require.True(t, buildPath.Join("libraries", "Bridge").Exist())
+		require.True(t, buildPath.Join("libraries", "dummy_file").Exist())
 	})
 
 	t.Run("Preprocess", func(t *testing.T) {