"deploy/operator/internal/controller/suite_test.go" did not exist on "602352ce190bcb02013c62c2337e8b8678015699"
mock.rs 8.63 KB
Newer Older
1
2
3
4
// SPDX-FileCopyrightText: Copyright (c) 2024-2025 NVIDIA CORPORATION & AFFILIATES. All rights reserved.
// SPDX-License-Identifier: Apache-2.0

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

/// 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()
    }
}

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

30
impl MockDiscovery {
31
32
33
34
35
36
37
38
39
40
41
42
43
44
    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,
        }
    }
}

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

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

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

#[async_trait]
125
impl Discovery for MockDiscovery {
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
    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)
    }

142
    async fn list(&self, query: DiscoveryQuery) -> Result<Vec<DiscoveryInstance>> {
143
144
145
        let instances = self.registry.instances.lock().unwrap();
        Ok(instances
            .iter()
146
            .filter(|instance| matches_query(instance, &query))
147
148
149
150
            .cloned()
            .collect())
    }

151
152
153
154
155
    async fn list_and_watch(
        &self,
        query: DiscoveryQuery,
        _cancel_token: Option<CancellationToken>,
    ) -> Result<DiscoveryStream> {
156
157
158
159
160
161
162
163
164
165
166
167
        use std::collections::HashSet;

        let registry = self.registry.clone();

        let stream = async_stream::stream! {
            let mut known_instances = HashSet::new();

            loop {
                let current: Vec<_> = {
                    let instances = registry.instances.lock().unwrap();
                    instances
                        .iter()
168
                        .filter(|instance| matches_query(instance, &query))
169
170
171
172
173
174
                        .cloned()
                        .collect()
                };

                let current_ids: HashSet<_> = current.iter().map(|i| {
                    match i {
175
                        DiscoveryInstance::Endpoint(inst) => inst.instance_id,
176
                        DiscoveryInstance::Model { instance_id, .. } => *instance_id,
177
178
179
180
181
182
                    }
                }).collect();

                // Emit Added events for new instances
                for instance in current {
                    let id = match &instance {
183
                        DiscoveryInstance::Endpoint(inst) => inst.instance_id,
184
                        DiscoveryInstance::Model { instance_id, .. } => *instance_id,
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
                    };
                    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<_>>() {
                    yield Ok(DiscoveryEvent::Removed(id));
                    known_instances.remove(&id);
                }

                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();
213
214
        let client1 = MockDiscovery::new(Some(1), registry.clone());
        let client2 = MockDiscovery::new(Some(2), registry.clone());
215
216
217
218
219

        let spec = DiscoverySpec::Endpoint {
            namespace: "test-ns".to_string(),
            component: "test-comp".to_string(),
            endpoint: "test-ep".to_string(),
220
            transport: crate::component::TransportType::NatsTcp("test-subject".to_string()),
221
222
        };

223
        let query = DiscoveryQuery::Endpoint {
224
225
226
227
228
229
            namespace: "test-ns".to_string(),
            component: "test-comp".to_string(),
            endpoint: "test-ep".to_string(),
        };

        // Start watching
230
        let mut stream = client1.list_and_watch(query.clone(), None).await.unwrap();
231
232
233
234
235
236

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

        let event = stream.next().await.unwrap().unwrap();
        match event {
237
238
            DiscoveryEvent::Added(DiscoveryInstance::Endpoint(inst)) => {
                assert_eq!(inst.instance_id, 1);
239
240
241
242
243
244
245
246
247
            }
            _ => 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 {
248
249
            DiscoveryEvent::Added(DiscoveryInstance::Endpoint(inst)) => {
                assert_eq!(inst.instance_id, 2);
250
251
252
253
254
255
            }
            _ => panic!("Expected Added event for instance-2"),
        }

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

        let event = stream.next().await.unwrap().unwrap();
        match event {
            DiscoveryEvent::Removed(instance_id) => {
                assert_eq!(instance_id, 1);
            }
            _ => panic!("Expected Removed event for instance-1"),
        }
    }
}