Opened 5 years ago

Closed 5 years ago

Last modified 4 years ago

#7992 enhancement closed fixed (fixed)

Stop relying on SSLv23_METHOD having SSLv3 in it

Reported by: hawkowl Owned by: hawkowl
Priority: normal Milestone:
Component: core Keywords:
Cc: Branch: branches/sslv23method-7992
branch-diff, diff-cov, branch-cov, buildbot
Author: hawkowl


Traceback (most recent call last):
  File "/buildslave/fedora22-py2.7/Twisted/twisted/test/", line 1239, in test_SSLv23
  File "/buildslave/fedora22-py2.7/Twisted/twisted/trial/", line 437, in assertEqual
    super(_Assertions, self).assertEqual(first, second, msg)
  File "/usr/lib64/python2.7/unittest/", line 553, in assertEqual
    assertion_func(first, second, msg=msg)
  File "/usr/lib64/python2.7/unittest/", line 836, in assertSetEqual, standardMsg))
twisted.trial.unittest.FailTest: Items in the first set but not the second:


Change History (5)

comment:1 Changed 5 years ago by hawkowl

Author: hawkowl
Branch: branches/sslv23method-7992

(In [45417]) Branching to sslv23method-7992.

comment:2 Changed 5 years ago by hawkowl

Keywords: review added

Removed the test, since it's worthless. shows green F22, the rest of the builders are green, please review.

comment:3 Changed 5 years ago by Adi Roiban

Keywords: review removed
Owner: set to hawkowl

Looks good. I am ok with removing the tests as it looks like a test for the content of the SSL.SSLv23_METHOD constant.

Don't forget to add the news file... then merge :)


comment:4 Changed 5 years ago by hawkowl

Resolution: fixed
Status: newclosed

(In [45420]) Merge sslv23method-7992: Remove useless test that checks the contents of the SSLv23_METHOD constant

Author: hawkowl Reviewer: adiroiban Fixes: #7992

comment:5 Changed 4 years ago by hawkowl

Milestone: Twisted 15.4

Ticket retargeted after milestone deleted

Note: See TracTickets for help on using tickets.