diff options
| author | Teddy Wing | 2017-06-08 14:40:26 +0200 |
|---|---|---|
| committer | Teddy Wing | 2017-06-08 14:40:26 +0200 |
| commit | fa04c7ecc9dff0a06ad5a9c793a7b17defcf4300 (patch) | |
| tree | 335c6a20bd6d430f1d78874c4b26a6e8451946d4 /spec | |
| parent | ebd01ff27d3a4b2a57566632ce37873de4acf810 (diff) | |
| download | chouette-core-fa04c7ecc9dff0a06ad5a9c793a7b17defcf4300.tar.bz2 | |
TableBuilder spec: Add a couple TODOs
It's not actually a collection of `workbenches` as we originally
thought, it's a collection of referentials. The name of the variable
should be changed to reflect that.
Also, in the real code, this value is not an array, but an
`ActiveRecord::Relation`, thus the `sortable_columns` method is able to
call `.model` on it. Since the test uses an array, that breaks. Need to
figure out what we want to do there. I don't like the dynamic-ness of
the way this is done now, but we need to figure out if it's possible to
factor that out in a pleasant way.
Refs #3479
Diffstat (limited to 'spec')
| -rw-r--r-- | spec/helpers/table_builder_helper_spec.rb | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/spec/helpers/table_builder_helper_spec.rb b/spec/helpers/table_builder_helper_spec.rb index 085ed5a81..f936e9303 100644 --- a/spec/helpers/table_builder_helper_spec.rb +++ b/spec/helpers/table_builder_helper_spec.rb @@ -3,6 +3,8 @@ require 'spec_helper' describe TableBuilderHelper, type: :helper do describe "#table_builder_2" do it "builds a table" do + # TODO: Rename to `referentials` + # TODO: `sortable_columns` calls `#model` on this collection workbenches = [ build_stubbed(:referential) ] |