Browse Source

fix:roa request can get real actionname rather than empty

wenzuochao 6 years ago
parent
commit
72fbe7420a
2 changed files with 8 additions and 6 deletions
  1. 6 4
      integration/core_test.go
  2. 2 2
      sdk/requests/roa_request.go

+ 6 - 4
integration/core_test.go

@@ -120,12 +120,14 @@ func Test_DescribeClustersWithCommonRequestWithROA(t *testing.T) {
 	request.Method = "GET"
 	request.Domain = "cs.aliyuncs.com"
 	request.Version = "2015-12-15"
-	request.PathPattern = "/clusters/[ClusterId]"
+	request.PathPattern = "/clusters"
+	request.ApiName = "DescribeClusters"
 	request.QueryParams["RegionId"] = os.Getenv("REGION_ID")
 	request.TransToAcsRequest()
-	_, err = client.ProcessCommonRequest(request)
-	assert.NotNil(t, err)
-	assert.Contains(t, err.Error(), "Request url is invalid")
+	resp, err := client.ProcessCommonRequest(request)
+	assert.Nil(t, err)
+	assert.Equal(t, 200, resp.GetHttpStatus())
+
 }
 
 func Test_DescribeClustersWithCommonRequestWithSignatureDostNotMatch(t *testing.T) {

+ 2 - 2
sdk/requests/roa_request.go

@@ -136,7 +136,7 @@ func (request *RoaRequest) InitWithApiInfo(product, version, action, uriPattern,
 	request.locationEndpointType = endpointType
 	request.product = product
 	//request.version = version
-	//request.actionName = action
+	request.actionName = action
 }
 
 func (request *RoaRequest) initWithCommonRequest(commonRequest *CommonRequest) {
@@ -145,7 +145,7 @@ func (request *RoaRequest) initWithCommonRequest(commonRequest *CommonRequest) {
 	request.product = commonRequest.Product
 	//request.version = commonRequest.Version
 	request.Headers["x-acs-version"] = commonRequest.Version
-	//request.actionName = commonRequest.ApiName
+	request.actionName = commonRequest.ApiName
 	request.pathPattern = commonRequest.PathPattern
 	request.locationServiceCode = commonRequest.ServiceCode
 	request.locationEndpointType = ""