The ElasticIndexer should never be asked to index an empty list

wellcomecollection · Oct 20, 2020 · 8e0cafd · 8e0cafd
1 parent 8163184
commit 8e0cafd
Showing 1 changed file with 1 addition and 0 deletions.
diff --git a/common/pipeline_storage/src/main/scala/uk/ac/wellcome/pipeline_storage/ElasticIndexer.scala b/common/pipeline_storage/src/main/scala/uk/ac/wellcome/pipeline_storage/ElasticIndexer.scala
@@ -22,6 +22,7 @@ class ElasticIndexer[T: Indexable](
     new ElasticsearchIndexCreator(client, index, config).create
 
   final def index(documents: Seq[T]): Future[Either[Seq[T], Seq[T]]] = {
+    assert(documents.nonEmpty, "Cannot index an empty list of documents")
 
     debug(s"Indexing ${documents.map(doc => indexable.id(doc)).mkString(", ")}")