Self review changes

Author: krishnangovindraj

docs/modules/ROOT/partials/rust/analyze/VariableInfo.adoc

@@ -0,0 +1,21 @@
+[#_struct_VariableInfo]
+=== VariableInfo
+
+*Implements traits:*
+
+* `Clone`
+* `Debug`
+
+Holds information about variables in a ``Pipeline``.
+
+[caption=""]
+.Fields
+// tag::properties[]
+[cols=",,"]
+[options="header"]
+|===
+|Name |Type |Description
+a| `name` a| `String` a| 
+|===
+// end::properties[]
+

docs/modules/ROOT/partials/rust/transaction/Transaction.adoc

@@ -8,12 +8,12 @@
 A transaction with a TypeDB database.
 
 // tag::methods[]
-[#_struct_Transaction_analyze_query_query_impl_AsRef_str_]
-==== analyze_query
+[#_struct_Transaction_analyze_query_impl_AsRef_str_]
+==== analyze
 
 [source,rust]
 ----
-pub fn analyze_query(
+pub fn analyze(
     &self,
     query: impl AsRef<str>,
 ) -> impl Promise<'static, Result<AnalyzedQuery>>
@@ -41,7 +41,7 @@ impl Promise<'static, Result<AnalyzedQuery>>
 .Code examples
 [source,rust]
 ----
-transaction.analyze_query(query)
+transaction.analyze(query)
 ----
 
 [#_struct_Transaction_close_]

rust/src/analyze/conjunction.rs

@@ -92,7 +92,7 @@ pub enum Constraint {
     },
     Expression {
         text: String,
-        assigned: Vec<ConstraintVertex>,
+        assigned: ConstraintVertex,
         arguments: Vec<ConstraintVertex>,
     },
     Is {

rust/src/connection/network/proto/analyze.rs

@@ -292,7 +292,9 @@ impl TryFromProto<conjunction_proto::StructureConstraint> for Constraint {
             }
             ConstraintProto::Expression(constraint_proto::Expression { assigned, arguments, text }) => {
                 Constraint::Expression {
-                    assigned: vec_from_proto(assigned)?,
+                    assigned: vec_from_proto(assigned)?.first().cloned().ok_or_else(|| {
+                        AnalyzeError::MissingResponseField { field: "Expression.assigned was an empty array" }
+                    })?,
                     arguments: vec_from_proto(arguments)?,
                     text,
                 }

rust/src/transaction.rs

@@ -93,9 +93,9 @@ impl Transaction {
     /// # Examples
     ///
     /// ```rust
-    /// transaction.analyze_query(query)
+    /// transaction.analyze(query)
     /// ```
-    pub fn analyze_query(&self, query: impl AsRef<str>) -> impl Promise<'static, Result<AnalyzedQuery>> {
+    pub fn analyze(&self, query: impl AsRef<str>) -> impl Promise<'static, Result<AnalyzedQuery>> {
         self.transaction_stream.analyze(query.as_ref())
     }
 

rust/tests/behaviour/steps/analyze.rs

@@ -33,7 +33,7 @@ pub(crate) async fn run_analyze_query(
     transaction: &Transaction,
     query: impl AsRef<str>,
 ) -> TypeDBResult<AnalyzedQuery> {
-    transaction.analyze_query(query).await
+    transaction.analyze(query).await
 }
 
 #[apply(generic_step)]