Config tests - Githubissues

FRC2706 / 2019-2706-Robot-Code

The main robot code for the FIRST 2019 challenge: Deep Space

MIT License

2 stars 0 forks source link

Config tests #62

Closed KyleRAnderson closed 5 years ago

KyleRAnderson commented 5 years ago

Closes #48 since all classes we have now have tests associated with them.

Summary of Changes

Fixed problems with concurrent access to the STATE_LISTENERS subscription list to avoid issues.
Added more tests for the Config class.
Changed to use a real file for Config tests.
Automatically formatted code.

Testing Performed

Environment: Plyboy. Changes to robot classes were not significant enough to require retest. Also drove around and made sure the camera worked on plyboy.

KyleRAnderson commented 5 years ago

@eandr127 I've got the tests working. It isn't pretty but I figured out the reason they were failing on the Linux build was because of the Config and Robot static stuff not being properly re-initialized for each test.

As a result of these findings I've created #63 to try and fix up the two classes but this PR won't include those changes since they will be quite substantial and will require changes outside of the scope of the intent of this PR.