patricklindsay / wice_grid

A Rails grid plugin to create grids with sorting, pagination, and (automatically generated) filters
http://wicegrid.herokuapp.com/
MIT License
33 stars 29 forks source link

Port test suite from https://github.com/leikind/wice_grid_testbed #12

Closed JasonBarnabe closed 6 years ago

JasonBarnabe commented 6 years ago

Linked to #6 - Closes first part

~/wice_grid (test-suite) $ rspec
[...]
Finished in 5 minutes 32 seconds (files took 3.63 seconds to load)
215 examples, 23 failures

Failed examples:

rspec ./spec/features/auto_reloads2_request_spec.rb:293 # auto reloads WiceGrid should negate the semantics of the text  filter
rspec ./spec/features/auto_reloads2_request_spec.rb:304 # auto reloads WiceGrid should reload the title filter
rspec ./spec/features/auto_reloads_request_spec.rb:297 # auto reloads WiceGrid should negate the semantics of the text  filter
rspec ./spec/features/custom_filters1_request_spec.rb:53 # custom_ordering WiceGrid should filter by custom filter
rspec ./spec/features/custom_filters4_request_spec.rb:9 # custom_ordering WiceGrid should not be possible to expand custom filters
rspec ./spec/features/custom_ordering_request_spec.rb:9 # custom_ordering WiceGrid should be sorted by the length of the word
rspec ./spec/features/dates_request_spec.rb[1:11] # dates WiceGrid should filter by created_at
rspec ./spec/features/dates_request_spec.rb:25 # dates WiceGrid should filter by multiple fields
rspec ./spec/features/integration_with_application_view_request_spec.rb:9 # current_page_records & all_pages_records WiceGrid should return records displayed on the page and throughout all pages
rspec ./spec/features/integration_with_forms_request_spec.rb:27 # dump_filter_parameters_as_hidden_fields WiceGrid should keep the state of an integer filter while reloading another form
rspec ./spec/features/integration_with_forms_request_spec.rb:50 # dump_filter_parameters_as_hidden_fields WiceGrid should keep the state of a string filter while reloading another form
rspec ./spec/features/integration_with_forms_request_spec.rb:72 # dump_filter_parameters_as_hidden_fields WiceGrid should keep the state of a string filter with negation while reloading another form
rspec ./spec/features/integration_with_forms_request_spec.rb:95 # dump_filter_parameters_as_hidden_fields WiceGrid should keep the state of a custom filter with negation while reloading another form
rspec ./spec/features/integration_with_forms_request_spec.rb:118 # dump_filter_parameters_as_hidden_fields WiceGrid should keep the state of a date filter  while reloading another form
rspec ./spec/features/localization_request_spec.rb:9 # localization WiceGrid should switch to different languages
rspec ./spec/features/many_grids_on_page_request_spec.rb:39 # many_grids_on_page WiceGrid should paginate independantly
rspec ./spec/features/negation_request_spec.rb:9 # negation WiceGrid should negate the semantics of the text  filter
rspec ./spec/features/resultset_processings2_request_spec.rb:9 # with_resultset WiceGrid should show records displayed on all pages
rspec ./spec/features/resultset_processings2_request_spec.rb:29 # with_resultset WiceGrid should show records displayed on all pages with a custom filter selection
rspec ./spec/features/resultset_processings_request_spec.rb:9 # with_resultset WiceGrid should return records displayed on the page
rspec ./spec/features/saved_queries_request_spec.rb:17 # auto reloads WiceGrid should filter by Added
rspec ./spec/features/saved_queries_request_spec.rb:72 # auto reloads WiceGrid should filter by Archived and Project Name
rspec ./spec/wice/wice_grid_misc_spec.rb:149 # build_includes get_string_matching_operators

[Coveralls] Outside the CI environment, not sending data.
Coverage report generated for RSpec to /Users/jasonbarnabe/wice_grid/coverage. 4358 / 4907 LOC (88.81%) covered.

Can run the test app with cd spec/support/test_app/bin && rails s but you don't get fixture data this way.

patricklindsay commented 6 years ago

Thanks @JasonBarnabe ! Although it doesn't close #6. That issue includes fixing the failures :p

patricklindsay commented 6 years ago

@JasonBarnabe I'm not able to run the specs on master - could you have a look & possibly write a small testing section in the README?

JasonBarnabe commented 6 years ago

Trying on a fresh computer, the only requirements outside of running bundle were having Postgres set up and installing phantomjs. Were you having issues outside of these two things?

JasonBarnabe commented 6 years ago

13 will remove the Postgres requirement.

JasonBarnabe commented 6 years ago

Instructions in #15.