Skip to content

Commit

Permalink
Merge pull request #32 from pawl/add_issue_30_test
Browse files Browse the repository at this point in the history
add unit test for #30
  • Loading branch information
coleifer committed Dec 2, 2015
2 parents 487d90a + 287d37d commit 3032645
Showing 1 changed file with 20 additions and 0 deletions.
20 changes: 20 additions & 0 deletions wtfpeewee/tests.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@
from peewee import *
from wtforms import fields as wtfields
from wtforms.form import Form as WTForm
from wtforms.validators import Regexp
from wtfpeewee.fields import *
from wtfpeewee.orm import model_form
from wtfpeewee._compat import PY2
Expand Down Expand Up @@ -111,6 +112,25 @@ def test_defaults(self):
form = BlogFormDef(obj=self.blog_a)
self.assertEqual(form.data, {'title': 'a'})

def test_duplicate_validators(self):
''' Test whether validators are duplicated when forms share field_args
'''
shared_field_args = {'id': {'validators': [Regexp('test')]}}

ValueIncludedForm = model_form(NonIntPKModel,
field_args=shared_field_args,
allow_pk=True)
ValueExcludedForm = model_form(NonIntPKModel,
field_args=shared_field_args,
allow_pk=True,
exclude=['value'])

value_included_form = ValueIncludedForm()
self.assertEqual(len(value_included_form.id.validators), 2)

value_excluded_form = ValueExcludedForm()
self.assertEqual(len(value_excluded_form.id.validators), 2)

def test_non_int_pk(self):
form = NonIntPKForm()
self.assertEqual(form.data, {'value': None, 'id': None})
Expand Down

0 comments on commit 3032645

Please sign in to comment.