✨ Set up CI with Azure Pipelines on windows and macos, resolve #122. :wheel_chair: skip the file permission related tests for now. #141

Author: chfw

.azure-pipelines-steps-macos.yml

@@ -0,0 +1,11 @@
+steps:
+- task: UsePythonVersion@0
+  displayName: 'Use Python 3.x'
+- script: |
+    python -m pip install --upgrade pip setuptools wheel
+    pip install -r requirements.txt
+    pip install -r tests/requirements.txt
+  displayName: 'Setup dependencies'
+- script: |
+    make
+  displayName: 'Run tests'

.azure-pipelines-steps.yml

@@ -0,0 +1,11 @@
+steps:
+- task: UsePythonVersion@0
+  displayName: 'Use Python 3.x'
+- script: |
+    python -m pip install --upgrade pip setuptools wheel
+    pip install -r requirements.txt
+    pip install -r tests\requirements.txt
+  displayName: 'Setup dependencies'
+- script: |
+    test.bat
+  displayName: 'Run tests'

.isort.cfg

@@ -1,6 +1,6 @@
 [settings]
 line_length=79
-known_first_party=lml, jinja2, ruamel.yaml
+known_first_party=lml, jinja2, ruamel.yaml, mock, nose
 indent='    '
 multi_line_output=3
 length_sort=1

README.rst

@@ -5,6 +5,9 @@ moban - 模板 Yet another jinja2 cli command for static text generation
 .. image:: https://api.travis-ci.org/moremoban/moban.svg?branch=master
    :target: http://travis-ci.org/moremoban/moban
 
+.. image:: GET https://dev.azure.com/moremoban/moban/_apis/build/repos/{repoType}/badge?api-version=5.0-preview.2
+   :target: https://dev.azure.com/moremoban/moban/_build?definitionId=1&_a=summary
+
 .. image:: https://codecov.io/gh/moremoban/moban/branch/master/graph/badge.svg
     :target: https://codecov.io/gh/moremoban/moban
 

azure-pipelines.yml

@@ -0,0 +1,15 @@
+# Starter pipeline
+# Start with a minimal pipeline that you can customize to build and deploy your code.
+# Add steps that build, run tests, deploy, and more:
+# https://aka.ms/yaml
+jobs:
+- job: 'Windows'
+  pool:
+    vmImage: vs2017-win2016
+  steps:
+  - template: '.azure-pipelines-steps.yml'
+- job: 'MacOS'
+  pool:
+    vmImage: macOS-10.13
+  steps:
+  - template: '.azure-pipelines-steps-macos.yml'

test.bat

@@ -3,5 +3,5 @@ pip freeze
 cd tests\moban-mako
 python setup.py install
 cd ..\..\
-nosetests --with-coverage --with-doctest --doctest-extension=.rst --cover-package=moban --cover-package=tests
-flake8 . --exclude=.moban.d --ignore=E203,E121,E123,E126,E226,E24,E704,W503,W504
+nosetests --with-coverage --cover-package=moban --cover-package=tests
+flake8 . --exclude=docs --exclude=.moban.d --ignore=E203,E121,E123,E126,E226,E24,E704,W503,W504

tests/test_context.py

@@ -1,6 +1,7 @@
 import os
 
 from nose.tools import eq_
+
 from moban.plugins import Context
 
 

tests/test_copier.py

@@ -3,6 +3,7 @@
 
 from mock import patch
 from nose.tools import eq_
+
 from moban.copier import Copier
 from moban.mobanfile import handle_copy
 

tests/test_docs.py

@@ -2,9 +2,10 @@
 import sys
 
 from mock import patch
-from moban.main import main
 from nose.tools import eq_
 
+from moban.main import main
+
 
 class TestTutorial:
     def setUp(self):
@@ -88,7 +89,8 @@ def test_level_7(self):
     def test_level_8(self):
         expected = "it is a test\n"
         folder = "level-8-pass-a-folder-full-of-templates"
-        self._raw_moban(["moban"], folder, expected, "templated-folder/my")
+        check_file = os.path.join("templated-folder", "my")
+        self._raw_moban(["moban"], folder, expected, check_file)
 
     def test_level_9(self):
         expected = "pypi-mobans: moban dependency as pypi package"

tests/test_engine.py

@@ -1,10 +1,10 @@
 import os
 
+from mock import patch
 from lml.plugin import PluginInfo
+from nose.tools import eq_, raises
 
 import moban.exceptions as exceptions
-from mock import patch
-from nose.tools import eq_, raises
 from moban.plugins import (
     ENGINES,
     Context,
@@ -13,6 +13,8 @@
 )
 from moban.jinja2.engine import Engine
 
+USER_HOME = os.path.join("user", "home", ".moban", "repos")
+
 
 @PluginInfo("library", tags=["testmobans"])
 class TestPypkg:
@@ -27,12 +29,12 @@ def test_expand_pypi_dir():
         assert os.path.exists(directory)
 
 
-@patch("moban.utils.get_moban_home", return_value="/user/home/.moban/repos")
+@patch("moban.utils.get_moban_home", return_value=USER_HOME)
 @patch("os.path.exists", return_value=True)
 def test_expand_repo_dir(_, __):
     dirs = list(expand_template_directories("git_repo:template"))
 
-    expected = ["/user/home/.moban/repos/git_repo/template"]
+    expected = [os.path.join(USER_HOME, "git_repo", "template")]
     eq_(expected, dirs)
 
 

tests/test_filter_github.py

@@ -1,4 +1,5 @@
 from nose.tools import eq_
+
 from moban.jinja2.filters.github import github_expand
 
 

tests/test_filter_repr.py

@@ -1,4 +1,5 @@
 from nose.tools import eq_
+
 from moban.jinja2.filters.repr import repr as repr_function
 
 

tests/test_hash_store.py

@@ -1,6 +1,8 @@
 import os
+import sys
 
 from moban.hashstore import HashStore
+from nose import SkipTest
 
 
 class TestHashStore:
@@ -13,7 +15,8 @@ def setUp(self):
         )
 
     def tearDown(self):
-        os.unlink(".moban.hashes")
+        if os.path.exists(".moban.hashes"):
+            os.unlink(".moban.hashes")
 
     def test_simple_use_case(self):
         hs = HashStore()
@@ -76,6 +79,8 @@ def test_dest_file_file_permision_changed(self):
         Save as above, but this time,
         the generated file had file permision change
         """
+        if sys.platform == 'win32':
+            raise SkipTest('No actual chmod on windows')
         hs = HashStore()
         flag = hs.is_file_changed(*self.fixture)
         if flag:

tests/test_jinja2_engine.py

@@ -1,6 +1,7 @@
 import os
 
 from nose.tools import eq_
+
 from moban.jinja2.engine import Engine
 
 

tests/test_jinja2_extensions.py

@@ -1,6 +1,7 @@
 import os
 
 from nose.tools import eq_
+
 from moban.plugins import BaseEngine
 from moban.jinja2.engine import Engine
 from moban.jinja2.extensions import jinja_global

tests/test_main.py

@@ -2,10 +2,11 @@
 import sys
 from shutil import copyfile
 
-import moban.exceptions as exceptions
 from mock import patch
 from nose.tools import raises, assert_raises
 
+import moban.exceptions as exceptions
+
 
 class TestException:
     def setUp(self):

tests/test_reporter.py

@@ -1,9 +1,10 @@
 import sys
 
-import moban.reporter as reporter
 from mock import patch
 from nose.tools import eq_
 
+import moban.reporter as reporter
+
 PY2 = sys.version_info[0] == 2
 if PY2:
     from StringIO import StringIO

tests/test_template.py

@@ -1,6 +1,7 @@
 import os
 
 from mock import patch
+
 from moban.plugins import BaseEngine
 from moban.jinja2.engine import Engine
 

tests/test_text_filter.py

@@ -1,4 +1,5 @@
 from nose.tools import eq_
+
 from moban.jinja2.filters.text import split_length
 
 

tests/test_utils.py

@@ -1,9 +1,11 @@
 import os
+import sys
 import stat
 from shutil import rmtree
 
 from mock import patch
 from nose.tools import eq_, raises
+from nose import SkipTest
 from moban.utils import (
     mkdir_p,
     get_repo_name,
@@ -25,6 +27,8 @@ def create_file(test_file, permission):
 
 
 def test_file_permission_copy():
+    if sys.platform == 'win32':
+        raise SkipTest('No actual chmod on windows')
     test_source = "test_file_permission_copy1"
     test_dest = "test_file_permission_copy2"
     create_file(test_source, 0o046)
@@ -44,6 +48,8 @@ def test_file_permissions_file_not_found():
 
 
 def test_file_permission_copy_symlink():
+    if sys.platform == 'win32':
+        raise SkipTest('No symlink on windows')
     test_source = "test_file_permission_copy1"
     test_dest = "test_file_permission_copy2"
     test_symlink = "test_file_permission_symlink"
@@ -92,15 +98,28 @@ def test_expand_dir():
     file_list = [("template-tests", "abc", "abc")]
     template_dirs = [os.path.join("tests", "fixtures")]
     results = list(expand_directories(file_list, template_dirs))
-    expected = [("template-tests/a.jj2", "abc", "abc/a")]
+    expected = [
+        (
+            os.path.join("template-tests", "a.jj2"),
+            "abc",
+            os.path.join("abc", "a"),
+        )
+    ]
     eq_(results, expected)
 
 
 def test_get_template_path():
-    temp_dirs = ["tests/fixtures/template-tests", "tests/abc", "tests/abc"]
+    temp_dirs = [
+        os.path.join("tests", "fixtures", "template-tests"),
+        os.path.join("tests", "abc"),
+        os.path.join("tests", "abc"),
+    ]
     template = "a.jj2"
     template_path = get_template_path(temp_dirs, template)
-    expected = os.path.join(os.getcwd(), "tests/fixtures/template-tests/a.jj2")
+    expected = os.path.join(
+        os.getcwd(),
+        os.path.join("tests", "fixtures", "template-tests", "a.jj2"),
+    )
     eq_(template_path, expected)
 
 

tests/test_yaml_loader.py

@@ -1,6 +1,7 @@
 import os
 
 from nose.tools import eq_, raises
+
 from moban.utils import open_yaml