Skip to content

Commit c235cd4

Browse files
committed
fix broken tests
1 parent b4a3052 commit c235cd4

File tree

3 files changed

+41
-3
lines changed

3 files changed

+41
-3
lines changed

components/gantry/fake/gantry.go

Lines changed: 23 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,8 @@ package fake
44
import (
55
"context"
66

7+
"github.com/golang/geo/r3"
8+
79
"go.viam.com/rdk/components/gantry"
810
"go.viam.com/rdk/logging"
911
"go.viam.com/rdk/referenceframe"
@@ -30,11 +32,29 @@ func init() {
3032

3133
// NewGantry returns a new fake gantry.
3234
func NewGantry(name resource.Name, logger logging.Logger) gantry.Gantry {
33-
m, err := referenceframe.KinematicModelFromFile("../test_gantry_model.json", "test_gantry_model")
35+
m := referenceframe.NewSimpleModel("test_gantry")
36+
pose := spatialmath.NewZeroPose()
37+
baseRailGeom, err := spatialmath.NewBox(pose, r3.Vector{500, 100, 100}, "base_rail")
38+
if err != nil {
39+
logger.CErrorf(context.Background(), "could not create base rail geometry: %v", err)
40+
}
41+
f, err := referenceframe.NewStaticFrameWithGeometry("base_rail", pose, baseRailGeom)
3442
if err != nil {
35-
logger.CWarnf(context.Background(), "failed to load kinematics from file: %v", err)
36-
m = nil
43+
logger.CErrorf(context.Background(), "could not create static frame: %v", err)
3744
}
45+
m.SetOrdTransforms(append(m.OrdTransforms(), f))
46+
47+
carriageGeom, err := spatialmath.NewBox(pose, r3.Vector{150, 120, 10}, "carriage")
48+
if err != nil {
49+
logger.CErrorf(context.Background(), "could not create carriage geometry: %v", err)
50+
}
51+
f, err = referenceframe.NewTranslationalFrameWithGeometry(
52+
"carriage", r3.Vector{1, 0, 0}, referenceframe.Limit{Min: 0, Max: 500}, carriageGeom)
53+
if err != nil {
54+
logger.CErrorf(context.Background(), "could not create translational frame: %v", err)
55+
}
56+
m.SetOrdTransforms(append(m.OrdTransforms(), f))
57+
3858
return &Gantry{
3959
testutils.NewUnimplementedResource(name),
4060
resource.TriviallyReconfigurable{},

components/gantry/multiaxis/multiaxis_test.go

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -48,14 +48,23 @@ func createFakeDeps() resource.Dependencies {
4848
fakeGantry1.LengthsFunc = func(ctx context.Context, extra map[string]interface{}) ([]float64, error) {
4949
return []float64{1}, nil
5050
}
51+
fakeGantry1.KinematicsFunc = func(ctx context.Context) (referenceframe.Model, error) {
52+
return referenceframe.NewSimpleModel(""), nil
53+
}
5154
fakeGantry2 := inject.NewGantry("2")
5255
fakeGantry2.LengthsFunc = func(ctx context.Context, extra map[string]interface{}) ([]float64, error) {
5356
return []float64{1}, nil
5457
}
58+
fakeGantry2.KinematicsFunc = func(ctx context.Context) (referenceframe.Model, error) {
59+
return referenceframe.NewSimpleModel(""), nil
60+
}
5561
fakeGantry3 := inject.NewGantry("3")
5662
fakeGantry3.LengthsFunc = func(ctx context.Context, extra map[string]interface{}) ([]float64, error) {
5763
return []float64{1}, nil
5864
}
65+
fakeGantry3.KinematicsFunc = func(ctx context.Context) (referenceframe.Model, error) {
66+
return referenceframe.NewSimpleModel(""), nil
67+
}
5968
fakeMotor := &fm.Motor{
6069
Named: motor.Named("fm1").AsNamed(),
6170
}
@@ -305,6 +314,7 @@ func TestKinematics(t *testing.T) {
305314
Named: gantry.Named("foo").AsNamed(),
306315
subAxes: twoAxes,
307316
lengthsMm: []float64{1, 1},
317+
model: referenceframe.NewSimpleModel(""),
308318
opMgr: operation.NewSingleOperationManager(),
309319
}
310320
model, err := fakemultiaxis.Kinematics(context.Background())
@@ -315,6 +325,7 @@ func TestKinematics(t *testing.T) {
315325
Named: gantry.Named("foo").AsNamed(),
316326
subAxes: threeAxes,
317327
lengthsMm: []float64{1, 1, 1},
328+
model: referenceframe.NewSimpleModel(""),
318329
opMgr: operation.NewSingleOperationManager(),
319330
}
320331
model, err = fakemultiaxis.Kinematics(context.Background())
@@ -342,6 +353,9 @@ func createComplexDeps() resource.Dependencies {
342353
mAx1.StopFunc = func(ctx context.Context, extra map[string]interface{}) error {
343354
return nil
344355
}
356+
mAx1.KinematicsFunc = func(ctx context.Context) (referenceframe.Model, error) {
357+
return referenceframe.NewSimpleModel("mAx1"), nil
358+
}
345359

346360
position2 := []float64{9, 8, 7}
347361
mAx2 := inject.NewGantry("2")
@@ -362,6 +376,9 @@ func createComplexDeps() resource.Dependencies {
362376
mAx2.StopFunc = func(ctx context.Context, extra map[string]interface{}) error {
363377
return nil
364378
}
379+
mAx2.KinematicsFunc = func(ctx context.Context) (referenceframe.Model, error) {
380+
return referenceframe.NewSimpleModel("mAx2"), nil
381+
}
365382

366383
fakeMotor := &fm.Motor{
367384
Named: motor.Named("foo").AsNamed(),

components/gantry/singleaxis/singleaxis_test.go

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -730,6 +730,7 @@ func TestKinematics(t *testing.T) {
730730
Board: boardName,
731731
LimitPinEnabled: &setTrue,
732732
GantryMmPerSec: float64(300),
733+
Kinematics: "../test_gantry_model.json",
733734
},
734735
}
735736
fakegantry, _ := newSingleAxis(ctx, deps, fakecfg, logger)

0 commit comments

Comments
 (0)