             {:short-name "s1"}
 
             "short-name all-revisions true"
-            [coll1-1 coll1-tombstone coll1-3 coll3]
+            [coll1-1 coll1-2-tombstone coll1-3 coll3]
             {:short-name "s1" :all-revisions true}
 
             "concept-id all-revisions false"

            {:version "v2"}

            "version all-revisions=true"
            [coll1-3 coll2-2 coll2-tombstone]
            {:version "v2" :all-revisions true}

            ;; verify that "finding latest", i.e., all-revisions=false, does not return old revisions
            "version all-revsions=false - no match to latest"
            []
            {:version "v1" :all-revisions false}


            {:concept-id "C1200000000-PROV1"}

            "concept-id all-revisions true"
            [coll1-1 coll1-2-tombstone coll1-3]
            {:concept-id "C1200000000-PROV1" :all-revisions true}

            "all-revisions true"
            [coll1-1 coll1-2-tombstone coll1-3 coll2-1 coll2-2 coll2-3-tombstone coll3]
            {:all-revisions true}))))

(deftest search-all-revisions-error-cases
  (testing "collection search with all_revisions bad value"
    (let [{:keys [status errors]} (search/get-search-failure-data
                                    (search/find-refs :collection {:all-revisions "foo"}))]
      (is (= [400 ["Parameter all_revisions must take value of true, false, or unset, but was [foo]"]]
             [status errors]))))
  (testing "granule search with all_revisions parameter is not supported"
    (let [{:keys [status errors]} (search/get-search-failure-data
                                    (search/find-refs :granule {:provider-id "PROV1"
                                                                :all-revisions false}))]
      (is (= [400 ["Parameter [all_revisions] was not recognized."]]
             [status errors]))))
  (testing "granule search with all_revisions bad value"
    (let [{:keys [status errors]} (search/get-search-failure-data
                                    (search/find-refs :granule {:provider-id "PROV1"
                                                                :all-revisions "foo"}))]
      (is (= [400 ["Parameter [all_revisions] was not recognized."
                   "Parameter all_revisions must take value of true, false, or unset, but was [foo]"]]
             [status errors])))))

