From d6cd4d3e260d076dd00e239258dac945f759f427 Mon Sep 17 00:00:00 2001 From: saji Date: Wed, 28 Feb 2024 01:07:28 -0600 Subject: [PATCH] fix tests, add packets stuff for db --- internal/db/db.go | 30 +- internal/db/db_test.go | 15 +- internal/db/migration.go | 19 +- internal/db/migrations/5_add_packets_down.sql | 2 + internal/db/migrations/5_add_packets_up.sql | 13 + internal/db/packets.go | 3 + skylab/skylab_gen.go | 2 +- skylab/skylab_gen_test.go | 270 ++++++------------ skylab/templates/golang_tests.go.tmpl | 3 +- socketcan/socketcan_test.go | 14 +- 10 files changed, 157 insertions(+), 214 deletions(-) create mode 100644 internal/db/migrations/5_add_packets_down.sql create mode 100644 internal/db/migrations/5_add_packets_up.sql create mode 100644 internal/db/packets.go diff --git a/internal/db/db.go b/internal/db/db.go index 4266882..e436001 100644 --- a/internal/db/db.go +++ b/internal/db/db.go @@ -28,16 +28,12 @@ type TelemDb struct { // TelemDbOption lets you customize the behavior of the sqlite database type TelemDbOption func(*TelemDb) error -// this string is used to open the read-write db. -// the extra options improve performance significantly. -const rwDbPathFmt = "file:%s?_journal_mode=wal&mode=rwc&_txlock=immediate&_timeout=10000" - -func OpenTelemDb(path string, options ...TelemDbOption) (tdb *TelemDb, err error) { +// this function is internal use. It actually opens the database, but uses +// a raw path string instead of formatting one like the exported functions. +func openRawDb(rawpath string, options ...TelemDbOption) (tdb *TelemDb, err error) { tdb = &TelemDb{} - - dbStr := fmt.Sprintf(rwDbPathFmt, path) - tdb.db, err = sqlx.Connect("sqlite3", dbStr) + tdb.db, err = sqlx.Connect("sqlite3", rawpath) if err != nil { return } @@ -62,6 +58,17 @@ func OpenTelemDb(path string, options ...TelemDbOption) (tdb *TelemDb, err error return tdb, err } + +// this string is used to open the read-write db. +// the extra options improve performance significantly. +const rwDbPathFmt = "file:%s?_journal_mode=wal&mode=rwc&_txlock=immediate&_timeout=10000" + +// OpenTelemDb opens a new telemetry database at the given path. +func OpenTelemDb(path string, options ...TelemDbOption) (*TelemDb, error) { + dbStr := fmt.Sprintf(rwDbPathFmt, path) + return openRawDb(dbStr, options...) +} + func (tdb *TelemDb) GetVersion() (int, error) { var version int err := tdb.db.Get(&version, "PRAGMA user_version") @@ -79,7 +86,10 @@ const sqlInsertEvent =`INSERT INTO "bus_events" (ts, name, data) VALUES ` // AddEvent adds the bus event to the database. func (tdb *TelemDb) AddEventsCtx(ctx context.Context, events ...skylab.BusEvent) (n int64, err error) { - // + // edge case - zero events. + if len(events) == 0 { + return 0, nil + } n = 0 tx, err := tdb.db.BeginTx(ctx, nil) defer tx.Rollback() @@ -109,7 +119,7 @@ func (tdb *TelemDb) AddEventsCtx(ctx context.Context, events ...skylab.BusEvent) // construct the full statement now sqlStmt = sqlStmt + strings.Join(inserts[:idx], ",") stmt, err := tx.PrepareContext(ctx, sqlStmt) - defer stmt.Close() + // defer stmt.Close() if err != nil { return } diff --git a/internal/db/db_test.go b/internal/db/db_test.go index 9e7de3a..4785393 100644 --- a/internal/db/db_test.go +++ b/internal/db/db_test.go @@ -1,6 +1,7 @@ package db import ( + "fmt" "reflect" "testing" "time" @@ -19,7 +20,6 @@ func GetRandomBusEvent() skylab.BusEvent { Timestamp: time.Now(), Data: &data, } - ev.Id, _ = data.CANId() return ev } @@ -30,11 +30,20 @@ func TestTelemDb(t *testing.T) { t.Run("test opening database", func(t *testing.T) { var err error - tdb, err = OpenTelemDb("file::memory:?cache=shared") + // we use the underlying raw database to avoid the options. + tdb, err = openRawDb("file::memory:?cache=shared") if err != nil { t.Errorf("could not open db: %v", err) } tdb.db.Ping() + res, _ := tdb.db.Query("SELECT name FROM sqlite_master WHERE type='table'") + + var table string + + for res.Next() { + res.Scan(&table) + fmt.Println(table) + } }) t.Run("test inserting bus event", func(t *testing.T) { @@ -63,7 +72,7 @@ func TestTelemDb(t *testing.T) { } for _, tt := range tests { t.Run(tt.name, func(t *testing.T) { - if err := tdb.AddEvents(tt.args.events...); (err != nil) != tt.wantErr { + if _, err := tdb.AddEvents(tt.args.events...); (err != nil) != tt.wantErr { t.Errorf("TelemDb.AddEvents() error = %v, wantErr %v", err, tt.wantErr) } }) diff --git a/internal/db/migration.go b/internal/db/migration.go index ef0c355..05b8684 100644 --- a/internal/db/migration.go +++ b/internal/db/migration.go @@ -87,7 +87,7 @@ func RunMigrations(tdb *TelemDb) (finalVer int, err error) { for _, v := range vers { if v != expectedVer { err = errors.New("missing update between") - return + return 0, err // invalid } expectedVer = v + 1 @@ -97,35 +97,36 @@ func RunMigrations(tdb *TelemDb) (finalVer int, err error) { // now apply the mappings based on current ver. tx, err := tdb.db.Begin() + defer tx.Rollback() if err != nil { - return + return 0, err } for v := currentVer + 1; v <= finalVer; v++ { // attempt to get the "up" migration. mMap, ok := migrations[v] if !ok { err = errors.New("could not find migration for version") - goto rollback + return 0, err } upMigration, ok := mMap["up"] if !ok { err = errors.New("could not get up migration") - goto rollback + return 0, err } upFile, err := migrationsFs.Open(path.Join("migrations", upMigration.FileName)) if err != nil { - goto rollback + return 0, err } upStmt, err := io.ReadAll(upFile) if err != nil { - goto rollback + return 0, err } // open the file name // execute the file. _, err = tx.Exec(string(upStmt)) if err != nil { - goto rollback + return 0, err } } @@ -134,8 +135,4 @@ func RunMigrations(tdb *TelemDb) (finalVer int, err error) { err = tdb.SetVersion(finalVer) return - // yeah, we use goto. Deal with it. -rollback: - tx.Rollback() - return } diff --git a/internal/db/migrations/5_add_packets_down.sql b/internal/db/migrations/5_add_packets_down.sql new file mode 100644 index 0000000..caa4f29 --- /dev/null +++ b/internal/db/migrations/5_add_packets_down.sql @@ -0,0 +1,2 @@ +DROP TABLE "packet_definitions"; +DROP TABLE "field_definitions"; diff --git a/internal/db/migrations/5_add_packets_up.sql b/internal/db/migrations/5_add_packets_up.sql new file mode 100644 index 0000000..c486f63 --- /dev/null +++ b/internal/db/migrations/5_add_packets_up.sql @@ -0,0 +1,13 @@ +CREATE TABLE "packet_definitions" ( + "name" TEXT NOT NULL, + "description" TEXT, + "id" INTEGER NOT NULL, +); + +CREATE TABLE "field_definitions" ( + "name" TEXT NOT NULL, + "subname" TEXT, -- if the data type is a bitfield, we can use subname to identify the bit. + "packet_name" TEXT NOT NULL, + "type" TEXT NOT NULL, + FOREIGN KEY("packet_name") REFERENCES packet_definitions(name) +); diff --git a/internal/db/packets.go b/internal/db/packets.go new file mode 100644 index 0000000..3e17c0f --- /dev/null +++ b/internal/db/packets.go @@ -0,0 +1,3 @@ +package db +// This file implements Packet modelling, which allows us to look up fields by name + diff --git a/skylab/skylab_gen.go b/skylab/skylab_gen.go index f9938c5..96e1191 100644 --- a/skylab/skylab_gen.go +++ b/skylab/skylab_gen.go @@ -1,4 +1,4 @@ -// generated by gen_skylab.go at 2024-02-12 10:42:55.884248761 -0600 CST m=+0.003327461 DO NOT EDIT! +// generated by gen_skylab.go at 2024-02-27 19:26:47.373116343 -0600 CST m=+0.002925968 DO NOT EDIT! package skylab diff --git a/skylab/skylab_gen_test.go b/skylab/skylab_gen_test.go index a1a4c49..33bd1a4 100644 --- a/skylab/skylab_gen_test.go +++ b/skylab/skylab_gen_test.go @@ -28,8 +28,7 @@ func TestJSONBmsMeasurement(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -64,8 +63,7 @@ func TestJSONBatteryStatus(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -100,8 +98,7 @@ func TestJSONBmsKillReason(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -136,8 +133,7 @@ func TestJSONBmsModuleMinMax(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -172,8 +168,7 @@ func TestJSONBmsSoc(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -208,8 +203,7 @@ func TestJSONBmsCapacity(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -244,8 +238,7 @@ func TestJSONBmsCurrentlimit(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -280,8 +273,7 @@ func TestJSONBmsFanInfo(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -316,8 +308,7 @@ func TestJSONBmsSetMinFanSpeed(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -352,8 +343,7 @@ func TestJSONBmsModule(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -388,8 +378,7 @@ func TestJSONBmsChargerResponse(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -424,8 +413,7 @@ func TestJSONChassisIsolationFault(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -460,8 +448,7 @@ func TestJSONBmsImdInfo(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -496,8 +483,7 @@ func TestJSONDashboardPedalPercentages(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -532,8 +518,7 @@ func TestJSONCarState(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -568,8 +553,7 @@ func TestJSONDashboardPedalFault(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -604,8 +588,7 @@ func TestJSONDashboardSystemTimeoutTest(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -640,8 +623,7 @@ func TestJSONCarSpeed(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -676,8 +658,7 @@ func TestJSONFlightComputerLvBoardDisconnectCounts(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -712,8 +693,7 @@ func TestJSONFlightComputerHvBoardDisconnectCounts(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -748,8 +728,7 @@ func TestJSONFlightComputerInternalState(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -784,8 +763,7 @@ func TestJSONPowerToDrive(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -820,8 +798,7 @@ func TestJSONArrayPower(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -856,8 +833,7 @@ func TestJSONArrayEnergy(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -892,8 +868,7 @@ func TestJSONArrayEnergyReset(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -928,8 +903,7 @@ func TestJSONVisionTurnSignalsCommand(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -964,8 +938,7 @@ func TestJSONVisionBrakeLightsCommand(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -1000,8 +973,7 @@ func TestJSONVisionHeadlightsCommand(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -1036,8 +1008,7 @@ func TestJSONVisionHornCommand(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -1072,8 +1043,7 @@ func TestJSONVisionArrayLatchesCommand(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -1108,8 +1078,7 @@ func TestJSONVisionRearviewCommand(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -1144,8 +1113,7 @@ func TestJSONTrackerEnable(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -1180,8 +1148,7 @@ func TestJSONDistanceTraveled(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -1216,8 +1183,7 @@ func TestJSONChargerState(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -1252,8 +1218,7 @@ func TestJSONChargerBmsRequest(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -1288,8 +1253,7 @@ func TestJSONChargerCurrentVoltage(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -1324,8 +1288,7 @@ func TestJSONChargerPower(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -1360,8 +1323,7 @@ func TestJSONThunderstruckControlMessage(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -1396,8 +1358,7 @@ func TestJSONVisionStatusFront(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -1432,8 +1393,7 @@ func TestJSONVisionStatusRear(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -1468,8 +1428,7 @@ func TestJSONLightsFrontId(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -1504,8 +1463,7 @@ func TestJSONLightsBackId(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -1540,8 +1498,7 @@ func TestJSONVisionId(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -1576,8 +1533,7 @@ func TestJSONSteeringPressCount1(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -1612,8 +1568,7 @@ func TestJSONSteeringPressCount2(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -1648,8 +1603,7 @@ func TestJSONSteeringButtonColors1(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -1684,8 +1638,7 @@ func TestJSONSteeringButtonColors2(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -1720,8 +1673,7 @@ func TestJSONSteeringHorn(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -1756,8 +1708,7 @@ func TestJSONThunderstruckStatusMessage(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -1792,8 +1743,7 @@ func TestJSONTrackerData(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -1828,8 +1778,7 @@ func TestJSONTritiumMotorDriveL(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -1864,8 +1813,7 @@ func TestJSONTritiumMotorPowerL(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -1900,8 +1848,7 @@ func TestJSONTritiumResetL(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -1936,8 +1883,7 @@ func TestJSONTritiumMotorDriveR(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -1972,8 +1918,7 @@ func TestJSONTritiumMotorPowerR(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -2008,8 +1953,7 @@ func TestJSONTritiumResetR(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -2044,8 +1988,7 @@ func TestJSONBmsAhSet(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -2080,8 +2023,7 @@ func TestJSONBmsWhSet(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -2116,8 +2058,7 @@ func TestJSONBmsKill(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -2152,8 +2093,7 @@ func TestJSONTelemetryRtcReset(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -2188,8 +2128,7 @@ func TestJSONWsrIdentification(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -2224,8 +2163,7 @@ func TestJSONWsrStatusInformation(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -2260,8 +2198,7 @@ func TestJSONWsrBusMeasurement(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -2296,8 +2233,7 @@ func TestJSONWsrVelocity(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -2332,8 +2268,7 @@ func TestJSONWsrPhaseCurrent(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -2368,8 +2303,7 @@ func TestJSONWsrMotorVoltageVector(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -2404,8 +2338,7 @@ func TestJSONWsrMotorCurrentVector(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -2440,8 +2373,7 @@ func TestJSONWsrMotorBackemf(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -2476,8 +2408,7 @@ func TestJSONWsr15165VoltageRail(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -2512,8 +2443,7 @@ func TestJSONWsr2512VoltageRail(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -2548,8 +2478,7 @@ func TestJSONWsrHeatsinkMotorTemp(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -2584,8 +2513,7 @@ func TestJSONWsrDspBoardTemp(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -2620,8 +2548,7 @@ func TestJSONWsrReserved(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -2656,8 +2583,7 @@ func TestJSONWsrOdometerBusAmphoursMeasurement(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -2692,8 +2618,7 @@ func TestJSONWsrSlipSpeedMeasurement(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -2728,8 +2653,7 @@ func TestJSONWslIdentification(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -2764,8 +2688,7 @@ func TestJSONWslStatusInformation(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -2800,8 +2723,7 @@ func TestJSONWslBusMeasurement(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -2836,8 +2758,7 @@ func TestJSONWslVelocity(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -2872,8 +2793,7 @@ func TestJSONWslPhaseCurrent(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -2908,8 +2828,7 @@ func TestJSONWslMotorVoltageVector(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -2944,8 +2863,7 @@ func TestJSONWslMotorCurrentVector(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -2980,8 +2898,7 @@ func TestJSONWslMotorBackemf(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -3016,8 +2933,7 @@ func TestJSONWsl15165VoltageRail(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -3052,8 +2968,7 @@ func TestJSONWsl2512VoltageRail(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -3088,8 +3003,7 @@ func TestJSONWslHeatsinkMotorTemp(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -3124,8 +3038,7 @@ func TestJSONWslDspBoardTemp(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -3160,8 +3073,7 @@ func TestJSONWslOdometerBusAmphoursMeasurement(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -3196,8 +3108,7 @@ func TestJSONWslReserved(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } @@ -3232,8 +3143,7 @@ func TestJSONWslSlipSpeedMeasurement(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } diff --git a/skylab/templates/golang_tests.go.tmpl b/skylab/templates/golang_tests.go.tmpl index 238ce61..5f6432c 100644 --- a/skylab/templates/golang_tests.go.tmpl +++ b/skylab/templates/golang_tests.go.tmpl @@ -29,8 +29,7 @@ func TestJSON{{$structName}}(t *testing.T) { t.Fatal(err) } - id, _ := v.CANId() - p, err := FromJson(id, rawData) + p, err := FromJson(v.String(), rawData) if err != nil { t.Fatal(err) } diff --git a/socketcan/socketcan_test.go b/socketcan/socketcan_test.go index 79c357e..945b614 100644 --- a/socketcan/socketcan_test.go +++ b/socketcan/socketcan_test.go @@ -7,7 +7,7 @@ import ( "net" "testing" - "github.com/kschamplin/gotelem" + "github.com/kschamplin/gotelem/internal/can" ) func TestCanSocket(t *testing.T) { @@ -43,9 +43,9 @@ func TestCanSocket(t *testing.T) { defer sock.Close() // make a packet. - testFrame := &gotelem.Frame{ - Id: 0x123, - Kind: gotelem.CanSFFFrame, + testFrame := &can.Frame{ + Id: can.CanID{Id: 0x123, Extended: false}, + Kind: can.CanDataFrame, Data: []byte{0, 1, 2, 3, 4, 5, 6, 7}, } err := sock.Send(testFrame) @@ -61,9 +61,9 @@ func TestCanSocket(t *testing.T) { defer sock.Close() defer rsock.Close() - testFrame := &gotelem.Frame{ - Id: 0x234, - Kind: gotelem.CanSFFFrame, + testFrame := &can.Frame{ + Id: can.CanID{Id: 0x234, Extended: false}, + Kind: can.CanDataFrame, Data: []byte{0, 1, 2, 3, 4, 5, 6, 7}, } _ = sock.Send(testFrame)