Skip to content

Commit

Permalink
Merge pull request #55 from plone/fix-richtextwidget-tests
Browse files Browse the repository at this point in the history
Fix RichTextWidget tests for Plone 4.3.
  • Loading branch information
garbas committed Feb 13, 2014
2 parents fcd7ffe + 327beb6 commit 93dc0d3
Show file tree
Hide file tree
Showing 6 changed files with 35 additions and 10 deletions.
6 changes: 6 additions & 0 deletions .travis.yml
Expand Up @@ -7,6 +7,8 @@ addons:
- access_key: $SAUCE_ACCESS_KEY
install:
- mkdir -p buildout-cache/downloads
- mkdir src
- git clone git://github.com/plone/plone.app.contenttypes.git -b mockup-issue-220 src/plone.app.contenttypes
- python bootstrap.py -c travis.cfg
- bin/buildout -N -t 3 -c travis.cfg
script:
Expand Down Expand Up @@ -38,8 +40,12 @@ env:
- ROBOT_REMOTE_URL=http://$SAUCE_USERNAME:$SAUCE_ACCESS_KEY@ondemand.saucelabs.com:80/wd/hub
matrix:
allow_failures:
- env: BROWSER="platform:OS X 10.8,browserName:iphone,version:6.1,device-orientation:portrait"
- env: BROWSER="platform:OS X 10.8,browserName:ipad,version:6.1,device-orientation:portrait"
- env: BROWSER="platform:Linux,browserName:android,version:4.0,device-type:tablet,device-orientation:portrait"
- env: BROWSER="platform:Windows 8.1,browserName:internet explorer,version:11"
- env: BROWSER="platform:Windows 7,browserName:internet explorer,version:10"
- env: BROWSER="platform:Windows 7,browserName:internet explorer,version:9"
- env: BROWSER="platform:Windows 7,browserName:internet explorer,version:8"
- env: BROWSER="platform:Windows 7,browserName:opera,version:12"
- env: BROWSER="platform:OS X 10.8,browserName:safari,version:6"
2 changes: 1 addition & 1 deletion plone/app/widgets/at.py
Expand Up @@ -569,7 +569,7 @@ def _base_args(self, context, field, request):
args.setdefault('pattern_options', {})
merged = dict_merge(get_tinymce_options(context, field, request),
args['pattern_options'])
args['pattern_options'] = merged
args['pattern_options'] = merged['pattern_options']

return args

Expand Down
16 changes: 7 additions & 9 deletions plone/app/widgets/dx.py
Expand Up @@ -6,9 +6,9 @@
from datetime import datetime

from plone.app.textfield.widget import IRichTextWidget \
as patextfield_IRichTextWidget
as patextfield_IRichTextWidget
from plone.app.textfield.widget import RichTextWidget \
as patextfield_RichTextWidget
as patextfield_RichTextWidget
from plone.app.textfield.value import RichTextValue
from plone.app.widgets.base import InputWidget
from plone.app.widgets.base import SelectWidget as BaseSelectWidget
Expand Down Expand Up @@ -67,7 +67,7 @@

try:
from plone.app.contenttypes.behaviors.collection import ICollection as IDXCollection # noqa
from plone.app.contenttypes.behaviors.richtext import IRichText # noqa
from plone.app.contenttypes.behaviors.richtext import IRichText # noqa
HAS_PAC = True
except ImportError:
HAS_PAC = False
Expand Down Expand Up @@ -727,14 +727,13 @@ def _base_args(self):
charset = properties.site_properties.getProperty('default_charset',
'utf-8')
value = self.value and self.value.raw or u''
args['value'] = (self.request.get(self.field.getName(),
value)
).decode(charset)
args['value'] = (self.request.get(
self.field.getName(), value)).decode(charset)

args.setdefault('pattern_options', {})
merged = dict_merge(get_tinymce_options(self.context, self.field, self.request),
merged = dict_merge(get_tinymce_options(self.context, self.field, self.request), # noqa
args['pattern_options'])
args['pattern_options'] = merged
args['pattern_options'] = merged['pattern_options']

return args

Expand Down Expand Up @@ -779,7 +778,6 @@ def RelatedItemsFieldWidget(field, request):
def QueryStringFieldWidget(field, request):
return FieldWidget(field, QueryStringWidget(request))


@adapter(getSpecification(IRichText['text']), IFormLayer)
@implementer(IFieldWidget)
def RichTextFieldWidget(field, request):
Expand Down
2 changes: 2 additions & 0 deletions plone/app/widgets/tests/test_dx.py
Expand Up @@ -914,6 +914,8 @@ def test_widget_params(self):
from plone.app.widgets.dx import RichTextWidget

widget = FieldWidget(self.field, RichTextWidget(self.request))
# set the context so we can get tinymce settings
widget.context = self.portal
widget.update()
base_args = widget._base_args()
self.assertEqual(base_args['name'], 'text')
Expand Down
3 changes: 3 additions & 0 deletions travis.cfg
Expand Up @@ -10,6 +10,8 @@ package-extras = [test,archetypes,dexterity]
test-eggs = Pillow
parts += extra

develop += src/plone.app.contenttypes

[extra]
recipe = zc.recipe.egg
eggs =
Expand All @@ -20,3 +22,4 @@ eggs =
[versions]
coverage = 3.7
plone.app.robotframework = 0.7.5
plone.app.contenttypes =
16 changes: 16 additions & 0 deletions versions.cfg
Expand Up @@ -11,3 +11,19 @@ plone.app.querystring = 1.1.0
plone.app.vocabularies = 2.1.12
plone.formwidget.querystring = 1.1.0
plone.formwidget.recurrence = 1.1

# versions copied from https://github.com/plone/buildout.coredev/blob/5.0/versions.cfg#L39
plone.app.robotframework = 0.7.5
plone.app.testing = 4.2.4
robotframework = 2.8.4
robotframework-selenium2library = 1.5.0
robotsuite = 1.4.3
selenium = 2.39.0
collective.xmltestreport = 1.3.1
mock = 1.0.1
zope.testrunner = 4.1.1
pep8 = 1.4.6
Babel = 1.3
robotframework-selenium2screenshots = 0.4.0
sphinxcontrib-robotframework = 0.4.3
Pygments = 1.6

1 comment on commit 93dc0d3

@mister-roboto
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TESTS FAILED
Mr.roboto url : http://jenkins.plone.org/roboto/get_info?push=06c20abea67440a5b714581043600e12
plone-5.0-python-2.7 [FAILURE]

Please sign in to comment.