tempesta-tech / tempesta-test

Test suite for Tempesta FW
10 stars 4 forks source link

Fix framework tearDown #505

Closed const-t closed 2 months ago

const-t commented 10 months ago

Stop deproxy manager and release lock first, before stopping clients. Stopping clients during polling and under lock looks wrong and leads to hanging on tearDown phase.

const-t commented 10 months ago

Master branch:

 ----------------------------------------------------------------------
 Running functional tests...
 ----------------------------------------------------------------------

 test_chains (forwarding.test_match_host_forwarded.TestMatchHost) ... ok

 ----------------------------------------------------------------------
 Ran 1 test in 3.303s

This branch:

 ----------------------------------------------------------------------
 Running functional tests...
 ----------------------------------------------------------------------

 test_chains (forwarding.test_match_host_forwarded.TestMatchHost) ... Root privileges are required: need access to module loading on localhost.
 ok

 ----------------------------------------------------------------------
 Ran 1 test in 1.267s
const-t commented 10 months ago

Master branch:

 ----------------------------------------------------------------------
 Running functional tests...
 ----------------------------------------------------------------------

 test_chains (forwarding.test_match_host_forwarded.TestMatchHost) ... ok

 ----------------------------------------------------------------------
 Ran 1 test in 3.303s

This branch:

 ----------------------------------------------------------------------
 Running functional tests...
 ----------------------------------------------------------------------

 test_chains (forwarding.test_match_host_forwarded.TestMatchHost) ... Root privileges are required: need access to module loading on localhost.
 ok

 ----------------------------------------------------------------------
 Ran 1 test in 1.267s