/*
 * Copyright 2019 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
 *
 *     https://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.android.gnd.inject;

import android.content.Context;
import androidx.annotation.NonNull;
import androidx.annotation.Nullable;
import androidx.work.ListenableWorker;
import androidx.work.WorkerFactory;
import androidx.work.WorkerParameters;
import com.google.android.gnd.persistence.local.LocalDataStore;
import com.google.android.gnd.persistence.remote.RemoteDataStore;
import com.google.android.gnd.persistence.sync.LocalMutationSyncWorker;
import com.google.android.gnd.workers.FileDownloadWorker;
import javax.inject.Inject;

/** Custom {@code WorkerFactory} to allow Dagger 2 injection into Ground Workers. */
public class GndWorkerFactory extends WorkerFactory {
  // TODO(github.com/google/dagger/issues/1183): Remove this factory once direct injection
  // supported on Workers. If this doesn't happen, replace with more robust solution, e.g.
  // https://github.com/nlgtuankiet/dagger-workmanager./
  private final LocalDataStore localDataStore;
  private final RemoteDataStore remoteDataStore;

  @Inject
  public GndWorkerFactory(LocalDataStore localDataStore, RemoteDataStore remoteDataStore) {
    this.localDataStore = localDataStore;
    this.remoteDataStore = remoteDataStore;
  }

  @Nullable
  @Override
  public ListenableWorker createWorker(
      @NonNull Context appContext,
      @NonNull String workerClassName,
      @NonNull WorkerParameters params) {
    // There are more generic and robust ways of doing this, but individual constructors are
    // hard-coded for simplicity. If and when github.com/google/dagger/issues/1183 is implemented
    // this class with go away completely.
    if (workerClassName.equals(LocalMutationSyncWorker.class.getName())) {
      return new LocalMutationSyncWorker(appContext, params, localDataStore, remoteDataStore);
    } else if (workerClassName.equals(FileDownloadWorker.class.getName())) {
      return new FileDownloadWorker(appContext, params, localDataStore);
    } else {
      throw new IllegalArgumentException("Unknown worker class " + workerClassName);
    }
  }
}

@@ -1397,129 +1397,8 @@
 ers.
- If this doesn't happen, replace with more robust solution, e.g.%0A  // https://github.com/nlgtuankiet/dagger-workmanager./
 %0A  p
@@ -2077,31 +2077,37 @@
 ass 
-with go away completely
+can be removed in favor of DI
 .%0A  

Tweak comments