Issue 4374 - Made Python code part of jshydra comply with our coding practices

autotest.py

 # This Source Code is subject to the terms of the Mozilla Public License
 # version 2.0 (the "License"). You can obtain a copy of the License at
 # http://mozilla.org/MPL/2.0/.
 
 from __future__ import print_function
 
 import sys
 import os
 import re
 import difflib
@@ skipping 13 matching lines @@
         name = None
         args = None
 
         with open(filename, 'r') as file:
             for line in file:
                 match = re.search(r'^//\s*([A-Za-z]+):\s*(.*?)\s*$', line)
                 if match:
                     key = match.group(1).lower()
                     if key == 'name':
                         name = match.group(2)
-                    elif key == 'args':
+                    elif key == 'arguments':

[Sebastian Noack, 2016/08/30 12:03:06]

I accidentally replaced this string as well when renaming the variable, causing
no tests to be found. All other changes in this patch set are due to rebasing.

[Vasily Kuznetsov, 2016/08/30 13:25:30]

Oops, I missed it too. Good catch!

                         args = match.group(2).split()
 
         if args is None:
             continue
 
         output = abp_rewrite.rewrite_js(args, filename)
         with open(filename + '.expected', 'rU') as file:
             expected = file.read()
 
         if output == expected:
             print(name + ' passed')
         else:
             succeed = False
             print(name + ' failed! Log:')
             for line in difflib.unified_diff(expected.splitlines(),
                                              output.splitlines(),
                                              fromfile=filename + '.expected',
                                              tofile=filename + '.output'):
                 print(line)
             print()
 
     return succeed
 
 if __name__ == '__main__':
     sys.exit(not run_tests())