-
Notifications
You must be signed in to change notification settings - Fork 58
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
Problem access device info #15
Comments
So, this does look like it is related to lxml being able to parse URLs directly, in which case the doc string is wrong as are the tests. As we have requests anyway, we should be able to use to to get the resource before handing it on for parsing.
|
OK, the first exception is triggered even by the |
FWIW the way nodes work makes debugging and testing really hard because it's so depedent upon side-effects. I generally find that init s themselves should do nothing that isn't directly testable, ie. encapsulated in a relevant method. |
If you have a better idea about the nodes please let me known. I'd thought about alternatives to implementing it this way, but didn't have had a better idea back then. Instead of rewriting |
I think we can get similar behaviour that is easier to test and debug with minor changes, which I think the current one isn't: the errors are not showing up at parse/scan time so are difficult to pin down. I'd be tempted to use descriptors for the class-specific attributes because they document the API much better and it's not as if you want really, really dynamic classes. |
I like your idea how to use requests in the description module. About desciptors I would like to think twice, how the dynamic nature of the description files can be handled better this way. Unfortunately in the next two weeks I don‘t have time to dive into coding. Hopefully I may have some ideas thought. |
No hurry on either front. I think a separate function for calling the box to get the info is the way to go and we can add logging to it for debugging purposes. |
That's about the entry point, actually |
I like to close this because of replacing nodes.py with processor.py 4e4cfb1 |
I can't replicate that error on the 1.0 branch but I do have trouble connecting even with the correct password from the CLI. Testing on a Fritz 6490 Unitymedia.
Moved from #12
The text was updated successfully, but these errors were encountered: