BIND 10 #433: [kean] run msgq when already running removes socket file

[BIND 10 Development]

#433: [kean] run msgq when already running removes socket file
-------------------------------------+-------------------------------------
            Reporter:  jreed         |                        Owner:  kean
                Type:  defect        |                       Status:
            Priority:  medium        |  reviewing
           Component:  msgq          |                    Milestone:
            Keywords:                |  Sprint-20131015
           Sensitive:  0             |                   Resolution:
         Sub-Project:  Core          |                 CVSS Scoring:
Estimated Difficulty:  0.0           |              Defect Severity:  N/A
         Total Hours:  0             |  Feature Depending on Ticket:
                                     |          Add Hours to Ticket:  0
                                     |                    Internal?:  0
-------------------------------------+-------------------------------------

Comment (by kean):

 Thanks for the review Muks.

 I have reduced the scope of _existing to be local to the condition in
 which it is used.

 I assign self._session to None so that it is in the same state that it
 would have been before the change. It was not intended to have any side
 effects, and I didn't check with CPython because that is not on the list
 of supported platforms or requirements, at least not as documented at
 http://bind10.isc.org/docs/bind10-guide.html#idm65352880.

 Changed the messages as requested.

 Added the pre-condition to the test, and check for the socket after the
 test to make sure it exists. There is no need to test that the existing
 queue is usable as the call will fail if a session cannot be started so
 that in itself checks that it is working.

-- 
Ticket URL: <http://bind10.isc.org/ticket/433#comment:6>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development