Update lints for parent-less batch methods

[apasel422]

Now that aip-dev/google.aip.dev#599 is fixed, we should update the following lints to account for the optional parent field:

• https://linter.aip.dev/136/http-uri-suffix
• https://linter.aip.dev/231/http-uri-suffix
• https://linter.aip.dev/231/request-parent-field
• https://linter.aip.dev/233/http-uri-suffix
• https://linter.aip.dev/233/request-parent-field
• https://linter.aip.dev/234/http-uri-suffix
• https://linter.aip.dev/234/request-parent-field

[lukesneeringer]

Starting to work on this, but why is the AIP-136 rule affected? (I can not see any obvious change to make.)

[apasel422]

api-linter/rules/aip0136/http_uri_suffix.go

Lines 54 to 61 in 9a793b5

	if strings.Contains(httpRule.URI, "{name=") || strings.Contains(httpRule.URI, "{parent=") {
	rpcSlice := strings.Split(strcase.SnakeCase(m.GetName()), "_")
	if rpcSlice[0] == "batch" {
	want = ":" + strcase.LowerCamelCase(rpcSlice[0]+"_"+rpcSlice[1])
	} else {
	want = ":" + rpcSlice[0]
	}
	}

doesn't work right if the batch method doesn't have the parent field in the URI.

[lukesneeringer]

Got it, thanks. Added to #619.

[apasel422]

This issue should be reopened, as the http-uri-suffix rules for AIPs 231, 233, and 234 weren't updated to account for parent-less paths. For example, the following still triggers a lint error:

service Library {
  rpc BatchGetBooks(BatchGetBooksRequest) returns (BatchGetBooksResponse) {
    option (google.api.http) = {
      get: "/v1/books:batchGet"
    }
  }
}