<@UDB0VFBPV> <https://github.com/kittinunf/fuel/is...
# kotlin-fuel
i
@Derk-Jan Karrenbeld https://github.com/kittinunf/fuel/issues/601#issuecomment-467058763 I am noticed result is actually Triple while the first one is from DefaultRequest, the second one is -1 but third one is actually 410..
🤪 1
d
so we just dont assign it correctly once
k
I think it is this line right?
I guess we can fix this quickly
d
Is that the culprit? I don't see any of this code throwing http status exception, so I don't think it's here!
k
🤔 maybe you are right ….
d
I think it happens in the process response thing
transform*
When we validate the status code
k
yeah it is definitely in
SuspendableRequest
file
, oh I lied …
man, I think you should have just used my
Result
instead 🤪
🤣 1
there is one thing i learned as a lesson from this bug is that we should test not only the error is not null but also the value inside the error object as well.
d
This is great!
com.github.[secure].fuel.coroutines.ObjectTest > captureDeserializationException FAILED
java.lang.ClassCastException Caused by: java.lang.ClassCastException
^_^
k
Oh ???!
d
(Probably wrapping or error.xxx issue when there is no response?)
k
d
This is why I wrap in fuelerror everywhere
It could be this, I don't know
you should check the failing test 😛
k
yeah …. it is painful …
maybe my
Result
is not so good after all 😢
d
I think it's nice and fine, but now that there is a native result, it's less helpful
k
OK! I think it should be ready 🙂
@Derk-Jan Karrenbeld when you have time, please send me some love for #603
❤️ 1