Comment 3 for bug 1758806

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/41059
Committed: http://github.com/Juniper/contrail-controller/commit/1a9d64504342228d5f4ce70a892343808ae9a19a
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit 1a9d64504342228d5f4ce70a892343808ae9a19a
Author: Ananth Suryanarayana <email address hidden>
Date: Sun Mar 25 21:26:27 2018 -0700

Remove invalid test in XmppStateMachineTest

In the test Established_EvTcpClose_Then_EvXmppMessageStreamHeader,
EvXmppMessageStreamHeader is injected into a session_ XmppSession *, after
injecting EvTcpClose event. This is an invalid scenario because as a result of
processing EvTcpClose, XmppSession *session pointer can/will get destroyed.
Hence, we should not enqueue EvXmppMessageStreamHeader over a deleted session
later on. This can cause the test to crash and hence makes this test case in
xmpp_server_sm_test flaky.

Change-Id: I66b257c49ede25e4142d93aa0264f729b1896031
Closes-Bug: 1758806