Apply suggestions from code review

test/helper.py

@@ -88,6 +88,7 @@ class FakeYDL(YoutubeDL):
             old_report_warning(message)
         self.report_warning = types.MethodType(report_warning, self)
 
+
 class FakeLogger(object):
     def debug(self, msg):
         pass
@@ -98,6 +99,7 @@ class FakeLogger(object):
     def error(self, msg):
         pass
 
+
 def gettestcases(include_onlymatching=False):
     for ie in youtube_dl.extractor.gen_extractors():
         for tc in ie.get_testcases(include_onlymatching):

test/test_downloader_http.py

@@ -9,7 +9,11 @@ import sys
 import unittest
 sys.path.insert(0, os.path.dirname(os.path.dirname(os.path.abspath(__file__))))
 
-from test.helper import FakeLogger, http_server_port, try_rm
+from test.helper import (
+    FakeLogger,
+    http_server_port,
+    try_rm,
+)
 from youtube_dl import YoutubeDL
 from youtube_dl.compat import compat_http_server
 from youtube_dl.downloader.http import HttpFD

test/test_external_downloader.py

@@ -10,7 +10,11 @@ import subprocess
 import unittest
 sys.path.insert(0, os.path.dirname(os.path.dirname(os.path.abspath(__file__))))
 
-from test.helper import FakeLogger, http_server_port, try_rm
+from test.helper import (
+    FakeLogger,
+    http_server_port,
+    try_rm,
+)
 from youtube_dl import YoutubeDL
 from youtube_dl.compat import compat_http_server
 from youtube_dl.utils import encodeFilename
@@ -22,6 +26,7 @@ TEST_DIR = os.path.dirname(os.path.abspath(__file__))
 
 TEST_SIZE = 10 * 1024
 
+
 class HTTPTestRequestHandler(compat_http_server.BaseHTTPRequestHandler):
     def log_message(self, format, *args):
         pass

test/test_http.py

@@ -8,7 +8,10 @@ import sys
 import unittest
 sys.path.insert(0, os.path.dirname(os.path.dirname(os.path.abspath(__file__))))
 
-from test.helper import http_server_port, FakeLogger
+from test.helper import (
+    FakeLogger,
+    http_server_port,
+)
 from youtube_dl import YoutubeDL
 from youtube_dl.compat import compat_http_server, compat_urllib_request
 import ssl

youtube_dl/downloader/external.py

@@ -233,11 +233,7 @@ class Aria2pFD(ExternalFD):
             'max-connection-per-server': 4,
             'auto-file-renaming': 'false',
         }
-        download_dir = os.path.dirname(tmpfilename)
-        if download_dir:
-            options['dir'] = download_dir
-        else:
-            options['dir'] = os.path.abspath('.')
+        options['dir'] = os.path.dirname(tmpfilename) or os.path.abspath('.')
         options['out'] = os.path.basename(tmpfilename)
         options['header'] = []
         for key, val in info_dict['http_headers'].items():
@@ -259,7 +255,7 @@ class Aria2pFD(ExternalFD):
             })
             self._hook_progress(status)
             time.sleep(.5)
-        return not download.status == 'complete'
+        return download.status != 'complete'
 
 
 class HttpieFD(ExternalFD):