tree f2dd581853426b3cffcc877dce53e0bfb79d1538
parent 88303d8505ce80787730c35b5716df3a1393c1c2
author Tobias Thierer <tobiast@google.com> 1466613121 +0100
committer Tobias Thierer <tobiast@google.com> 1466613878 +0100

Disable leak detection in URLConnectionTest by extending TestCase.

URLConnectionTest previously extended
AbstractResourceLeakageDetectorTestCase which supplied the
leak detection logic, but that logic was unreliable.

This created more boilerplate and maintenance work in unrelated
tests (which are not focused on leak detection) exceeding the
value it provides:

  - The leak detection doesn't work at all under CTS (29533469)
    and is very flaky on vogar.
  - Currently URLConnectionTest constructs 19 instances of
    TestSSLContext and 127 instances of Http(s)URLConnection,
    most of which aren't closed/disconnected. Closing all of
    these would add excessive boilerplate and pollute the
    purpose of these tests.

Tested:
 - Test now passes under vogar

Bug: 29533469
Change-Id: I5ef6a51e651b52d975a4cd0dfe4548b83a328c72