Week 8 ready for review

[OscarJohnson6]

Hi, @pawaitemadisoncollege week 8 is complete.

• My main takeaway was how REST is used and implemented in projects to do crud operations/stuff on APIs.
• My main challenge was mapping the response to the pogo object. When I used the pogo generator it made 2 classes, but the main pogo was just an array list to the other class. Then when I tried to map to it I got the error Cannot deserialize instance of ‘java.util.ArrayList<fit.app.pojo.ExerciseDbJson>’ out of START_OBJECT token. So, I tried a few different ways and got it to work. By having an array list of the other class that had all the JSON variables/objects and setting the readValue to new TypeReference.
• I watched this video on APIs. I also did other searching/googling on the error I had but I looked at a few sources and tried a lot of things.

[pawaitemadisoncollege]

Hi @OscarJohnson6 - You've done a lot of work consuming apis with java code!

I was curious about how to avoid having the generic List as the root element, so I tried this out and was able to create it such that there's a more meaningful java object as the root. It still has a list in it, but at least the object is somewhat meaningful.

Here is what I did:

I went to the API Ninjas site, found the Exercises service, and ran a request. Then I copied the entire response and ran it through the robo pojo generator - note that I gave the root object the name "Exercises", which gave me two classes: Exercises which has a list of ExercisesItem. I hope this helps you see how you can avoid having that List as the root.

• [ ] I was poking around and noticed this: https://github.com/OscarJohnson6/IndieProject/blob/4d5d663b5137f0cfe09d93ebf08b08ce015f032b/src/test/java/unitTests/UserAccountsTest.java#L94-L98. I think the issue might be that you don't have a userid on the WeightRecord, but instead a user.

[OscarJohnson6]

Hi, @pawaitemadisoncollege I still got the error below when I tried that on the API ninjas response.

The pojos/classes it generated are the same.

Here is my unit test failing.

I searched on Google and found this StackOverflow article. The top reply said

JsonMappingException: out of START_ARRAY token exception is thrown by Jackson object mapper as it's expecting an Object {} whereas it found an Array [{}] in response. This can be solved by replacing Object with Object[] in the argument for geForObject("url",Object[].class).

Which I think follows the error I am having as the response from API Ninja is a collection of objects or [{}] but I have the readValue saying that it's an object for the ApiNinjaResult class. I was also thinking would it be less complex to just have it be an array instead of creating an object for a class that can only be an array? Because won't I end up doing this: ApiNinjaResult results = ninjasExercise.createApiResponse( "", "", "", "", 5); List<ApiNinjaResultItem> resultItems = results.getApiNinjaResult(); instead of: List<ApiNinjaResultItem> resultItems = ninjasExercise.createApiResponse( "", "", "", "", 5);

For the user accounts test I got (weightDao.getByPropertyEqual("user", idNumber) to work but since it was an object of the user instead of just the id number and I was sure how to tell it to get the user id because user.Id didn't work; I just changed to from getByPropertyEqual to getById.

Also to keep my response in one spot, when I tried running a coverage test on my HealthCalculations class I get No coverage results. I can't figure out why, when I looked at the configurations it wasn't any different than what I had in week 3's exercise which has the coverage results show up. One thing I tried was generating a unit test class from the HealthCalculations and copying my code but that also didn't have coverage results display. Here is a picture of the configuration settings:

[pawaitemadisoncollege]

Hey Oscar, I poked around at this too and found the same thing you did with the array. Sorry I sent you down a rabbit hole there! I'm learning too :)

I am not sure about the coverage issue, but this might be a dialog to check:

[OscarJohnson6]

Hi, I got my coverage results to show. I modified the coverage config to include the HealthCalculations class, but I am not sure why I needed to because I went back and checked week 3 and it works without it. I added tests for another user and got my coverage to be 100%.

[pawaitemadisoncollege]

Awesome! Thanks for sending that on!