Skip to content
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

PsFacebook seems not to support UTF8 content #876

Closed
victornoel opened this issue Nov 28, 2018 · 22 comments
Closed

PsFacebook seems not to support UTF8 content #876

victornoel opened this issue Nov 28, 2018 · 22 comments
Labels

Comments

@victornoel
Copy link
Contributor

Since PsFacebookTest relies on a random id to do its test, sometimes it retrieves UTF8 content which can't be parsed by Takes and/or restfb:

canLogin(org.takes.facets.auth.social.PsFacebookTest)  Time elapsed: 0.056 sec  <<< ERROR!
java.lang.IllegalArgumentException: com.restfb.exception.FacebookJsonMappingException: Unable to map JSON to Java. Offending JSON is '{"id":"JD6U6vubd1","name":"ⲉ鯨ɯ摓㻦恛҉祐፲"}'.
	at org.takes.facets.auth.social.PsFacebook.fetch(PsFacebook.java:199)
	at org.takes.facets.auth.social.PsFacebook.enter(PsFacebook.java:157)
	at org.takes.facets.auth.social.PsFacebookTest.canLogin(PsFacebookTest.java:83)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
	at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
	at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
	at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
	at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:325)
	at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:78)
	at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:57)
	at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
	at org.apache.maven.surefire.junitcore.pc.Scheduler$1.run(Scheduler.java:393)
	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at java.lang.Thread.run(Thread.java:748)
Caused by: com.restfb.exception.FacebookJsonMappingException: Unable to map JSON to Java. Offending JSON is '{"id":"JD6U6vubd1","name":"ⲉ鯨ɯ摓㻦恛҉祐፲"}'.
	at com.restfb.DefaultJsonMapper.toJavaObject(DefaultJsonMapper.java:333)
	at com.restfb.DefaultFacebookClient.fetchObject(DefaultFacebookClient.java:408)
	at org.takes.facets.auth.social.PsFacebook.fetch(PsFacebook.java:195)
	... 20 more
Caused by: com.restfb.json.ParseException: Expected valid string character at 1:35
	at com.restfb.json.JsonParser.error(JsonParser.java:370)
	at com.restfb.json.JsonParser.expected(JsonParser.java:363)
	at com.restfb.json.JsonParser.readStringInternal(JsonParser.java:199)
	at com.restfb.json.JsonParser.readString(JsonParser.java:187)
	at com.restfb.json.JsonParser.readValue(JsonParser.java:84)
	at com.restfb.json.JsonParser.readObject(JsonParser.java:139)
	at com.restfb.json.JsonParser.readValue(JsonParser.java:88)
	at com.restfb.json.JsonParser.parse(JsonParser.java:67)
	at com.restfb.json.Json.parse(Json.java:289)
	at com.restfb.DefaultJsonMapper.toJavaObject(DefaultJsonMapper.java:263)
	... 22 more

I think Takes should be improved to be able to parse such content. So the test shouldn't simply be modified to pass, more tests should be added and the behaviour fixed.

@0crat
Copy link
Collaborator

0crat commented Nov 28, 2018

@paulodamaso/z please, pay attention to this issue

@0crat
Copy link
Collaborator

0crat commented Nov 29, 2018

Job #876 is now in scope, role is DEV

@0crat
Copy link
Collaborator

0crat commented Nov 29, 2018

Bug was reported, see §29: +15 point(s) just awarded to @victornoel/z

@0crat
Copy link
Collaborator

0crat commented Nov 30, 2018

The job #876 assigned to @krzyk/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be a monetary reward for this job

@0crat
Copy link
Collaborator

0crat commented Dec 5, 2018

@krzyk/z this job was assigned to you 5days ago. It will be taken away from you soon, unless you close it, see §8. Read this and this, please.

krzyk added a commit to krzyk/takes that referenced this issue Dec 7, 2018
@krzyk
Copy link
Contributor

krzyk commented Dec 7, 2018

@0crat waiting for review of PR #885

@0crat
Copy link
Collaborator

0crat commented Dec 7, 2018

@0crat waiting for review of PR #885 (here)

@krzyk The impediment for #876 was registered successfully by @krzyk/z

krzyk added a commit to krzyk/takes that referenced this issue Dec 12, 2018
@krzyk
Copy link
Contributor

krzyk commented Dec 12, 2018

@victornoel Fix has been merged in #885, please close this issue

@victornoel
Copy link
Contributor Author

@krzyk thanks!

@0crat
Copy link
Collaborator

0crat commented Dec 12, 2018

@ypshenychka/z please review this job completed by @krzyk/z, as in §30; the job will be fully closed and all payments will be made when the quality review is completed

@0crat 0crat removed the scope label Dec 12, 2018
@0crat
Copy link
Collaborator

0crat commented Dec 12, 2018

The job #876 is now out of scope

@ypshenychka
Copy link

@0crat quality good

@0crat
Copy link
Collaborator

0crat commented Dec 12, 2018

@0crat quality good (here)

@ypshenychka The project doesn't have enough funds, can't make a payment

@0crat
Copy link
Collaborator

0crat commented Dec 12, 2018

@0crat quality good (here)

@ypshenychka The project doesn't have enough funds, can't make a payment

@0crat
Copy link
Collaborator

0crat commented Dec 12, 2018

Order was finished, quality is "good": +35 point(s) just awarded to @krzyk/z

@0crat
Copy link
Collaborator

0crat commented Dec 12, 2018

Quality review completed: +8 point(s) just awarded to @ypshenychka/z

@krzyk
Copy link
Contributor

krzyk commented Dec 13, 2018

@paulodamaso error with funds here? If the estimate was when the project had funds it should pay them also after completion.

@victornoel
Copy link
Contributor Author

@krzyk it seems that 0crat complains but pays anyway if the points are awarded…

@paulodamaso
Copy link
Contributor

@krzyk Looks like QA and REV jobs does not enter on estimates. But in this case I think that you've been assigned before we ran out of funds.

@krzyk
Copy link
Contributor

krzyk commented Dec 13, 2018

@paulodamaso OK, I assume so

@paulodamaso
Copy link
Contributor

@krzyk Have you been paid for this one?

@krzyk
Copy link
Contributor

krzyk commented Dec 13, 2018

@paulodamaso I don't know :) But let's assume I was.
The only way to check that is to look at the user page, but 0crat did payment already, and in paypal comment there is no mention of the issue number :(

Waiting for https://github.com/zerocracy/farm/issues/1556 or https://github.com/zerocracy/farm/issues/1613 to be done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants