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

More complete testing thoughts ... #15

Open
Apollon77 opened this issue Jan 24, 2018 · 2 comments
Open

More complete testing thoughts ... #15

Apollon77 opened this issue Jan 24, 2018 · 2 comments

Comments

@Apollon77
Copy link
Contributor

Hey Sebastian,

I have checked a bit. "Complete" testing for CUL-Net is kind of easy. Setup Net-Server and communication can be handled from both sides.

For serial it is more interesting:
One option is to use the Mock-Features from Serialport ... but as I tested for this we need access to the serialport instance of the cul instance ... because of strict mode and such it is not accessible directly, it would be needed to add some kind of "getSerialport" method ... for testing only ... not great :-(
But only alternative I found would be some kind of virtual serialports using socat or something like this. But this would be needed to be installed and started anytime tests are executed. Not very comfortable

What do you think about the options ?! ;-)

@hobbyquaker
Copy link
Owner

hobbyquaker commented Jan 24, 2018

To be honest - CUL doesn't play a big role in my smarthome anymore. Only 3 temp/humid sensors, a EM for gas and a doorbell sensor are left atm. This is working "rock stable" for years now, so I don't have that much motivation to put much time into the cul module ;)
If you have motivation and time - do it like you think it's best, I will not stand in your way as long as we try our best to keep the module stable and backwards compatible :)

@Apollon77
Copy link
Contributor Author

I personally still have the CUL stick on my CCU with cuxd :-) Other prios till now than change this "running system" :-)
I will postpone the topic for now I think because other things are more important. You could leave this issue open, so we can continue later if needed. There were requests in ioBroker community to maybe add missing stuff ... but don't looked deep into it till now.

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

No branches or pull requests

2 participants