-
Notifications
You must be signed in to change notification settings - Fork 56
TestRail API was changed - may you update it to the latest structure? #67
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
Comments
+1 same issue here. If anyone creates a patch for this it would be greatly appreciated |
Seconded, Unable to parse any responses from Test Rail with the latest update to their API. |
+1 same issue here |
@kunal546 @apolaskey @ChenChiaHung as contributors - may you take a look? |
+1
So the root node is not an |
@pwebster-cpi yep. But not only for get_cases. But in some cases the old List is still actual. |
Hi @maksimsarychau, I'm just interesting if you are going to use another library for testrail integration or wait for the fix? |
Hi @kbilokura, I didn't find any up-to-date and with such wide functionality :( If you find something - please let me know. I'm really waiting for a fix but in parallel trying to add direct API calls in my code as a workaround (without this library) |
I just saw this fork with changes made yesterday: I have not yet looked closely at it, but there is a new file named "PageDeserializer.java". Sounds promising... |
Thank you @pwebster-cpi |
The https://github.com/pablo-iglesias/testrail-api-java-client fork worked for me, with a small bug fix re: _next URL's in the Request.java execute() method |
@pwebster-cpi Did you have a chance to check? |
|
I only needed the fix because my baseurl is https://www.demo.test/testing?/api/v2/ so when combined by getURL it would end up being https://www.demo.test/testing?/api/v2//api/v2/some/testrails/url just a problem with not using an actual base baseurl |
Hey, |
Really big thanks @pablo-iglesias. It works! Btw, where to I can order a coffee or cake (beer) for you? :) |
Lol, you are welcome. |
Thanks for your help! |
Sorry for the delay. I have merged this and made a new release v2.0.2. |
Hi. I am having the same problem with the revised version 2.0.2.
The code I use is as below. How should I use it? |
Hello.
TestRail release the new API everywhere last weekend.
Changes are based on https://discuss.gurock.com/t/bulk-api-endpoint-changes-coming-soon/19537
And now all parses from this library stop worked. :(
I got errors like:
Caused by: com.fasterxml.jackson.databind.JsonMappingException: Can not deserialize instance of java.util.ArrayList out of START_OBJECT token
at [Source: (BufferedInputStream); line: 1, column: 1]
at com.fasterxml.jackson.databind.JsonMappingException.from(JsonMappingException.java:164)
at com.fasterxml.jackson.databind.DeserializationContext.mappingException(DeserializationContext.java:691)
at com.fasterxml.jackson.databind.DeserializationContext.mappingException(DeserializationContext.java:685)
at com.fasterxml.jackson.databind.deser.std.CollectionDeserializer.handleNonArray(CollectionDeserializer.java:256)
at com.fasterxml.jackson.databind.deser.std.CollectionDeserializer.deserialize(CollectionDeserializer.java:214)
at com.fasterxml.jackson.databind.deser.std.CollectionDeserializer.deserialize(CollectionDeserializer.java:204)
at com.fasterxml.jackson.databind.deser.std.CollectionDeserializer.deserialize(CollectionDeserializer.java:23)
at com.fasterxml.jackson.databind.ObjectReader._bindAndClose(ObjectReader.java:1270)
at com.fasterxml.jackson.databind.ObjectReader.readValue(ObjectReader.java:865)
at com.codepine.api.testrail.Request.execute(Request.java:164)
The text was updated successfully, but these errors were encountered: