RedHat-Middleware-Workshops / dayinthelife-integration

Day in the Life Agile Integration Workshop
73 stars 93 forks source link

Citizen integrator lab02 - endpoints doesn't work #94

Open VinayBhalerao opened 5 years ago

VinayBhalerao commented 5 years ago

After the mock is completed, the /locations works as expected, but the GET /locations/id doesn't return appropriate response


curl "http://microcks.dil.opentry.me/rest/Locations%20User%2032/1.0.0/locations/1" -v
*   Trying 18.208.43.86...
* Connected to microcks.dil.opentry.me (127.0.0.1) port 80 (#0)
> GET /rest/Locations%20User%2032/1.0.0/locations/1 HTTP/1.1
> Host: microcks.dil.opentry.me
> User-Agent: curl/7.43.0
> Accept: */*
>
< HTTP/1.1 404 Not Found
< Content-Length: 0
< X-Application-Context: application:prod
< Date: Wed, 24 Oct 2018 15:48:13 GMT
< Set-Cookie: 56bd75657f4963aff4ab4fa37d05260a=caac68128482c087ec26629aab45c106; path=/; HttpOnly
< Cache-control: private
<
* Connection #0 to host microcks.dil.opentry.me left intact```

**Expectation**
We need to add a sample response type in lab01  for`/locations/{id}` in order for the mock to return correct response in lab02
VinayBhalerao commented 5 years ago

We need to make a decision

1) Fix the lab 01 to add the response for locations/{id} 2) Remove the contents for GET /locations/{id} and POST /locations since that lab is getting unnecessary long and the participants might lose interest. We can only keep the steps for GET /locations

wdyt @sigreen ? and your preference?

sigreen commented 5 years ago

My preference is for (1). I've spent a lot of time adding extra paths at the request of @weimeilin79 and @hguerrero , so to delete all that content now would be a real shame. Can we just fix lab01?

weimeilin79 commented 5 years ago

I vote (1). These endpoint are used in the Dev Track. Is it hard to fix? I can take a look at it tomorrow.