Ensure failed LRO returns HTTP response (#155)

Omit the early-exit from an LRO sender as it prevents the underlying HTTP response from being returned to the caller (the LRO state machine can handle the success/failure status codes). Update test dependencies.
Azure · Oct 16, 2018 · a5f3812 · a5f3812
1 parent b8734b8
commit a5f3812
Show file tree

Hide file tree

Showing 8 changed files with 4 additions and 316 deletions.
diff --git a/src/Templates/MethodTemplate.cshtml b/src/Templates/MethodTemplate.cshtml
@@ -227,10 +227,6 @@ func (client @(Model.Owner)) @(Model.Name)(@Model.MethodParametersSignature) (@M
         if err != nil {
             return
         }
-        err = autorest.Respond(resp, azure.WithErrorUnlessStatusCode(@(string.Join(",", Model.ResponseCodes.ToArray()))))
-        if err != nil {
-            return
-        }
         future.Future, err = azure.NewFutureFromResponse(resp)
         return
         </text>

diff --git a/test/src/tests/Gopkg.lock b/test/src/tests/Gopkg.lock
diff --git a/test/src/tests/Gopkg.toml b/test/src/tests/Gopkg.toml
@@ -22,7 +22,7 @@
 
 [[constraint]]
   name = "github.com/Azure/go-autorest"
-  version = "10.15.3"
+  version = "11.1.1"
 
 [[constraint]]
   name = "github.com/satori/go.uuid"

diff --git a/test/src/tests/generated/lro/lroretrys.go b/test/src/tests/generated/lro/lroretrys.go