Merge pull request #135 from ippei-fukamatsu/ALIS-1137

ALIS-1137: テストが通らなくなってたので修正

Author: keillera

.circleci/config.yml

@@ -23,8 +23,8 @@ jobs:
       # Download and cache dependencies
       - restore_cache:
           keys:
-          - v1-dependencies-{{ checksum "requirements.txt" }}
-          - v1-dependencies-
+          - v2-dependencies-{{ checksum "requirements.txt" }}
+          - v2-dependencies-
 
       - run:
           name: install dependencies
@@ -37,7 +37,7 @@ jobs:
       - save_cache:
           paths:
             - ./venv
-          key: v1-dependencies-{{ checksum "requirements.txt" }}
+          key: v2-dependencies-{{ checksum "requirements.txt" }}
 
       - run:
           name: run checkstyle for python code

exec_test.py

@@ -21,7 +21,7 @@
     # テストの実行ディレクトリを追加
     execute_dir.append(test_dir)
     # テスト対象ソースを複製（対象ソースは tests 配下と同一構造の src ディレクトリ配下が対象）
-    copy_tree(re.sub('^\./tmp_tests', './src', test_dir), test_dir)
+    copy_tree(re.sub(r'^\./tmp_tests', './src', test_dir), test_dir)
     # 共通ライブラリを複製
     copy_tree('./src/common', test_dir)
     # 共通ライブラリを複製

src/common/settings.py

@@ -13,13 +13,13 @@
         'type': 'string',
         'minLength': 3,
         'maxLength': 30,
-        'pattern': '^(?!.*--)[a-zA-Z0-9][a-zA-Z0-9-]+[a-zA-Z0-9]$'
+        'pattern': r'^(?!.*--)[a-zA-Z0-9][a-zA-Z0-9-]+[a-zA-Z0-9]$'
     },
     'phone_number': {
         'type': 'string',
         'minLength': 13,
         'maxLength': 13,
-        'pattern': '^\+81[6-9]0\d{8}$'
+        'pattern': r'^\+81[6-9]0\d{8}$'
     },
     'icon_image': {
         'type': 'string',

tests/handlers/me/articles/comments/create/test_me_articles_comments_create.py

@@ -13,6 +13,8 @@ def setUp(self):
         TestsUtil.set_all_tables_name_to_env()
         TestsUtil.delete_all_tables(self.dynamodb)
 
+        os.environ['SALT_FOR_ARTICLE_ID'] = 'test_salt'
+
         self.article_info_table = self.dynamodb.Table(os.environ['ARTICLE_INFO_TABLE_NAME'])
         self.article_info_table_items = [
             {