Issue #450: unit tests to verify the fix

tests/unit/conftest.py

@@ -32,3 +32,14 @@ def foo():
         return {}
 
     return app
+
+
+@fixture
+def sample_app_incompatible_with_cf():
+    app = Chalice('sample_invalid_cf')
+
+    @app.route('/')
+    def foo_invalid():
+        return {}
+
+    return app

tests/unit/test_package.py

@@ -1,6 +1,7 @@
 import mock
 
 import pytest
+import re
 from chalice.config import Config
 from chalice import package
 from chalice import __version__ as chalice_version
@@ -273,3 +274,21 @@ def test_fails_with_custom_auth(sample_app_with_auth,
         app_name='myapp', manage_iam_role=False, iam_role_arn='role-arn')
     with pytest.raises(package.UnsupportedFeatureError):
         p.generate_sam_template(config)
+
+
+def test_app_incompatible_with_cf(sample_app_incompatible_with_cf,
+                                  mock_swagger_generator,
+                                  mock_policy_generator):
+    p = package.SAMTemplateGenerator(
+        mock_swagger_generator, mock_policy_generator)
+    mock_swagger_generator.generate_swagger.return_value = {
+        'swagger': 'document'
+    }
+    config = Config.create(chalice_app=sample_app_incompatible_with_cf,
+                           api_gateway_stage='dev',
+                           app_name='sample_invalid_cf')
+    template = p.generate_sam_template(config)
+    check_exp = re.compile(r'[^A-Za-z0-9]+')
+    events = template['Resources']['APIHandler']['Properties']['Events']
+    for k in events.keys():
+        assert not check_exp.search(k)