fossasia / knitlib

Knitting backend library for knitting machines
GNU Lesser General Public License v3.0
1.54k stars 14 forks source link

Full testsuite for AYAB plugin #19

Open tian2992 opened 9 years ago

tian2992 commented 9 years ago

Will require mocking and emulation of Serial Port

shiluka commented 9 years ago

Hi @tian2992, I am currently working on this issue by creating tests for ayab_image and ayab_communication. Can I work on this issue ?

tian2992 commented 9 years ago

There are two main issues that make this full stack testing hard.

If you only do tests for ayab_communication, bear in mind that this tests will only cover that code instead of being a full stack test. However learning from those tests can be useful and can help to make ayab_communication more generic.

I would also not recommend creating a test for ayab_image as it is up to a certain way, a way more primitive version of the holding object of what can be in the future a Knitpat object.

shiluka commented 9 years ago

ok thanks for the comment.

tian2992 commented 9 years ago

Also please see #25 Cheers :)

shiluka commented 9 years ago

Thanks :)

tian2992 commented 9 years ago

Except for full device emulation, I'd consider this mostly done.