Ensure the correct url generation with leading/trailing slashes in config.

Fixes: #1

Author: Nick Joyce

flask_assetrev/__init__.py

@@ -144,7 +144,10 @@ def get_asset_url(self, asset):
             asset_file = os.path.join(self.base_path, asset_file)
 
         if self.base_url:
-            return self.base_url + asset_file
+            return '/'.join([
+                self.base_url.rstrip('/'),
+                asset_file.lstrip('/'),
+            ])
 
         return url_for(
             'static',

test/test_assetrev.py

@@ -112,3 +112,21 @@ def test_missing_asset(self):
                 asset_url('missing.js')
 
             self.assertEqual(ctx.exception.asset_file, 'missing.js')
+
+    # https://github.com/njoyce/flask-assetrev/issues/1
+    def test_trailing_slash(self):
+        """
+        Handle trailing slashes in config
+        """
+        app = make_app(config={
+            'ASSETREV_BASE_URL': '//cdn.myapp.com/',
+            'ASSETREV_BASE_PATH': '/foobar',
+        })
+
+        with app.test_request_context():
+            import flask_assetrev as assetrev
+
+            self.assertEqual(
+                assetrev.asset_url('app.js'),
+                '//cdn.myapp.com/foobar/app.deadb33f.js'
+            )