trailblazer 0.2.2 → 0.3.0

data/test/rails/controller_test.rb

@@ -52,12 +52,24 @@ class ResponderRespondTest < ActionController::TestCase
     assert_redirected_to song_path(Song.last)
   end
 
+  test "Create [html/valid/location]" do
+    post :other_create, {song: {title: "You're Going Down"}}
+    assert_redirected_to other_create_songs_path
+  end
+
   test "Create [html/invalid]" do
     post :create, {song: {title: ""}}
     assert_response 200
     assert_equal @response.body, "{:title=&gt;[&quot;can&#39;t be blank&quot;]}"
   end
 
+  test "Create [html/invalid/action]" do
+    post :other_create, {song: {title: ""}}
+    assert_response 200
+    assert_equal @response.body, "OTHER SONG\n{:title=&gt;[&quot;can&#39;t be blank&quot;]}\n"
+    assert_template "songs/another_view"
+  end
+
   test "Delete [html/valid]" do
     song = Song::Create[song: {title: "You're Going Down"}].model
     delete :destroy, id: song.id
@@ -170,6 +182,24 @@ class ControllerPresentTest < ActionController::TestCase
   end
 end
 
+#collection
+class ControllerCollectionTest < ActionController::TestCase
+  tests BandsController
+
+  # let (:band) { }
+
+  test "#collection" do
+    Band.destroy_all
+    Band::Create[band: {name: "Nofx"}]
+    Band::Create[band: {name: "Ramones"}]
+
+
+    get :index
+
+    assert_equal "bands/index.html: Nofx Ramones \n", response.body
+  end
+end
+
 # #form.
 class ControllerFormTest < ActionController::TestCase
   tests BandsController
@@ -203,4 +233,25 @@ class ActiveRecordPresentTest < ActionController::TestCase
 
     assert_equal "active_record_bands/show.html: Band, Band, true, Band::Update", response.body
   end
+
+  test "#collection" do
+    Band.destroy_all
+    Band::Create[band: {name: "Nofx"}]
+
+    get :index
+
+    assert_equal "active_record_bands/index.html: Band::ActiveRecord_Relation, Band::ActiveRecord_Relation, Band::Index", response.body
+  end
 end
+
+class PrefixedTablenameControllerTest < ActionController::TestCase
+  tests TenantsController
+
+  test "show" do
+    tenant = Tenant.create(name: "My Tenant") # yepp, I am not using an operation! blasphemy!!!
+    get :show, id: tenant.id
+
+    assert_equal "My Tenant", response.body
+  end
+end
+

data/test/rails/endpoint_test.rb

@@ -0,0 +1,86 @@
+require "test_helper"
+
+module RailsEndpoint
+  # this tests "the rails way" where both JSON and HTML operation use the pre-parsed params hash.
+  class UnconfiguredTest < ActionDispatch::IntegrationTest
+    class Create < Trailblazer::Operation
+      include CRUD
+      model Band
+
+      def process(params)
+        @model = Band.create(params["band"].permit(:name)) # how ridiculous is strong_parameters?
+      end
+    end
+
+    class BandsController < ApplicationController
+      include Trailblazer::Operation::Controller
+      respond_to :html, :json
+      # missing document_formats.
+
+      def create
+        run Create if request.format == :json
+        run Create if request.format == :html
+        render text: ""
+      end
+    end
+
+    test "Create" do
+      post "/rails_endpoint/unconfigured_test/bands", {band: {name: "SNFU"}}
+      assert_response 200
+      assert_equal "SNFU", Band.last.name
+    end
+
+    test "Create: JSON" do
+      headers = { 'CONTENT_TYPE' => 'application/json' } # hahaha, oh god, rails, good bye!
+      post "/rails_endpoint/unconfigured_test/bands", {band: {name: "Strike Anywhere"}}.to_json, headers
+      assert_response 200
+      assert_equal "Strike Anywhere", Band.last.name
+    end
+  end
+
+
+  class ConfiguredTest < ActionDispatch::IntegrationTest
+    class Create < Trailblazer::Operation
+      include CRUD
+      model Band
+
+      def process(params)
+        @model = Band.create(params["band"].permit(:name)) # how ridiculous is strong_parameters?
+      end
+    end
+
+    class JSONCreate < Trailblazer::Operation
+      include CRUD
+      model Band
+
+      def process(params)
+        @model = Band.create(JSON.parse(params["band"])) # document comes in keyed as "band".
+      end
+
+    end
+
+    class BandsController < ApplicationController
+      include Trailblazer::Operation::Controller
+      respond_to :html, :json
+      operation document_formats: :json
+
+      def create
+        run Create if request.format == :html
+        run JSONCreate if request.format == :json
+        render text: ""
+      end
+    end
+
+    test "Create" do
+      post "/rails_endpoint/configured_test/bands", {band: {name: "All"}}
+      assert_response 200
+      assert_equal "All", Band.last.name
+    end
+
+    test "Create: JSON" do
+      post "/rails_endpoint/configured_test/bands.json", {name: "NOFX"}.to_json#, headers # FIXME: headers do not work
+      assert_response 200
+      assert_equal "NOFX", Band.last.name
+    end
+  end
+end

data/test/rails/fake_app/cells.rb

@@ -4,7 +4,7 @@ class UserCell < Cell::Rails
   def show(users)
     @users = users
 
-    render :inline => <<-ERB
+    render inline: <<-ERB
 <%= paginate @users %>
 ERB
   end
@@ -14,7 +14,7 @@ class ViewModelCell < Cell::ViewModel
   include Kaminari::Cells
 
   def show
-    render :inline => <<-ERB
+    render inline: <<-ERB
 <%= paginate model %>
 ERB
   end

data/test/rails/fake_app/config.rb

@@ -1,3 +1,3 @@
 # database
 ActiveRecord::Base.configurations = {'test' => {:adapter => 'sqlite3', :database => ':memory:'}}
-ActiveRecord::Base.establish_connection('test')
+ActiveRecord::Base.establish_connection(:test)

data/test/rails/fake_app/controllers.rb

@@ -20,6 +20,10 @@ ERB
     respond Song::Create
   end
 
+  def other_create
+    respond Song::Create, params, { location: other_create_songs_path, action: :another_view }
+  end
+
   def create_with_params
     respond Song::Create, song: {title: "A Beautiful Indifference"}
   end
@@ -44,6 +48,11 @@ end
 class BandsController < ApplicationController
   include Trailblazer::Operation::Controller
   respond_to :html, :json
+  operation document_formats: :json
+
+  def index
+    collection Band::Index
+  end
 
   def show
     present Band::Update do |op|
@@ -110,9 +119,27 @@ class ActiveRecordBandsController < ApplicationController
   include Trailblazer::Operation::Controller::ActiveRecord
   respond_to :html
 
+  def index
+    collection Band::Index
+    render text: "active_record_bands/index.html: #{@collection.class}, #{@bands.class}, #{@operation.class}"
+  end
+
   def show
     present Band::Update
 
     render text: "active_record_bands/show.html: #{@model.class}, #{@band.class}, #{@form.is_a?(Reform::Form)}, #{@operation.class}"
   end
 end
+
+require 'trailblazer/operation/controller/active_record'
+class TenantsController < ApplicationController
+  include Trailblazer::Operation::Controller
+  include Trailblazer::Operation::Controller::ActiveRecord
+  respond_to :html
+
+  def show
+    present Tenant::Show
+    render text: "#{@tenant.name}" # model ivar doesn't contain table prefix `bla.xxx`.
+  end
+end
+

data/test/rails/fake_app/models.rb

@@ -6,6 +6,10 @@ class Band < ActiveRecord::Base
   has_many :songs
 end
 
+class Tenant < ActiveRecord::Base
+  self.table_name = 'public.tenants'
+end
+
 # migrations
 class CreateAllTables < ActiveRecord::Migration
   def self.up
@@ -19,7 +23,12 @@ class CreateAllTables < ActiveRecord::Migration
       t.string :name
       t.string :locality
     end
+
+    create_table(:"public.tenants") do |t|
+      t.string :name
+    end
   end
 end
 ActiveRecord::Migration.verbose = false
-CreateAllTables.up
+CreateAllTables.up
+

data/test/rails/fake_app/rails_app.rb

@@ -5,6 +5,7 @@
 require 'action_controller/railtie'
 require 'action_view/railtie'
 require 'active_record'
+require 'responders'
 
 require 'fake_app/config'
 
@@ -12,7 +13,7 @@ require 'fake_app/config'
 # config
 app = Class.new(Rails::Application)
 app.config.secret_token = '3b7cd727ee24e8444053437c36cc66c4'
-app.config.session_store :cookie_store, :key => '_myapp_session'
+app.config.session_store :cookie_store, key: '_myapp_session'
 app.config.active_support.deprecation = :log
 app.config.eager_load = false
 # Rais.root
@@ -22,12 +23,14 @@ app.initialize!
 
 # routes
 app.routes.draw do
+
   resources :songs do
     member do # argh.
       delete :destroy_with_formats
     end
 
     collection do
+      post :other_create
       post :create_with_params
       post :create_with_block
     end
@@ -45,6 +48,17 @@ app.routes.draw do
       post :update_with_block
     end
   end
+
+  namespace :rails_endpoint do
+    namespace :unconfigured_test do
+      resources :bands
+    end
+    namespace :configured_test do
+      resources :bands
+    end
+  end
+
+  resources :tenants, only: [:show]
 end
 
 require 'trailblazer/operation/responder'

data/test/rails/fake_app/song/operations.rb

@@ -1,3 +1,5 @@
+require 'trailblazer/autoloading'
+
 class Song < ActiveRecord::Base
   class Create < Trailblazer::Operation
     include CRUD
@@ -30,7 +32,7 @@ end
 
 class Band < ActiveRecord::Base
   class Create < Trailblazer::Operation
-    include CRUD, Responder, Representer
+    include CRUD, Responder#, Representer
     model Band, :create
 
     contract do
@@ -56,6 +58,7 @@ class Band < ActiveRecord::Base
     end
 
     class JSON < self
+      include Representer
       require "reform/form/json"
       contract do
         include Reform::Form::JSON # this allows deserialising JSON.
@@ -93,6 +96,7 @@ class Band < ActiveRecord::Base
 
     # TODO: infer stuff per default.
     class JSON < self
+      include Representer
       self.contract_class = Create::JSON.contract_class
       self.representer_class = Create::JSON.representer_class
     end
@@ -101,4 +105,36 @@ class Band < ActiveRecord::Base
       JSON if params[:format] == "json"
     end
   end
-end
+
+  class Index < Trailblazer::Operation
+    include Collection
+
+    def model!(params)
+      Band.all
+    end
+
+    builds do |params|
+      JSON if params[:format] == "json"
+    end
+
+    class JSON < self
+      include Representer
+
+      module BandRepresenter
+        include Representable::JSON
+        property :name
+        property :locality
+      end
+
+      self.representer_class = BandRepresenter
+    end
+  end
+end
+
+class Tenant < ActiveRecord::Base
+  class Show < Trailblazer::Operation
+    include CRUD
+    model Tenant, :update
+  end
+end
+

data/test/rails/fake_app/views/bands/index.html.erb

@@ -0,0 +1 @@
+bands/index.html: <% @collection.each do |element| %><%= "#{element.name} " %><% end %>

data/test/rails/fake_app/views/songs/another_view.html.erb

@@ -0,0 +1,2 @@
+OTHER SONG
+<%= @form.errors.to_s %>

data/test/representer_test.rb

@@ -0,0 +1,126 @@
+require "test_helper"
+
+require "representable/json"
+
+class RepresenterTest < MiniTest::Spec
+  Album  = Struct.new(:title, :artist)
+  Artist = Struct.new(:name)
+
+  class Create < Trailblazer::Operation
+    require "trailblazer/operation/representer"
+    include Representer
+
+    contract do
+      property :title
+      validates :title, presence: true
+      property :artist, populate_if_empty: Artist do
+        property :name
+        validates :name, presence: true
+      end
+    end
+
+    def process(params)
+      @model = Album.new # NO artist!!!
+      validate(params[:album], @model)
+    end
+  end
+
+
+  # Infers representer from contract, no customization.
+  class Show < Create
+    def process(params)
+      @model = Album.new("After The War", Artist.new("Gary Moore"))
+      @contract = @model
+    end
+  end
+
+
+  # Infers representer, adds hypermedia.
+  require "roar/json/hal"
+  class HypermediaCreate < Create
+    representer do
+      include Roar::JSON::HAL
+
+      link(:self) { "//album/#{represented.title}" }
+    end
+  end
+
+  class HypermediaShow < HypermediaCreate
+    def process(params)
+      @model = Album.new("After The War", Artist.new("Gary Moore"))
+      @contract = @model
+    end
+  end
+
+
+  # rendering
+  # generic contract -> representer
+  it do
+    res, op = Show.run({})
+    op.to_json.must_equal %{{"title":"After The War","artist":{"name":"Gary Moore"}}}
+  end
+
+  # contract -> representer with hypermedia
+  it do
+    res, op = HypermediaShow.run({})
+    op.to_json.must_equal %{{"title":"After The War","artist":{"name":"Gary Moore"},"_links":{"self":{"href":"//album/After The War"}}}}
+  end
+
+
+  # parsing
+  it do
+    res, op = Create.run(album: %{{"title":"Run For Cover","artist":{"name":"Gary Moore"}}})
+    op.contract.title.must_equal "Run For Cover"
+    op.contract.artist.name.must_equal "Gary Moore"
+  end
+
+  it do
+    res, op = HypermediaCreate.run(album: %{{"title":"After The War","artist":{"name":"Gary Moore"},"_links":{"self":{"href":"//album/After The War"}}}})
+    op.contract.title.must_equal "After The War"
+    op.contract.artist.name.must_equal "Gary Moore"
+  end
+
+
+
+
+
+  # explicit representer set with ::representer_class=.
+  require "roar/decorator"
+  class JsonApiCreate < Trailblazer::Operation
+    include Representer
+
+    contract do # we still need contract as the representer writes to the contract twin.
+      property :title
+    end
+
+    class AlbumRepresenter < Roar::Decorator
+      include Roar::JSON
+      property :title
+    end
+    self.representer_class = AlbumRepresenter
+
+    def process(params)
+      @model = Album.new # NO artist!!!
+      validate(params[:album], @model)
+    end
+  end
+
+  class JsonApiShow < JsonApiCreate
+    def process(params)
+      @model = Album.new("After The War", Artist.new("Gary Moore"))
+      @contract = @model
+    end
+  end
+
+  # render.
+  it do
+    res, op = JsonApiShow.run({})
+    op.to_json.must_equal %{{"title":"After The War"}}
+  end
+
+  # parse.
+  it do
+    res, op = JsonApiCreate.run(album: %{{"title":"Run For Cover"}})
+    op.contract.title.must_equal "Run For Cover"
+  end
+end