blacklight-spotlight 0.4.1 → 0.5.0

Sign up to get free protection for your applications and to get access to all the features.
Files changed (367) hide show
  1. checksums.yaml +4 -4
  2. data/README.md +22 -8
  3. data/Rakefile +40 -39
  4. data/app/assets/javascripts/spotlight/blacklight_configuration.js +16 -0
  5. data/app/assets/javascripts/spotlight/blocks/resources_block.js +3 -1
  6. data/app/assets/stylesheets/spotlight/_blacklight_configuration.scss +2 -2
  7. data/app/assets/stylesheets/spotlight/_blacklight_overrides.scss +24 -0
  8. data/app/assets/stylesheets/spotlight/_header.scss +150 -47
  9. data/app/assets/stylesheets/spotlight/_pages.scss +62 -11
  10. data/app/assets/stylesheets/spotlight/_report_a_problem.scss +1 -0
  11. data/app/assets/stylesheets/spotlight/_spotlight.scss +1 -0
  12. data/app/builders/spotlight/bootstrap_breadcrumbs_builder.rb +28 -21
  13. data/app/controllers/spotlight/about_pages_controller.rb +25 -14
  14. data/app/controllers/spotlight/appearances_controller.rb +34 -30
  15. data/app/controllers/spotlight/application_controller.rb +2 -1
  16. data/app/controllers/spotlight/attachments_controller.rb +9 -7
  17. data/app/controllers/spotlight/browse_controller.rb +13 -9
  18. data/app/controllers/spotlight/catalog_controller.rb +179 -161
  19. data/app/controllers/spotlight/concerns/application_controller.rb +22 -20
  20. data/app/controllers/spotlight/concerns/catalog_search_context.rb +44 -0
  21. data/app/controllers/spotlight/confirmations_controller.rb +5 -2
  22. data/app/controllers/spotlight/contact_forms_controller.rb +13 -7
  23. data/app/controllers/spotlight/contacts_controller.rb +13 -4
  24. data/app/controllers/spotlight/custom_fields_controller.rb +48 -40
  25. data/app/controllers/spotlight/dashboards_controller.rb +17 -8
  26. data/app/controllers/spotlight/default_controller.rb +3 -1
  27. data/app/controllers/spotlight/exhibits_controller.rb +69 -62
  28. data/app/controllers/spotlight/feature_pages_controller.rb +21 -11
  29. data/app/controllers/spotlight/home_pages_controller.rb +6 -5
  30. data/app/controllers/spotlight/lock_controller.rb +3 -1
  31. data/app/controllers/spotlight/metadata_configurations_controller.rb +49 -0
  32. data/app/controllers/spotlight/pages_controller.rb +28 -18
  33. data/app/controllers/spotlight/resources/upload_controller.rb +53 -46
  34. data/app/controllers/spotlight/resources_controller.rb +9 -5
  35. data/app/controllers/spotlight/roles_controller.rb +26 -22
  36. data/app/controllers/spotlight/search_configurations_controller.rb +66 -0
  37. data/app/controllers/spotlight/searches_controller.rb +91 -82
  38. data/app/controllers/spotlight/solr_controller.rb +20 -13
  39. data/app/controllers/spotlight/tags_controller.rb +5 -3
  40. data/app/controllers/spotlight/versions_controller.rb +15 -12
  41. data/app/controllers/spotlight/view_configurations_controller.rb +17 -0
  42. data/app/helpers/spotlight/application_helper.rb +44 -32
  43. data/app/helpers/spotlight/browse_helper.rb +13 -1
  44. data/app/helpers/spotlight/crud_link_helpers.rb +22 -14
  45. data/app/helpers/spotlight/jcrop_helper.rb +4 -2
  46. data/app/helpers/spotlight/main_app_helpers.rb +45 -41
  47. data/app/helpers/spotlight/navbar_helper.rb +6 -0
  48. data/app/helpers/spotlight/pages_helper.rb +23 -15
  49. data/app/helpers/spotlight/roles_helper.rb +10 -4
  50. data/app/helpers/spotlight/search_configurations_helper.rb +22 -0
  51. data/app/helpers/spotlight/title_helper.rb +16 -13
  52. data/app/jobs/spotlight/add_uploads_from_csv.rb +13 -9
  53. data/app/jobs/spotlight/reindex_job.rb +9 -9
  54. data/app/jobs/spotlight/rename_sidecar_field_job.rb +3 -0
  55. data/app/mailers/spotlight/confirmation_mailer.rb +2 -0
  56. data/app/mailers/spotlight/indexing_complete_mailer.rb +4 -2
  57. data/app/models/ability.rb +3 -0
  58. data/app/models/concerns/spotlight/ar_light.rb +49 -25
  59. data/app/models/concerns/spotlight/blacklight_configuration_defaults.rb +25 -10
  60. data/app/models/concerns/spotlight/exhibit_analytics.rb +28 -0
  61. data/app/models/concerns/spotlight/image_derivatives.rb +25 -26
  62. data/app/models/concerns/spotlight/resources/open_graph.rb +23 -19
  63. data/app/models/concerns/spotlight/resources/web.rb +25 -24
  64. data/app/models/concerns/spotlight/solr_document.rb +40 -34
  65. data/app/models/concerns/spotlight/solr_document/active_model_concern.rb +29 -40
  66. data/app/models/concerns/spotlight/solr_document/atomic_updates.rb +22 -18
  67. data/app/models/concerns/spotlight/solr_document/finder.rb +23 -18
  68. data/app/models/concerns/spotlight/solr_document/spotlight_images.rb +43 -39
  69. data/app/models/concerns/spotlight/solr_document/uploaded_resource.rb +41 -29
  70. data/app/models/concerns/spotlight/user.rb +25 -19
  71. data/app/models/sir_trevor_rails/blocks/browse_block.rb +36 -30
  72. data/app/models/sir_trevor_rails/blocks/featured_pages_block.rb +24 -18
  73. data/app/models/sir_trevor_rails/blocks/oembed_block.rb +8 -4
  74. data/app/models/sir_trevor_rails/blocks/search_results_block.rb +29 -23
  75. data/app/models/sir_trevor_rails/blocks/solr_documents_block.rb +43 -42
  76. data/app/models/sir_trevor_rails/blocks/solr_documents_carousel_block.rb +16 -12
  77. data/app/models/sir_trevor_rails/blocks/solr_documents_embed_block.rb +7 -3
  78. data/app/models/sir_trevor_rails/blocks/solr_documents_features_block.rb +7 -3
  79. data/app/models/sir_trevor_rails/blocks/solr_documents_grid_block.rb +7 -3
  80. data/app/models/sir_trevor_rails/blocks/textable.rb +18 -18
  81. data/app/models/spotlight/ability.rb +32 -33
  82. data/app/models/spotlight/about_page.rb +3 -1
  83. data/app/models/spotlight/analytics/ga.rb +46 -27
  84. data/app/models/spotlight/attachment.rb +3 -1
  85. data/app/models/spotlight/blacklight_configuration.rb +146 -68
  86. data/app/models/spotlight/contact.rb +35 -30
  87. data/app/models/spotlight/contact_email.rb +6 -3
  88. data/app/models/spotlight/contact_form.rb +4 -3
  89. data/app/models/spotlight/custom_field.rb +41 -33
  90. data/app/models/spotlight/exhibit.rb +104 -130
  91. data/app/models/spotlight/feature_page.rb +9 -7
  92. data/app/models/spotlight/featured_image.rb +10 -10
  93. data/app/models/spotlight/field_metadata.rb +46 -0
  94. data/app/models/spotlight/home_page.rb +8 -5
  95. data/app/models/spotlight/lock.rb +4 -3
  96. data/app/models/spotlight/main_navigation.rb +9 -3
  97. data/app/models/spotlight/masthead.rb +17 -16
  98. data/app/models/spotlight/page.rb +16 -16
  99. data/app/models/spotlight/resource.rb +69 -22
  100. data/app/models/spotlight/resources/csv_upload.rb +8 -4
  101. data/app/models/spotlight/resources/upload.rb +68 -57
  102. data/app/models/spotlight/role.rb +30 -34
  103. data/app/models/spotlight/search.rb +74 -63
  104. data/app/models/spotlight/solr_document_sidecar.rb +17 -15
  105. data/app/serializers/spotlight/exhibit_export_serializer.rb +65 -29
  106. data/app/serializers/spotlight/page_representer.rb +10 -6
  107. data/app/uploaders/spotlight/attachment_uploader.rb +4 -6
  108. data/app/uploaders/spotlight/avatar_uploader.rb +4 -3
  109. data/app/uploaders/spotlight/featured_image_uploader.rb +15 -14
  110. data/app/uploaders/spotlight/item_uploader.rb +3 -1
  111. data/app/uploaders/spotlight/masthead_uploader.rb +4 -2
  112. data/app/views/catalog/_search_form.html.erb +24 -0
  113. data/app/views/shared/_browse_navbar.html.erb +1 -1
  114. data/app/views/shared/_exhibit_navbar.html.erb +1 -1
  115. data/app/views/spotlight/appearances/edit.html.erb +13 -45
  116. data/app/views/spotlight/catalog/_edit_default.html.erb +1 -1
  117. data/app/views/spotlight/custom_fields/_form.html.erb +1 -1
  118. data/app/views/spotlight/exhibits/edit.html.erb +4 -4
  119. data/app/views/spotlight/home_pages/_empty.html.erb +1 -3
  120. data/app/views/spotlight/home_pages/_page_options.html.erb +1 -1
  121. data/app/views/spotlight/{blacklight_configurations → metadata_configurations}/_metadata_field.html.erb +0 -0
  122. data/app/views/spotlight/{blacklight_configurations/edit_metadata_fields.html.erb → metadata_configurations/edit.html.erb} +1 -1
  123. data/app/views/spotlight/pages/show.html.erb +2 -2
  124. data/app/views/spotlight/search_configurations/_default_per_page.html.erb +5 -0
  125. data/app/views/spotlight/search_configurations/_document_index_view_types.html.erb +7 -0
  126. data/app/views/spotlight/search_configurations/_facets.html.erb +38 -0
  127. data/app/views/spotlight/search_configurations/_search_fields.html.erb +58 -0
  128. data/app/views/spotlight/search_configurations/_sort.html.erb +53 -0
  129. data/app/views/spotlight/search_configurations/edit.html.erb +46 -0
  130. data/app/views/spotlight/searches/_search.html.erb +1 -1
  131. data/app/views/spotlight/shared/_admin_sidebar.html.erb +0 -2
  132. data/app/views/spotlight/shared/_curation_sidebar.html.erb +2 -3
  133. data/app/views/spotlight/shared/_report_a_problem.html.erb +2 -2
  134. data/app/views/spotlight/sir_trevor/blocks/_oembed_block.html.erb +5 -1
  135. data/app/views/spotlight/sir_trevor/blocks/_solr_documents_block.html.erb +7 -5
  136. data/app/views/spotlight/sir_trevor/blocks/_solr_documents_carousel_block.html.erb +2 -2
  137. data/app/views/spotlight/sir_trevor/blocks/_solr_documents_embed_block.html.erb +7 -5
  138. data/app/views/spotlight/sir_trevor/blocks/_solr_documents_features_block.html.erb +2 -2
  139. data/app/views/spotlight/sir_trevor/blocks/_solr_documents_grid_block.html.erb +14 -9
  140. data/config/locales/spotlight.en.yml +94 -49
  141. data/config/routes.rb +11 -20
  142. data/db/migrate/20150410180014_add_taggings_counter_cache_to_tags.acts_as_taggable_on_engine.rb +15 -0
  143. data/db/migrate/20150410180015_add_missing_taggable_index.acts_as_taggable_on_engine.rb +10 -0
  144. data/db/migrate/20150410180016_change_collation_for_tag_names.acts_as_taggable_on_engine.rb +10 -0
  145. data/db/migrate/20150713160101_change_spotlight_searches_to_published.rb +5 -0
  146. data/db/migrate/20151016092343_remove_searchable_from_exhibit.rb +20 -0
  147. data/lib/blacklight/spotlight.rb +1 -1
  148. data/lib/generators/spotlight/install_generator.rb +27 -25
  149. data/lib/generators/spotlight/templates/catalog_controller.rb +7 -5
  150. data/lib/generators/spotlight/templates/jetty.rake +8 -2
  151. data/lib/generators/spotlight/templates/spotlight_helper.rb +3 -1
  152. data/lib/spotlight.rb +4 -2
  153. data/lib/spotlight/base.rb +21 -14
  154. data/lib/spotlight/catalog.rb +10 -9
  155. data/lib/spotlight/catalog/access_controls_enforcement.rb +30 -15
  156. data/lib/spotlight/config.rb +4 -2
  157. data/lib/spotlight/controller.rb +12 -11
  158. data/lib/spotlight/engine.rb +30 -22
  159. data/lib/spotlight/rails/routes.rb +7 -4
  160. data/lib/spotlight/version.rb +2 -1
  161. data/lib/tasks/spotlight_tasks.rake +56 -50
  162. data/spec/controllers/application_controller_spec.rb +13 -13
  163. data/spec/controllers/spotlight/about_pages_controller_spec.rb +63 -59
  164. data/spec/controllers/spotlight/appearances_controller_spec.rb +29 -24
  165. data/spec/controllers/spotlight/application_controller_spec.rb +3 -3
  166. data/spec/controllers/spotlight/attachments_controller_spec.rb +10 -10
  167. data/spec/controllers/spotlight/browse_controller_spec.rb +23 -25
  168. data/spec/controllers/spotlight/catalog_controller_spec.rb +82 -87
  169. data/spec/controllers/spotlight/confirmations_controller_spec.rb +13 -11
  170. data/spec/controllers/spotlight/contact_forms_controller_spec.rb +17 -12
  171. data/spec/controllers/spotlight/contacts_controller_spec.rb +36 -36
  172. data/spec/controllers/spotlight/custom_fields_controller_spec.rb +26 -26
  173. data/spec/controllers/spotlight/dashboards_controller_spec.rb +14 -14
  174. data/spec/controllers/spotlight/exhibits_controller_spec.rb +59 -54
  175. data/spec/controllers/spotlight/feature_pages_controller_spec.rb +59 -59
  176. data/spec/controllers/spotlight/home_pages_controller_spec.rb +28 -29
  177. data/spec/controllers/spotlight/metadata_configurations_controller_spec.rb +84 -0
  178. data/spec/controllers/spotlight/resources/upload_controller_spec.rb +42 -33
  179. data/spec/controllers/spotlight/resources_controller_spec.rb +25 -29
  180. data/spec/controllers/spotlight/roles_controller_spec.rb +37 -19
  181. data/spec/controllers/spotlight/search_configurations_controller_spec.rb +125 -0
  182. data/spec/controllers/spotlight/searches_controller_spec.rb +84 -63
  183. data/spec/controllers/spotlight/solr_controller_spec.rb +13 -14
  184. data/spec/controllers/spotlight/tags_controller_spec.rb +16 -16
  185. data/spec/controllers/spotlight/versions_controller_spec.rb +15 -18
  186. data/spec/controllers/spotlight/view_configurations_controller_spec.rb +31 -0
  187. data/spec/factories/contacts.rb +1 -3
  188. data/spec/factories/custom_fields.rb +2 -4
  189. data/spec/factories/exhibits.rb +4 -2
  190. data/spec/factories/pages.rb +8 -11
  191. data/spec/factories/resources.rb +4 -4
  192. data/spec/factories/searches.rb +10 -13
  193. data/spec/factories/tags.rb +0 -3
  194. data/spec/factories/users.rb +2 -4
  195. data/spec/features/about_page_spec.rb +19 -19
  196. data/spec/features/add_contacts_spec.rb +34 -34
  197. data/spec/features/add_custom_field_metadata_spec.rb +24 -26
  198. data/spec/features/add_item_bookmarklet_spec.rb +12 -12
  199. data/spec/features/browse_category_admin_spec.rb +39 -39
  200. data/spec/features/catalog_spec.rb +9 -10
  201. data/spec/features/confirm_email_spec.rb +10 -10
  202. data/spec/features/create_exhibit_spec.rb +18 -15
  203. data/spec/features/create_page_spec.rb +8 -8
  204. data/spec/features/curator_items.rb +3 -3
  205. data/spec/features/dashboard_spec.rb +15 -16
  206. data/spec/features/edit_contact_spec.rb +8 -10
  207. data/spec/features/edit_search_fields_spec.rb +79 -0
  208. data/spec/features/exhibits/add_tags_spec.rb +15 -16
  209. data/spec/features/exhibits/administration_spec.rb +25 -25
  210. data/spec/features/exhibits/custom_metadata_fields_spec.rb +41 -43
  211. data/spec/features/exhibits/edit_metadata_fields_spec.rb +19 -19
  212. data/spec/features/feature_page_spec.rb +49 -48
  213. data/spec/features/home_page_spec.rb +38 -38
  214. data/spec/features/import_exhibit_spec.rb +23 -20
  215. data/spec/features/item_admin_spec.rb +28 -29
  216. data/spec/features/javascript/about_page_admin_spec.rb +9 -9
  217. data/spec/features/javascript/block_controls_spec.rb +16 -18
  218. data/spec/features/javascript/blocks/featured_browse_categories_block_spec.rb +11 -12
  219. data/spec/features/javascript/blocks/featured_pages_block_spec.rb +12 -13
  220. data/spec/features/javascript/blocks/search_result_block_spec.rb +17 -17
  221. data/spec/features/javascript/blocks/solr_documents_block_spec.rb +67 -65
  222. data/spec/features/javascript/edit_in_place_spec.rb +30 -56
  223. data/spec/features/javascript/feature_page_admin_spec.rb +73 -59
  224. data/spec/features/javascript/home_page_edit_spec.rb +6 -6
  225. data/spec/features/javascript/metadata_admin_spec.rb +15 -15
  226. data/spec/features/javascript/multi_image_select_spec.rb +17 -18
  227. data/spec/features/javascript/preview_block_spec.rb +11 -12
  228. data/spec/features/javascript/rule_block_spec.rb +7 -7
  229. data/spec/features/javascript/search_config_admin_spec.rb +144 -0
  230. data/spec/features/javascript/search_context_spec.rb +41 -42
  231. data/spec/features/main_navigation_spec.rb +22 -22
  232. data/spec/features/metadata_admin_spec.rb +8 -8
  233. data/spec/features/multiple_exhibits_spec.rb +15 -17
  234. data/spec/features/report_a_problem_spec.rb +23 -17
  235. data/spec/features/site_masthead_spec.rb +46 -46
  236. data/spec/features/slideshow_spec.rb +8 -9
  237. data/spec/features/tags_admin_spec.rb +11 -11
  238. data/spec/features/upload_non_repository_item_spec.rb +32 -32
  239. data/spec/features/user_admin_spec.rb +7 -7
  240. data/spec/helpers/spotlight/application_helper_spec.rb +63 -56
  241. data/spec/helpers/spotlight/browse_helper_spec.rb +6 -6
  242. data/spec/helpers/spotlight/crud_link_helpers_spec.rb +58 -52
  243. data/spec/helpers/spotlight/main_app_helpers_spec.rb +44 -24
  244. data/spec/helpers/spotlight/navbar_helper_spec.rb +4 -4
  245. data/spec/helpers/spotlight/pages_helper_spec.rb +56 -37
  246. data/spec/helpers/spotlight/roles_helper_spec.rb +3 -3
  247. data/spec/helpers/spotlight/search_configurations_helper_spec.rb +24 -0
  248. data/spec/helpers/spotlight/title_helper_spec.rb +33 -35
  249. data/spec/jobs/spotlight/reindex_job_spec.rb +6 -6
  250. data/spec/jobs/spotlight/rename_sidecar_field_job_spec.rb +7 -7
  251. data/spec/lib/spotlight/catalog/access_controls_enforcement_spec.rb +41 -26
  252. data/spec/lib/spotlight/controller_spec.rb +10 -10
  253. data/spec/lib/spotlight/routes_spec.rb +4 -4
  254. data/spec/mailers/spotlight/indexing_complete_mailer_spec.rb +13 -13
  255. data/spec/models/sir_trevor_rails/blocks/solr_documents_block_spec.rb +8 -8
  256. data/spec/models/{spotlight/sir_trevor_blocks → sir_trevor_rails/blocks}/textable_spec.rb +7 -8
  257. data/spec/models/solr_document_spec.rb +49 -50
  258. data/spec/models/spotlight/ability_spec.rb +17 -20
  259. data/spec/models/spotlight/about_page_spec.rb +6 -6
  260. data/spec/models/spotlight/analytics/ga_spec.rb +2 -2
  261. data/spec/models/spotlight/blacklight_configuration_spec.rb +196 -133
  262. data/spec/models/spotlight/contact_email_spec.rb +28 -24
  263. data/spec/models/spotlight/contact_spec.rb +6 -7
  264. data/spec/models/spotlight/custom_field_spec.rb +51 -52
  265. data/spec/models/spotlight/exhibit_spec.rb +102 -60
  266. data/spec/models/spotlight/feature_page_spec.rb +32 -32
  267. data/spec/models/spotlight/featured_image_spec.rb +5 -5
  268. data/spec/models/spotlight/field_metadata_spec.rb +78 -0
  269. data/spec/models/spotlight/home_page_spec.rb +11 -20
  270. data/spec/models/spotlight/{spotlight_image_derivatives_spec.rb → image_derivatives_spec.rb} +2 -2
  271. data/spec/models/spotlight/main_navigation_spec.rb +7 -9
  272. data/spec/models/spotlight/masthead_spec.rb +6 -6
  273. data/spec/models/spotlight/page_spec.rb +26 -29
  274. data/spec/models/spotlight/resource_spec.rb +56 -26
  275. data/spec/models/spotlight/resources/open_graph_spec.rb +16 -18
  276. data/spec/models/spotlight/resources/upload_spec.rb +32 -35
  277. data/spec/models/spotlight/resources/web_spec.rb +9 -11
  278. data/spec/models/spotlight/role_spec.rb +20 -20
  279. data/spec/models/spotlight/search_spec.rb +32 -15
  280. data/spec/models/spotlight/solr_document/atomic_updates_spec.rb +34 -0
  281. data/spec/models/spotlight/{solr_document_images_spec.rb → solr_document/spotlight_images_spec.rb} +18 -10
  282. data/spec/models/spotlight/solr_document/uploaded_resource_spec.rb +52 -0
  283. data/spec/models/spotlight/solr_document_sidecar_spec.rb +3 -4
  284. data/spec/routing/spotlight/exhibit_catalog_spec.rb +5 -5
  285. data/spec/routing/spotlight/pages_routing_spec.rb +24 -25
  286. data/spec/serializers/spotlight/exhibit_export_serializer_spec.rb +88 -68
  287. data/spec/spec_helper.rb +13 -22
  288. data/spec/support/features/test_features_helpers.rb +50 -46
  289. data/spec/support/views/test_view_helpers.rb +1 -1
  290. data/spec/test_app_templates/Gemfile.extra +0 -1
  291. data/spec/test_app_templates/carrierwave.rb +2 -2
  292. data/spec/test_app_templates/catalog_controller.rb +42 -42
  293. data/spec/test_app_templates/lib/generators/test_app_generator.rb +20 -11
  294. data/spec/test_app_templates/spotlight_test.rake +3 -3
  295. data/spec/uploaders/spotlight/item_uploader_spec.rb +68 -0
  296. data/spec/views/_user_util_links.html.erb_spec.rb +14 -14
  297. data/spec/views/shared/_analytics.html.erb_spec.rb +4 -6
  298. data/spec/views/shared/_exhibit_masthead.html.erb_spec.rb +19 -20
  299. data/spec/views/shared/_exhibit_navbar.html.erb_spec.rb +129 -131
  300. data/spec/views/shared/_footer.html.erb_spec.rb +6 -7
  301. data/spec/views/shared/_header_navbar.html.erb_spec.rb +5 -5
  302. data/spec/views/spotlight/about_pages/_empty.html.erb_spec.rb +4 -4
  303. data/spec/views/spotlight/about_pages/_sidebar.html.erb_spec.rb +9 -11
  304. data/spec/views/spotlight/about_pages/index.html.erb_spec.rb +31 -28
  305. data/spec/views/spotlight/browse/_search.html.erb_spec.rb +6 -6
  306. data/spec/views/spotlight/browse/_sort_and_per_page.html.erb_spec.rb +10 -12
  307. data/spec/views/spotlight/browse/index.html.erb_spec.rb +6 -6
  308. data/spec/views/spotlight/browse/show.html.erb_spec.rb +19 -20
  309. data/spec/views/spotlight/catalog/_edit_default.html.erb_spec.rb +7 -7
  310. data/spec/views/spotlight/catalog/admin.html.erb_spec.rb +7 -7
  311. data/spec/views/spotlight/catalog/edit.html.erb_spec.rb +3 -6
  312. data/spec/views/spotlight/catalog/new.html.erb_spec.rb +8 -10
  313. data/spec/views/spotlight/contacts/edit.html.erb_spec.rb +8 -8
  314. data/spec/views/spotlight/dashboards/_analytics.html.erb_spec.rb +22 -22
  315. data/spec/views/spotlight/exhibits/edit.html.erb_spec.rb +7 -8
  316. data/spec/views/spotlight/feature_pages/_empty.html.erb_spec.rb +5 -5
  317. data/spec/views/spotlight/feature_pages/_sidebar.html.erb_spec.rb +24 -26
  318. data/spec/views/spotlight/home_pages/_empty.html.erb_spec.rb +13 -9
  319. data/spec/views/spotlight/home_pages/_sidebar.html.erb_spec.rb +6 -6
  320. data/spec/views/spotlight/metadata_configurations/_metadata_field.html.erb_spec.rb +26 -0
  321. data/spec/views/spotlight/metadata_configurations/edit.html.erb_spec.rb +22 -0
  322. data/spec/views/spotlight/pages/edit.html.erb_spec.rb +16 -16
  323. data/spec/views/spotlight/pages/index.html.erb_spec.rb +19 -17
  324. data/spec/views/spotlight/pages/new.html.erb_spec.rb +6 -6
  325. data/spec/views/spotlight/pages/show.html.erb_spec.rb +27 -28
  326. data/spec/views/spotlight/roles/index.html.erb_spec.rb +6 -6
  327. data/spec/views/spotlight/search_configurations/_search_fields.html.erb_spec.rb +54 -0
  328. data/spec/views/spotlight/search_configurations/_sort.html.erb_spec.rb +28 -0
  329. data/spec/views/spotlight/searches/_search.html.erb_spec.rb +20 -10
  330. data/spec/views/spotlight/searches/edit.html.erb_spec.rb +12 -11
  331. data/spec/views/spotlight/searches/index.html.erb_spec.rb +12 -7
  332. data/spec/views/spotlight/sir_trevor/blocks/_iframe_block.html.erb_spec.rb +7 -8
  333. data/spec/views/spotlight/sir_trevor/blocks/_rule_block.html.erb_spec.rb +4 -4
  334. data/spec/views/spotlight/sir_trevor/blocks/_solr_documents_block.html.erb_spec.rb +38 -0
  335. data/spec/views/spotlight/sir_trevor/blocks/_solr_documents_carousel_block.html.erb_spec.rb +7 -8
  336. data/spec/views/spotlight/sir_trevor/blocks/_solr_documents_embed_block.html.erb_spec.rb +18 -13
  337. data/spec/views/spotlight/sir_trevor/blocks/_solr_documents_features_block.html.erb_spec.rb +14 -12
  338. data/spec/views/spotlight/sir_trevor/blocks/_solr_documents_grid_block.html.erb_spec.rb +11 -12
  339. data/spec/views/spotlight/tags/index.html.erb_spec.rb +9 -9
  340. metadata +167 -83
  341. data/app/controllers/spotlight/blacklight_configurations_controller.rb +0 -101
  342. data/app/controllers/spotlight/resources/csv_controller.rb +0 -23
  343. data/app/forms/spotlight/appearance.rb +0 -86
  344. data/app/helpers/spotlight/attachments_helper.rb +0 -4
  345. data/app/helpers/spotlight/blacklight_configurations_helper.rb +0 -15
  346. data/app/jobs/spotlight/set_default_thumbnail_job.rb +0 -16
  347. data/app/models/spotlight/resources/csv.rb +0 -80
  348. data/app/uploaders/spotlight/csv_uploader.rb +0 -15
  349. data/app/views/spotlight/blacklight_configurations/edit_facet_fields.html.erb +0 -47
  350. data/app/views/spotlight/blacklight_configurations/edit_sort_fields.html.erb +0 -65
  351. data/app/views/spotlight/resources/csv/_form.html.erb +0 -17
  352. data/app/views/spotlight/resources/csv/new.html.erb +0 -3
  353. data/spec/controllers/spotlight/blacklight_configurations_controller_spec.rb +0 -167
  354. data/spec/features/edit_sort_fields_spec.rb +0 -42
  355. data/spec/features/exhibits/edit_facet_fields_spec.rb +0 -37
  356. data/spec/features/javascript/facet_admin_spec.rb +0 -36
  357. data/spec/features/search_facets_admin_spec.rb +0 -19
  358. data/spec/features/update_appearance_spec.rb +0 -58
  359. data/spec/helpers/spotlight/blacklight_configurations_helper_spec.rb +0 -12
  360. data/spec/models/spotlight/appearance_spec.rb +0 -33
  361. data/spec/models/spotlight/resources/csv_spec.rb +0 -101
  362. data/spec/models/spotlight/solr_document_atomic_updates_spec.rb +0 -29
  363. data/spec/models/spotlight/solr_document_uploaded_resource_spec.rb +0 -45
  364. data/spec/uploaders/item_uploader_spec.rb +0 -62
  365. data/spec/views/spotlight/blacklight_configuration/_metadata_field.html.erb_spec.rb +0 -27
  366. data/spec/views/spotlight/blacklight_configuration/edit_metadata_fields.html.erb_spec.rb +0 -23
  367. data/spec/views/spotlight/blacklight_configuration/edit_sort_fields.html.erb_spec.rb +0 -19
@@ -1,77 +1,73 @@
1
1
  require 'spec_helper'
2
2
 
3
- describe Spotlight::ResourcesController, :type => :controller do
3
+ describe Spotlight::ResourcesController, type: :controller do
4
4
  routes { Spotlight::Engine.routes }
5
5
  let(:exhibit) { FactoryGirl.create(:exhibit) }
6
6
 
7
- describe "when not logged in" do
8
-
9
- describe "GET new" do
10
- it "should not be allowed" do
7
+ describe 'when not logged in' do
8
+ describe 'GET new' do
9
+ it 'does not be allowed' do
11
10
  get :new, exhibit_id: exhibit
12
11
  expect(response).to redirect_to main_app.new_user_session_path
13
12
  end
14
13
  end
15
14
 
16
- describe "POST create" do
17
- it "should not be allowed" do
15
+ describe 'POST create' do
16
+ it 'does not be allowed' do
18
17
  post :create, exhibit_id: exhibit
19
18
  expect(response).to redirect_to main_app.new_user_session_path
20
19
  end
21
20
  end
22
21
 
23
- describe "POST reindex_all" do
24
- it "should not be allowed" do
22
+ describe 'POST reindex_all' do
23
+ it 'does not be allowed' do
25
24
  post :reindex_all, exhibit_id: exhibit
26
25
  expect(response).to redirect_to main_app.new_user_session_path
27
26
  end
28
27
  end
29
28
  end
30
29
 
31
-
32
- describe "when signed in as a curator" do
30
+ describe 'when signed in as a curator' do
33
31
  let(:user) { FactoryGirl.create(:exhibit_curator, exhibit: exhibit) }
34
- before {sign_in user }
32
+ before { sign_in user }
35
33
 
36
- describe "GET new" do
37
-
38
- it "should render form" do
34
+ describe 'GET new' do
35
+ it 'renders form' do
39
36
  get :new, exhibit_id: exhibit
40
- expect(response).to render_template "spotlight/resources/new"
37
+ expect(response).to render_template 'spotlight/resources/new'
41
38
  end
42
39
 
43
- it "should populate the resource with parameters from the url" do
44
- get :new, exhibit_id: exhibit, resource: { url: "info:uri"}
45
- expect(assigns[:resource].url).to eq "info:uri"
40
+ it 'populates the resource with parameters from the url' do
41
+ get :new, exhibit_id: exhibit, resource: { url: 'info:uri' }
42
+ expect(assigns[:resource].url).to eq 'info:uri'
46
43
  end
47
44
 
48
- describe "Within a popup" do
49
- it "should render with the simplified popup layout" do
45
+ describe 'Within a popup' do
46
+ it 'renders with the simplified popup layout' do
50
47
  get :new, exhibit_id: exhibit, popup: true
51
- expect(response).to render_template "layouts/spotlight/popup"
48
+ expect(response).to render_template 'layouts/spotlight/popup'
52
49
  end
53
50
  end
54
51
  end
55
52
 
56
- describe "POST create" do
53
+ describe 'POST create' do
57
54
  let(:blacklight_solr) { double }
58
- it "create a resource" do
55
+ it 'create a resource' do
59
56
  allow_any_instance_of(Spotlight::Resource).to receive(:reindex)
60
57
  expect(blacklight_solr).to receive(:commit)
61
58
  allow_any_instance_of(Spotlight::Resource).to receive(:blacklight_solr).and_return blacklight_solr
62
- post :create, exhibit_id: exhibit, resource: { url: "info:uri" }
59
+ post :create, exhibit_id: exhibit, resource: { url: 'info:uri' }
63
60
  expect(assigns[:resource]).to be_persisted
64
61
  end
65
62
  end
66
63
 
67
- describe "POST reindex_all" do
68
- it "should trigger a reindex" do
64
+ describe 'POST reindex_all' do
65
+ it 'triggers a reindex' do
69
66
  expect_any_instance_of(Spotlight::Exhibit).to receive(:reindex_later)
70
67
  post :reindex_all, exhibit_id: exhibit
71
68
  expect(response).to redirect_to admin_exhibit_catalog_index_path(exhibit)
72
- expect(flash[:notice]).to match /Reindexing/
69
+ expect(flash[:notice]).to include 'Reindexing'
73
70
  end
74
71
  end
75
72
  end
76
-
77
73
  end
@@ -1,57 +1,75 @@
1
1
  require 'spec_helper'
2
2
 
3
- describe Spotlight::RolesController, :type => :controller do
3
+ describe Spotlight::RolesController, type: :controller do
4
4
  routes { Spotlight::Engine.routes }
5
- let(:exhibit) {FactoryGirl.create(:exhibit) }
5
+ let(:exhibit) { FactoryGirl.create(:exhibit) }
6
6
 
7
- describe "when user does not have access" do
7
+ describe 'when user does not have access' do
8
8
  before { sign_in FactoryGirl.create(:exhibit_visitor) }
9
- it "should not allow index" do
9
+ it 'does not allow index' do
10
10
  get :index, exhibit_id: exhibit
11
11
  expect(response).to redirect_to main_app.root_path
12
12
  end
13
13
  end
14
14
 
15
- describe "when user is an admin" do
15
+ describe 'when user is an admin' do
16
16
  let(:admin) { FactoryGirl.create(:exhibit_admin, exhibit: exhibit) }
17
17
  let(:role) { admin.roles.first }
18
18
  before { sign_in admin }
19
- it "should allow index" do
20
- expect(controller).to receive(:add_breadcrumb).with("Home", exhibit)
21
- expect(controller).to receive(:add_breadcrumb).with("Administration", exhibit_dashboard_path(exhibit))
22
- expect(controller).to receive(:add_breadcrumb).with("Users", exhibit_roles_path(exhibit))
19
+ it 'allows index' do
20
+ expect(controller).to receive(:add_breadcrumb).with('Home', exhibit)
21
+ expect(controller).to receive(:add_breadcrumb).with('Administration', exhibit_dashboard_path(exhibit))
22
+ expect(controller).to receive(:add_breadcrumb).with('Users', exhibit_roles_path(exhibit))
23
23
  get :index, exhibit_id: exhibit
24
24
  expect(response).to be_successful
25
25
  expect(assigns[:roles].to_a).to eq [admin.roles.first]
26
26
  end
27
27
 
28
- describe "PATCH update_all" do
29
- it "should be successful" do
30
- patch :update_all, exhibit_id: exhibit, "exhibit"=>{"roles_attributes"=>{"0"=>{"user_key"=>"cbeer@cbeer.io", "role"=>"curator", "id"=>role.id}, '1' => {"user_key"=>"", "role"=>"admin"}}}
28
+ describe 'PATCH update_all' do
29
+ it 'is successful' do
30
+ patch :update_all, exhibit_id: exhibit, 'exhibit' => {
31
+ 'roles_attributes' => {
32
+ '0' => { 'user_key' => 'cbeer@cbeer.io', 'role' => 'curator', 'id' => role.id },
33
+ '1' => { 'user_key' => '', 'role' => 'admin' }
34
+ }
35
+ }
31
36
  expect(response).to redirect_to exhibit_roles_path(exhibit)
32
37
  expect(flash[:notice]).to eq 'User has been updated.'
33
38
  expect(admin.reload.roles.first.role).to eq 'curator'
34
39
  expect(admin.reload.roles.first.user.email).to eq 'cbeer@cbeer.io'
35
40
  end
36
41
 
37
- it "should authorize records" do
42
+ it 'authorizes records' do
38
43
  allow(controller).to receive(:authorize!).and_raise(CanCan::AccessDenied)
39
- patch :update_all, exhibit_id: exhibit, "exhibit"=>{"roles_attributes"=>{"0"=>{"user_key"=>"cbeer@cbeer.info", "role"=>"curator", "id"=>role.id}}}
40
- expect(response).to redirect_to main_app.root_path
44
+ patch :update_all, exhibit_id: exhibit, 'exhibit' => {
45
+ 'roles_attributes' => {
46
+ '0' => { 'user_key' => 'cbeer@cbeer.info', 'role' => 'curator', 'id' => role.id }
47
+ }
48
+ }
49
+ expect(response).to redirect_to main_app.root_path
41
50
  expect(flash[:alert]).to eq 'You are not authorized to access this page.'
42
51
  expect(admin.reload.roles.first.role).to eq 'admin'
43
52
  end
44
53
 
45
- it "should destroy records" do
46
- patch :update_all, exhibit_id: exhibit, "exhibit"=>{"roles_attributes"=>{"0"=>{"user_key"=>"cbeer@cbeer.info", "role"=>"curator", "id"=>role.id, '_destroy' => '1'}}}
54
+ it 'destroys records' do
55
+ patch :update_all, exhibit_id: exhibit, 'exhibit' => {
56
+ 'roles_attributes' => {
57
+ '0' => { 'user_key' => 'cbeer@cbeer.info', 'role' => 'curator', 'id' => role.id, '_destroy' => '1' }
58
+ }
59
+ }
60
+
47
61
  expect(response).to redirect_to exhibit_roles_path(exhibit)
48
62
  expect(admin.reload.roles).to be_empty
49
63
  expect(flash[:notice]).to eq 'User has been removed.'
50
64
  end
51
65
 
52
- it "should handle failure" do
66
+ it 'handles failure' do
53
67
  allow_any_instance_of(Spotlight::Exhibit).to receive_messages(update: false)
54
- patch :update_all, exhibit_id: exhibit, "exhibit"=>{"roles_attributes"=>{"0"=>{"user_key"=>"cbeer@cbeer.info", "role"=>"curator", "id"=>role.id}}}
68
+ patch :update_all, exhibit_id: exhibit, 'exhibit' => {
69
+ 'roles_attributes' => {
70
+ '0' => { 'user_key' => 'cbeer@cbeer.info', 'role' => 'curator', 'id' => role.id }
71
+ }
72
+ }
55
73
  expect(response).to be_successful
56
74
  expect(flash[:alert]).to eq 'There was a problem saving the users.'
57
75
  end
@@ -0,0 +1,125 @@
1
+ require 'spec_helper'
2
+ describe Spotlight::SearchConfigurationsController, type: :controller do
3
+ routes { Spotlight::Engine.routes }
4
+ let(:exhibit) { FactoryGirl.create(:exhibit) }
5
+
6
+ describe 'when the user is not authorized' do
7
+ before do
8
+ sign_in FactoryGirl.create(:exhibit_visitor)
9
+ end
10
+
11
+ it 'denies access' do
12
+ get :edit, exhibit_id: exhibit
13
+ expect(response).to redirect_to main_app.root_path
14
+ expect(flash[:alert]).to be_present
15
+ end
16
+ end
17
+
18
+ describe 'when not logged in' do
19
+ describe '#update' do
20
+ it 'denies access' do
21
+ patch :update, exhibit_id: exhibit
22
+ expect(response).to redirect_to main_app.new_user_session_path
23
+ end
24
+ end
25
+
26
+ describe '#edit' do
27
+ it 'denies access' do
28
+ get :edit, exhibit_id: exhibit
29
+ expect(response).to redirect_to main_app.new_user_session_path
30
+ end
31
+ end
32
+ end
33
+
34
+ describe 'when signed in' do
35
+ let(:user) { FactoryGirl.create(:exhibit_admin, exhibit: exhibit) }
36
+ before { sign_in user }
37
+
38
+ describe '#edit' do
39
+ it 'is successful' do
40
+ expect(controller).to receive(:add_breadcrumb).with('Home', exhibit)
41
+ expect(controller).to receive(:add_breadcrumb).with('Curation', exhibit_dashboard_path(exhibit))
42
+ expect(controller).to receive(:add_breadcrumb).with('Search', edit_exhibit_search_configuration_path(exhibit))
43
+ get :edit, exhibit_id: exhibit
44
+ expect(response).to be_successful
45
+ end
46
+
47
+ it 'assigns the field metadata' do
48
+ get :edit, exhibit_id: exhibit
49
+ expect(assigns(:field_metadata)).to be_an_instance_of(Spotlight::FieldMetadata)
50
+ expect(assigns(:field_metadata).repository).to eq controller.repository
51
+ expect(assigns(:field_metadata).blacklight_config).to eq controller.blacklight_config
52
+ end
53
+ end
54
+
55
+ describe '#update' do
56
+ it 'updates facet fields' do
57
+ patch :update, exhibit_id: exhibit, blacklight_configuration: {
58
+ facet_fields: { 'genre_ssim' => { enabled: '1', label: 'Label' } }
59
+ }
60
+ expect(flash[:notice]).to eq 'The exhibit was successfully updated.'
61
+ expect(response).to redirect_to edit_exhibit_search_configuration_path(exhibit)
62
+ assigns[:exhibit].tap do |saved|
63
+ expect(saved.blacklight_configuration.facet_fields.keys).to eq ['genre_ssim']
64
+ end
65
+ end
66
+
67
+ it 'updates sort fields' do
68
+ patch :update, exhibit_id: exhibit, blacklight_configuration: {
69
+ sort_fields: {
70
+ 'relevance' => { 'enabled' => '1', 'label' => 'Relevance' },
71
+ 'title' => { 'enabled' => '1', 'label' => 'Title' },
72
+ 'type' => { 'enabled' => '1', 'label' => 'Type' },
73
+ 'date' => { 'enabled' => '0', 'label' => 'Date' },
74
+ 'source' => { 'enabled' => '0', 'label' => 'Source' },
75
+ 'identifier' => { 'enabled' => '0', 'label' => 'Identifier' }
76
+ }
77
+ }
78
+ expect(flash[:notice]).to eq 'The exhibit was successfully updated.'
79
+ expect(response).to redirect_to edit_exhibit_search_configuration_path(exhibit)
80
+ assigns[:exhibit].tap do |saved|
81
+ expect(saved.blacklight_configuration.sort_fields).to eq(
82
+ 'relevance' => { 'label' => 'Relevance', 'enabled' => true },
83
+ 'title' => { 'label' => 'Title', 'enabled' => true },
84
+ 'type' => { 'label' => 'Type', 'enabled' => true },
85
+ 'date' => { 'label' => 'Date', 'enabled' => false },
86
+ 'source' => { 'label' => 'Source', 'enabled' => false },
87
+ 'identifier' => { 'label' => 'Identifier', 'enabled' => false }
88
+ )
89
+ end
90
+ end
91
+
92
+ it 'updates search fields' do
93
+ patch :update, exhibit_id: exhibit, blacklight_configuration: {
94
+ search_fields: {
95
+ 'all_fields' => { 'enabled' => '1' },
96
+ 'title' => { 'enabled' => '0', 'label' => 'Title' },
97
+ 'author' => { 'enabled' => '1', 'label' => 'Primary Author' }
98
+ }
99
+ }
100
+ expect(flash[:notice]).to eq 'The exhibit was successfully updated.'
101
+ expect(response).to redirect_to edit_exhibit_search_configuration_path(exhibit)
102
+ assigns[:exhibit].tap do |saved|
103
+ expect(saved.blacklight_configuration.search_fields).to eq(
104
+ 'all_fields' => { 'label' => 'All fields', 'enabled' => true },
105
+ 'title' => { 'label' => 'Title', 'enabled' => false },
106
+ 'author' => { 'label' => 'Primary Author', 'enabled' => true }
107
+ )
108
+ end
109
+ end
110
+
111
+ it 'updates appearance fields' do
112
+ patch :update, exhibit_id: exhibit, blacklight_configuration: {
113
+ document_index_view_types: { 'list' => '1', 'gallery' => '1', 'map' => '0' },
114
+ default_per_page: '50'
115
+ }
116
+ expect(flash[:notice]).to eq 'The exhibit was successfully updated.'
117
+ expect(response).to redirect_to edit_exhibit_search_configuration_path(exhibit)
118
+ assigns[:exhibit].tap do |saved|
119
+ expect(saved.blacklight_configuration.document_index_view_types).to eq %w(list gallery)
120
+ expect(saved.blacklight_configuration.default_per_page).to eq 50
121
+ end
122
+ end
123
+ end
124
+ end
125
+ end
@@ -1,95 +1,100 @@
1
1
  require 'spec_helper'
2
2
 
3
- describe Spotlight::SearchesController, :type => :controller do
3
+ describe Spotlight::SearchesController, type: :controller do
4
4
  routes { Spotlight::Engine.routes }
5
5
  let(:exhibit) { FactoryGirl.create(:exhibit) }
6
6
 
7
- describe "when the user is not authorized" do
8
-
7
+ describe 'when the user is not authorized' do
9
8
  before do
10
9
  sign_in FactoryGirl.create(:exhibit_visitor)
11
10
  end
12
11
 
13
- it "should raise an error" do
14
- post :create, exhibit_id: exhibit
12
+ it 'raises an error' do
13
+ post :create, exhibit_id: exhibit
15
14
  expect(response).to redirect_to main_app.root_path
16
15
  expect(flash[:alert]).to be_present
17
16
  end
18
17
 
19
- it "should raise an error" do
18
+ it 'raises an error' do
20
19
  get :index, exhibit_id: exhibit
21
20
  expect(response).to redirect_to main_app.root_path
22
21
  expect(flash[:alert]).to be_present
23
22
  end
24
23
  end
25
24
 
26
- describe "when the user is a curator" do
25
+ describe 'when the user is a curator' do
27
26
  before do
28
27
  sign_in FactoryGirl.create(:exhibit_curator, exhibit: exhibit)
29
28
  end
30
29
  let(:search) { FactoryGirl.create(:search, exhibit: exhibit) }
31
30
 
32
- it "should create a saved search" do
33
- request.env["HTTP_REFERER"] = "/referring_url"
34
- post :create, "search"=>{"title"=>"A bunch of maps"}, "f"=>{"genre_ssim"=>["map"]}, exhibit_id: exhibit
35
- expect(response).to redirect_to "/referring_url"
36
- expect(flash[:notice]).to eq "The search was created."
37
- expect(assigns[:search].title).to eq "A bunch of maps"
38
- expect(assigns[:search].query_params).to eq("f"=>{"genre_ssim"=>["map"]})
31
+ it 'creates a saved search' do
32
+ request.env['HTTP_REFERER'] = '/referring_url'
33
+ post :create, 'search' => { 'title' => 'A bunch of maps' }, 'f' => { 'genre_ssim' => ['map'] }, exhibit_id: exhibit
34
+ expect(response).to redirect_to '/referring_url'
35
+ expect(flash[:notice]).to eq 'The search was created.'
36
+ expect(assigns[:search].title).to eq 'A bunch of maps'
37
+ expect(assigns[:search].query_params).to eq('f' => { 'genre_ssim' => ['map'] })
39
38
  end
40
39
 
41
- describe "GET index" do
40
+ describe 'GET index' do
42
41
  let!(:search) { FactoryGirl.create(:search, exhibit: exhibit) }
43
- it "should show all the items" do
44
- expect(controller).to receive(:add_breadcrumb).with("Home", exhibit)
45
- expect(controller).to receive(:add_breadcrumb).with("Curation", exhibit_dashboard_path(exhibit))
46
- expect(controller).to receive(:add_breadcrumb).with("Browse", exhibit_searches_path(exhibit))
47
- get :index, exhibit_id: search.exhibit_id
42
+ it 'shows all the items' do
43
+ expect(controller).to receive(:add_breadcrumb).with('Home', exhibit)
44
+ expect(controller).to receive(:add_breadcrumb).with('Curation', exhibit_dashboard_path(exhibit))
45
+ expect(controller).to receive(:add_breadcrumb).with('Browse', exhibit_searches_path(exhibit))
46
+ get :index, exhibit_id: search.exhibit_id
48
47
  expect(response).to be_successful
49
48
  expect(assigns[:exhibit]).to eq search.exhibit
50
49
  expect(assigns[:searches]).to include search
51
50
  end
52
51
 
53
- it "should have a JSON response with published resources" do
54
- search.on_landing_page = true
52
+ it 'has a JSON response with published resources' do
53
+ search.published = true
55
54
  search.save!
56
55
 
57
56
  get :index, exhibit_id: exhibit, format: 'json'
58
57
  expect(response).to be_successful
59
58
  json = JSON.parse(response.body)
60
59
  expect(json.size).to eq 1
61
- expect(json.last).to include({'id' => search.id, "title"=>"Search1"})
60
+ expect(json.last).to include('id' => search.id, 'title' => 'Search1')
62
61
  end
63
62
  end
64
63
 
65
- describe "GET autocomplete" do
66
- let(:search) { FactoryGirl.create(:search, exhibit: exhibit, title: "New Mexico Maps", query_params: {q: "New Mexico"} ) }
67
- let(:search_fq) { FactoryGirl.create(:search, exhibit: exhibit, title: "New Mexico Maps", query_params: {f: { subject_geographic_ssim: ["Pacific Ocean"]}} ) }
68
- it "should show all the items returned search's query_params" do
64
+ describe 'GET autocomplete' do
65
+ let(:search) do
66
+ FactoryGirl.create(:search, exhibit: exhibit, title: 'New Mexico Maps', query_params: { q: 'New Mexico' })
67
+ end
68
+
69
+ let(:search_fq) do
70
+ FactoryGirl.create(:search, exhibit: exhibit, title: 'New Mexico Maps', query_params: { f: { subject_geographic_ssim: ['Pacific Ocean'] } })
71
+ end
72
+
73
+ it "shows all the items returned search's query_params" do
69
74
  pending("A search defined by a query doesn't work with autocomplete correctly.")
70
75
  get :autocomplete, exhibit_id: exhibit, id: search, format: 'json'
71
76
  expect(response).to be_successful
72
77
  docs = JSON.parse(response.body)['docs']
73
- doc_ids = docs.map{|d| d["id"] }
78
+ doc_ids = docs.map { |d| d['id'] }
74
79
  expect(docs.length).to eq 2
75
- expect(doc_ids).to include "cz507zk0531"
76
- expect(doc_ids).to include "rz818vx8201"
80
+ expect(doc_ids).to include 'cz507zk0531'
81
+ expect(doc_ids).to include 'rz818vx8201'
77
82
  end
78
- it "should search within the items returned in the query_params" do
79
- get :autocomplete, exhibit_id: exhibit, id: search_fq, q: "California",format: 'json'
83
+ it 'searches within the items returned in the query_params' do
84
+ get :autocomplete, exhibit_id: exhibit, id: search_fq, q: 'California', format: 'json'
80
85
  expect(response).to be_successful
81
86
  docs = JSON.parse(response.body)['docs']
82
87
  expect(docs.length).to eq 1
83
- expect(docs.first["id"]).to eq "sn161bw2027"
84
- expect(docs.first["description"]).to eq "sn161bw2027"
85
- expect(docs.first["title"]).to match /Pas-caart van Zuyd-Zee/
86
- expect(docs.first).to have_key("thumbnail")
87
- expect(docs.first).to have_key("url")
88
+ expect(docs.first['id']).to eq 'sn161bw2027'
89
+ expect(docs.first['description']).to eq 'sn161bw2027'
90
+ expect(docs.first['title']).to match(/Pas-caart van Zuyd-Zee/)
91
+ expect(docs.first).to have_key('thumbnail')
92
+ expect(docs.first).to have_key('url')
88
93
  end
89
94
  end
90
95
 
91
- describe "GET edit" do
92
- it "should show edit page" do
96
+ describe 'GET edit' do
97
+ it 'shows edit page' do
93
98
  get :edit, id: search, exhibit_id: search.exhibit
94
99
  expect(response).to be_successful
95
100
  expect(assigns[:search]).to eq search
@@ -97,44 +102,60 @@ describe Spotlight::SearchesController, :type => :controller do
97
102
  end
98
103
  end
99
104
 
100
- describe "PATCH update" do
101
- it "should show edit page" do
102
- patch :update, id: search, exhibit_id: search.exhibit, search: {title: 'Hey man', long_description: 'long', featured_image: 'http://lorempixel.com/64/64/'}
105
+ describe 'PATCH update' do
106
+ it 'shows edit page' do
107
+ patch :update, id: search, exhibit_id: search.exhibit, search: {
108
+ title: 'Hey man',
109
+ long_description: 'long',
110
+ featured_image: 'http://lorempixel.com/64/64/'
111
+ }
112
+
103
113
  expect(assigns[:search].title).to eq 'Hey man'
104
- expect(response).to redirect_to exhibit_searches_path(search.exhibit)
114
+ expect(response).to redirect_to exhibit_searches_path(search.exhibit)
105
115
  end
106
116
 
107
- it "should render edit if there's an error" do
117
+ it "renders edit if there's an error" do
108
118
  expect_any_instance_of(Spotlight::Search).to receive(:update).and_return(false)
109
- patch :update, id: search, exhibit_id: search.exhibit, search: {title: 'Hey man', long_description: 'long', featured_image: 'http://lorempixel.com/64/64/'}
110
- expect(response).to be_successful
119
+ patch :update, id: search, exhibit_id: search.exhibit, search: {
120
+ title: 'Hey man',
121
+ long_description: 'long',
122
+ featured_image: 'http://lorempixel.com/64/64/'
123
+ }
124
+
125
+ expect(response).to be_successful
111
126
  expect(response).to render_template 'edit'
112
127
  end
113
128
  end
114
129
 
115
- describe "DELETE destroy" do
130
+ describe 'DELETE destroy' do
116
131
  let!(:search) { FactoryGirl.create(:search, exhibit: exhibit) }
117
- it "should remove it" do
118
- expect {
132
+ it 'removes it' do
133
+ expect do
119
134
  delete :destroy, id: search, exhibit_id: search.exhibit
120
- }.to change { Spotlight::Search.count }.by(-1)
121
- expect(response).to redirect_to exhibit_searches_path(search.exhibit)
122
- expect(flash[:alert]).to eq "The search was deleted."
135
+ end.to change { Spotlight::Search.count }.by(-1)
136
+ expect(response).to redirect_to exhibit_searches_path(search.exhibit)
137
+ expect(flash[:alert]).to eq 'The search was deleted.'
123
138
  end
124
139
  end
125
140
 
126
- describe "POST update_all" do
127
- let!(:search2) { FactoryGirl.create(:search, exhibit: exhibit, on_landing_page: true ) }
128
- let!(:search3) { FactoryGirl.create(:search, exhibit: exhibit, on_landing_page: true ) }
129
- before { request.env["HTTP_REFERER"] = "http://example.com" }
130
- it "should update whether they are on the landing page" do
131
- post :update_all, exhibit_id: exhibit, exhibit: {searches_attributes: [{id: search.id, on_landing_page: true, weight: '1' }, {id: search2.id, on_landing_page: false, weight: '0'}]}
132
- expect(search.reload.on_landing_page).to be_truthy
141
+ describe 'POST update_all' do
142
+ let!(:search2) { FactoryGirl.create(:search, exhibit: exhibit, published: true) }
143
+ let!(:search3) { FactoryGirl.create(:search, exhibit: exhibit, published: true) }
144
+ before { request.env['HTTP_REFERER'] = 'http://example.com' }
145
+ it 'updates whether they are on the landing page' do
146
+ post :update_all, exhibit_id: exhibit, exhibit: {
147
+ searches_attributes: [
148
+ { id: search.id, published: true, weight: '1' },
149
+ { id: search2.id, published: false, weight: '0' }
150
+ ]
151
+ }
152
+
153
+ expect(search.reload.published).to be_truthy
133
154
  expect(search.weight).to eq 1
134
- expect(search2.reload.on_landing_page).to be_falsey
135
- expect(search3.reload.on_landing_page).to be_truthy # should remain untouched since it wasn't present
136
- expect(response).to redirect_to "http://example.com"
137
- expect(flash[:notice]).to eq "Searches were successfully updated."
155
+ expect(search2.reload.published).to be_falsey
156
+ expect(search3.reload.published).to be_truthy # should remain untouched since it wasn't present
157
+ expect(response).to redirect_to 'http://example.com'
158
+ expect(flash[:notice]).to eq 'Searches were successfully updated.'
138
159
  end
139
160
  end
140
161
  end