System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification calling Autorest generated code

亡梦爱人 提交于 2020-01-06 17:58:05

问题


I have a unit test, calling a service that makes use of Autorest generated code to call my Api.

I want my unit test to display the error that my Api is throwing, but there seems to be an error in the service's error handling.

I am using the following command to generate code to consume my api.

autorest --input-file=https://mywebsite.com.au:4433/myapi/api-docs/v1/swagger.json --output-folder=generated --csharp --namespace=MyConnector

The generated "client code" contains

    /// <param name='request'>
    /// </param>
    /// <param name='customHeaders'>
    /// Headers that will be added to request.
    /// </param>
    /// <param name='cancellationToken'>
    /// The cancellation token.
    /// </param>
    /// <exception cref="HttpOperationException">
    /// Thrown when the operation returned an invalid status code
    /// </exception>
    /// <exception cref="SerializationException">
    /// Thrown when unable to deserialize the response
    /// </exception>
    /// <return>
    /// A response object containing the response body and response headers.
    /// </return>
    public async Task<HttpOperationResponse<GetAvailableCarriersResponse>> GetAvailableCarriersByJobHeaderIdWithHttpMessagesAsync(GetAvailableCarriersRequest request = default(GetAvailableCarriersRequest), Dictionary<string, List<string>> customHeaders = null, CancellationToken cancellationToken = default(CancellationToken))
    {
        // Tracing
        bool _shouldTrace = ServiceClientTracing.IsEnabled;
        string _invocationId = null;
        if (_shouldTrace)
        {
            _invocationId = ServiceClientTracing.NextInvocationId.ToString();
            Dictionary<string, object> tracingParameters = new Dictionary<string, object>();
            tracingParameters.Add("request", request);
            tracingParameters.Add("cancellationToken", cancellationToken);
            ServiceClientTracing.Enter(_invocationId, this, "GetAvailableCarriersByJobHeaderId", tracingParameters);
        }
        // Construct URL
        var _baseUrl = BaseUri.AbsoluteUri;
        var _url = new System.Uri(new System.Uri(_baseUrl + (_baseUrl.EndsWith("/") ? "" : "/")), "api/shipping-management/Get-Available-Carriers").ToString();
        // Create HTTP transport objects
        var _httpRequest = new HttpRequestMessage();
        HttpResponseMessage _httpResponse = null;
        _httpRequest.Method = new HttpMethod("POST");
        _httpRequest.RequestUri = new System.Uri(_url);
        // Set Headers


        if (customHeaders != null)
        {
            foreach(var _header in customHeaders)
            {
                if (_httpRequest.Headers.Contains(_header.Key))
                {
                    _httpRequest.Headers.Remove(_header.Key);
                }
                _httpRequest.Headers.TryAddWithoutValidation(_header.Key, _header.Value);
            }
        }

        // Serialize Request
        string _requestContent = null;
        if(request != null)
        {
            _requestContent = SafeJsonConvert.SerializeObject(request, SerializationSettings);
            _httpRequest.Content = new StringContent(_requestContent, System.Text.Encoding.UTF8);
            _httpRequest.Content.Headers.ContentType =System.Net.Http.Headers.MediaTypeHeaderValue.Parse("application/json-patch+json; charset=utf-8");
        }
        // Send Request
        if (_shouldTrace)
        {
            ServiceClientTracing.SendRequest(_invocationId, _httpRequest);
        }
        cancellationToken.ThrowIfCancellationRequested();
        _httpResponse = await HttpClient.SendAsync(_httpRequest, cancellationToken).ConfigureAwait(false);
        if (_shouldTrace)
        {
            ServiceClientTracing.ReceiveResponse(_invocationId, _httpResponse);
        }
        HttpStatusCode _statusCode = _httpResponse.StatusCode;
        cancellationToken.ThrowIfCancellationRequested();
        string _responseContent = null;
        if ((int)_statusCode != 200)
        {
            var ex = new HttpOperationException(string.Format("Operation returned an invalid status code '{0}'", _statusCode));
            if (_httpResponse.Content != null) {
                _responseContent = await _httpResponse.Content.ReadAsStringAsync().ConfigureAwait(false);
            }
            else {
                _responseContent = string.Empty;
            }
            ex.Request = new HttpRequestMessageWrapper(_httpRequest, _requestContent);
            ex.Response = new HttpResponseMessageWrapper(_httpResponse, _responseContent);
            if (_shouldTrace)
            {
                ServiceClientTracing.Error(_invocationId, ex);
            }
            _httpRequest.Dispose();
            if (_httpResponse != null)
            {
                _httpResponse.Dispose();
            }
            throw ex;
        }
        // Create Result
        var _result = new HttpOperationResponse<GetAvailableCarriersResponse>();
        _result.Request = _httpRequest;
        _result.Response = _httpResponse;
        // Deserialize Response
        if ((int)_statusCode == 200)
        {
            _responseContent = await _httpResponse.Content.ReadAsStringAsync().ConfigureAwait(false);
            try
            {
                _result.Body = SafeJsonConvert.DeserializeObject<GetAvailableCarriersResponse>(_responseContent, DeserializationSettings);
            }
            catch (JsonException ex)
            {
                _httpRequest.Dispose();
                if (_httpResponse != null)
                {
                    _httpResponse.Dispose();
                }
                throw new SerializationException("Unable to deserialize the response.", _responseContent, ex);
            }
        }
        if (_shouldTrace)
        {
            ServiceClientTracing.Exit(_invocationId, _result);
        }
        return _result;
    }

I have a unit test to call the generated code using

var api = MakeApi();
var task=api.GetAvailableCarriersByJobHeaderIdWithHttpMessagesAsync(req);
var carriers  = task.Result.Body.Carriers;

where

    private static MyApiService MakeApi()
    {
        var setting = new MyAPISettings(false);
        var api = new MyApiService(setting);
        return api;
    }

and MyApiService contains (with altered namespaces)

    public Task<HttpOperationResponse<GetAvailableCarriersResponse>> GetAvailableCarriersByJobHeaderIdWithHttpMessagesAsync(
        GetAvailableCarriersRequest request = default(GetAvailableCarriersRequest), Dictionary<string, List<string>> customHeaders = null,
        CancellationToken cancellationToken = default(CancellationToken))
    {
        return ApiCaller.ExecuteAsync(
            async headers => await API.GetAvailableCarriersByJobHeaderIdWithHttpMessagesAsync(request, headers, cancellationToken),
            async () => await GetTokenHeadersAsync(customHeaders));
    }

where apicaller is

using System;
using System.Collections.Generic;
using System.Threading.Tasks;

namespace MyServices
{
    public static class ApiCaller
    {



        private static Dictionary<string, List<string>> Headers { get; set; }

            private static string GetHeadersMessage()
            {
                string ret = "";

                if (Headers != null)
                {
                    foreach (string key in Headers.Keys)
                    {
                        if (Headers[key] != null)
                        {
                            foreach (string value in Headers[key])
                            {
                                ret = $"{key}-{value}\n";
                            }
                        }
                    }
                }

                return ret;
            }


        public async static Task<T> ExecuteAsync<T>(Func<Dictionary<string, List<string>>, Task<T>> f,
                Func<Task<Dictionary<string, List<string>>>> getHeaders)
            {
                T ret = default(T);

                try
                {
                    try
                    {
                        if (getHeaders != null && Headers == null)
                        {
                            Headers = await getHeaders();
                        }

                        ret = await f(Headers);
                    }
                    catch (Microsoft.Rest.HttpOperationException ex1)
                    {
                        if (ex1.Response?.StatusCode == System.Net.HttpStatusCode.Unauthorized && getHeaders != null)
                        {
                            Headers = await getHeaders();
                            ret = await f(Headers);
                        }
                        else
                        {
                            throw;
                        }
                    }
                }
                catch (Exception ex)
                {
                    //Log.Error(ex, $"... API CALL ERROR ...\nHEADERS:{GetHeadersMessage()}");
                    throw new Exception($"Error calling the API. {ex.Message}", ex);
                }

                return ret;
            }
        }
    }

My Api throws an InternalServerError However when I run the unit test, I get an error in the client code.

The error occurs at

        // Create Result
        var _result = new HttpOperationResponse<GetAvailableCarriersResponse>();

And is

System.Exception: Error calling the API. Operation returned an invalid status code 'InternalServerError' ---> Microsoft.Rest.HttpOperationException: Operation returned an invalid status code 'InternalServerError'
   at MyConnector.MyApi.<GetAvailableCarriersByJobHeaderIdWithHttpMessagesAsync>d__49.MoveNext() 

How can I work around this?

I note that the code for HttpOperationResponse is

namespace Microsoft.Rest
{
  /// <summary>
  /// Represents the base return type of all ServiceClient REST operations.
  /// </summary>
  public class HttpOperationResponse<T> : HttpOperationResponse, IHttpOperationResponse<T>, IHttpOperationResponse
  {
    /// <summary>Gets or sets the response object.</summary>
    public T Body { get; set; }
  }
}

Here is the structure for GetAvailableCarriersResponse

using Newtonsoft.Json;
using System.Collections.Generic;

public partial class GetAvailableCarriersResponse
{
    public GetAvailableCarriersResponse()
    {
        CustomInit();
    }

    public GetAvailableCarriersResponse(IList<DeliverBy> carriers = default(IList<DeliverBy>))
    {
        Carriers = carriers;
        CustomInit();
    }

    partial void CustomInit();

    [JsonProperty(PropertyName = "carriers")]
    public IList<DeliverBy> Carriers { get; set; }

}

[Update]

In ApiCaller ExecuteAsync the following executes.

throw;

If I catch the error at this point, it's (edited) ToString() returns

"Microsoft.Rest.HttpOperationException: Operation returned an invalid status code 'InternalServerError' at MyAPI.

<GetAvailableCarriersByJobHeaderIdWithHttpMessagesAsync>d__49.MoveNext() in 
MyConnector\\generated\\MyAPI.cs:line 4018

End of stack trace from previous location where exception was thrown at 
System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() at
System.Runtime.CompilerServices.TaskAwaiter.
HandleNonSuccessAndDebuggerNotification(Task task) 
at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
at MyApiService.<>c__DisplayClass39_0.<<GetAvailableCarriersByJobHeaderIdWithHttpMessagesAsync>b__0>d.MoveNext() 
in MyApiService.cs:line 339
End of stack trace from previous location where exception was thrown  

at System.Runtime.ExceptionServices.ExceptionDispatch
Info.Throw() at System.Runtime.CompilerServices.TaskAwaiter.
HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()  
at MyServices.ApiCaller.<ExecuteAsync>d__5`1.MoveNext() 
in ApiCaller.cs:line 50"

I edited some of the names in the above code to simplify and obfuscate.

[Update]

The problem seems to be to do with the getHeaders parameter to ApiCaller.ExecuteAsync

[Update]

If I examine ex1 thrown in ExecuteAsync, I can get my Api Error type using

ex1.Response.StatusCode

but how do I get the error description?


回答1:


What I did to get error description is to cast it to one of the error types generated by Autorest.

if (myRawResponseObject is My422Response my422Response)
{
    // Response has HTTP Status Code 422
    Console.WriteLine($"Error Response Type: {my422Response.ToString()}");
}

If you OpenAPI document defines error properties for a 422 response, then you will find them on the My422Response object.



来源:https://stackoverflow.com/questions/56402878/system-runtime-compilerservices-taskawaiter-handlenonsuccessanddebuggernotificat

标签
易学教程内所有资源均来自网络或用户发布的内容,如有违反法律规定的内容欢迎反馈
该文章没有解决你所遇到的问题?点击提问,说说你的问题,让更多的人一起探讨吧!