Stefan S

Results 43 comments of Stefan S

> > > Finally, RaspyFM 👍 > > Can you change the license to MPLv2? > > Also, you do have the posibility to make a unit test for it...

> > > This is my 433gpio unittest: > https://github.com/pilight/pilight/blob/rewrite/tests/hardware_433gpio_receive.c How do I run this unittest?

>Also, you do have the posibility to make a unit test for it as well on the rewrite branch? Why having 2 PRs with the same content? What is the...

>https://github.com/pilight/pilight/blob/rewrite/ci/travis-script.sh Running this on the PI, nothing happens >https://github.com/pilight/pilight/blob/rewrite/tests/hardware_433gpio_receive.c Sorry I did not really understand how this works. How could I write a unit test for the raspyrfm?

Thank you for the detailed explaination! I see a bit clearer now. However writing a unittest for the RaspyRFM could be complex as I would have to emulate the behavior...

Running the unittest for the gpio I get: `sudo LD_PRELOAD=libgpio.so ./pilight-unittest test_hardware_433gpio_receive [ test_hardware_433gpio_receive ] F There was 1 failure: 1) test_hardware_433gpio_receive: /home/pi/dev/pilight/tests/hardware_433gpio_receive.c:226: assert failed !!!FAILURES!!! Runs: 1 Passes: 0...

I tried to implement a skeleton of a unittest for the RaspyRFM but the test does not appear in the list. What did I miss? See https://github.com/Phunkafizer/pilight/commit/1ac80b544c55f44545535f25e34a207dd9b68dea

Ok, I updated my branch https://github.com/Phunkafizer/pilight/commit/b0ba17a7d62da69d64151e584d1074443dbd35a6 Meanwhile my hardwaremodule's init is called but the unittest failes at this point: `if(config_setting_get_string("gpio-platform", 0, &platform) != 0) { logprintf(LOG_ERR, "no gpio-platform configured"); return...

Meanwhile my IRQ routine in the hardwaremodule is called by the unittest. >Also, writing i2c unittests isn't that difficult as i2c can be emulated pretty easily. The RaspyRfm is actually...

Thank you very much for the snipped! I'll give it a try! Later this extension will be included in the current libgpio?