Skip to content

Commit 417e951

Browse files
alessio-peruginicmaglie
alessio-perugini
authored and
cmaglie
committed
use detector FindIncludes in tests
1 parent ea37d54 commit 417e951

6 files changed

+28
-28
lines changed

‎legacy/builder/builder.go

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -223,7 +223,7 @@ func RunPreprocess(ctx *types.Context) error {
223223

224224
func findIncludes(ctx *types.Context) types.BareCommand {
225225
return types.BareCommand(func(ctx *types.Context) error {
226-
ctx.SketchLibrariesDetector.FindIncludes(
226+
return ctx.SketchLibrariesDetector.FindIncludes(
227227
ctx.BuildPath,
228228
ctx.BuildProperties.GetPath("build.core.path"),
229229
ctx.BuildProperties.GetPath("build.variant.path"),
@@ -233,6 +233,5 @@ func findIncludes(ctx *types.Context) types.BareCommand {
233233
ctx.BuildProperties,
234234
ctx.TargetPlatform.Platform.Architecture,
235235
)
236-
return nil
237236
})
238237
}

‎legacy/builder/test/builder_test.go

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@ import (
2222
"time"
2323

2424
bldr "github.com/arduino/arduino-cli/arduino/builder"
25+
"github.com/arduino/arduino-cli/arduino/builder/detector"
2526
"github.com/arduino/arduino-cli/arduino/cores/packagemanager"
2627
"github.com/arduino/arduino-cli/arduino/sketch"
2728
"github.com/arduino/arduino-cli/legacy/builder"
@@ -125,14 +126,14 @@ func prepareBuilderTestContext(t *testing.T, ctx *types.Context, sketchPath *pat
125126
}
126127

127128
if !stepToSkip[skipLibraries] {
128-
lm, libsResolver, _, err := bldr.LibrariesLoader(
129+
lm, libsResolver, _, err := detector.LibrariesLoader(
129130
false, nil,
130131
ctx.BuiltInLibrariesDirs, ctx.LibraryDirs, ctx.OtherLibrariesDirs,
131132
ctx.ActualPlatform, ctx.TargetPlatform,
132133
)
133134
NoError(t, err)
134135

135-
ctx.SketchLibrariesDetector = bldr.NewSketchLibrariesDetector(
136+
ctx.SketchLibrariesDetector = detector.NewSketchLibrariesDetector(
136137
lm, libsResolver,
137138
ctx.Verbose,
138139
false,

‎legacy/builder/test/builder_utils_test.go

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ import (
2020
"testing"
2121
"time"
2222

23-
"github.com/arduino/arduino-cli/legacy/builder/builder_utils"
23+
"github.com/arduino/arduino-cli/arduino/builder/utils"
2424
paths "github.com/arduino/go-paths-helper"
2525
"github.com/stretchr/testify/require"
2626
)
@@ -42,7 +42,7 @@ func TestObjFileIsUpToDateObjMissing(t *testing.T) {
4242
sourceFile := tempFile(t, "source")
4343
defer sourceFile.RemoveAll()
4444

45-
upToDate, err := builder_utils.ObjFileIsUpToDate(sourceFile, nil, nil)
45+
upToDate, err := utils.ObjFileIsUpToDate(sourceFile, nil, nil)
4646
NoError(t, err)
4747
require.False(t, upToDate)
4848
}
@@ -54,7 +54,7 @@ func TestObjFileIsUpToDateDepMissing(t *testing.T) {
5454
objFile := tempFile(t, "obj")
5555
defer objFile.RemoveAll()
5656

57-
upToDate, err := builder_utils.ObjFileIsUpToDate(sourceFile, objFile, nil)
57+
upToDate, err := utils.ObjFileIsUpToDate(sourceFile, objFile, nil)
5858
NoError(t, err)
5959
require.False(t, upToDate)
6060
}
@@ -70,7 +70,7 @@ func TestObjFileIsUpToDateObjOlder(t *testing.T) {
7070
sourceFile := tempFile(t, "source")
7171
defer sourceFile.RemoveAll()
7272

73-
upToDate, err := builder_utils.ObjFileIsUpToDate(sourceFile, objFile, depFile)
73+
upToDate, err := utils.ObjFileIsUpToDate(sourceFile, objFile, depFile)
7474
NoError(t, err)
7575
require.False(t, upToDate)
7676
}
@@ -86,7 +86,7 @@ func TestObjFileIsUpToDateObjNewer(t *testing.T) {
8686
depFile := tempFile(t, "dep")
8787
defer depFile.RemoveAll()
8888

89-
upToDate, err := builder_utils.ObjFileIsUpToDate(sourceFile, objFile, depFile)
89+
upToDate, err := utils.ObjFileIsUpToDate(sourceFile, objFile, depFile)
9090
NoError(t, err)
9191
require.True(t, upToDate)
9292
}
@@ -110,7 +110,7 @@ func TestObjFileIsUpToDateDepIsNewer(t *testing.T) {
110110
data := objFile.String() + ": \\\n\t" + sourceFile.String() + " \\\n\t" + headerFile.String()
111111
depFile.WriteFile([]byte(data))
112112

113-
upToDate, err := builder_utils.ObjFileIsUpToDate(sourceFile, objFile, depFile)
113+
upToDate, err := utils.ObjFileIsUpToDate(sourceFile, objFile, depFile)
114114
NoError(t, err)
115115
require.False(t, upToDate)
116116
}
@@ -132,7 +132,7 @@ func TestObjFileIsUpToDateDepIsOlder(t *testing.T) {
132132
res := objFile.String() + ": \\\n\t" + sourceFile.String() + " \\\n\t" + headerFile.String()
133133
depFile.WriteFile([]byte(res))
134134

135-
upToDate, err := builder_utils.ObjFileIsUpToDate(sourceFile, objFile, depFile)
135+
upToDate, err := utils.ObjFileIsUpToDate(sourceFile, objFile, depFile)
136136
NoError(t, err)
137137
require.True(t, upToDate)
138138
}
@@ -156,7 +156,7 @@ func TestObjFileIsUpToDateDepIsWrong(t *testing.T) {
156156
res := sourceFile.String() + ": \\\n\t" + sourceFile.String() + " \\\n\t" + headerFile.String()
157157
depFile.WriteFile([]byte(res))
158158

159-
upToDate, err := builder_utils.ObjFileIsUpToDate(sourceFile, objFile, depFile)
159+
upToDate, err := utils.ObjFileIsUpToDate(sourceFile, objFile, depFile)
160160
NoError(t, err)
161161
require.False(t, upToDate)
162162
}

‎legacy/builder/test/includes_finder_with_regexp_test.go

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ package test
1818
import (
1919
"testing"
2020

21-
"github.com/arduino/arduino-cli/legacy/builder"
21+
"github.com/arduino/arduino-cli/arduino/builder/detector"
2222
"github.com/stretchr/testify/require"
2323
)
2424

@@ -27,13 +27,13 @@ func TestIncludesFinderWithRegExp(t *testing.T) {
2727
"#include <SPI.h>\n" +
2828
"^\n" +
2929
"compilation terminated."
30-
include := builder.IncludesFinderWithRegExp(output)
30+
include := detector.IncludesFinderWithRegExp(output)
3131

3232
require.Equal(t, "SPI.h", include)
3333
}
3434

3535
func TestIncludesFinderWithRegExpEmptyOutput(t *testing.T) {
36-
include := builder.IncludesFinderWithRegExp("")
36+
include := detector.IncludesFinderWithRegExp("")
3737

3838
require.Equal(t, "", include)
3939
}
@@ -43,7 +43,7 @@ func TestIncludesFinderWithRegExpPaddedIncludes(t *testing.T) {
4343
" # include <Wire.h>\n" +
4444
" ^\n" +
4545
"compilation terminated.\n"
46-
include := builder.IncludesFinderWithRegExp(output)
46+
include := detector.IncludesFinderWithRegExp(output)
4747

4848
require.Equal(t, "Wire.h", include)
4949
}
@@ -53,7 +53,7 @@ func TestIncludesFinderWithRegExpPaddedIncludes2(t *testing.T) {
5353
" #\t\t\tinclude <Wire.h>\n" +
5454
" ^\n" +
5555
"compilation terminated.\n"
56-
include := builder.IncludesFinderWithRegExp(output)
56+
include := detector.IncludesFinderWithRegExp(output)
5757

5858
require.Equal(t, "Wire.h", include)
5959
}
@@ -62,7 +62,7 @@ func TestIncludesFinderWithRegExpPaddedIncludes3(t *testing.T) {
6262
output := "/some/path/sketch.ino:1:33: fatal error: SPI.h: No such file or directory\n" +
6363
"compilation terminated.\n"
6464

65-
include := builder.IncludesFinderWithRegExp(output)
65+
include := detector.IncludesFinderWithRegExp(output)
6666

6767
require.Equal(t, "SPI.h", include)
6868
}
@@ -71,7 +71,7 @@ func TestIncludesFinderWithRegExpPaddedIncludes4(t *testing.T) {
7171
output := "In file included from /tmp/arduino_modified_sketch_815412/binouts.ino:52:0:\n" +
7272
"/tmp/arduino_build_static/sketch/regtable.h:31:22: fatal error: register.h: No such file or directory\n"
7373

74-
include := builder.IncludesFinderWithRegExp(output)
74+
include := detector.IncludesFinderWithRegExp(output)
7575

7676
require.Equal(t, "register.h", include)
7777
}

‎legacy/builder/test/libraries_loader_test.go

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -20,8 +20,8 @@ import (
2020
"sort"
2121
"testing"
2222

23+
"github.com/arduino/arduino-cli/arduino/builder/detector"
2324
"github.com/arduino/arduino-cli/arduino/libraries"
24-
"github.com/arduino/arduino-cli/arduino/builder"
2525
"github.com/arduino/arduino-cli/legacy/builder/constants"
2626
"github.com/arduino/arduino-cli/legacy/builder/types"
2727
paths "github.com/arduino/go-paths-helper"
@@ -47,7 +47,7 @@ func TestLoadLibrariesAVR(t *testing.T) {
4747
ctx = prepareBuilderTestContext(t, ctx, nil, "arduino:avr:leonardo")
4848
defer cleanUpBuilderTestContext(t, ctx)
4949

50-
lm, libsResolver, _, err := builder.LibrariesLoader(
50+
lm, libsResolver, _, err := detector.LibrariesLoader(
5151
false, nil,
5252
ctx.BuiltInLibrariesDirs, ctx.LibraryDirs, ctx.OtherLibrariesDirs,
5353
ctx.ActualPlatform, ctx.TargetPlatform,
@@ -151,7 +151,7 @@ func TestLoadLibrariesSAM(t *testing.T) {
151151
ctx = prepareBuilderTestContext(t, ctx, nil, "arduino:sam:arduino_due_x_dbg")
152152
defer cleanUpBuilderTestContext(t, ctx)
153153

154-
lm, libsResolver, _, err := builder.LibrariesLoader(
154+
lm, libsResolver, _, err := detector.LibrariesLoader(
155155
false, nil,
156156
ctx.BuiltInLibrariesDirs, ctx.LibraryDirs, ctx.OtherLibrariesDirs,
157157
ctx.ActualPlatform, ctx.TargetPlatform,
@@ -228,7 +228,7 @@ func TestLoadLibrariesAVRNoDuplicateLibrariesFolders(t *testing.T) {
228228
ctx = prepareBuilderTestContext(t, ctx, nil, "arduino:avr:leonardo")
229229
defer cleanUpBuilderTestContext(t, ctx)
230230

231-
lm, _, _, err := builder.LibrariesLoader(
231+
lm, _, _, err := detector.LibrariesLoader(
232232
false, nil,
233233
ctx.BuiltInLibrariesDirs, ctx.LibraryDirs, ctx.OtherLibrariesDirs,
234234
ctx.ActualPlatform, ctx.TargetPlatform,
@@ -251,7 +251,7 @@ func TestLoadLibrariesMyAVRPlatform(t *testing.T) {
251251
ctx = prepareBuilderTestContext(t, ctx, nil, "my_avr_platform:avr:custom_yun")
252252
defer cleanUpBuilderTestContext(t, ctx)
253253

254-
lm, _, _, err := builder.LibrariesLoader(
254+
lm, _, _, err := detector.LibrariesLoader(
255255
false, nil,
256256
ctx.BuiltInLibrariesDirs, ctx.LibraryDirs, ctx.OtherLibrariesDirs,
257257
ctx.ActualPlatform, ctx.TargetPlatform,

‎legacy/builder/test/unused_compiled_libraries_remover_test.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ package test
1818
import (
1919
"testing"
2020

21-
bldr "github.com/arduino/arduino-cli/arduino/builder"
21+
"github.com/arduino/arduino-cli/arduino/builder/detector"
2222
"github.com/arduino/arduino-cli/arduino/libraries"
2323
"github.com/arduino/arduino-cli/legacy/builder"
2424
"github.com/arduino/arduino-cli/legacy/builder/types"
@@ -37,7 +37,7 @@ func TestUnusedCompiledLibrariesRemover(t *testing.T) {
3737

3838
ctx := &types.Context{}
3939
ctx.LibrariesBuildPath = temp
40-
ctx.SketchLibrariesDetector = bldr.NewSketchLibrariesDetector(
40+
ctx.SketchLibrariesDetector = detector.NewSketchLibrariesDetector(
4141
nil, nil, false, false, false, nil, nil, nil, nil,
4242
)
4343
ctx.SketchLibrariesDetector.AppendImportedLibraries(&libraries.Library{Name: "Bridge"})
@@ -60,7 +60,7 @@ func TestUnusedCompiledLibrariesRemover(t *testing.T) {
6060
func TestUnusedCompiledLibrariesRemoverLibDoesNotExist(t *testing.T) {
6161
ctx := &types.Context{}
6262
ctx.LibrariesBuildPath = paths.TempDir().Join("test")
63-
ctx.SketchLibrariesDetector = bldr.NewSketchLibrariesDetector(
63+
ctx.SketchLibrariesDetector = detector.NewSketchLibrariesDetector(
6464
nil, nil, false, false, false, nil, nil, nil, nil,
6565
)
6666
ctx.SketchLibrariesDetector.AppendImportedLibraries(&libraries.Library{Name: "Bridge"})
@@ -80,7 +80,7 @@ func TestUnusedCompiledLibrariesRemoverNoUsedLibraries(t *testing.T) {
8080
NoError(t, temp.Join("dummy_file").WriteFile([]byte{}))
8181

8282
ctx := &types.Context{}
83-
ctx.SketchLibrariesDetector = bldr.NewSketchLibrariesDetector(
83+
ctx.SketchLibrariesDetector = detector.NewSketchLibrariesDetector(
8484
nil, nil, false, false, false, nil, nil, nil, nil,
8585
)
8686
ctx.LibrariesBuildPath = temp

0 commit comments

Comments
 (0)