search

samvera / questioning_authority

Question your authorities
Other
54 stars 30 forks source link

Add search method to see if wants the controller with the request. #119

Closed andjsmit closed 7 years ago

andjsmit commented 7 years ago

Needed for a local QA that uses a SearchBuilder.

jcoyne commented 7 years ago

Can you please add a test?

andjsmit commented 7 years ago

@jcoyne I'm stuck on how to test this as no included authorities would require two arguments for its search method. Not sure how to go about mocking an authority if that is what is needed. Could you give me a clue on how to get started. Thanks.

jcoyne commented 7 years ago

You can define an authority class that takes two args in the test.

jcoyne commented 7 years ago

This looks good @andjsmit. Can you squash these commits?

andjsmit commented 7 years ago

@jcoyne Squashed it. Thanks.