test: add smart-default values tests

Author: x0rw

examples/lock.rs

@@ -1,4 +1,4 @@
-use rs_consul::{Config, Consul, CreateACLTokenPayload, types::*};
+use rs_consul::{Config, Consul, types::*};
 
 #[tokio::main] // Enables async main
 async fn main() {

src/types.rs

@@ -241,15 +241,15 @@ pub struct LockRequest<'a> {
     /// When locks are forcibly expired, such as when following the leader election pattern in an application, sessions may not be reaped for up to double this TTL, so long TTL values (> 1 hour) should be avoided.
     /// Defaults to 10 seconds.
     #[default(_code = "Duration::from_secs(10)")]
-    #[builder(default)]
+    #[builder(default = Duration::from_secs(10))]
     pub timeout: Duration,
     /// Controls the behavior to take when a session is invalidated. See also [LockExpirationBehavior](consul::types::LockExpirationBehavior)
     #[builder(default)]
     pub behavior: LockExpirationBehavior,
     /// Specifies the duration for the lock delay.
     /// Defaults to 1 second.
     #[default(_code = "Duration::from_secs(1)")]
-    #[builder(default)]
+    #[builder(default = Duration::from_secs(1))]
     pub lock_delay: Duration,
 }
 
@@ -294,7 +294,7 @@ pub(crate) struct SessionResponse {
     pub(crate) id: String,
 }
 
-#[derive(Clone, Debug, SmartDefault, Serialize, Deserialize, PartialEq, bon::Builder)]
+#[derive(Clone, Debug, SmartDefault, Serialize, Deserialize, PartialEq)]
 #[serde(rename_all = "PascalCase")]
 pub(crate) struct CreateSessionRequest {
     #[default(_code = "Duration::from_secs(0)")]

tests/test_runner.rs

@@ -576,14 +576,16 @@ mod tests {
 
 /// tests to ensure that Bon the builder can Build with minimal fields set
 mod minimal_setters_build_test {
+    use std::time::Duration;
+
     use rs_consul::{
         ACLTokenPolicyLink, CreateACLPolicyRequest, CreateACLTokenPayload, DeleteKeyRequest,
         DeregisterEntityPayload, GetNodesRequest, GetServiceNodesRequest, LockRequest,
         ReadKeyRequest, RegisterEntityCheck, RegisterEntityPayload, RegisterEntityService,
     };
 
     #[test]
-    fn test() {
+    fn test_builder_default() {
         let _ = ACLTokenPolicyLink::builder().build();
         let _ = CreateACLTokenPayload::builder()
             .policies(vec![
@@ -601,7 +603,11 @@ mod minimal_setters_build_test {
         let _ = CreateACLPolicyRequest::builder()
             .name("hehe".to_owned())
             .build();
-        let _ = LockRequest::builder().key("key-test").build();
+
+        let l_req = LockRequest::builder().key("key-test").build();
+        assert_eq!(l_req.timeout, Duration::from_secs(10));
+        assert_eq!(l_req.lock_delay, Duration::from_secs(1));
+
         let _ = RegisterEntityPayload::builder()
             .Node("node-ddd".to_owned())
             .Address("2.2.2.2".to_owned())