Comments on: Sprytne testowanie modelu w Ruby on Rails

By: Tomek

Tomek — Sat, 13 Oct 2007 12:02:15 +0000

Wszystko fajnie tylko testowanie tego to jest troche overkill. Wlasne metody to rozumiem, popieram i stosuje. Ale takie rzeczy jak validates_* to juz mi pachnie nadgorliwoscia podobna do testowania np. attr_reader.

By: wiktor

wiktor — Thu, 11 Oct 2007 09:21:24 +0000

@Marek

Faktycznie, twój sposób jest elegancki, ale pozbądźmy się class_eval. Można wtedy zapisać to w ten sposób:

def self.test_validates_presence_of(*args)
args.each do |name|
define_method(:”test_validation_#{name}”) do

u = users(‘user1′)

u.send :”#{name}=”, ”
assert !u.valid?
assert u.errors.invalid?(:”#{name}”)

u.send :”#{name}=”, nil
assert !u.valid?
assert u.errors.invalid?(:”#{name}”)
end
end
end

Żeby się pozbyć redundancji to można blok send, assert, assert wyekstraktować do osobnej metody i jest ‘zahardcodowane’ używanie z fixtures user1, które to też należałoby zmienić.

By: wiktor

wiktor — Thu, 11 Oct 2007 09:05:17 +0000

Wstyd wstyd wstyd, ale błędu już nie ma

By: Daj Pan Spokój

Daj Pan Spokój — Thu, 11 Oct 2007 06:31:21 +0000

A ja się doczepię o ortografię.
Powinno być “załóżmy”, a nie “zaużmy”.

By: Marek Kirejczyk

Marek Kirejczyk — Wed, 10 Oct 2007 22:30:46 +0000

It’s more rails style

By: Marek

Marek — Wed, 10 Oct 2007 22:22:12 +0000

No niezle, ale mozna lepiej:
Zróbmy uniwersalną metodę do, testowania validates_presence_of we wszystkich modelach jakie kiedys napiszemy (Najlepiej włożyć do modułu):

def self.test_validates_presence_of(*args)
args.each do |name|

class_eval(”

def test_validation_#{name}
u = users(‘user1′)
u.send :”#{name}=”, ”
assert !u.valid?
assert u.errors.invalid?(:”#{name}”)
u.send :”#{name}=”, nil
assert !u.valid?
assert u.errors.invalid?(:”#{name}”)
end
“)
end
end

I jej użyjmy:
test_validates_presence_of :name, :middlename, :surname, :sex, :birthday