Tests: style.

1 files changed, 124 insertions, 80 deletions

diff --git a/test/test_ruby_application.py b/test/test_ruby_application.py
index 262fc497..0f2f6185 100644
--- a/test/test_ruby_application.py
+++ b/test/test_ruby_application.py
@@ -1,8 +1,8 @@
 import unittest
 import unit
 
-class TestUnitRubyApplication(unit.TestUnitApplicationRuby):
 
+class TestUnitRubyApplication(unit.TestUnitApplicationRuby):
     def setUpClass():
         unit.TestUnit().check_modules('ruby')
 
@@ -11,43 +11,56 @@ class TestUnitRubyApplication(unit.TestUnitApplicationRuby):
 
         body = 'Test body string.'
 
-        resp = self.post(headers={
-            'Host': 'localhost',
-            'Content-Type': 'text/html',
-            'Custom-Header': 'blah',
-            'Connection': 'close'
-        }, body=body)
+        resp = self.post(
+            headers={
+                'Host': 'localhost',
+                'Content-Type': 'text/html',
+                'Custom-Header': 'blah',
+                'Connection': 'close',
+            },
+            body=body,
+        )
 
         self.assertEqual(resp['status'], 200, 'status')
         headers = resp['headers']
         header_server = headers.pop('Server')
         self.assertRegex(header_server, r'Unit/[\d\.]+', 'server header')
-        self.assertEqual(headers.pop('Server-Software'), header_server,
-            'server software header')
+        self.assertEqual(
+            headers.pop('Server-Software'),
+            header_server,
+            'server software header',
+        )
 
         date = headers.pop('Date')
         self.assertEqual(date[-4:], ' GMT', 'date header timezone')
-        self.assertLess(abs(self.date_to_sec_epoch(date) - self.sec_epoch()), 5,
-            'date header')
-
-        self.assertDictEqual(headers, {
-            'Connection': 'close',
-            'Content-Length': str(len(body)),
-            'Content-Type': 'text/html',
-            'Request-Method': 'POST',
-            'Request-Uri': '/',
-            'Http-Host': 'localhost',
-            'Server-Protocol': 'HTTP/1.1',
-            'Custom-Header': 'blah',
-            'Rack-Version': '13',
-            'Rack-Url-Scheme': 'http',
-            'Rack-Multithread': 'false',
-            'Rack-Multiprocess': 'true',
-            'Rack-Run-Once': 'false',
-            'Rack-Hijack-Q': 'false',
-            'Rack-Hijack': '',
-            'Rack-Hijack-IO': ''
-        }, 'headers')
+        self.assertLess(
+            abs(self.date_to_sec_epoch(date) - self.sec_epoch()),
+            5,
+            'date header',
+        )
+
+        self.assertDictEqual(
+            headers,
+            {
+                'Connection': 'close',
+                'Content-Length': str(len(body)),
+                'Content-Type': 'text/html',
+                'Request-Method': 'POST',
+                'Request-Uri': '/',
+                'Http-Host': 'localhost',
+                'Server-Protocol': 'HTTP/1.1',
+                'Custom-Header': 'blah',
+                'Rack-Version': '13',
+                'Rack-Url-Scheme': 'http',
+                'Rack-Multithread': 'false',
+                'Rack-Multiprocess': 'true',
+                'Rack-Run-Once': 'false',
+                'Rack-Hijack-Q': 'false',
+                'Rack-Hijack': '',
+                'Rack-Hijack-IO': '',
+            },
+            'headers',
+        )
         self.assertEqual(resp['body'], body, 'body')
 
     def test_ruby_application_query_string(self):
@@ -55,8 +68,11 @@ class TestUnitRubyApplication(unit.TestUnitApplicationRuby):
 
         resp = self.get(url='/?var1=val1&var2=val2')
 
-        self.assertEqual(resp['headers']['Query-String'], 'var1=val1&var2=val2',
-            'Query-String header')
+        self.assertEqual(
+            resp['headers']['Query-String'],
+            'var1=val1&var2=val2',
+            'Query-String header',
+        )
 
     def test_ruby_application_query_string_empty(self):
         self.load('query_string')
@@ -64,8 +80,9 @@ class TestUnitRubyApplication(unit.TestUnitApplicationRuby):
         resp = self.get(url='/?')
 
         self.assertEqual(resp['status'], 200, 'query string empty status')
-        self.assertEqual(resp['headers']['Query-String'], '',
-            'query string empty')
+        self.assertEqual(
+            resp['headers']['Query-String'], '', 'query string empty'
+        )
 
     @unittest.expectedFailure
     def test_ruby_application_query_string_absent(self):
@@ -74,15 +91,17 @@ class TestUnitRubyApplication(unit.TestUnitApplicationRuby):
         resp = self.get()
 
         self.assertEqual(resp['status'], 200, 'query string absent status')
-        self.assertEqual(resp['headers']['Query-String'], '',
-            'query string absent')
+        self.assertEqual(
+            resp['headers']['Query-String'], '', 'query string absent'
+        )
 
     @unittest.expectedFailure
     def test_ruby_application_server_port(self):
         self.load('server_port')
 
-        self.assertEqual(self.get()['headers']['Server-Port'], '7080',
-            'Server-Port header')
+        self.assertEqual(
+            self.get()['headers']['Server-Port'], '7080', 'Server-Port header'
+        )
 
     def test_ruby_application_status_int(self):
         self.load('status_int')
@@ -97,20 +116,29 @@ class TestUnitRubyApplication(unit.TestUnitApplicationRuby):
     def test_ruby_application_input_read_parts(self):
         self.load('input_read_parts')
 
-        self.assertEqual(self.post(body='0123456789')['body'], '012345678',
-            'input read parts')
+        self.assertEqual(
+            self.post(body='0123456789')['body'],
+            '012345678',
+            'input read parts',
+        )
 
     def test_ruby_application_input_read_buffer(self):
         self.load('input_read_buffer')
 
-        self.assertEqual(self.post(body='0123456789')['body'], '0123456789',
-            'input read buffer')
+        self.assertEqual(
+            self.post(body='0123456789')['body'],
+            '0123456789',
+            'input read buffer',
+        )
 
     def test_ruby_application_input_read_buffer_not_empty(self):
         self.load('input_read_buffer_not_empty')
 
-        self.assertEqual(self.post(body='0123456789')['body'], '0123456789',
-            'input read buffer not empty')
+        self.assertEqual(
+            self.post(body='0123456789')['body'],
+            '0123456789',
+            'input read buffer not empty',
+        )
 
     def test_ruby_application_input_gets(self):
         self.load('input_gets')
@@ -122,8 +150,9 @@ class TestUnitRubyApplication(unit.TestUnitApplicationRuby):
     def test_ruby_application_input_gets_2(self):
         self.load('input_gets')
 
-        self.assertEqual(self.post(body='01234\n56789\n')['body'], '01234\n',
-            'input gets 2')
+        self.assertEqual(
+            self.post(body='01234\n56789\n')['body'], '01234\n', 'input gets 2'
+        )
 
     def test_ruby_application_input_gets_all(self):
         self.load('input_gets_all')
@@ -149,12 +178,14 @@ class TestUnitRubyApplication(unit.TestUnitApplicationRuby):
 
     @unittest.expectedFailure
     def test_ruby_application_syntax_error(self):
-        self.skip_alerts.extend([
-            r'Failed to parse rack script',
-            r'syntax error',
-            r'new_from_string',
-            r'parse_file'
-        ])
+        self.skip_alerts.extend(
+            [
+                r'Failed to parse rack script',
+                r'syntax error',
+                r'new_from_string',
+                r'parse_file',
+            ]
+        )
         self.load('syntax_error')
 
         self.assertEqual(self.get()['status'], 500, 'syntax error')
@@ -168,7 +199,8 @@ class TestUnitRubyApplication(unit.TestUnitApplicationRuby):
 
         self.assertIsNotNone(
             self.search_in_log(r'\[error\].+Error in application'),
-            'errors puts')
+            'errors puts',
+        )
 
     def test_ruby_application_errors_puts_int(self):
         self.load('errors_puts_int')
@@ -178,8 +210,8 @@ class TestUnitRubyApplication(unit.TestUnitApplicationRuby):
         self.stop()
 
         self.assertIsNotNone(
-            self.search_in_log(r'\[error\].+1234567890'),
-            'errors puts int')
+            self.search_in_log(r'\[error\].+1234567890'), 'errors puts int'
+        )
 
     def test_ruby_application_errors_write(self):
         self.load('errors_write')
@@ -190,13 +222,13 @@ class TestUnitRubyApplication(unit.TestUnitApplicationRuby):
 
         self.assertIsNotNone(
             self.search_in_log(r'\[error\].+Error in application'),
-            'errors write')
+            'errors write',
+        )
 
     def test_ruby_application_errors_write_to_s_custom(self):
         self.load('errors_write_to_s_custom')
 
-        self.assertEqual(self.get()['status'], 200,
-            'errors write to_s custom')
+        self.assertEqual(self.get()['status'], 200, 'errors write to_s custom')
 
     def test_ruby_application_errors_write_int(self):
         self.load('errors_write_int')
@@ -206,38 +238,40 @@ class TestUnitRubyApplication(unit.TestUnitApplicationRuby):
         self.stop()
 
         self.assertIsNotNone(
-            self.search_in_log(r'\[error\].+1234567890'),
-            'errors write int')
+            self.search_in_log(r'\[error\].+1234567890'), 'errors write int'
+        )
 
     def test_ruby_application_at_exit(self):
         self.load('at_exit')
 
         self.get()
 
-        self.conf({
-            "listeners": {},
-            "applications": {}
-        })
+        self.conf({"listeners": {}, "applications": {}})
 
         self.stop()
 
         self.assertIsNotNone(
-            self.search_in_log(r'\[error\].+At exit called\.'), 'at exit')
+            self.search_in_log(r'\[error\].+At exit called\.'), 'at exit'
+        )
 
     def test_ruby_application_header_custom(self):
         self.load('header_custom')
 
         resp = self.post(body="\ntc=one,two\ntc=three,four,\n\n")
 
-        self.assertEqual(resp['headers']['Custom-Header'],
-            ['', 'tc=one,two', 'tc=three,four,', '', ''], 'header custom')
+        self.assertEqual(
+            resp['headers']['Custom-Header'],
+            ['', 'tc=one,two', 'tc=three,four,', '', ''],
+            'header custom',
+        )
 
     @unittest.expectedFailure
     def test_ruby_application_header_custom_non_printable(self):
         self.load('header_custom')
 
-        self.assertEqual(self.post(body='\b')['status'], 500,
-            'header custom non printable')
+        self.assertEqual(
+            self.post(body='\b')['status'], 500, 'header custom non printable'
+        )
 
     def test_ruby_application_header_status(self):
         self.load('header_status')
@@ -277,7 +311,8 @@ class TestUnitRubyApplication(unit.TestUnitApplicationRuby):
 
         self.assertIsNotNone(
             self.search_in_log(r'\[error\].+Failed to run ruby script'),
-            'body each error')
+            'body each error',
+        )
 
     def test_ruby_application_body_file(self):
         self.load('body_file')
@@ -287,21 +322,30 @@ class TestUnitRubyApplication(unit.TestUnitApplicationRuby):
     def test_ruby_keepalive_body(self):
         self.load('mirror')
 
-        (resp, sock) = self.post(headers={
-            'Host': 'localhost',
-            'Connection': 'keep-alive',
-            'Content-Type': 'text/html'
-        }, start=True, body='0123456789' * 500)
+        (resp, sock) = self.post(
+            headers={
+                'Host': 'localhost',
+                'Connection': 'keep-alive',
+                'Content-Type': 'text/html',
+            },
+            start=True,
+            body='0123456789' * 500,
+        )
 
         self.assertEqual(resp['body'], '0123456789' * 500, 'keep-alive 1')
 
-        resp = self.post(headers={
-            'Host': 'localhost',
-            'Connection': 'close',
-            'Content-Type': 'text/html'
-        }, sock=sock, body='0123456789')
+        resp = self.post(
+            headers={
+                'Host': 'localhost',
+                'Connection': 'close',
+                'Content-Type': 'text/html',
+            },
+            sock=sock,
+            body='0123456789',
+        )
 
         self.assertEqual(resp['body'], '0123456789', 'keep-alive 2')
 
+
 if __name__ == '__main__':
     TestUnitRubyApplication.main()