[MAINTENANCE] Unpin snowflake-sqlalchemy

[NathanFarmer]

Unpin snowflake-sqlalchemy

They didn't yank the broken version (1.7.0), so we just exclude it.

There are some other breaking changes, but they only affect tests in a meaningful way.

• Description of PR changes above includes a link to an existing GitHub issue
• PR title is prefixed with one of: [BUGFIX], [FEATURE], [DOCS], [MAINTENANCE], [CONTRIB]
• Code is linted - run invoke lint (uses ruff format + ruff check)
• Appropriate tests and docs have been updated

[netlify]

✅ Deploy Preview for niobium-lead-7998 ready!

Name	Link
🔨 Latest commit	543aeaa
🔍 Latest deploy log	https://app.netlify.com/sites/niobium-lead-7998/deploys/67883e9a01905900088ab8b1
😎 Deploy Preview	https://deploy-preview-10838.docs.greatexpectations.io
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[NathanFarmer]

They didn't yank the broken version

[codecov]

❌ 20 Tests Failed:

Tests completed	Failed	Passed	Skipped
21219	20	21199	5858

View the top 3 failed tests by shortest run time

tests.data_context.test_data_context_in_code_config::test_DataContext_construct_data_context_id_uses_id_of_currently_configured_expectations_store

Stack Traces | 0.152s run time

aws_credentials = None

    @pytest.mark.aws_deps
    @mock_s3
    def test_DataContext_construct_data_context_id_uses_id_of_currently_configured_expectations_store(
        aws_credentials,
    ):
        """
        What does this test and why?
    
        A DataContext should have an id. This ID should come from either:
        1. configured expectations store store_backend_id
        2. great_expectations.yml
        3. new generated id from DataContextConfig
        This test verifies that DataContext._construct_data_context_id
        uses the store_backend_id from the currently configured expectations store
        when instantiating the DataContext
        """
    
        store_backend_id_filename = StoreBackend.STORE_BACKEND_ID_KEY[0]
        bucket = "leakybucket"
        expectations_store_prefix = "expectations_store_prefix"
        validation_results_store_prefix = "validation_results_store_prefix"
        data_docs_store_prefix = "data_docs_store_prefix"
        data_context_prefix = ""
    
        # Create a bucket in Moto's mock AWS environment
        conn = boto3.resource("s3", region_name="us-east-1")
        conn.create_bucket(Bucket=bucket)
    
        # Create a TupleS3StoreBackend
        # Initialize without store_backend_id and check that the store_backend_id is generated correctly
        s3_expectations_store_backend = TupleS3StoreBackend(
            filepath_template="my_file_{0}",
            bucket=bucket,
            prefix=expectations_store_prefix,
        )
        # Make sure store_backend_id is not the error string
        store_error_uuid = uuid.UUID("00000000-0000-0000-0000-00000000e003")
        s3_expectations_store_backend_id = s3_expectations_store_backend.store_backend_id
>       assert s3_expectations_store_backend_id != store_error_uuid
E       AssertionError: assert UUID('00000000-0000-0000-0000-00000000e003') != UUID('00000000-0000-0000-0000-00000000e003')

tests/data_context/test_data_context_in_code_config.py:160: AssertionError

tests.data_context.store.test_store_backends::test_TupleS3StoreBackend_with_s3_put_options

Stack Traces | 0.166s run time

aws_credentials = None

    @mock_s3
    @pytest.mark.aws_deps
    def test_TupleS3StoreBackend_with_s3_put_options(aws_credentials):
        bucket = "leakybucket"
        conn = boto3.client("s3", region_name="us-east-1")
        conn.create_bucket(Bucket=bucket)
    
        my_store = TupleS3StoreBackend(
            bucket=bucket,
            # Since not all out options are supported in moto, only Metadata and StorageClass is passed here.  # noqa: E501 # FIXME CoP
            s3_put_options={
                "Metadata": {"test": "testMetadata"},
                "StorageClass": "REDUCED_REDUNDANCY",
            },
        )
    
        assert my_store.config["s3_put_options"] == {
            "Metadata": {"test": "testMetadata"},
            "StorageClass": "REDUCED_REDUNDANCY",
        }
    
        my_store.set(("AAA",), "aaa")
    
        res = conn.get_object(Bucket=bucket, Key="AAA")
    
        assert res["Metadata"] == {"test": "testMetadata"}
        assert res["StorageClass"] == "REDUCED_REDUNDANCY"
    
>       assert my_store.get(("AAA",)) == "aaa"

.../data_context/store/test_store_backends.py:838: 
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 
.../data_context/store/_store_backend.py:121: in get
    value = self._get(key, **kwargs)
.../data_context/store/tuple_store_backend.py:545: in _get
    return self._get_by_s3_object_key(client, s3_object_key)
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 

self = <great_expectations.data_context.store.tuple_store_backend.TupleS3StoreBackend object at 0x7f756ad3b490>
s3_client = <botocore.client.S3 object at 0x7f756ae5d520>, s3_object_key = 'AAA'

    def _get_by_s3_object_key(self, s3_client, s3_object_key):
        try:
            s3_response_object = s3_client.get_object(Bucket=self.bucket, Key=s3_object_key)
        except (s3_client.exceptions.NoSuchKey, s3_client.exceptions.NoSuchBucket) as e:
            raise InvalidKeyError(  # noqa: TRY003 # FIXME CoP
                f"Unable to retrieve object from TupleS3StoreBackend with the following Key: {s3_object_key!s}"  # noqa: E501 # FIXME CoP
            ) from e
    
        return (
>           s3_response_object["Body"]
            .read()
            .decode(s3_response_object.get("ContentEncoding", "utf-8"))
        )
E       LookupError: unknown encoding: utf-8,aws-chunked

.../data_context/store/tuple_store_backend.py:569: LookupError

tests.data_context.store.test_store_backends::test_TupleS3StoreBackend_with_prefix

Stack Traces | 0.192s run time

aws_credentials = None

    @mock_s3
    @pytest.mark.aws_deps
    def test_TupleS3StoreBackend_with_prefix(aws_credentials):
        """
        What does this test test and why?
    
        We will exercise the store backend's set method twice and then verify
        that the we calling get and list methods will return the expected keys.
    
        We will also check that the objects are stored on S3 at the expected location,
        and that the correct S3 URL for the object can be retrieved.
    
        """
        bucket = "leakybucket"
        prefix = "this_is_a_test_prefix"
        base_public_path = "http://www.test.com/"
    
        # create a bucket in Moto's mock AWS environment
        conn = boto3.resource("s3", region_name="us-east-1")
        conn.create_bucket(Bucket=bucket)
    
        my_store = TupleS3StoreBackend(
            filepath_template="my_file_{0}",
            bucket=bucket,
            prefix=prefix,
        )
    
        # We should be able to list keys, even when empty
        keys = my_store.list_keys()
        assert len(keys) == 1
    
        my_store.set(("AAA",), "aaa", content_type="text/html; charset=utf-8")
>       assert my_store.get(("AAA",)) == "aaa"

.../data_context/store/test_store_backends.py:491: 
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 
.../data_context/store/_store_backend.py:121: in get
    value = self._get(key, **kwargs)
.../data_context/store/tuple_store_backend.py:545: in _get
    return self._get_by_s3_object_key(client, s3_object_key)
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 

self = <great_expectations.data_context.store.tuple_store_backend.TupleS3StoreBackend object at 0x7f756c6dcd90>
s3_client = <botocore.client.S3 object at 0x7f756c6f1fa0>
s3_object_key = 'this_is_a_test_prefix/my_file_AAA'

    def _get_by_s3_object_key(self, s3_client, s3_object_key):
        try:
            s3_response_object = s3_client.get_object(Bucket=self.bucket, Key=s3_object_key)
        except (s3_client.exceptions.NoSuchKey, s3_client.exceptions.NoSuchBucket) as e:
            raise InvalidKeyError(  # noqa: TRY003 # FIXME CoP
                f"Unable to retrieve object from TupleS3StoreBackend with the following Key: {s3_object_key!s}"  # noqa: E501 # FIXME CoP
            ) from e
    
        return (
>           s3_response_object["Body"]
            .read()
            .decode(s3_response_object.get("ContentEncoding", "utf-8"))
        )
E       LookupError: unknown encoding: utf-8,aws-chunked

.../data_context/store/tuple_store_backend.py:569: LookupError

To view more test analytics, go to the Test Analytics Dashboard
📢 Thoughts on this report? Let us know!

[NathanFarmer]

This call started returning dict_keys instead of list. We only call it in tests. Open issue here: snowflakedb/snowflake-sqlalchemy#566

[NathanFarmer]

I'm looking into how to reproduce this for a snowflake-sqlalchemy GH issue, but in the meantime xfailing this test allows us to un-pin.