Ticket #1730 defect closed fixed

Opened 7 years ago

Last modified 6 years ago

twisted.test.test_tcp.HalfClose3TestCase.testReadNotificationRaises fails intermittently

Reported by: radix Owned by:
Priority: highest Milestone:
Component: core Keywords:
Cc: therve Branch:
Author: Launchpad Bug:

Description

At least on my x86/OS X machine.

  File "/Users/radix/tmp/lib/python2.4/site-packages/twisted/test/test_tcp.py", line 1124, in testReadNotificationRaises
    self.f.protocol.readConnectionLost = self.aBug
exceptions.AttributeError: 'NoneType' object has no attribute 'readConnectionLost'

Change History

1

Changed 6 years ago by exarkun

#2256 was a duplicate of this.

2

Changed 6 years ago by exarkun

#2450 was a duplicate of this.

3

Changed 6 years ago by exarkun

  • priority changed from normal to high

This fails on Windows sometimes, too.

*bump*

4

Changed 6 years ago by exarkun

  • owner changed from jknight to exarkun
  • status changed from new to assigned

5

Changed 6 years ago by exarkun

  • status changed from assigned to new
  • owner exarkun deleted
  • keywords review added
  • priority changed from high to highest

Fails in trunk on neutron (OS X host) within 100 runs pretty reliably. In read-notification-raises-1730, ran 4000+ times without failing. Please review!

6

Changed 6 years ago by therve

  • keywords review removed
  • cc therve added
  • owner set to exarkun

That looks OK, I couldn't reproduce the problem anymore under my vmware win32 machine.

Just one thing: aBug mentions readConnectionLost, it should probably mention writeConnectionLost too.

7

Changed 6 years ago by exarkun

  • status changed from new to closed
  • resolution set to fixed

(In [21268]) Merge read-notification-raises-1730

Author: exarkun Reviewer: therve Fixes #1730

Rewrite two unit tests for a feature of half-close support so that they do not contain a race condition which sometimes results in the tests failing due to setup having been only partially completed.

8

Changed 2 years ago by <automation>

  • owner exarkun deleted
Note: See TracTickets for help on using tickets.