removed unneded expressions

Author: daniel-sanche

packages/google-cloud-firestore/google/cloud/firestore_v1/pipeline_expressions.py

@@ -1176,35 +1176,6 @@ def storage_size(self) -> "Expression":
         """
         return FunctionExpression("storage_size", [self])
 
-    @expose_as_static
-    def namespace(self) -> "Expression":
-        """Extracts the namespace from a document reference.
-
-        Example:
-            >>> Field.of("__path__").namespace()
-
-        Returns:
-            A new `Expression` representing the namespace extraction.
-        """
-        return FunctionExpression("namespace", [self])
-
-    @expose_as_static
-    def has_ancestor(self, ancestor: Expression | CONSTANT_TYPE) -> "BooleanExpression":
-        """Checks if the current document has the specified ancestor.
-
-        Example:
-            >>> Field.of("__path__").has_ancestor("projects/my-project/databases/(default)/documents/users/user1")
-
-        Args:
-            ancestor: The DocumentReference of the potential ancestor.
-
-        Returns:
-            A new `BooleanExpression` representing the has_ancestor check.
-        """
-        return BooleanExpression(
-            "has_ancestor", [self, self._cast_to_expr_or_convert_to_constant(ancestor)]
-        )
-
     @expose_as_static
     def reference_slice(
         self, start: int | Expression, end: int | Expression | None = None

packages/google-cloud-firestore/tests/system/pipeline_e2e/general.yaml

@@ -848,62 +848,6 @@ tests:
                     - fieldReferenceValue: __path__
                     name: storage_size
           name: select
-  - description: testNamespace
-    pipeline:
-      - Collection: books
-      - Limit: 1
-      - Select:
-        - AliasedExpression:
-            - FunctionExpression.namespace:
-                - Field: __path__
-            - res
-    assert_proto:
-      pipeline:
-        stages:
-        - args:
-          - referenceValue: /books
-          name: collection
-        - args:
-          - integerValue: '1'
-          name: limit
-        - args:
-          - mapValue:
-              fields:
-                res:
-                  functionValue:
-                    args:
-                    - fieldReferenceValue: __path__
-                    name: namespace
-          name: select
-  - description: testHasAncestor
-    pipeline:
-      - Collection: books
-      - Limit: 1
-      - Select:
-        - AliasedExpression:
-            - FunctionExpression.has_ancestor:
-                - Field: __path__
-                - Constant: "projects/test/databases/(default)/documents"
-            - res
-    assert_proto:
-      pipeline:
-        stages:
-        - args:
-          - referenceValue: /books
-          name: collection
-        - args:
-          - integerValue: '1'
-          name: limit
-        - args:
-          - mapValue:
-              fields:
-                res:
-                  functionValue:
-                    args:
-                    - fieldReferenceValue: __path__
-                    - stringValue: "projects/test/databases/(default)/documents"
-                    name: has_ancestor
-          name: select
   - description: testReferenceSlice
     pipeline:
       - Collection: books

packages/google-cloud-firestore/tests/unit/v1/test_pipeline_expressions.py

@@ -967,25 +967,6 @@ def test_storage_size(self):
         infix_instance = arg1.storage_size()
         assert infix_instance == instance
 
-    def test_namespace(self):
-        arg1 = self._make_arg("Input")
-        instance = Expression.namespace(arg1)
-        assert instance.name == "namespace"
-        assert instance.params == [arg1]
-        assert repr(instance) == "Input.namespace()"
-        infix_instance = arg1.namespace()
-        assert infix_instance == instance
-
-    def test_has_ancestor(self):
-        arg1 = self._make_arg("Input")
-        arg2 = self._make_arg("Ancestor")
-        instance = Expression.has_ancestor(arg1, arg2)
-        assert instance.name == "has_ancestor"
-        assert instance.params == [arg1, arg2]
-        assert repr(instance) == "Input.has_ancestor(Ancestor)"
-        infix_instance = arg1.has_ancestor(arg2)
-        assert infix_instance == instance
-
     def test_reference_slice(self):
         arg1 = self._make_arg("Input")
         arg2 = self._make_arg("Start")
@@ -1751,7 +1732,7 @@ def test_array_transform(self):
         ]
         assert (
             repr(instance_with_idx)
-            == "ArrayField.array_transform(Constant.of('element_alias'), TransformExpr, Constant.of('index_alias'))"
+            == "ArrayField.array_transform(TransformExpr, Constant.of('element_alias'), Constant.of('index_alias'))"
         )
         infix_instance_with_idx = arg1.array_transform(arg3, arg2, arg4)
         assert infix_instance_with_idx == instance_with_idx