Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Look into continuous integration and unit testing #14

Open
gbear605 opened this issue Sep 20, 2016 · 3 comments
Open

Look into continuous integration and unit testing #14

gbear605 opened this issue Sep 20, 2016 · 3 comments

Comments

@gbear605
Copy link
Member

gbear605 commented Sep 20, 2016

In 2016 we made minimal use of continuous integration, but it didn't really help. To be more useful in the future, it would need to have more unit tests than the basic ones that I implemented. Those unit tests would require a version of WPILib that doesn't require the presence of the robot. I know that @Team254 has done this in the past, so it might be possible to use their version, or another team's version. If not, we will need to create a version ourselves. I suspect that that difficulty combined with the difficulty of writing unit tests for an environment like FRC will outweigh the benefit of unit tests.

@gbear605 gbear605 changed the title Look into continuous integration for 2017 and unit testing Look into continuous integration and unit testing Sep 20, 2016
@Ipsum
Copy link

Ipsum commented Sep 20, 2016

I am 100% for testing always but without a decent test harness I would have to vote to make this lower priority. We do need to figure out some way of controlling the quality of code even under pressure though.

@arlevin
Copy link
Contributor

arlevin commented Sep 20, 2016

@Ipsum Agreed. It's the 2nd lowest item. @gbear605 please let us know if you disagree

@gbear605
Copy link
Member Author

I agree with it being very low priority.
On Mon, Sep 19, 2016 at 23:02 arlevin notifications@github.com wrote:

@Ipsum https://github.com/Ipsum Agreed. It's the 2nd lowest item.
@gbear605 https://github.com/gbear605 please let us know if you disagree


You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub
#14 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAqKHKjpUrNYZP3srSC20M3FiUmt0KNfks5qr0zAgaJpZM4KBJxm
.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants