class User
  validates_uniqueness_of :username
end

class RegistrationController < ApplicationController
  def create
    user = User.where(username: params[:username]).first
    user ||= User.new.tap do |new_user|
      new_user.username = params[:username]
      new_user.save!
    end
    render json: user
  end
end

ここで我々が必要としているのは、ユーザーを登録するためのユーザー名だ。あと、もしもユーザーが既に登録していた際は、単にユーザーの情報を返すだけだ。

すっごくシンプルだね。

だけど、そのロジックはControllerに属していない。それだと、うんこみたいなスタブやモックを使わずに、データベースに接続せずにコントローラーをテストすることは不可能だ。 any_instanceをテストで使うのは、「あなたは間違ったことをしている」ことを示す良い目印だ。

まず最初のカイゼンは次のようになるだろう

class User
  validates_uniqueness_of :username

  def self.register(username)
    user = User.where(username: username).first
    user ||= User.new.tap do |new_user|
      new_user.username = username
      new_user.save!
    end
  end
end

class RegistrationController < ApplicationController
  def create
    user = User.register(params[:username])
    render json: user
  end
end

これは既に元よりもはるかに優れたコードだ、君はコントローラーをテストするためにUser.registerをスタブすればいい。それでうまく行く。

もしも君がそこで足をとめ、君のアプリが大きく成長しすぎなければ、それでうまいこと出来るだろう。

だけど、もし君のアプリが成長しはじめたら、Userモデルはもっと大きく大きく育っていって、君はより多くの責務を与えていくだろう、

君はUserモデルにもっと責務を与えていくだろう。そして、君が知っていることを超えたあたりで「僕はただしいのかな?」と自分自身に問いかけるんだ。

コードをよりよく切り分ける道、実際にスタブを取り除く最終的なコードがこれだ:

class User
  validate_uniqueness_of :username
end

class RegistrationService
  def register(username)
    user = User.where(username: username).first
    user ||= User.new.tap do |new_user|
      new_user.username = username
      new_user.save!
    end
  end
end

class RegistrationController < ApplicationController
  before_filter :load_registration_service

  def create
    user = @registration_service.register(params[:username])
    render json: user
  end

  def load_registration_service(service = RegistrationService.new)
    @registration_service ||= service
  end
end

RegistrationServiceは、データベースに関連する登録に関してのみ、責務をおう。このサービス自体はデータベースにアクセスを行わずに、HTTPサービスを使うような別のものに取り替えることが可能なことに注意して欲しい。

その上、あなたは今テストから完全にスタブを取り除くことが出来るようになった。次のような感じだ

describe RegistrationController do
  before do
    @fake_registration_service = controller.
        load_registration_service(FakeRegistrationService.new)
  end

  describe "POST #create" do
    it "delegates to the registration service and renders the returned user" do
      expected_user = User.new.tap {|u| u.username = "damien"}
      @fake_registration_service.registered_user = expected_user

      post :create, username: "damien"

      expect(response.status).to eq(200)
      expect(response.body).to eq(expected_user.to_json)
      expect(@fake_registration_service.username_registered).to eq("damien")
    end
  end
end

class FakeRegistrationService
  attr_accessor :registered_user
  attr_reader :username_registered

  def register(username)
    @username_registered = username
    registered_user
  end
end

哀しいことに、我々はRailsがどうやってControllerをインスタンス化するのかコントロールしていない。だから我々はbefore_filterの中でこんな感じのトリックを使って、サービスのデフォルト値を使うようにしなくちゃならない。 PORO(Plain Old Ruby Object、素のRuby)でこれをやるならコンストラクターを作って似たようなことをやればい。

AndroidやiOSのアプリでテストを走らせるとき、依存性の注入を使うのは非常に一般的だ。私は個人的にはこれらのプラットフォーム上でモックを使わないことを好む。

そして、私はRuby on Railsのアプリケーションでもこのテクニックをもっと試してみたい。それはあなたによりよいオブジェクト設計について考えること、より特定の目的を持ったオブジェクトを持つこと、値オブジェクト上にロジックを持つことを制限(モデルとプレゼンターのように)することを強制するだろう。

あなたがこの記事について考えていることを、ぜひ教えて下さい! :)

2013-07-04

Ruby5 - Episode #383 - July 2, 2013

Ruby関連の情報はもっぱらRuby Weekly(週一で届く無料のRubyニュースメルマガ)に頼ってます。

Ruby5も聞いてみようと思いましたが、日本語じゃない(｀；ω；´)

日本語記事を探してみたら

Ruby5をひたすら翻訳する日記

すごく良いブログなんだけど更新止まってるみたい。せっかくなので僕もやってみたいな〜。と思って聞いてみました。

一旦英語で聴きとって書きだす
意訳

の順番で行こうと思ったがそんなに甘くなかった。普通に知らない単語出てくるし、聞き取れない。なので

聞き取れた範囲で意訳

のみで頑張ろうと思います。TopRubyJobsは翻訳しない方向で。

Ruby5 - Episode #383 - July 2, 2013

ワンライナーウェブサーバー、コントローラーへの提案、きちっとしたモック、モッキングライブラリを作る、Rails4への用意、オープンソースプロジェクトの地図

ワンライナーウェブサーバー

nobu (Nobuyoshi Nakada)さん(Rubyコントリビュータ)による、Rubyだけを使って、ワーキングディレクトリでウェブサーバーを立ち上げる美しくてシンプルな方法。

ruby -run -e httpd . -p 5000

を呼ぶだけ。Rubyの標準ライブラリのWEBRickを使ったサーバーがhttp://localhost:5000/で立ち上がる。(index.htmlがあればそれが表示される) 静的なファイル配信のに特化してるすごい技w

よいコントローラを書く

週末にPivotal Labsのダミアンレベゴッド(※訳注: 発音難しいのでカタカナでちゃんと書けてない可能性高いです)が書いた興味深いブログについて。

コントローラをテストするときモックとスタブを使わず、オブジェクト指向な設計を育てよう。

コントローラをテストするよりもインテグレーションテストをする方にみんな興味持ってそうなのに、Controllerのテストの話。 any_instanceを使ってオブジェクトのメソッドをスタブするのは、コードの臭いぷんぷん。

まずクラスメソッドに切り出して、それをサービスクラスに切り出す。 (※このあとの部分聞き取りづらかった。ブログ記事のコメント欄の方も盛り上がっていて面白い)

Object Oriented Rails - Writing better controllers - Pivotal Labs

verified_double

rspec-fireと似てるモック用のライブラリらしい。違うのは、そのモックがスタブしたメソッドが正しい値を返すかテストで保証出来ること。スタブが返す値をテストしていない場合warningがでるらしい。

gsmendoza/verified_double

モッキングライブラリを作る

7/23にあったAncient City RubyでのAndy Lindemanさんのモックライブラリを作るついての話。YouTubeで配信中。(今まで見た中で一番のライブコーディングらしい) モックが出来ること、レッド・グリーン・リファクタのサイクルを高速に回して、どれほどパワフルなことが出来るか理解出来るよう。(※ぼくもあとでみる)

Andy Lindeman - Building a mocking library - Ancient City Ruby 2013 - YouTube

Rails4 Ready?

Francesco RodríguezさんとFlorent Guilleuxが作ったready4rails4.netの話。 gemがRails 4に対応してるかしてないかが調べられる。

Ready for Rails 4?

オープンソース地図

世界オープンソースコントリビュータ地図。 David FischerがGitHub's Date Challengeに投稿した超ヤバいツール。 GitHubで最も活発な200のオープンソースリポジトリのコントリビューターが、世界のどこにいるのか地図にプロットした。

サイト

Open source contributions by location

2013-07-02

Rails 4.0.0を入れた後でRails 3.2.13を入れ直したいとき

当方rbenvを使っています。 gem install railsしてうっかり4.0.0を入れてしまったあと、3.2.13を入れなおしたのですが、rails -v、rails --versionしても4.0.0のままで困っていました。

状況

gem install rails             # 4.0.0が入る
gem uninstall rails           # 3.2.13が使いたいので、4.0.0を消す
gem install rails -v '3.2.13' # 3.2.13を指定して入れる
rails -v  # 3.2.13だと思った?? ざーんねんっｗｗ4.0.0でしたっwww

解決策

railtiesを消そう

gem uninstall railties -v '4.0.0'

なんで?

railsコマンドはrailsgemじゃなくてrailtiesgemのコマンドだからです。 https://github.com/rails/rails/blob/master/railties/bin/rails

有頂天Ruby

ビールを飲みながらRubyについて書きます。

Understanding Computation (1) Part1 プログラムと機械

Part1の章について

雑感

Understanding Computation (0)

オブジェクト指向Rails - よりよいコントローラーを書く

Ruby5 - Episode #383 - July 2, 2013

Ruby5 - Episode #383 - July 2, 2013

ワンライナーウェブサーバー

よいコントローラを書く

verified_double

モッキングライブラリを作る

Rails4 Ready?

オープンソース地図

サイト

Rails 4.0.0を入れた後でRails 3.2.13を入れ直したいとき

状況

解決策

なんで?