// 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;

import static java.util.stream.Collectors.toList;

import com.google.auto.value.AutoValue;
import com.google.common.net.InetAddresses;
import com.google.gerrit.common.Nullable;
import com.google.gerrit.mail.Address;
import com.google.gerrit.reviewdb.client.Account;
import java.net.InetSocketAddress;
import java.util.Arrays;
import java.util.List;
import org.apache.http.client.utils.URIBuilder;
import org.eclipse.jgit.lib.PersonIdent;

@AutoValue
public abstract class TestAccount {
  public static List<Account.Id> ids(List<TestAccount> accounts) {
    return accounts.stream().map(TestAccount::id).collect(toList());
  }

  public static List<String> names(List<TestAccount> accounts) {
    return accounts.stream().map(TestAccount::fullName).collect(toList());
  }

  public static List<String> names(TestAccount... accounts) {
    return names(Arrays.asList(accounts));
  }

  static TestAccount create(
      Account.Id id,
      @Nullable String username,
      @Nullable String email,
      @Nullable String fullName,
      @Nullable String httpPassword) {
    return new AutoValue_TestAccount(id, username, email, fullName, httpPassword);
  }

  public abstract Account.Id id();

  @Nullable
  public abstract String username();

  @Nullable
  public abstract String email();

  @Nullable
  public abstract String fullName();

  @Nullable
  public abstract String httpPassword();

  public PersonIdent newIdent() {
    return new PersonIdent(fullName(), email());
  }

  public String getHttpUrl(GerritServer server) {
    InetSocketAddress addr = server.getHttpAddress();
    return new URIBuilder()
        .setScheme("http")
        .setUserInfo(username(), httpPassword())
        .setHost(InetAddresses.toUriString(addr.getAddress()))
        .setPort(addr.getPort())
        .toString();
  }

  public Address getEmailAddress() {
    // Address is weird enough that it's safer and clearer to create a new instance in a
    // non-abstract method rather than, say, having an abstract emailAddress() as part of this
    // AutoValue class. Specifically:
    //  * Email is not specified as @Nullable in Address, but it is nullable in this class. If this
    //    is a problem, at least it's a problem only for users of TestAccount that actually call
    //    emailAddress().
    //  * Address#equals only considers email, not name, whereas TestAccount#equals should include
    //    name.
    return new Address(fullName(), email());
  }
}

@@ -659,83 +659,194 @@
 tic 
-java.util.stream.Collectors.toList;%0A%0Aimport com.google.auto.value.AutoValue
+com.google.common.collect.ImmutableList.toImmutableList;%0A%0Aimport com.google.auto.value.AutoValue;%0Aimport com.google.common.collect.ImmutableList;%0Aimport com.google.common.collect.Streams
 ;%0Aim
@@ -1082,31 +1082,8 @@
 ys;%0A
-import java.util.List;%0A
 impo
@@ -1227,24 +1227,33 @@
 blic static 
+Immutable
 List%3CAccount
@@ -1261,20 +1261,24 @@
 Id%3E ids(
-List
+Iterable
 %3CTestAcc
@@ -1298,39 +1298,38 @@
 s) %7B%0A    return 
-account
+Stream
 s.stream().map(T
@@ -1313,32 +1313,40 @@
  Streams.stream(
+accounts
 ).map(TestAccoun
@@ -1354,32 +1354,41 @@
 ::id).collect(to
+Immutable
 List());%0A  %7D%0A%0A  
@@ -1393,32 +1393,41 @@
   public static 
+Immutable
 List%3CString%3E nam
@@ -1429,20 +1429,24 @@
 %3E names(
-List
+Iterable
 %3CTestAcc
@@ -1474,23 +1474,22 @@
  return 
-account
+Stream
 s.stream
@@ -1489,16 +1489,24 @@
 .stream(
+accounts
 ).map(Te
@@ -1536,16 +1536,25 @@
 llect(to
+Immutable
 List());
@@ -1575,16 +1575,25 @@
  static 
+Immutable
 List%3CStr

Improve signatures of static methods