stlehmann / pyads

Python wrapper for TwinCAT ADS
MIT License
247 stars 93 forks source link

Test for pull request #304 (Added try-catch to notification close in symbol destructor) #369

Open chhinze opened 5 months ago

chhinze commented 5 months ago

This is an extension to #304 (fixing #303) adding a test for the specific behavior. I have this problem repeatedly, when using auto_update=true for AdsSymbols and the server disconnects and, afterwards the AdsSymbols instances get garbage-collected.

I would love to see pull request #304 merged, so I added the requested test. The additional commit is directly added on top of the commit in the branch in #304. Without the try-catch in #304 the test fails. I hope the test is useful.

Is this what you had in mind @stlehmann?