Skip to content

Conversation

@Nachtalb
Copy link
Contributor

@Nachtalb Nachtalb commented Sep 28, 2020

Code copied from p.a.testing version with the email changed from
"noreply@example.com" to "test@localhost" so that tests that depend on that don't
unnecessarily break.

@Nachtalb Nachtalb requested review from a team and buchi September 28, 2020 09:33
@Nachtalb
Copy link
Contributor Author

Nachtalb commented Sep 28, 2020

@buchi can you tell me what I am doing wrong that the 5.2 test fail with

Traceback (most recent call last):
  File "bin/test", line 279, in <module>
    '--test-path', '/var/lib/jenkins/zope/eggs/Pillow-6.2.2-py2.7-linux-x86_64.egg',
  File "/var/lib/jenkins/zope/eggs/collective.xmltestreport-2.0.1-py2.7.egg/collective/xmltestreport/runner.py", line 66, in run
    failed = run_internal(defaults, args, script_parts=script_parts)
  File "/var/lib/jenkins/zope/eggs/collective.xmltestreport-2.0.1-py2.7.egg/collective/xmltestreport/runner.py", line 79, in run_internal
    runner.run()
  File "/var/lib/jenkins/zope/eggs/zope.testrunner-5.2-py2.7.egg/zope/testrunner/runner.py", line 191, in run
    self.run_tests()
  File "/var/lib/jenkins/zope/eggs/zope.testrunner-5.2-py2.7.egg/zope/testrunner/runner.py", line 293, in run_tests
    self.skipped, self.import_errors)
  File "/var/lib/jenkins/zope/eggs/zope.testrunner-5.2-py2.7.egg/zope/testrunner/runner.py", line 474, in run_layer
    import_errors)
  File "/var/lib/jenkins/zope/eggs/zope.testrunner-5.2-py2.7.egg/zope/testrunner/runner.py", line 396, in run_tests
    test(result)
  File "/usr/local/python/2.7.16/lib/python2.7/unittest/case.py", line 393, in __call__
    return self.run(*args, **kwds)
  File "/usr/local/python/2.7.16/lib/python2.7/unittest/case.py", line 333, in run
    result.addFailure(self, sys.exc_info())
  File "/var/lib/jenkins/zope/eggs/zope.testrunner-5.2-py2.7.egg/zope/testrunner/runner.py", line 966, in addFailure
    stdout=stdout, stderr=stderr)
TypeError: test_failure() got an unexpected keyword argument 'stdout'

Code copied from p.a.testing version with the email changed from
"noreply@example.com" to "test@localhost" so that tests that depend on that don't
unnecessarily break.
@Nachtalb Nachtalb force-pushed the ne/fix-setting-default-email-in-plone5 branch from 32a6c28 to 83ccd6d Compare September 28, 2020 09:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants