Allow POST/PUT/etc methods in Query operations to utilize a body.

[nderscore]

• Remove restriction that only allows request bodies to be built for Mutation operations.

---

This PR resolves #154.

Since v0.5.0, POST and other "Mutation"-y HTTP methods (PUT, PATCH, etc) are now permitted in Query operations.

Currently, when a query is defined using the @rest directive to wrap a POST call which includes a body (eg: input argument), the body is silently ignored, despite the request being made as otherwise expected.

This is because a restriction is still in place which prevents them having a body added to the request unless they are part of a Mutation operation. If we're going to allow a POST request to be made from a Query, it makes sense to include support for a body, since in most cases, these requests need them. This PR relaxes that restriction, instead relying solely on the HTTP method to determine whether we should attempt to build a body for the request.

[codecov-io]

Codecov Report

Merging #173 into master will increase coverage by 0.25%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #173      +/-   ##
==========================================
+ Coverage   92.65%   92.91%   +0.25%     
==========================================
  Files           2        2              
  Lines         395      395              
  Branches      117      116       -1     
==========================================
+ Hits          366      367       +1     
+ Misses         27       26       -1     
  Partials        2        2

Impacted Files	Coverage Δ
src/restLink.ts	92.7% <100%> (+0.26%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 48af562...30fcb92. Read the comment docs.

[fbartho]

@nderscore would you mind adding some unit-tests to make sure we don't break this in the future? Thanks!

[nderscore]

@fbartho No problem. I added a test to cover this usage.

I also added a test for the error state for when no body input is provided, since I touched it and it was missing from coverage. Turns out, it was unreachable code when no args are passed to the query, so I fixed that as well. 😄

[icebox]

Awesome. LGTM

[Falieson]

This PR fixes my issue. I guess until it gets released I'll throw it on npm under @tgrx/apollo-link-rest@0.6.1

[fbartho]

Woo!

[nderscore]

Thanks @fbartho

[fbartho]

Release Shipped:
https://github.com/apollographql/apollo-link-rest/releases/tag/v0.7.0