TableBuilderHelper: Fix specs which mock `current_referential`

These two specs were failing because of the line:

    current_referential ||= nil

which would set `current_referential` to `nil` even after updating the
object being stubbed to `helper`.

Change the specs to stub the method on the correct object now that
`TableBuilderHelper::URL` doesn't depend on `current_referential`.

Update the faulty line in question to not clobber `current_referential`,
but still give us the `nil` value we were looking for when the helper
method isn't defined.

Refs #3479

1 files changed, 4 insertions, 4 deletions

diff --git a/app/helpers/table_builder_helper.rb b/app/helpers/table_builder_helper.rb
index 1ddd629f8..e1b8b406d 100644
--- a/app/helpers/table_builder_helper.rb
+++ b/app/helpers/table_builder_helper.rb
@@ -104,9 +104,9 @@ module TableBuilderHelper
   end
 
   def tbody(collection, columns, selectable, links)
-    # Certain controllers don't define a `#current_referential`. If the method
-    # is unavailable, give it a default value to avoid a `NoMethodError`.
-    current_referential ||= nil
+    # Certain controllers don't define a `#current_referential`. In these
+    # cases, avoid a `NoMethodError`.
+    referential = current_referential if respond_to?(:current_referential)
 
     content_tag :tbody do
       collection.map do |item|
@@ -128,7 +128,7 @@ module TableBuilderHelper
               # Build a link to the `item`
               polymorph_url = URL.polymorphic_url_parts(
                 item,
-                current_referential
+                referential
               )
               bcont << content_tag(:td, link_to(value, polymorph_url), title: 'Voir')
             else