Skip to content

Commit

Permalink
test AdminAdminSetMemberSearchBuilder for collection search
Browse files Browse the repository at this point in the history
update specs to ensure the correct models are included in the search:

  - always include `Collection`,
  - include configured collection model name,
  - include registered curationconcerns

should the implementation be updated to search using the indexed generic type?
  • Loading branch information
tamsin johnson committed Oct 28, 2021
1 parent 8e6cba6 commit 435063c
Showing 1 changed file with 11 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,17 @@

it 'searches for valid work types' do
expect(builder.filter_models(solr_params))
.to contain_exactly("{!terms f=has_model_ssim}Monograph,#{Hyrax.config.collection_class}")
.to contain_exactly(include("{!terms f=has_model_ssim}Monograph"))
end

it 'searches for collections indexed as ActiveFedora' do
expect(builder.filter_models(solr_params))
.to contain_exactly(include("Collection"))
end

it 'searches for collections indexed as valkyrie' do
expect(builder.filter_models(solr_params))
.to contain_exactly(include(Hyrax.config.collection_model))
end

it 'does not limit to active only' do
Expand Down

0 comments on commit 435063c

Please sign in to comment.