aboutsummaryrefslogtreecommitdiffstats
path: root/spec/helpers
diff options
context:
space:
mode:
authorTeddy Wing2017-07-07 16:42:58 +0200
committerTeddy Wing2017-07-12 11:06:03 +0200
commit9ab672a31cd3a1aeea59e6ae3dab96e4057ee997 (patch)
tree73632f15c1c107b88fb27b848eab1fa6cc0f8ac8 /spec/helpers
parentd4be863478181b126faaafe25cfb293dca058570 (diff)
downloadchouette-core-9ab672a31cd3a1aeea59e6ae3dab96e4057ee997.tar.bz2
CustomLinks: Pass referential directly when initialising
Instead of getting the referential to use when building the polymorphic URL from the `UserContext`, pass in a referential directly. The old code that used `user_context.context[:referential]` relied on the fact that `ApplicationController#pundit_user` was defined as follows: def pundit_user UserContext.new(current_user, referential: self.try(:current_referential)) end (We pass `pundit_user` into `CustomLinks` from `TableBuilderHelper#build_links`.) However, Robert's change 747d333ffbcc8ee0c9f1daf93ccca32799434e04 removes the `current_referential` call from `#pundit_user`. In `CustomLinks`, we actually always want to be using `current_referential`. For example, on `Companies#index` (/line_referentials/:id/companies), `CustomLinks` fails to build a correct #show link because `user_context.context[:referential]` is `nil`, when it should instead be a `LineReferential` object, that gets returned by the `#current_referential` helper method. Sure, `#current_referential` is hard to understand, so maybe we'll change that around in the future, but this at least allows us to use the current referential in `CustomLinks`. Refs #3479
Diffstat (limited to 'spec/helpers')
-rw-r--r--spec/helpers/table_builder_helper/custom_links_spec.rb16
1 files changed, 16 insertions, 0 deletions
diff --git a/spec/helpers/table_builder_helper/custom_links_spec.rb b/spec/helpers/table_builder_helper/custom_links_spec.rb
index d6fbd2c64..ac60c7da3 100644
--- a/spec/helpers/table_builder_helper/custom_links_spec.rb
+++ b/spec/helpers/table_builder_helper/custom_links_spec.rb
@@ -1,4 +1,20 @@
describe TableBuilderHelper::CustomLinks do
+ describe "#polymorphic_url" do
+ it "returns the correct URL path for Companies#show" do
+ company = build_stubbed(:company)
+ user_context = UserContext.new(build_stubbed(:user))
+
+ expect(
+ TableBuilderHelper::CustomLinks.new(
+ company,
+ user_context,
+ [:show],
+ company.line_referential
+ ).polymorphic_url(:show)
+ ).to eq([company.line_referential, company])
+ end
+ end
+
describe "#authorized_actions" do
it "includes :show" do
referential = build_stubbed(:referential)