Fill out AddToStats handler and test
This patch makes it so we can submit data scouting data to the
database. All tests are updated.
This patch also adds more error checking code to the database code to
make sure we get notified of some important edge cases.
The majority of the patch is written by Sabina. I amended the
`cli_test.py` file.
Signed-off-by: Philipp Schrader <philipp.schrader@gmail.com>
Signed-off-by: Sabina Leaver <100027607@mvla.net>
Change-Id: I9c70b8b02bbd3ff434fbbb857c2ed65f2684a50e
diff --git a/scouting/db/db.go b/scouting/db/db.go
index 5d68e0f..788e6e3 100644
--- a/scouting/db/db.go
+++ b/scouting/db/db.go
@@ -2,6 +2,7 @@
import (
"database/sql"
+ "errors"
"fmt"
_ "github.com/mattn/go-sqlite3"
@@ -102,11 +103,20 @@
fmt.Println(error_)
return (error_)
}
- _, error_ = statement.Exec(s.TeamNumber, s.MatchNumber, s.ShotsMissed, s.UpperGoalShots, s.LowerGoalShots, s.ShotsMissedAuto, s.UpperGoalAuto, s.LowerGoalAuto, s.PlayedDefense, s.Climbing, s.MatchNumber, s.TeamNumber)
+ result, error_ := statement.Exec(s.TeamNumber, s.MatchNumber, s.ShotsMissed, s.UpperGoalShots, s.LowerGoalShots, s.ShotsMissedAuto, s.UpperGoalAuto, s.LowerGoalAuto, s.PlayedDefense, s.Climbing, s.MatchNumber, s.TeamNumber)
if error_ != nil {
fmt.Println(error_)
return (error_)
}
+ numRowsAffected, error_ := result.RowsAffected()
+ if error_ != nil {
+ return errors.New(fmt.Sprint("Failed to query rows affected: ", error_))
+ }
+ if numRowsAffected == 0 {
+ return errors.New(fmt.Sprint(
+ "Failed to find team ", s.TeamNumber,
+ " in match ", s.MatchNumber, " in the schedule."))
+ }
return nil
}
@@ -128,7 +138,10 @@
}
func (database *Database) ReturnStats() ([]Stats, error) {
- rows, _ := database.Query("SELECT * FROM team_match_stats")
+ rows, error_ := database.Query("SELECT * FROM team_match_stats")
+ if error_ != nil {
+ return nil, errors.New(fmt.Sprint("Failed to SELECT * FROM team_match_stats: ", error_))
+ }
defer rows.Close()
teams := make([]Stats, 0)
var id int