petstore server does not update pet data for updatePetWithForm

[xhh]

It seems the petstore server does not make any changes for pet at all.

Steps to reproduce the issue:

1. create a pet and use getPetById to get the current pet data (e.g. via Swagger UI with pet id: 1429180158721)
2. use updatePetWithForm or updatePet to update the pet's name and status to new values
3. use getPetById to get the data again, which is not updated

It's better to make it work so that add test cases for them in codegen (the current test cases in petstore java then can be changed to reflect the actual changes).

[fehguy]

Are you sure? The java & objective c tests mutate the pets for testing purposes.

[xhh]

@fehguy yes, I'm sure. As you can see in the java test:
https://github.com/swagger-api/swagger-codegen/blob/master/samples/client/petstore/java/src/test/java/io/swagger/petstore/test/PetApiTest.java#L97
it does not compare names between "frank" and "furt", which actually proves that the name is not changed at all.
The updatePet test does not compare name changes either.

[wing328]

I did some quick tests with swagger ui. The following works for me

addPet (post)
getPet (get)
updatePet (post model)

However, updatePetWithForm does not work for me even though it returns 200

[xhh]

I confirmed that updatePet works well. Only updatePetWithForm has issue.

[webron]

@xhh - to clarify, does the online pet store server not update? The generated server (which would make sense)? Or does the client not execute the operation properly?

[xhh]

@webron it's the server that has issue, the client sends the correct data but the server does not update the data in the database

[webron]

@xhh - can you open the issue on swagger-core instead then? That's the project containing the online pet store. If you want to go over the logic of it, the specific sample is this one - https://github.com/swagger-api/swagger-core/tree/master/samples/java-jaxrs.

[xhh]

done