Merge 3ce7fac into 246a48e

.bumpversion.cfg

@@ -1,5 +1,5 @@
 [bumpversion]
-current_version = 0.3.1
+current_version = 0.3.2
 commit = True
 tag = True
 

.travis.yml

@@ -34,6 +34,14 @@ matrix:
     - python: '3.6'
       env:
         - TOXENV=3.6-nocov
+    - python: '3.7'
+      dist: xenial
+      env:
+        - TOXENV=3.7-cover,report,coveralls,codecov
+    - python: '3.7'
+      dist: xenial
+      env:
+        - TOXENV=3.7-nocov
     - python: 'pypy-5.4'
       env:
         - TOXENV=pypy-cover,report,coveralls,codecov

CHANGELOG.rst

@@ -2,6 +2,14 @@
 Changelog
 =========
 
+0.3.2 (2019-08-05)
+------------------
+
+* Fixed broken builds due to changes in warning output
+* Changed tests to check error messages
+* Added python 3.7 to testing matrix
+* Added requirement to verify SNI when checking URLs in docs
+
 0.3.1 (2018-08-20)
 ------------------
 

HISTORY.rst

@@ -2,6 +2,14 @@
 History
 =======
 
+0.3.2 (2019-08-05)
+------------------
+
+* Fixed broken builds due to changes in warning output
+* Changed tests to check error messages
+* Added python 3.7 to testing matrix
+* Added requirement to verify SNI when checking URLs in docs
+
 0.3.1 (2018-08-20)
 ------------------
 

ci/templates/.travis.yml

@@ -12,9 +12,12 @@ matrix:
   include:
 {%- for env, config in tox_environments|dictsort %}{{ '' }}
     - python: '{{ '{0[0]}-5.4'.format(env.split('-')) if env.startswith('pypy') else env.split('-')[0] }}'
+      {% if not env.startswith('pypy') and env.split('-')[0] >= "3.7" %}
+      dist: xenial
+      {% endif %}
       env:
-        - TOXENV={{ env }}{% if "true" in config.cover %},report,coveralls,codecov{% endif -%}
-{% endfor %}
+        - TOXENV={{ env }}{% if "true" in config.cover %},report,coveralls,codecov{% endif %}
+{%- endfor %}
 
 before_install:
   - python --version

docs/conf.py

@@ -28,7 +28,7 @@
 year = '2018'
 author = u'Anderson Toshiyuki Sasaki <ansasaki@redhat.com>'
 copyright = '{0}, {1}'.format(year, author)
-version = release = u'0.3.1'
+version = release = u'0.3.2'
 
 pygments_style = 'trac'
 templates_path = ['.']

docs/requirements.txt

@@ -1,3 +1,4 @@
 sphinx>=1.3
 sphinx-rtd-theme
+requests[security]
 -e .

setup.cfg

@@ -70,6 +70,7 @@ python_versions =
     3.4
     3.5
     3.6
+    3.7
     pypy
 
 coverage_flags =

src/abimap/_version.py

@@ -1 +1 @@
-__version__ = '0.3.1'
+__version__ = '0.3.2'

src/abimap/symver.py

@@ -970,9 +970,9 @@ def clean_symbols(symbols):
                     duplicates.add(previous)
                 previous = i
         if duplicates:
-            dup_list = "".join((" " * 4 + dup + "\n" for dup in
+            dup_list = "".join((dup + ", " for dup in
                                 sorted(duplicates)))
-            logger.warning("Duplicated symbols provided:\n%s", dup_list)
+            logger.warning("Duplicated symbols provided: %s", dup_list)
 
     return clean
 
@@ -1009,7 +1009,7 @@ def check_files(out_arg, out_name, in_arg, in_name, dry):
                     # preserve the content
                     shutil.copy2(str(in_name), str(in_name) + ".old")
                 except Exception as e:
-                    logger.error("Could no copy \'%s\' to \'%s.old\'."
+                    logger.error("Could not copy \'%s\' to \'%s.old\'."
                                  " Aborting.", str(in_name), str(in_name))
                     raise e
 

tests/conftest.py

@@ -1,5 +1,6 @@
 import filecmp
 import os
+import re
 from distutils import dir_util
 
 import pytest
@@ -56,7 +57,8 @@ def testcases(datadir, capsys):
             if file_type.lower() == ".yml" or file_type.lower() == ".yaml":
                 with open(str(test_input), 'r') as stream:
                     try:
-                        all_tests.extend(yaml.load(stream))
+                        all_tests.extend(yaml.load(stream,
+                                                   Loader=yaml.FullLoader))
                     except yaml.YAMLError as e:
                         with capsys.disabled():
                             print(e)
@@ -87,6 +89,50 @@ def __exit__(self, etype, value, traceback):
         os.chdir(self.saved_path)
 
 
+def is_warning_in_log(expected, log):
+    """
+    Search for a warning containing the expected message in the log. Returns
+    True if found; False otherwise.
+
+    :param expected: The regular expression
+    "param log: The input string
+    """
+
+    # Escape possibly existing special characters
+    escaped = re.escape(expected)
+
+    regex = "WARNING.*" + escaped
+
+    to_match = re.compile(regex)
+    found = to_match.search(log)
+    if found:
+        return True
+
+    return False
+
+
+def is_error_in_log(expected, log):
+    """
+    Search for an error containing the expected message in the log. Returns
+    True if found; False otherwise.
+
+    :param expected: The regular expression
+    "param log: The input string
+    """
+
+    # Escape possibly existing special characters
+    escaped = re.escape(expected)
+
+    regex = "ERROR.*" + escaped
+
+    to_match = re.compile(regex)
+    found = to_match.search(log)
+    if found:
+        return True
+
+    return False
+
+
 def run_tc(tc, datadir, capsys, caplog):
     """
     Run a command test case (for update and new commands)
@@ -166,10 +212,15 @@ class C(object):
                 # Fail
                 assert 0
 
-        # Check if the expected messages are in the log
+        # Check if the expected warning messages are in the log
         if tc_out["warnings"]:
             for expected in tc_out["warnings"]:
-                assert expected in caplog.text
+                assert is_warning_in_log(expected, caplog.text)
+
+        # Check if the expected error messages are in the log
+        if tc_out["errors"]:
+            for expected in tc_out["errors"]:
+                assert is_error_in_log(expected, caplog.text)
 
         # If a log file was supposed to exist, check the content
         if args.logfile:
@@ -178,7 +229,10 @@ class C(object):
                     logged = log.read()
                     if tc_out["warnings"]:
                         for expected in tc_out["warnings"]:
-                            assert expected in logged
+                            assert is_warning_in_log(expected, caplog.text)
+                    if tc_out["errors"]:
+                        for expected in tc_out["errors"]:
+                            assert is_error_in_log(expected, caplog.text)
                     if tc_out["exceptions"]:
                         for expected in tc_out["exceptions"]:
                             assert expected in logged

tests/data_template/test_new/guess_names.yaml

@@ -10,6 +10,7 @@
     file:
     stdout: "separated.stdout"
     warnings:
+    errors:
     exceptions:
 -
   input:
@@ -21,6 +22,7 @@
     file:
     stdout: "given_release.stdout"
     warnings:
+    errors:
     exceptions:
 -
   input:
@@ -33,6 +35,7 @@
     file:
     stdout: "release_and_version.stdout"
     warnings:
+    errors:
     exceptions:
 -
   input:
@@ -47,6 +50,7 @@
     file:
     stdout: "release_and_name.stdout"
     warnings:
+    errors:
     exceptions:
 -
   input:
@@ -61,5 +65,7 @@
       - "Release provided is not well formed (a well formed release contain
         the library identifier and the version information). \
         Suggested: something like LIBNAME_1_2_3"
+    errors:
+      - "Please provide the release name."
     exceptions:
       - "Please provide the release name."

tests/data_template/test_new/options.yaml

@@ -10,6 +10,7 @@
     file:
     stdout: "sanity.stdout"
     warnings:
+    errors:
     exceptions:
 -
   input:
@@ -23,6 +24,7 @@
     file:
     stdout: "sanity_final.stdout"
     warnings:
+    errors:
     exceptions:
 -
   input:
@@ -37,6 +39,7 @@
     file: "with_out.outfile"
     stdout: ""
     warnings:
+    errors:
     exceptions:
 -
   input:
@@ -56,6 +59,7 @@
       - "Overwriting existing file 'overwrite.in'"
       - "Given paths in '--out' and '--in' are the same."
       - "Moving 'overwrite.in' to 'overwrite.in.old'."
+    errors:
     exceptions:
 -
   input:
@@ -75,6 +79,7 @@
     warnings:
       - "Overwriting existing file 'dry.in'"
       - "Given paths in '--out' and '--in' are the same."
+    errors:
     exceptions:
 -
   input:
@@ -90,6 +95,7 @@
     stdout:
     warnings:
       - "No valid symbols provided. Nothing done."
+    errors:
     exceptions:
 -
   input:
@@ -105,6 +111,8 @@
       - "Release provided is not well formed (a well formed release contain \
         the library identifier and the version information). \
         Suggested: something like LIBNAME_1_2_3"
+    errors:
+      - "Please provide the release name."
     exceptions:
       - "Please provide the release name."
 -
@@ -116,6 +124,8 @@
     file:
     stdout:
     warnings:
+    errors:
+      - "It is necessary to provide either release name or name and version"
     exceptions:
       - "It is necessary to provide either release name or name and version"
 -
@@ -129,5 +139,6 @@
     file:
     stdout: "duplicated_input.stdout"
     warnings:
-      - "Duplicated symbols provided:\n    a\n    b\n\n"
+      - "Duplicated symbols provided: a, b"
+    errors:
     exceptions:

tests/data_template/test_overwrite_protected/overwrite_protected.yaml

@@ -16,7 +16,8 @@
       - "Overwriting existing file 'overwrite_protected.in'"
       - "Given paths in '--out' and '--in' are the same."
       - "Moving 'overwrite_protected.in' to 'overwrite_protected.in.old'."
-      - "Could no copy 'overwrite_protected.in' to \
+    errors:
+      - "Could not copy 'overwrite_protected.in' to \
         'overwrite_protected.in.old'. Aborting."
     exceptions:
       - "Permission denied: 'overwrite_protected.in.old'"