RITAccess / JBrick

An accessible Lego Mindstorms NXT programming environment for students and hobbyists.
3 stars 0 forks source link

Get the unit tests compiling #10

Closed f4ngy closed 10 years ago

f4ngy commented 10 years ago

Right now lets focus on just getting them compiling, and then if they don't pass we can update them/fix bugs in the future.

f4ngy commented 10 years ago

I fixed the communications tests.

The hardware tests are failing because the classes they use (i.e. gamepadcontroller) aren't in the project any more. I am confused as to whether or not support for controllers was removed? We should clarify this with Dr. Ludi or a previous developer.

The same seems to be the problem with the UI tests...they're testing the piano functionality and it looks like the piano code has mostly been removed?

The MyTest folder doesn't actually have any unit tests in it. It has some code that a previous developer was using to test out creating the preferences page in SWT. I'm not sure if we're intentionally keeping the code there or if someone forgot to remove it.

retrogamer80s commented 10 years ago

Hi,

Please comment out the controller support for now, with a note that future support will be re evaluated.

Thanks, Steph

Sent from my iPad

On Feb 19, 2014, at 8:47 AM, Lindsey notifications@github.com wrote:

I fixed the communications tests.

The hardware tests are failing because the classes they use (i.e. gamepadcontroller) aren't in the project any more. I am confused as to whether or not support for controllers was removed? We should clarify this with Dr. Ludi or a previous developer.

The MyTest folder doesn't actually have any unit tests in it. It has some code that a previous developer was using to test out creating the preferences page in SWT. I'm not sure if we're intentionally keeping the code there or if someone forgot to remove it.

— Reply to this email directly or view it on GitHub.

f4ngy commented 10 years ago

Okay, in that case I will mark this issue as closed.