0047 - brittle and fragile tests

Screencast being a Value.rb

class Screencast
  attr_accessor :versions

  def initialize(params)
    params.each {|key, value| self.send "#{key}=", value }
  end
end

class ScreencastCategories
  def self.by_tool(screencasts)
    by_tool = Hash.new { |hash, key| hash[key] = [] }
    screencasts.each do |screencast|
      screencast.versions.each do |tool_name, version|
        by_tool[tool_name] << screencast
      end
    end
    by_tool
  end
end


require 'rspec/autorun'

describe ScreencastCategories do
  it 'groups by tool' do
    screencast = Screencast.new(:versions => {:vim => '7.3'})
    by_tool = ScreencastCategories.by_tool([screencast])
    by_tool.fetch(:vim).should == [screencast]
  end
end

Screencast being an Identity.rb

class ScreencastCategories
  def self.by_tool(screencasts)
    by_tool = Hash.new { |hash, key| hash[key] = [] }
    screencasts.each do |screencast|
      screencast.versions.each do |tool_name, version|
        by_tool[tool_name] << screencast
      end
    end
    by_tool
  end
end


require 'rspec/autorun'

describe ScreencastCategories do
  it 'groups by tool' do
    screencast = stub(:screencast, :versions => {:vim => '7.3'})
    by_tool = ScreencastCategories.by_tool([screencast])
    by_tool.fetch(:vim).should == [screencast]
  end
end