10000 Porting `legacy` tests to new integration-test infra (part 3...) by cmaglie · Pull Request #2300 · arduino/arduino-cli · GitHub
[go: up one dir, main page]
Skip to content

Porting legacy tests to new integration-test infra (part 3...) #2300

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 12 commits into from
Sep 8, 2023
Merged

Porting legacy tests to new integration-test infra (part 3...) #2300

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
ca89ce7
Ported TestCoreCaching to current integration test infra
cmaglie Sep 7, 2023
180d06f
Removed useless TestBuilderEmptySketch
cmaglie Sep 7, 2023
f0025bb
Ported TestBuilderWithBuildPathInSketchDir to current integration tes…
cmaglie Sep 7, 2023
18dfe0d
Ported TestLoadHardware to current integration test infra
cmaglie Sep 7, 2023
8727f67
Moved user_hardware testdata resources
cmaglie Sep 7, 2023
0deba1c
Ported TestLoadHardwareMixingUserHardwareFolder to current integratio…
cmaglie Sep 7, 2023
2fd7651
Removed redundant legacy tests
cmaglie Sep 7, 2023
a040517
Removed useless 'sleep' helper function
cmaglie Sep 7, 2023
45aa68b
Removed unused LoadAndInterpolate function
cmaglie Sep 7, 2023
3d979d3
Removed useless NoError helper function
cmaglie Sep 7, 2023
aa0e293
Grouped tests inside the same correct sub-group
cmaglie Sep 7, 2023
3d8d37d
Ported TestFailIfBuildPathEqualsSketchPath and TestFailIfBuildPathEqu…
cmaglie Sep 7, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Removed useless TestBuilderEmptySketch 
The same features are already tested in a number of other integration tests.
  • Loading branch information
@cmaglie
cmaglie committed Sep 7, 2023
commit 180d06f1f42aa4446cc1011b76d08680f4b40d2b
24 changes: 0 additions & 24 deletions legacy/builder/test/builder_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -146,30 +146,6 @@ func prepareBuilderTestContext(t *testing.T, ctx *types.Context, sketchPath *pat
return ctx
}

func TestBuilderEmptySketch(t *testing.T) {
ctx := prepareBuilderTestContext(t, nil, paths.New("sketch1", "sketch1.ino"), "arduino:avr:uno")
defer cleanUpBuilderTestContext(t, ctx)

// Run builder
command := builder.Builder{}
err := command.Run(ctx)
NoError(t, err)

buildPath := ctx.BuildPath
exist, err := buildPath.Join(constants.FOLDER_CORE, "HardwareSerial.cpp.o").ExistCheck()
NoError(t, err)
require.True(t, exist)
exist, err = buildPath.Join(constants.FOLDER_SKETCH, "sketch1.ino.cpp.o").ExistCheck()
NoError(t, err)
require.True(t, exist)
exist, err = buildPath.Join("sketch1.ino.elf").ExistCheck()
NoError(t, err)
require.True(t, exist)
exist, err = buildPath.Join("sketch1.ino.hex").ExistCheck()
NoError(t, err)
require.True(t, exist)
}

func TestBuilderWithBuildPathInSketchDir(t *testing.T) {
buildPath, err := paths.New("sketch1", "build").Abs()
NoError(t, err)
Expand Down
0