-
Notifications
You must be signed in to change notification settings - Fork 15
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
Updated for FRITZ-Dect 500 devices #16
Conversation
@@ -39,6 +39,7 @@ public Deserializer() { | |||
} | |||
|
|||
public <T> T parse(String data, Class<T> resultType) { | |||
//LOG.info("parsing data:"+data); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please don't add commented code
//LOG.info("parsing data:"+data); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
has been removed, sorry for that.
public void parseDeviceListFritzDect200() throws IOException { | ||
final String fileContent = Files.readAllLines(Paths.get("src/test/resources/deviceListConnectedFritzDect200Payload.xml")).stream() | ||
.collect(joining("\n")); | ||
new Deserializer().parse(fileContent, DeviceList.class); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for adding tests 👍
I will build a new release soon, maybe on the weekend. |
Published 1.2.1 to jcenter: https://jcenter.bintray.com/com/github/kaklakariada/fritzbox-java-api/1.2.1/ |
While playing around with this lib i noticed that some of the newer devices are not supported.