Unify part of code with logwrap

* coverage rules
* MANIFEST.in
* use _version.py
* part of setup.py
* coverage on azure

Author: penguinolog

.azure_pipelines/run_tests.yml

@@ -40,14 +40,13 @@ jobs:
 
   - script: |
       pip install -U wheel
-      python setup.py bdist_wheel
-      pip install --pre --no-index -f dist advanced-descriptors
+      python setup.py develop -v
     displayName: Build and install
 
   - script: |
-      pip install pytest pytest-sugar
+      pip install pytest pytest-sugar pytest-cov
 
-      pytest -vvv --junitxml=unit_result.xml test
+      pytest -vvv --junitxml=unit_result.xml --cov=advanced_descriptors --cov-report=xml --cov-report=html --cov-report term test
     displayName: PyTest
 
   - task: PublishTestResults@2
@@ -57,3 +56,10 @@ jobs:
       testResultsFormat: "JUnit"
       testResultsFiles: ${{ format('$(System.DefaultWorkingDirectory)/unit_result.xml') }}
       testRunTitle: ${{ format('{0}_{1}_{2}', parameters.name, parameters.architecture, parameters.kind) }}
+
+  - ${{ if ne(parameters.kind, 'cython') }}:
+    - task: PublishCodeCoverageResults@1
+      inputs:
+        codeCoverageTool: Cobertura
+        summaryFileLocation: ${{ format('$(System.DefaultWorkingDirectory)/coverage.xml') }}
+        reportDirectory: ${{ format('$(System.DefaultWorkingDirectory)/htmlcov') }}

.coveragerc

@@ -3,3 +3,36 @@ source =
     advanced_descriptors
 omit =
     test/*
+branch = True
+[report]
+exclude_lines =
+  # Have to re-enable the standard pragma
+  pragma: no cover
+
+  # Don't complain about missing debug-only code:
+  def __repr__
+
+  # Don't complain if tests don't hit defensive assertion code:
+  raise NotImplementedError
+
+  # Exclude methods marked as abstract
+  @abstractmethod
+
+  # Exclude import statements
+  ^from\b
+  ^import\b
+
+  # Exclude variable declarations that are executed when file is loaded
+  ^[a-zA-Z_]+\b\s=
+
+  # Code for static analysis is never covered:
+  if typing.TYPE_CHECKING:
+
+  # Fallback code with no installed deps is almost impossible to cover properly
+  except ImportError:
+
+  # Don't complain if non-runnable code isn't run:
+  if __name__ == .__main__.:
+
+  # OS Specific
+  if platform.system()

.gitignore

@@ -1,9 +1,12 @@
 ### Test results
 /*_result*.xml
 /report.html
-/assets/
+/assets/*
 .cache/*
-.pytest_cache/
+.pytest_cache/*
+
+### Generated version
+/advanced_descriptors/_version.py
 
 ### Generated code
 /advanced_descriptors/*.c

.travis.yml

@@ -47,7 +47,7 @@ _helpers:
 
 script:
   - python setup.py develop -v
-  - py.test -vv --cov-config .coveragerc --cov-report= --cov=advanced_descriptors test
+  - py.test -vvv --cov-config .coveragerc --cov-report= --cov=advanced_descriptors test
   - coverage report -m --fail-under 89
 after_success:
   - coveralls
@@ -59,8 +59,9 @@ jobs:
       install:
         - *upgrade_python_toolset
         - *install_deps
-        - pip install --upgrade "pylint > 2.3" isort[pyproject,requirements]
+        - pip install --upgrade "pylint >= 2.3" isort[pyproject,requirements]
       script:
+        - python setup.py --version
         - pylint advanced_descriptors
     - <<: *static_analysis
       name: "Bandit"
@@ -74,8 +75,9 @@ jobs:
       install:
         - *upgrade_python_toolset
         - *install_deps
-        - pip install --upgrade "mypy >= 0.670"
+        - pip install --upgrade "mypy >= 0.700"
       script:
+        - python setup.py --version
         - mypy --strict advanced_descriptors
     - <<: *static_analysis
       name: "PEP8"

LICENSE

@@ -186,7 +186,7 @@
       same "printed page" as the copyright notice for easier
       identification within third-party archives.
 
-   Copyright 2017-2018 Alexey Stepanov aka penguinolog
+   Copyright 2017-2019 Alexey Stepanov aka penguinolog
 
    Licensed under the Apache License, Version 2.0 (the "License");
    you may not use this file except in compliance with the License.

MANIFEST.in

@@ -1,4 +1,5 @@
-include *.rst LICENSE requirements.txt pyproject.toml
+include *.rst LICENSE requirements.txt
+global-include *.pyx *.pxd
 global-exclude *.c
 exclude Makefile
 prune tools

advanced_descriptors/__init__.py

@@ -13,24 +13,17 @@
 
 """Advanced descriptors for special cases."""
 
-import pkg_resources
-
+# Local Implementation
 from .advanced_property import AdvancedProperty
 from .log_on_access import LogOnAccess
 from .separate_class_method import SeparateClassMethod
 
 __all__ = ("SeparateClassMethod", "AdvancedProperty", "LogOnAccess")
 
 try:
-    __version__ = pkg_resources.get_distribution(__name__).version
-except pkg_resources.DistributionNotFound:
-    # package is not installed, try to get from SCM
-    try:
-        import setuptools_scm  # type: ignore
-
-        __version__ = setuptools_scm.get_version()
-    except ImportError:
-        setuptools_scm = None
+    from ._version import version as __version__
+except ImportError:
+    pass
 
 __author__ = "Alexey Stepanov"
 __author_email__ = "[email protected]"

advanced_descriptors/advanced_property.py

@@ -17,6 +17,7 @@
 
 __all__ = ("AdvancedProperty",)
 
+# Standard Library
 import typing
 
 

advanced_descriptors/log_on_access.py

@@ -17,6 +17,7 @@
 
 __all__ = ("LogOnAccess",)
 
+# Standard Library
 import logging
 import sys
 import traceback

advanced_descriptors/separate_class_method.py

@@ -17,6 +17,7 @@
 
 __all__ = ("SeparateClassMethod",)
 
+# Standard Library
 import functools
 import typing