Merge pull request #1550 from remear/url-helpers

Add Rails url_helpers

Author: remear

CHANGELOG.md

@@ -3,6 +3,8 @@
 Breaking changes:
 
 Features:
+- [#1550](https://github.com/rails-api/active_model_serializers/pull/1550) Add
+  Rails url_helpers to `SerializationContext` for use in links. (@remear, @bf4)
 - [#1004](https://github.com/rails-api/active_model_serializers/pull/1004) JSON API errors object implementation.
   - Only implements `detail` and `source` as derived from `ActiveModel::Error`
   - Provides checklist of remaining questions and remaining parts of the spec.

Rakefile

@@ -45,25 +45,23 @@ Rake::TestTask.new do |t|
 end
 
 desc 'Run isolated tests'
-task isolated: ['test:isolated:railtie']
+task isolated: ['test:isolated']
 namespace :test do
-  namespace :isolated do
+  task :isolated do
     desc 'Run isolated tests for Railtie'
-    task :railtie do
-      require 'shellwords'
-      dir = File.dirname(__FILE__)
-      file = Shellwords.shellescape("#{dir}/test/active_model_serializers/railtie_test_isolated.rb")
-      dir = Shellwords.shellescape(dir)
-
-      # https://github.com/rails/rails/blob/3d590add45/railties/lib/rails/generators/app_base.rb#L345-L363
-      _bundle_command = Gem.bin_path('bundler', 'bundle')
-      require 'bundler'
-      Bundler.with_clean_env do
-        command = "-w -I#{dir}/lib -I#{dir}/test #{file}"
+    require 'shellwords'
+    dir = File.dirname(__FILE__)
+    dir = Shellwords.shellescape(dir)
+    isolated_test_files = FileList['test/**/*_test_isolated.rb']
+    # https://github.com/rails/rails/blob/3d590add45/railties/lib/rails/generators/app_base.rb#L345-L363
+    _bundle_command = Gem.bin_path('bundler', 'bundle')
+    require 'bundler'
+    Bundler.with_clean_env do
+      isolated_test_files.all? do |test_file|
+        command = "-w -I#{dir}/lib -I#{dir}/test #{Shellwords.shellescape(test_file)}"
         full_command = %("#{Gem.ruby}" #{command})
-        system(full_command) or # rubocop:disable Style/AndOr
-          fail 'Failures'
-      end
+        system(full_command)
+      end or fail 'Failures' # rubocop:disable Style/AndOr
     end
   end
 end

docs/general/getting_started.md

@@ -96,3 +96,12 @@ class PostsController < ApplicationController
 
 end
 ```
+
+If you wish to use Rails url helpers for link generation, e.g., `link(:resources) { resources_url }`, ensure your application sets
+`Rails.application.routes.default_url_options`.
+
+```ruby
+Rails.application.routes.default_url_options = {
+    host: 'example.com'
+}
+```

docs/general/rendering.md

@@ -103,7 +103,10 @@ PR please :)
 
 #### links
 
-##### How to add top-level links
+If you wish to use Rails url helpers for link generation, e.g., `link(:resources) { resources_url }`, ensure your application sets
+`Rails.application.routes.default_url_options`.
+
+##### Top-level
 
 JsonApi supports a [links object](http://jsonapi.org/format/#document-links) to be specified at top-level, that you can specify in the `render`:
 
@@ -144,6 +147,33 @@ That's the result:
 
 This feature is specific to JsonApi, so you have to use the use the [JsonApi Adapter](adapters.md#jsonapi)
 
+
+##### Resource-level
+
+In your serializer, define each link in one of the following methods:
+
+As a static string
+
+```ruby
+link :link_name, 'https://example.com/resource'
+```
+
+As a block to be evaluated. When using Rails, URL helpers are available.
+Ensure your application sets `Rails.application.routes.default_url_options`.
+
+```ruby
+link :link_name_ do
+  "https://example.com/resource/#{object.id}"
+end
+
+link(:link_name) { "https://example.com/resource/#{object.id}" }
+
+link(:link_name) { resource_url(object) }
+
+link(:link_name) { url_for(controller: 'controller_name', action: 'index', only_path: false) }
+
+```
+
 ### serializer_opts
 
 #### include

docs/general/serializers.md

@@ -135,13 +135,15 @@ With the `:json_api` adapter, the previous serializers would be rendered as:
 
 #### ::link
 
-e.g.
-
 ```ruby
-link :other, 'https://example.com/resource'
 link :self do
- href "https://example.com/link_author/#{object.id}"
+  href "https://example.com/link_author/#{object.id}"
 end
+link :author { link_author_url(object) }
+link :link_authors { link_authors_url }
+link :other, 'https://example.com/resource'
+link :posts { link_author_posts_url(object) }
+```
 ```
 
 #### #object

lib/active_model/serializer/links.rb

@@ -20,9 +20,11 @@ def inherited(base)
 
         # Define a link on a serializer.
         # @example
-        #   link :self { "//example.com/posts/#{object.id}" }
+        #   link(:self) { resource_url(object) }
         # @example
-        #   link :self, "//example.com/user"
+        #   link(:self) { "http://example.com/resource/#{object.id}" }
+        # @example
+        #   link :resource, "http://example.com/resource"
         #
         def link(name, value = nil, &block)
           _links[name] = block || value

lib/active_model_serializers/adapter/json_api/link.rb

@@ -1,7 +1,12 @@
+require 'active_support/core_ext/module/delegation'
+
 module ActiveModelSerializers
   module Adapter
     class JsonApi
       class Link
+        include SerializationContext.url_helpers
+        delegate :default_url_options, to: SerializationContext
+
         def initialize(serializer, value)
           @object = serializer.object
           @scope = serializer.scope

lib/active_model_serializers/railtie.rb

@@ -15,6 +15,11 @@ class Railtie < Rails::Railtie
       end
     end
 
+    initializer 'active_model_serializers.prepare_serialization_context' do
+      SerializationContext.url_helpers = Rails.application.routes.url_helpers
+      SerializationContext.default_url_options = Rails.application.routes.default_url_options
+    end
+
     # This hook is run after the action_controller railtie has set the configuration
     # based on the *environment* configuration and before any config/initializers are run
     # and also before eager_loading (if enabled).

lib/active_model_serializers/serialization_context.rb

@@ -1,10 +1,24 @@
 module ActiveModelSerializers
   class SerializationContext
+    class << self
+      attr_writer :url_helpers, :default_url_options
+    end
+
     attr_reader :request_url, :query_parameters
 
-    def initialize(request)
+    def initialize(request, options = {})
       @request_url = request.original_url[/\A[^?]+/]
       @query_parameters = request.query_parameters
+      @url_helpers = options.delete(:url_helpers) || self.class.url_helpers
+      @default_url_options = options.delete(:default_url_options) || self.class.default_url_options
+    end
+
+    def self.url_helpers
+      @url_helpers ||= Module.new
+    end
+
+    def self.default_url_options
+      @default_url_options ||= {}
     end
   end
 end

test/active_model_serializers/railtie_test_isolated.rb

@@ -18,6 +18,12 @@ class WithRails < RailtieTest
         "ActionController::Serialization should be included in ActionController::Base, but isn't"
     end
 
+    test 'prepares url_helpers for SerializationContext' do
+      assert ActiveModelSerializers::SerializationContext.url_helpers.respond_to? :url_for
+      assert_equal Rails.application.routes.default_url_options,
+        ActiveModelSerializers::SerializationContext.default_url_options
+    end
+
     test 'sets the ActiveModelSerializers.logger to Rails.logger' do
       refute_nil Rails.logger
       refute_nil ActiveModelSerializers.logger