mock.rs 8.7 KB
Newer Older
1
// SPDX-FileCopyrightText: Copyright (c) 2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved.
2
3
4
// SPDX-License-Identifier: Apache-2.0

use super::{
5
6
    Discovery, DiscoveryEvent, DiscoveryInstance, DiscoveryInstanceId, DiscoveryQuery,
    DiscoverySpec, DiscoveryStream,
7
};
8
use anyhow::Result;
9
10
use async_trait::async_trait;
use std::sync::{Arc, Mutex};
11
use tokio_util::sync::CancellationToken;
12
13
14
15
16
17
18
19
20
21
22
23
24

/// Shared in-memory registry for mock discovery
#[derive(Clone, Default)]
pub struct SharedMockRegistry {
    instances: Arc<Mutex<Vec<DiscoveryInstance>>>,
}

impl SharedMockRegistry {
    pub fn new() -> Self {
        Self::default()
    }
}

25
26
27
/// Mock implementation of Discovery for testing
/// We can potentially remove this once we have KVStoreDiscovery fully tested
pub struct MockDiscovery {
28
29
30
31
    instance_id: u64,
    registry: SharedMockRegistry,
}

32
impl MockDiscovery {
33
34
35
36
37
38
39
40
41
42
43
44
45
46
    pub fn new(instance_id: Option<u64>, registry: SharedMockRegistry) -> Self {
        let instance_id = instance_id.unwrap_or_else(|| {
            use std::sync::atomic::{AtomicU64, Ordering};
            static COUNTER: AtomicU64 = AtomicU64::new(1);
            COUNTER.fetch_add(1, Ordering::SeqCst)
        });

        Self {
            instance_id,
            registry,
        }
    }
}

47
48
49
/// Helper function to check if an instance matches a discovery query
fn matches_query(instance: &DiscoveryInstance, query: &DiscoveryQuery) -> bool {
    match (instance, query) {
50
        // Endpoint matching
51
52
        (DiscoveryInstance::Endpoint(_), DiscoveryQuery::AllEndpoints) => true,
        (DiscoveryInstance::Endpoint(inst), DiscoveryQuery::NamespacedEndpoints { namespace }) => {
53
54
55
56
            &inst.namespace == namespace
        }
        (
            DiscoveryInstance::Endpoint(inst),
57
            DiscoveryQuery::ComponentEndpoints {
58
59
60
61
62
63
                namespace,
                component,
            },
        ) => &inst.namespace == namespace && &inst.component == component,
        (
            DiscoveryInstance::Endpoint(inst),
64
            DiscoveryQuery::Endpoint {
65
66
67
68
69
70
71
72
73
74
                namespace,
                component,
                endpoint,
            },
        ) => {
            &inst.namespace == namespace
                && &inst.component == component
                && &inst.endpoint == endpoint
        }

75
76
        // Model matching
        (DiscoveryInstance::Model { .. }, DiscoveryQuery::AllModels) => true,
77
        (
78
            DiscoveryInstance::Model {
79
                namespace: inst_ns, ..
80
            },
81
            DiscoveryQuery::NamespacedModels { namespace },
82
        ) => inst_ns == namespace,
83
        (
84
            DiscoveryInstance::Model {
85
86
                namespace: inst_ns,
                component: inst_comp,
87
88
                ..
            },
89
            DiscoveryQuery::ComponentModels {
90
91
92
                namespace,
                component,
            },
93
        ) => inst_ns == namespace && inst_comp == component,
94
        (
95
            DiscoveryInstance::Model {
96
97
98
                namespace: inst_ns,
                component: inst_comp,
                endpoint: inst_ep,
99
100
                ..
            },
101
            DiscoveryQuery::EndpointModels {
102
103
104
105
                namespace,
                component,
                endpoint,
            },
106
107
108
109
110
        ) => inst_ns == namespace && inst_comp == component && inst_ep == endpoint,

        // Cross-type matches return false
        (
            DiscoveryInstance::Endpoint(_),
111
112
113
114
            DiscoveryQuery::AllModels
            | DiscoveryQuery::NamespacedModels { .. }
            | DiscoveryQuery::ComponentModels { .. }
            | DiscoveryQuery::EndpointModels { .. },
115
116
        ) => false,
        (
117
118
119
120
121
            DiscoveryInstance::Model { .. },
            DiscoveryQuery::AllEndpoints
            | DiscoveryQuery::NamespacedEndpoints { .. }
            | DiscoveryQuery::ComponentEndpoints { .. }
            | DiscoveryQuery::Endpoint { .. },
122
        ) => false,
123
124
125
126
    }
}

#[async_trait]
127
impl Discovery for MockDiscovery {
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
    fn instance_id(&self) -> u64 {
        self.instance_id
    }

    async fn register(&self, spec: DiscoverySpec) -> Result<DiscoveryInstance> {
        let instance = spec.with_instance_id(self.instance_id);

        self.registry
            .instances
            .lock()
            .unwrap()
            .push(instance.clone());

        Ok(instance)
    }

144
145
146
147
148
149
150
151
152
153
154
155
    async fn unregister(&self, instance: DiscoveryInstance) -> Result<()> {
        let instance_id = instance.instance_id();

        self.registry
            .instances
            .lock()
            .unwrap()
            .retain(|i| i.instance_id() != instance_id);

        Ok(())
    }

156
    async fn list(&self, query: DiscoveryQuery) -> Result<Vec<DiscoveryInstance>> {
157
158
159
        let instances = self.registry.instances.lock().unwrap();
        Ok(instances
            .iter()
160
            .filter(|instance| matches_query(instance, &query))
161
162
163
164
            .cloned()
            .collect())
    }

165
166
167
168
169
    async fn list_and_watch(
        &self,
        query: DiscoveryQuery,
        _cancel_token: Option<CancellationToken>,
    ) -> Result<DiscoveryStream> {
170
171
172
173
174
        use std::collections::HashSet;

        let registry = self.registry.clone();

        let stream = async_stream::stream! {
175
            let mut known_instances: HashSet<DiscoveryInstanceId> = HashSet::new();
176
177
178
179
180
181

            loop {
                let current: Vec<_> = {
                    let instances = registry.instances.lock().unwrap();
                    instances
                        .iter()
182
                        .filter(|instance| matches_query(instance, &query))
183
184
185
186
                        .cloned()
                        .collect()
                };

187
                let current_ids: HashSet<DiscoveryInstanceId> = current.iter().map(|i| i.id()).collect();
188
189
190

                // Emit Added events for new instances
                for instance in current {
191
                    let id = instance.id();
192
193
194
195
196
197
198
199
                    if known_instances.insert(id) {
                        yield Ok(DiscoveryEvent::Added(instance));
                    }
                }

                // Emit Removed events for instances that are gone
                for id in known_instances.difference(&current_ids).cloned().collect::<Vec<_>>() {
                    known_instances.remove(&id);
200
                    yield Ok(DiscoveryEvent::Removed(id));
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
                }

                tokio::time::sleep(tokio::time::Duration::from_millis(10)).await;
            }
        };

        Ok(Box::pin(stream))
    }
}

#[cfg(test)]
mod tests {
    use super::*;
    use futures::StreamExt;

    #[tokio::test]
    async fn test_mock_discovery_add_and_remove() {
        let registry = SharedMockRegistry::new();
219
220
        let client1 = MockDiscovery::new(Some(1), registry.clone());
        let client2 = MockDiscovery::new(Some(2), registry.clone());
221
222
223
224
225

        let spec = DiscoverySpec::Endpoint {
            namespace: "test-ns".to_string(),
            component: "test-comp".to_string(),
            endpoint: "test-ep".to_string(),
226
            transport: crate::component::TransportType::Nats("test-subject".to_string()),
227
228
        };

229
        let query = DiscoveryQuery::Endpoint {
230
231
232
233
234
235
            namespace: "test-ns".to_string(),
            component: "test-comp".to_string(),
            endpoint: "test-ep".to_string(),
        };

        // Start watching
236
        let mut stream = client1.list_and_watch(query.clone(), None).await.unwrap();
237
238
239
240
241
242

        // Add first instance
        client1.register(spec.clone()).await.unwrap();

        let event = stream.next().await.unwrap().unwrap();
        match event {
243
244
            DiscoveryEvent::Added(DiscoveryInstance::Endpoint(inst)) => {
                assert_eq!(inst.instance_id, 1);
245
246
247
248
249
250
251
252
253
            }
            _ => panic!("Expected Added event for instance-1"),
        }

        // Add second instance
        client2.register(spec.clone()).await.unwrap();

        let event = stream.next().await.unwrap().unwrap();
        match event {
254
255
            DiscoveryEvent::Added(DiscoveryInstance::Endpoint(inst)) => {
                assert_eq!(inst.instance_id, 2);
256
257
258
259
260
261
            }
            _ => panic!("Expected Added event for instance-2"),
        }

        // Remove first instance
        registry.instances.lock().unwrap().retain(|i| match i {
262
            DiscoveryInstance::Endpoint(inst) => inst.instance_id != 1,
263
            DiscoveryInstance::Model { instance_id, .. } => *instance_id != 1,
264
265
266
267
        });

        let event = stream.next().await.unwrap().unwrap();
        match event {
268
269
270
            DiscoveryEvent::Removed(id) => {
                let endpoint_id = id.extract_endpoint_id().expect("Expected endpoint removal");
                assert_eq!(endpoint_id.instance_id, 1);
271
272
273
274
275
            }
            _ => panic!("Expected Removed event for instance-1"),
        }
    }
}