Skip to content

Commit

Permalink
fix: fix buildRequest setUrl order (#1255)
Browse files Browse the repository at this point in the history
* fix: fix buildRequest setUrl order

* chore: add unit test

* update

* format

* Update google-http-client/src/test/java/com/google/api/client/http/HttpRequestFactoryTest.java

* Update google-http-client/src/test/java/com/google/api/client/http/HttpRequestFactoryTest.java

Co-authored-by: Jeff Ching <chingor@google.com>

Co-authored-by: Jeff Ching <chingor@google.com>
  • Loading branch information
arithmetic1728 and chingor13 authored Feb 24, 2021
1 parent adad01e commit 97ffee1
Show file tree
Hide file tree
Showing 2 changed files with 44 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -84,13 +84,13 @@ public HttpRequestInitializer getInitializer() {
public HttpRequest buildRequest(String requestMethod, GenericUrl url, HttpContent content)
throws IOException {
HttpRequest request = transport.buildRequest();
if (url != null) {
request.setUrl(url);
}
if (initializer != null) {
initializer.initialize(request);
}
request.setRequestMethod(requestMethod);
if (url != null) {
request.setUrl(url);
}
if (content != null) {
request.setContent(content);
}
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,41 @@
/*
* Copyright 2021 Google LLC
*
* Licensed under the Apache License, Version 2.0 (the "License"); you may not use this file except
* in compliance with the License. You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software distributed under the License
* is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express
* or implied. See the License for the specific language governing permissions and limitations under
* the License.
*/

package com.google.api.client.http;

import com.google.api.client.http.javanet.NetHttpTransport;
import java.io.IOException;
import junit.framework.TestCase;

/** Tests {@link HttpRequestFactory}. */
public class HttpRequestFactoryTest extends TestCase {

public void testBuildRequest_urlShouldBeSet() throws IllegalArgumentException, IOException {
HttpRequestFactory requestFactory =
new NetHttpTransport()
.createRequestFactory(
new HttpRequestInitializer() {
@Override
public void initialize(HttpRequest request) {
// Url should be set by buildRequest method before calling initialize.
if (request.getUrl() == null) {
throw new IllegalArgumentException("url is not set in request");
}
}
});
GenericUrl url = new GenericUrl("https://foo.googleapis.com/");
HttpRequest request = requestFactory.buildRequest("GET", url, null);
assertEquals(url, request.getUrl());
}
}

0 comments on commit 97ffee1

Please sign in to comment.