Bug fix repo name (moremoban#227)

* 🐛 fix wrong repo name bug by using giturlparse package. resolves moremoban#226

* 🔥 remove moban from test requirements

.moban.cd/moban.yml

@@ -24,6 +24,6 @@ dependencies:
   - appdirs==1.4.3
   - crayons
   - GitPython==2.1.11
-  - git-url-parse
+  - giturlparse
 description: Yet another jinja2 cli command for static text generation
 scm_host: github.com

moban/repo.py

@@ -32,12 +32,14 @@ def git_clone(requires):
 
 def get_repo_name(repo_url):
     import giturlparse
-    from giturlparse.parser import ParserError
 
     try:
         repo = giturlparse.parse(repo_url)
-        return repo.name
-    except ParserError:
+        name = repo.repo
+        if name.endswith('/'):
+            name = name[:-1]
+        return name
+    except AttributeError:
         reporter.report_error_message(
             constants.MESSAGE_INVALID_GIT_URL % repo_url
         )

requirements.txt

@@ -4,4 +4,4 @@ lml>=0.0.9
 appdirs==1.4.3
 crayons
 GitPython==2.1.11
-git-url-parse
+giturlparse

tests/requirements.txt

@@ -3,7 +3,6 @@ codecov
 coverage
 mock
 flake8
-moban
 black;python_version>="3.6"
 isort;python_version>="3.6"
 moban-handlebars

tests/test_repo.py

@@ -86,13 +86,14 @@ def test_checkout_new_with_branch(
 
 def test_get_repo_name():
     repos = [
+        "https://github.com/sphinx-doc/sphinx",
         "https://github.com/abc/repo",
         "https://github.com/abc/repo.git",
         "https://github.com/abc/repo/",
         "git@github.com:moremoban/moban.git",
     ]
     actual = [get_repo_name(repo) for repo in repos]
-    expected = ["repo", "repo", "repo", "moban"]
+    expected = ["sphinx", "repo", "repo", "repo", "moban"]
     eq_(expected, actual)