// Copyright 2018 The Nomulus Authors. All Rights Reserved.
//
// 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 google.registry.tools;

import static google.registry.model.ofy.ObjectifyService.ofy;
import static google.registry.model.registry.Registries.assertTldsExist;

import com.beust.jcommander.Parameter;
import com.beust.jcommander.Parameters;
import com.google.common.collect.Iterators;
import google.registry.model.domain.DomainBase;
import google.registry.util.Clock;
import java.util.List;
import javax.inject.Inject;
import org.joda.time.DateTime;

/** Command to show the count of active domains on a given TLD. */
@Parameters(separators = " =", commandDescription = "Show count of domains on TLD")
final class CountDomainsCommand implements CommandWithRemoteApi {

  @Parameter(description = "TLD(s) to count domains on", required = true)
  private List<String> mainParameters;

  @Inject Clock clock;

  @Override
  public void run() {
    DateTime now = clock.nowUtc();
    assertTldsExist(mainParameters)
        .forEach(tld -> System.out.printf("%s,%d\n", tld, getCountForTld(tld, now)));
  }

  private int getCountForTld(String tld, DateTime now) {
    return Iterators.size(
        ofy()
            .load()
            .type(DomainBase.class)
            .filter("tld", tld)
            .filter("deletionTime >", now)
            .chunkAll()
            .keys()
            .iterator());
  }
}

@@ -897,19 +897,19 @@
 ct.Itera
-tor
+ble
 s;%0Aimpor
@@ -1694,19 +1694,19 @@
 rn Itera
-tor
+ble
 s.size(%0A
@@ -1897,32 +1897,8 @@
 ys()
-%0A            .iterator()
 );%0A 

Use simpler Iterables helper when counting domains