search

CrossTheRoadElec / Phoenix-api

CTRE Phoenix language API (targets FIRST Robotics Competition, Linux, RaspPi, Windows)
Other
39 stars 28 forks source link

Unit testing #55

Closed rgreenblatt closed 3 years ago

rgreenblatt commented 6 years ago

Main 5.8 pr. Specifically solves: https://github.com/CrossTheRoadElectronics/Phoenix-IssueTracker/issues/502 https://github.com/CrossTheRoadElectronics/Phoenix-IssueTracker/issues/499 https://github.com/CrossTheRoadElectronics/Phoenix-IssueTracker/issues/484 https://github.com/CrossTheRoadElectronics/Phoenix-IssueTracker/issues/501 https://github.com/CrossTheRoadElectronics/Phoenix-IssueTracker/issues/490

Implements part of: CrossTheRoadElectronics/Phoenix-IssueTracker#475 CrossTheRoadElectronics/Phoenix-IssueTracker#488 CrossTheRoadElectronics/Phoenix-IssueTracker#486

rgreenblatt commented 6 years ago

Also now solves https://github.com/CrossTheRoadElectronics/Phoenix-IssueTracker/issues/497 / https://github.com/CrossTheRoadElec/Phoenix-frc-lib/issues/18

tested here: https://github.com/CrossTheRoadElec/Phoenix-frc-lib/pull/55/commits/d56cfbecf7c8f634a0c96edeea3df68a0f8c3a33#diff-29661677fb334a4d55ce2ce9df8cccccR141 (tested that problem reproduces without change and change solves it)

rgreenblatt commented 6 years ago

In this branch, all hal reporting calls are commented out in java. ~This needs to be resolved before release.~

rgreenblatt commented 6 years ago

This pr solves https://github.com/CrossTheRoadElectronics/Phoenix-IssueTracker/issues/492 and https://github.com/CrossTheRoadElectronics/Phoenix-IssueTracker/issues/477

(I forgot to mention this earlier)