Make team number a string
Signed-off-by: Emily Markova <emily.markova@gmail.com>
Change-Id: Ia1cff2cfb854e2202cbcaee6f1fd22fc7efd1c15
diff --git a/scouting/webserver/requests/debug/cli/cli_test.py b/scouting/webserver/requests/debug/cli/cli_test.py
index b944564..d11b506 100644
--- a/scouting/webserver/requests/debug/cli/cli_test.py
+++ b/scouting/webserver/requests/debug/cli/cli_test.py
@@ -101,7 +101,7 @@
# First submit some data to be added to the database.
json_path = write_json_request({
- "team": 100,
+ "team": "100",
"notes": "A very inspiring and useful comment",
"good_driving": True,
"bad_driving": False,
@@ -124,7 +124,7 @@
self.assertIn(
textwrap.dedent("""\
{
- Team: (int32) 100,
+ Team: (string) (len=3) "100",
Notes: (string) (len=35) "A very inspiring and useful comment",
GoodDriving: (bool) true,
BadDriving: (bool) false,
@@ -140,10 +140,10 @@
# First submit some data to be added to the database.
json_path = write_json_request({
- "matchNumber": 100,
- "rank1": 101,
- "rank2": 202,
- "rank3": 303,
+ "match_number": 100,
+ "rank1": "101",
+ "rank2": "202",
+ "rank3": "303",
})
exit_code, _, stderr = run_debug_cli(
["-submitDriverRanking", json_path])
@@ -160,9 +160,9 @@
textwrap.dedent("""\
{
MatchNumber: (int32) 100,
- Rank1: (int32) 101,
- Rank2: (int32) 202,
- Rank3: (int32) 303
+ Rank1: (string) (len=3) "101",
+ Rank2: (string) (len=3) "202",
+ Rank3: (string) (len=3) "303"
}"""), stdout)
def test_request_all_matches(self):
diff --git a/scouting/webserver/requests/messages/request_all_driver_rankings_response.fbs b/scouting/webserver/requests/messages/request_all_driver_rankings_response.fbs
index dcc9712..8ce0ef2 100644
--- a/scouting/webserver/requests/messages/request_all_driver_rankings_response.fbs
+++ b/scouting/webserver/requests/messages/request_all_driver_rankings_response.fbs
@@ -1,10 +1,10 @@
namespace scouting.webserver.requests;
table Ranking {
- matchNumber:int (id: 0);
- rank1:int (id: 1);
- rank2:int (id: 2);
- rank3:int (id: 3);
+ match_number:int (id: 0);
+ rank1:string (id: 1);
+ rank2:string (id: 2);
+ rank3:string (id: 3);
}
table RequestAllDriverRankingsResponse {
diff --git a/scouting/webserver/requests/messages/request_all_notes_response.fbs b/scouting/webserver/requests/messages/request_all_notes_response.fbs
index 70504d3..1afdf25 100644
--- a/scouting/webserver/requests/messages/request_all_notes_response.fbs
+++ b/scouting/webserver/requests/messages/request_all_notes_response.fbs
@@ -1,7 +1,7 @@
namespace scouting.webserver.requests;
table Note {
- team:int (id: 0);
+ team:string (id: 0);
notes:string (id: 1);
good_driving:bool (id: 2);
bad_driving:bool (id: 3);
diff --git a/scouting/webserver/requests/messages/request_notes_for_team.fbs b/scouting/webserver/requests/messages/request_notes_for_team.fbs
index 9bda6d3..aa93a2b 100644
--- a/scouting/webserver/requests/messages/request_notes_for_team.fbs
+++ b/scouting/webserver/requests/messages/request_notes_for_team.fbs
@@ -1,7 +1,7 @@
namespace scouting.webserver.requests;
table RequestNotesForTeam {
- team:int (id: 0);
+ team:string (id: 0);
}
root_type RequestNotesForTeam;
diff --git a/scouting/webserver/requests/messages/submit_driver_ranking.fbs b/scouting/webserver/requests/messages/submit_driver_ranking.fbs
index ac1e218..052728c 100644
--- a/scouting/webserver/requests/messages/submit_driver_ranking.fbs
+++ b/scouting/webserver/requests/messages/submit_driver_ranking.fbs
@@ -1,10 +1,10 @@
namespace scouting.webserver.requests;
table SubmitDriverRanking {
- matchNumber:int (id: 0);
- rank1:int (id: 1);
- rank2:int (id: 2);
- rank3:int (id: 3);
+ match_number:int (id: 0);
+ rank1:string (id: 1);
+ rank2:string (id: 2);
+ rank3:string (id: 3);
}
root_type SubmitDriverRanking;
diff --git a/scouting/webserver/requests/messages/submit_notes.fbs b/scouting/webserver/requests/messages/submit_notes.fbs
index 845a601..332b612 100644
--- a/scouting/webserver/requests/messages/submit_notes.fbs
+++ b/scouting/webserver/requests/messages/submit_notes.fbs
@@ -1,7 +1,7 @@
namespace scouting.webserver.requests;
table SubmitNotes {
- team:int (id: 0);
+ team:string (id: 0);
notes:string (id: 1);
good_driving:bool (id: 2);
bad_driving:bool (id: 3);
diff --git a/scouting/webserver/requests/requests.go b/scouting/webserver/requests/requests.go
index 51612ff..af67386 100644
--- a/scouting/webserver/requests/requests.go
+++ b/scouting/webserver/requests/requests.go
@@ -84,7 +84,7 @@
ReturnStats2023() ([]db.Stats2023, error)
ReturnStats2023ForTeam(teamNumber string, matchNumber int32, setNumber int32, compLevel string, preScouting bool) ([]db.Stats2023, error)
QueryAllShifts(int) ([]db.Shift, error)
- QueryNotes(int32) ([]string, error)
+ QueryNotes(string) ([]string, error)
QueryPitImages(string) ([]db.RequestedPitImage, error)
AddNotes(db.NotesData) error
AddPitImage(db.PitImage) error
@@ -367,7 +367,7 @@
}
err = handler.db.AddNotes(db.NotesData{
- TeamNumber: request.Team(),
+ TeamNumber: string(request.Team()),
Notes: string(request.Notes()),
GoodDriving: bool(request.GoodDriving()),
BadDriving: bool(request.BadDriving()),
@@ -671,7 +671,7 @@
return
}
- notes, err := handler.db.QueryNotes(request.Team())
+ notes, err := handler.db.QueryNotes(string(request.Team()))
if err != nil {
respondWithError(w, http.StatusInternalServerError, fmt.Sprintf("Failed to query notes: %v", err))
return
@@ -790,9 +790,9 @@
err = handler.db.AddDriverRanking(db.DriverRankingData{
MatchNumber: request.MatchNumber(),
- Rank1: request.Rank1(),
- Rank2: request.Rank2(),
- Rank3: request.Rank3(),
+ Rank1: string(request.Rank1()),
+ Rank2: string(request.Rank2()),
+ Rank3: string(request.Rank3()),
})
if err != nil {
diff --git a/scouting/webserver/requests/requests_test.go b/scouting/webserver/requests/requests_test.go
index e50583b..0797bc1 100644
--- a/scouting/webserver/requests/requests_test.go
+++ b/scouting/webserver/requests/requests_test.go
@@ -449,7 +449,7 @@
builder := flatbuffers.NewBuilder(1024)
builder.Finish((&submit_notes.SubmitNotesT{
- Team: 971,
+ Team: "971",
Notes: "Notes",
GoodDriving: true,
BadDriving: false,
@@ -467,7 +467,7 @@
expected := []db.NotesData{
{
- TeamNumber: 971,
+ TeamNumber: "971",
Notes: "Notes",
GoodDriving: true,
BadDriving: false,
@@ -487,7 +487,7 @@
func TestRequestNotes(t *testing.T) {
database := MockDatabase{
notes: []db.NotesData{{
- TeamNumber: 971,
+ TeamNumber: "971A",
Notes: "Notes",
GoodDriving: true,
BadDriving: false,
@@ -505,7 +505,7 @@
builder := flatbuffers.NewBuilder(1024)
builder.Finish((&request_notes_for_team.RequestNotesForTeamT{
- Team: 971,
+ Team: "971A",
}).Pack(builder))
response, err := debug.RequestNotes("http://localhost:8080", builder.FinishedBytes())
if err != nil {
@@ -714,9 +714,9 @@
builder := flatbuffers.NewBuilder(1024)
builder.Finish((&submit_driver_ranking.SubmitDriverRankingT{
MatchNumber: 36,
- Rank1: 1234,
- Rank2: 1235,
- Rank3: 1236,
+ Rank1: "1234",
+ Rank2: "1235",
+ Rank3: "1236",
}).Pack(builder))
_, err := debug.SubmitDriverRanking("http://localhost:8080", builder.FinishedBytes())
@@ -725,7 +725,7 @@
}
expected := []db.DriverRankingData{
- {MatchNumber: 36, Rank1: 1234, Rank2: 1235, Rank3: 1236},
+ {MatchNumber: 36, Rank1: "1234", Rank2: "1235", Rank3: "1236"},
}
if !reflect.DeepEqual(database.driver_ranking, expected) {
@@ -739,15 +739,15 @@
driver_ranking: []db.DriverRankingData{
{
MatchNumber: 36,
- Rank1: 1234,
- Rank2: 1235,
- Rank3: 1236,
+ Rank1: "1234",
+ Rank2: "1235",
+ Rank3: "1236",
},
{
MatchNumber: 36,
- Rank1: 101,
- Rank2: 202,
- Rank3: 303,
+ Rank1: "101",
+ Rank2: "202",
+ Rank3: "303",
},
},
}
@@ -768,15 +768,15 @@
DriverRankingList: []*request_all_driver_rankings_response.RankingT{
{
MatchNumber: 36,
- Rank1: 1234,
- Rank2: 1235,
- Rank3: 1236,
+ Rank1: "1234",
+ Rank2: "1235",
+ Rank3: "1236",
},
{
MatchNumber: 36,
- Rank1: 101,
- Rank2: 202,
- Rank3: 303,
+ Rank1: "101",
+ Rank2: "202",
+ Rank3: "303",
},
},
}
@@ -795,7 +795,7 @@
db := MockDatabase{
notes: []db.NotesData{
{
- TeamNumber: 971,
+ TeamNumber: "971",
Notes: "Notes",
GoodDriving: true,
BadDriving: false,
@@ -806,7 +806,7 @@
EasilyDefended: false,
},
{
- TeamNumber: 972,
+ TeamNumber: "972",
Notes: "More Notes",
GoodDriving: false,
BadDriving: false,
@@ -834,7 +834,7 @@
expected := request_all_notes_response.RequestAllNotesResponseT{
NoteList: []*request_all_notes_response.NoteT{
{
- Team: 971,
+ Team: "971",
Notes: "Notes",
GoodDriving: true,
BadDriving: false,
@@ -845,7 +845,7 @@
EasilyDefended: false,
},
{
- Team: 972,
+ Team: "972",
Notes: "More Notes",
GoodDriving: false,
BadDriving: false,
@@ -1137,7 +1137,7 @@
return results, nil
}
-func (database *MockDatabase) QueryNotes(requestedTeam int32) ([]string, error) {
+func (database *MockDatabase) QueryNotes(requestedTeam string) ([]string, error) {
var results []string
for _, data := range database.notes {
if data.TeamNumber == requestedTeam {