// Copyright (C) 2013 The Android Open Source Project
//
// 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.gerrit.acceptance.rest.account;

import static com.google.common.truth.Truth.assertThat;
import static com.google.gerrit.acceptance.rest.account.AccountAssert.assertAccountInfo;

import com.google.gerrit.acceptance.AbstractDaemonTest;
import com.google.gerrit.acceptance.RestResponse;
import com.google.gerrit.acceptance.TestAccount;
import com.google.gerrit.extensions.common.AccountInfo;
import com.google.gerrit.extensions.restapi.Url;

import org.apache.http.HttpStatus;
import org.junit.Test;

import java.io.IOException;

public class GetAccountIT extends AbstractDaemonTest {
  @Test
  public void getNonExistingAccount_NotFound() throws Exception {
    assertThat(adminSession.get("/accounts/non-existing").getStatusCode())
      .isEqualTo(HttpStatus.SC_NOT_FOUND);
  }

  @Test
  public void getAccount() throws Exception {
    // by formatted string
    testGetAccount("/accounts/"
        + Url.encode(admin.fullName + " <" + admin.email + ">"), admin);

    // by email
    testGetAccount("/accounts/" + admin.email, admin);

    // by full name
    testGetAccount("/accounts/" + admin.fullName, admin);

    // by account ID
    testGetAccount("/accounts/" + admin.id.get(), admin);

    // by user name
    testGetAccount("/accounts/" + admin.username, admin);

    // by 'self'
    testGetAccount("/accounts/self", admin);
  }

  private void testGetAccount(String url, TestAccount expectedAccount)
      throws IOException {
    RestResponse r = adminSession.get(url);
    assertThat(r.getStatusCode()).isEqualTo(HttpStatus.SC_OK);
    assertAccountInfo(expectedAccount, newGson()
        .fromJson(r.getReader(), AccountInfo.class));
  }
}

@@ -658,64 +658,8 @@
 t;%0A%0A
-import static com.google.common.truth.Truth.assertThat;%0A
 impo
@@ -742,16 +742,53 @@
 untInfo;
+%0Aimport static org.junit.Assert.fail;
 %0A%0Aimport
@@ -873,28 +873,23 @@
 eptance.
-RestResponse
+NoHttpd
 ;%0Aimport
@@ -971,165 +971,76 @@
 ons.
-common.AccountInfo;%0Aimport com.google.gerrit.extensions.restapi.Url;%0A%0Aimport org.apache.http.HttpStatus;%0Aimport org.junit.Test;%0A%0Aimport java.io.IOException;%0A
+restapi.ResourceNotFoundException;%0A%0Aimport org.junit.Test;%0A%0A@NoHttpd
 %0Apub
@@ -1173,46 +1173,33 @@
     
-assertThat(adminSession.get(%22/
+try %7B%0A      gApi.
 accounts
 /non
@@ -1194,17 +1194,23 @@
 accounts
-/
+().id(%22
 non-exis
@@ -1223,64 +1223,132 @@
 .get
-StatusCode
 ()
-)
+;
 %0A      
-.isEqualTo(HttpStatus.SC_NOT_FOUND);
+fail(%22Expected account to not exist%22);%0A    %7D catch (ResourceNotFoundException expected) %7B%0A      // Expected.%0A    %7D
 %0A  %7D
@@ -1453,42 +1453,8 @@
 unt(
-%22/accounts/%22%0A        + Url.encode(
 admi
@@ -1490,17 +1490,16 @@
 il + %22%3E%22
-)
 , admin)
@@ -1532,39 +1532,24 @@
 tGetAccount(
-%22/accounts/%22 + 
 admin.email,
@@ -1593,39 +1593,24 @@
 tGetAccount(
-%22/accounts/%22 + 
 admin.fullNa
@@ -1658,39 +1658,41 @@
 tGetAccount(
-%22/accounts/%22 + 
+Integer.toString(
 admin.id.get
@@ -1693,16 +1693,17 @@
 id.get()
+)
 , admin)
@@ -1748,23 +1748,8 @@
 unt(
-%22/accounts/%22 + 
 admi
@@ -1810,18 +1810,8 @@
 nt(%22
-/accounts/
 self
@@ -1863,19 +1863,18 @@
 (String 
-url
+id
 , TestAc
@@ -1909,18 +1909,16 @@
  throws 
-IO
 Exceptio
@@ -1929,210 +1929,70 @@
     
-RestResponse r = adminSession.get(url);%0A    assertThat(r.getStatusCode()).isEqualTo(HttpStatus.SC_OK);%0A    assertAccountInfo(expectedAccount, newGson()%0A        .fromJson(r.getReader(), AccountInfo.class
+assertAccountInfo(expectedAccount, gApi.accounts().id(id).get(
 ));%0A

Convert GetAccountIT to extension API