Update contract tests to latest flask version (#176)

Our contract tests depend on flask v1, which in turn depends on Jinja 2.
Both of these are terribly dated and no longer supported.

Jinja depends on markupsafe. markupsafe recently updated its code to no longer provide
soft_unicode which in turn broke Jinja.

Updating to the latest flask keeps all transitive dependencies better
aligned and addresses this mismatch.

Author: keelerm84

.circleci/config.yml

@@ -10,6 +10,7 @@ workflows:
           name: Python 3.5
           docker-image: cimg/python:3.5
           skip-sse-contract-tests: true  # the test service app has dependencies that aren't available in 3.5, which is EOL anyway
+          skip-contract-tests: true  # the test service app has dependencies that aren't available in 3.5, which is EOL anyway
       - test-linux:
           name: Python 3.6
           docker-image: cimg/python:3.6
@@ -46,6 +47,9 @@ jobs:
       skip-sse-contract-tests:
         type: boolean
         default: false
+      skip-contract-tests:
+        type: boolean
+        default: false
     docker:
       - image: <<parameters.docker-image>>
       - image: redis
@@ -109,13 +113,16 @@ jobs:
               name: run SSE contract tests
               command: cd sse-contract-tests && make run-contract-tests
 
-      - run: make build-contract-tests
-      - run:
-          command: make start-contract-test-service
-          background: true
-      - run:
-          name: run contract tests
-          command: TEST_HARNESS_PARAMS="-junit test-reports/contract-tests-junit.xml" make run-contract-tests
+      - unless:
+          condition: <<parameters.skip-contract-tests>>
+          steps:
+          - run: make build-contract-tests
+          - run:
+              command: make start-contract-test-service
+              background: true
+          - run:
+              name: run contract tests
+              command: TEST_HARNESS_PARAMS="-junit test-reports/contract-tests-junit.xml" make run-contract-tests
 
       - store_test_results:
           path: test-reports

contract-tests/requirements.txt

@@ -1,2 +1,2 @@
-Flask==1.1.4
+Flask==2.0.3
 urllib3>=1.22.0

ldclient/impl/integrations/redis/redis_big_segment_store.py

@@ -26,7 +26,10 @@ def __init__(self, url: str, prefix: Optional[str], max_connections: int):
     def get_metadata(self) -> BigSegmentStoreMetadata:
         r = redis.Redis(connection_pool=self._pool)
         value = r.get(self._prefix + self.KEY_LAST_UP_TO_DATE)
-        return BigSegmentStoreMetadata(None if value is None else int(value))
+        if value is None:
+            return BigSegmentStoreMetadata(None)
+
+        return BigSegmentStoreMetadata(int(value))
 
     def get_membership(self, user_hash: str) -> Optional[dict]:
         r = redis.Redis(connection_pool=self._pool)

sse-contract-tests/requirements.txt

@@ -1,2 +1,2 @@
-Flask==2.0.2
+Flask==2.0.3
 urllib3>=1.22.0