andrewtch/SonataDoctrineORMAdminBundle — pull request #1 by franmomu (2012-03-01 12:08:41+00:00)
Title: Added Test to DateFilter

Hi,

I'm very interested in this feature (very nice work), I have tried to write one quick test, but maybe I'm totally wrong because I'm not used to write them (I should hehe, I'm trying), so this PR is about if the test is OK and two questions:
- I delete the white space al the end of IS NULL, IS NOT NULL, I don't know if it is right.
- And I think when the operator is NULL or NOT NULL the value doesn't mind, so I don't know if the operator is one of them the filter doesn't should check for the value.

If this is ok, I could try to write test for the other classes

Bye