MkVersionEye.java:37-38: Continue implementing the mock API.... about versioneye-api

[amihaiemil]

The jsonobject builder, the "storage", will be passed through all the mkclasses, just like an http request is passed in the real ones. And each class will add a json object or array in it, or read something from it.

[amihaiemil]

Well, you could do MkMe? :D open an isue for it and I'll assign it to you :)

[0pdd]

@amihaiemil please pay attention to this new issue.

[SherifWaly]

@amihaiemil Each of Users, Organizations, ... etc should be passed a MkServer object which is built in the unit test for example. Is that right or I'm missing something :D ?

[amihaiemil]

@SherifWaly Good question. Well, normally, we should also have POST methods, but as you see the API doesn't have many of those... most of them are only for GETting information from the server, not sending.

So it's going to be either as you said, or I think we could add POSTs on the interfaces (even if they are not yet available in the actual API) and the real implementations will throw UnsupportedOperationException :D Makes sense?

When testing, it would be something like this:

VersionEye versioneye = MkVersionEye();
versioneye.me().comments().post(...)//add the comment to the JSON structure
//..test code here

And if versioneye above would happen to be a real RtVersionEye then the post(...) method would throw the exception since there is no real endpoint to call yet. (well actually, the post method for comments would be on a Project interface somehow but nevermind :) )

[SherifWaly]

@amihaiemil Yes, I think I understand your idea :D
Just I want to note that for example Teams consists of JsonArray not JsonObject so would that conflict with MkServer as storage() returns only JsonObjectBuilder and no JsonArrayBuilder?

[amihaiemil]

@SherifWaly no, because the JsonObjectBuilder of MkServer will contain everything. It will be the parent of all sub-jsons (think of it as of a tree). So when you add a JsonArray of teams, it will be part of the big json object :D

[SherifWaly]

@amihaiemil Yes, I understand it now thank you for elaborating :D
Could you assign me a sub-task which is somehow easy so that I would understand it better by implementing :) ?

[0pdd]

The puzzle 13-64d6efce has disappeared from the source code, that's why I closed this issue. //cc @amihaiemil