Mårten Kongstad | 4d2b4b0 | 2023-04-27 16:05:58 +0200 | [diff] [blame] | 1 | /* |
| 2 | * Copyright (C) 2023 The Android Open Source Project |
| 3 | * |
| 4 | * Licensed under the Apache License, Version 2.0 (the "License"); |
| 5 | * you may not use this file except in compliance with the License. |
| 6 | * You may obtain a copy of the License at |
| 7 | * |
| 8 | * http://www.apache.org/licenses/LICENSE-2.0 |
| 9 | * |
| 10 | * Unless required by applicable law or agreed to in writing, software |
| 11 | * distributed under the License is distributed on an "AS IS" BASIS, |
| 12 | * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| 13 | * See the License for the specific language governing permissions and |
| 14 | * limitations under the License. |
| 15 | */ |
| 16 | |
Mårten Kongstad | e66b89f | 2023-05-15 10:29:25 +0200 | [diff] [blame] | 17 | use anyhow::{bail, ensure, Result}; |
Mårten Kongstad | 4d2b4b0 | 2023-04-27 16:05:58 +0200 | [diff] [blame] | 18 | use serde::{Deserialize, Serialize}; |
| 19 | use std::io::{Read, Write}; |
| 20 | |
Mårten Kongstad | fa23d29 | 2023-05-11 14:47:02 +0200 | [diff] [blame] | 21 | use crate::aconfig::{FlagDeclaration, FlagState, FlagValue, Permission}; |
Mårten Kongstad | 4d2b4b0 | 2023-04-27 16:05:58 +0200 | [diff] [blame] | 22 | use crate::commands::Source; |
| 23 | |
Mårten Kongstad | fa23d29 | 2023-05-11 14:47:02 +0200 | [diff] [blame] | 24 | const DEFAULT_FLAG_STATE: FlagState = FlagState::Disabled; |
| 25 | const DEFAULT_FLAG_PERMISSION: Permission = Permission::ReadWrite; |
| 26 | |
Mårten Kongstad | 416330b | 2023-05-05 11:10:01 +0200 | [diff] [blame] | 27 | #[derive(Serialize, Deserialize, Debug)] |
Mårten Kongstad | 3095078 | 2023-05-09 13:31:29 +0200 | [diff] [blame] | 28 | pub struct Tracepoint { |
Mårten Kongstad | 76adff2 | 2023-05-08 10:57:24 +0200 | [diff] [blame] | 29 | pub source: Source, |
| 30 | pub state: FlagState, |
| 31 | pub permission: Permission, |
| 32 | } |
| 33 | |
| 34 | #[derive(Serialize, Deserialize, Debug)] |
Mårten Kongstad | 4d2b4b0 | 2023-04-27 16:05:58 +0200 | [diff] [blame] | 35 | pub struct Item { |
Mårten Kongstad | 3095078 | 2023-05-09 13:31:29 +0200 | [diff] [blame] | 36 | // TODO: duplicating the Cache.namespace as Item.namespace makes the internal representation |
| 37 | // closer to the proto message `parsed_flag`; hopefully this will enable us to replace the Item |
| 38 | // struct and use a newtype instead once aconfig has matured. Until then, namespace should |
| 39 | // really be a Cow<String>. |
| 40 | pub namespace: String, |
| 41 | pub name: String, |
Mårten Kongstad | 4d2b4b0 | 2023-04-27 16:05:58 +0200 | [diff] [blame] | 42 | pub description: String, |
Mårten Kongstad | c68c4ea | 2023-05-05 16:20:09 +0200 | [diff] [blame] | 43 | pub state: FlagState, |
Mårten Kongstad | 416330b | 2023-05-05 11:10:01 +0200 | [diff] [blame] | 44 | pub permission: Permission, |
Mårten Kongstad | 3095078 | 2023-05-09 13:31:29 +0200 | [diff] [blame] | 45 | pub trace: Vec<Tracepoint>, |
Mårten Kongstad | 4d2b4b0 | 2023-04-27 16:05:58 +0200 | [diff] [blame] | 46 | } |
| 47 | |
Mårten Kongstad | 416330b | 2023-05-05 11:10:01 +0200 | [diff] [blame] | 48 | #[derive(Serialize, Deserialize, Debug)] |
Mårten Kongstad | 4d2b4b0 | 2023-04-27 16:05:58 +0200 | [diff] [blame] | 49 | pub struct Cache { |
Mårten Kongstad | 3095078 | 2023-05-09 13:31:29 +0200 | [diff] [blame] | 50 | namespace: String, |
Mårten Kongstad | 4d2b4b0 | 2023-04-27 16:05:58 +0200 | [diff] [blame] | 51 | items: Vec<Item>, |
| 52 | } |
| 53 | |
Mårten Kongstad | 2f95444 | 2023-05-17 16:51:16 +0200 | [diff] [blame^] | 54 | // TODO: replace this function with Iterator.is_sorted_by_key(...)) when that API becomes stable |
| 55 | fn iter_is_sorted_by_key<'a, T: 'a, F, K>(iter: impl Iterator<Item = &'a T>, f: F) -> bool |
| 56 | where |
| 57 | F: FnMut(&'a T) -> K, |
| 58 | K: PartialOrd<K>, |
| 59 | { |
| 60 | let mut last: Option<K> = None; |
| 61 | for current in iter.map(f) { |
| 62 | if let Some(l) = last { |
| 63 | if l > current { |
| 64 | return false; |
| 65 | } |
| 66 | } |
| 67 | last = Some(current); |
Mårten Kongstad | 4d2b4b0 | 2023-04-27 16:05:58 +0200 | [diff] [blame] | 68 | } |
Mårten Kongstad | 2f95444 | 2023-05-17 16:51:16 +0200 | [diff] [blame^] | 69 | true |
| 70 | } |
Mårten Kongstad | 4d2b4b0 | 2023-04-27 16:05:58 +0200 | [diff] [blame] | 71 | |
Mårten Kongstad | 2f95444 | 2023-05-17 16:51:16 +0200 | [diff] [blame^] | 72 | impl Cache { |
Mårten Kongstad | 4d2b4b0 | 2023-04-27 16:05:58 +0200 | [diff] [blame] | 73 | pub fn read_from_reader(reader: impl Read) -> Result<Cache> { |
Mårten Kongstad | 2f95444 | 2023-05-17 16:51:16 +0200 | [diff] [blame^] | 74 | let cache: Cache = serde_json::from_reader(reader)?; |
| 75 | ensure!( |
| 76 | iter_is_sorted_by_key(cache.iter(), |item| &item.name), |
| 77 | "internal error: flags in cache file not sorted" |
| 78 | ); |
| 79 | Ok(cache) |
Mårten Kongstad | 4d2b4b0 | 2023-04-27 16:05:58 +0200 | [diff] [blame] | 80 | } |
| 81 | |
| 82 | pub fn write_to_writer(&self, writer: impl Write) -> Result<()> { |
Mårten Kongstad | 2f95444 | 2023-05-17 16:51:16 +0200 | [diff] [blame^] | 83 | ensure!( |
| 84 | iter_is_sorted_by_key(self.iter(), |item| &item.name), |
| 85 | "internal error: flags in cache file not sorted" |
| 86 | ); |
Mårten Kongstad | 4d2b4b0 | 2023-04-27 16:05:58 +0200 | [diff] [blame] | 87 | serde_json::to_writer(writer, self).map_err(|e| e.into()) |
| 88 | } |
| 89 | |
Mårten Kongstad | 4d2b4b0 | 2023-04-27 16:05:58 +0200 | [diff] [blame] | 90 | pub fn iter(&self) -> impl Iterator<Item = &Item> { |
| 91 | self.items.iter() |
| 92 | } |
Mårten Kongstad | 4d2b4b0 | 2023-04-27 16:05:58 +0200 | [diff] [blame] | 93 | |
Mårten Kongstad | a102909 | 2023-05-08 11:51:59 +0200 | [diff] [blame] | 94 | pub fn into_iter(self) -> impl Iterator<Item = Item> { |
| 95 | self.items.into_iter() |
| 96 | } |
Mårten Kongstad | 6b9e382 | 2023-05-16 11:19:58 +0200 | [diff] [blame] | 97 | |
| 98 | pub fn namespace(&self) -> &str { |
Mårten Kongstad | e17ba5f | 2023-05-16 12:52:43 +0200 | [diff] [blame] | 99 | debug_assert!(!self.namespace.is_empty()); |
Mårten Kongstad | 6b9e382 | 2023-05-16 11:19:58 +0200 | [diff] [blame] | 100 | &self.namespace |
| 101 | } |
Mårten Kongstad | a102909 | 2023-05-08 11:51:59 +0200 | [diff] [blame] | 102 | } |
Mårten Kongstad | 4d2b4b0 | 2023-04-27 16:05:58 +0200 | [diff] [blame] | 103 | |
Mårten Kongstad | 2f95444 | 2023-05-17 16:51:16 +0200 | [diff] [blame^] | 104 | #[derive(Debug)] |
| 105 | pub struct CacheBuilder { |
| 106 | cache: Cache, |
| 107 | } |
| 108 | |
| 109 | impl CacheBuilder { |
| 110 | pub fn new(namespace: String) -> Result<CacheBuilder> { |
| 111 | ensure!(!namespace.is_empty(), "empty namespace"); |
| 112 | let cache = Cache { namespace, items: vec![] }; |
| 113 | Ok(CacheBuilder { cache }) |
| 114 | } |
| 115 | |
| 116 | pub fn add_flag_declaration( |
| 117 | &mut self, |
| 118 | source: Source, |
| 119 | declaration: FlagDeclaration, |
| 120 | ) -> Result<&mut CacheBuilder> { |
| 121 | ensure!(!declaration.name.is_empty(), "empty flag name"); |
| 122 | ensure!(!declaration.description.is_empty(), "empty flag description"); |
| 123 | ensure!( |
| 124 | self.cache.items.iter().all(|item| item.name != declaration.name), |
| 125 | "failed to declare flag {} from {}: flag already declared", |
| 126 | declaration.name, |
| 127 | source |
| 128 | ); |
| 129 | self.cache.items.push(Item { |
| 130 | namespace: self.cache.namespace.clone(), |
| 131 | name: declaration.name.clone(), |
| 132 | description: declaration.description, |
| 133 | state: DEFAULT_FLAG_STATE, |
| 134 | permission: DEFAULT_FLAG_PERMISSION, |
| 135 | trace: vec![Tracepoint { |
| 136 | source, |
| 137 | state: DEFAULT_FLAG_STATE, |
| 138 | permission: DEFAULT_FLAG_PERMISSION, |
| 139 | }], |
| 140 | }); |
| 141 | Ok(self) |
| 142 | } |
| 143 | |
| 144 | pub fn add_flag_value( |
| 145 | &mut self, |
| 146 | source: Source, |
| 147 | value: FlagValue, |
| 148 | ) -> Result<&mut CacheBuilder> { |
| 149 | ensure!(!value.namespace.is_empty(), "empty flag namespace"); |
| 150 | ensure!(!value.name.is_empty(), "empty flag name"); |
| 151 | ensure!( |
| 152 | value.namespace == self.cache.namespace, |
| 153 | "failed to set values for flag {}/{} from {}: expected namespace {}", |
| 154 | value.namespace, |
| 155 | value.name, |
| 156 | source, |
| 157 | self.cache.namespace |
| 158 | ); |
| 159 | let Some(existing_item) = self.cache.items.iter_mut().find(|item| item.name == value.name) else { |
| 160 | bail!("failed to set values for flag {}/{} from {}: flag not declared", value.namespace, value.name, source); |
| 161 | }; |
| 162 | existing_item.state = value.state; |
| 163 | existing_item.permission = value.permission; |
| 164 | existing_item.trace.push(Tracepoint { |
| 165 | source, |
| 166 | state: value.state, |
| 167 | permission: value.permission, |
| 168 | }); |
| 169 | Ok(self) |
| 170 | } |
| 171 | |
| 172 | pub fn build(mut self) -> Cache { |
| 173 | self.cache.items.sort_by_cached_key(|item| item.name.clone()); |
| 174 | self.cache |
| 175 | } |
| 176 | } |
| 177 | |
Mårten Kongstad | 4d2b4b0 | 2023-04-27 16:05:58 +0200 | [diff] [blame] | 178 | #[cfg(test)] |
| 179 | mod tests { |
| 180 | use super::*; |
Mårten Kongstad | fa23d29 | 2023-05-11 14:47:02 +0200 | [diff] [blame] | 181 | use crate::aconfig::{FlagState, Permission}; |
Mårten Kongstad | 4d2b4b0 | 2023-04-27 16:05:58 +0200 | [diff] [blame] | 182 | |
| 183 | #[test] |
Mårten Kongstad | fa23d29 | 2023-05-11 14:47:02 +0200 | [diff] [blame] | 184 | fn test_add_flag_declaration() { |
Mårten Kongstad | 2f95444 | 2023-05-17 16:51:16 +0200 | [diff] [blame^] | 185 | let mut builder = CacheBuilder::new("ns".to_string()).unwrap(); |
| 186 | builder |
Mårten Kongstad | fa23d29 | 2023-05-11 14:47:02 +0200 | [diff] [blame] | 187 | .add_flag_declaration( |
Mårten Kongstad | 4d2b4b0 | 2023-04-27 16:05:58 +0200 | [diff] [blame] | 188 | Source::File("first.txt".to_string()), |
Mårten Kongstad | fa23d29 | 2023-05-11 14:47:02 +0200 | [diff] [blame] | 189 | FlagDeclaration { name: "foo".to_string(), description: "desc".to_string() }, |
Mårten Kongstad | 4d2b4b0 | 2023-04-27 16:05:58 +0200 | [diff] [blame] | 190 | ) |
| 191 | .unwrap(); |
Mårten Kongstad | 2f95444 | 2023-05-17 16:51:16 +0200 | [diff] [blame^] | 192 | let error = builder |
Mårten Kongstad | fa23d29 | 2023-05-11 14:47:02 +0200 | [diff] [blame] | 193 | .add_flag_declaration( |
Mårten Kongstad | 4d2b4b0 | 2023-04-27 16:05:58 +0200 | [diff] [blame] | 194 | Source::File("second.txt".to_string()), |
Mårten Kongstad | fa23d29 | 2023-05-11 14:47:02 +0200 | [diff] [blame] | 195 | FlagDeclaration { name: "foo".to_string(), description: "desc".to_string() }, |
Mårten Kongstad | 4d2b4b0 | 2023-04-27 16:05:58 +0200 | [diff] [blame] | 196 | ) |
| 197 | .unwrap_err(); |
| 198 | assert_eq!( |
| 199 | &format!("{:?}", error), |
Mårten Kongstad | fa23d29 | 2023-05-11 14:47:02 +0200 | [diff] [blame] | 200 | "failed to declare flag foo from second.txt: flag already declared" |
Mårten Kongstad | 4d2b4b0 | 2023-04-27 16:05:58 +0200 | [diff] [blame] | 201 | ); |
Mårten Kongstad | 2f95444 | 2023-05-17 16:51:16 +0200 | [diff] [blame^] | 202 | builder |
| 203 | .add_flag_declaration( |
| 204 | Source::File("first.txt".to_string()), |
| 205 | FlagDeclaration { name: "bar".to_string(), description: "desc".to_string() }, |
| 206 | ) |
| 207 | .unwrap(); |
| 208 | |
| 209 | let cache = builder.build(); |
| 210 | |
| 211 | // check flags are sorted by name |
| 212 | assert_eq!( |
| 213 | cache.into_iter().map(|item| item.name).collect::<Vec<_>>(), |
| 214 | vec!["bar".to_string(), "foo".to_string()] |
| 215 | ); |
Mårten Kongstad | 4d2b4b0 | 2023-04-27 16:05:58 +0200 | [diff] [blame] | 216 | } |
| 217 | |
| 218 | #[test] |
Mårten Kongstad | fa23d29 | 2023-05-11 14:47:02 +0200 | [diff] [blame] | 219 | fn test_add_flag_value() { |
Mårten Kongstad | 2f95444 | 2023-05-17 16:51:16 +0200 | [diff] [blame^] | 220 | let mut builder = CacheBuilder::new("ns".to_string()).unwrap(); |
| 221 | let error = builder |
Mårten Kongstad | fa23d29 | 2023-05-11 14:47:02 +0200 | [diff] [blame] | 222 | .add_flag_value( |
Mårten Kongstad | 416330b | 2023-05-05 11:10:01 +0200 | [diff] [blame] | 223 | Source::Memory, |
Mårten Kongstad | fa23d29 | 2023-05-11 14:47:02 +0200 | [diff] [blame] | 224 | FlagValue { |
Mårten Kongstad | 3095078 | 2023-05-09 13:31:29 +0200 | [diff] [blame] | 225 | namespace: "ns".to_string(), |
| 226 | name: "foo".to_string(), |
Mårten Kongstad | c68c4ea | 2023-05-05 16:20:09 +0200 | [diff] [blame] | 227 | state: FlagState::Enabled, |
| 228 | permission: Permission::ReadOnly, |
| 229 | }, |
Mårten Kongstad | 416330b | 2023-05-05 11:10:01 +0200 | [diff] [blame] | 230 | ) |
Mårten Kongstad | 4d2b4b0 | 2023-04-27 16:05:58 +0200 | [diff] [blame] | 231 | .unwrap_err(); |
Mårten Kongstad | fa23d29 | 2023-05-11 14:47:02 +0200 | [diff] [blame] | 232 | assert_eq!( |
| 233 | &format!("{:?}", error), |
| 234 | "failed to set values for flag ns/foo from <memory>: flag not declared" |
| 235 | ); |
Mårten Kongstad | 4d2b4b0 | 2023-04-27 16:05:58 +0200 | [diff] [blame] | 236 | |
Mårten Kongstad | 2f95444 | 2023-05-17 16:51:16 +0200 | [diff] [blame^] | 237 | builder |
Mårten Kongstad | fa23d29 | 2023-05-11 14:47:02 +0200 | [diff] [blame] | 238 | .add_flag_declaration( |
Mårten Kongstad | 4d2b4b0 | 2023-04-27 16:05:58 +0200 | [diff] [blame] | 239 | Source::File("first.txt".to_string()), |
Mårten Kongstad | fa23d29 | 2023-05-11 14:47:02 +0200 | [diff] [blame] | 240 | FlagDeclaration { name: "foo".to_string(), description: "desc".to_string() }, |
Mårten Kongstad | 4d2b4b0 | 2023-04-27 16:05:58 +0200 | [diff] [blame] | 241 | ) |
| 242 | .unwrap(); |
Mårten Kongstad | 4d2b4b0 | 2023-04-27 16:05:58 +0200 | [diff] [blame] | 243 | |
Mårten Kongstad | 2f95444 | 2023-05-17 16:51:16 +0200 | [diff] [blame^] | 244 | builder |
Mårten Kongstad | fa23d29 | 2023-05-11 14:47:02 +0200 | [diff] [blame] | 245 | .add_flag_value( |
Mårten Kongstad | 416330b | 2023-05-05 11:10:01 +0200 | [diff] [blame] | 246 | Source::Memory, |
Mårten Kongstad | fa23d29 | 2023-05-11 14:47:02 +0200 | [diff] [blame] | 247 | FlagValue { |
Mårten Kongstad | 3095078 | 2023-05-09 13:31:29 +0200 | [diff] [blame] | 248 | namespace: "ns".to_string(), |
| 249 | name: "foo".to_string(), |
Mårten Kongstad | c68c4ea | 2023-05-05 16:20:09 +0200 | [diff] [blame] | 250 | state: FlagState::Disabled, |
Mårten Kongstad | fa23d29 | 2023-05-11 14:47:02 +0200 | [diff] [blame] | 251 | permission: Permission::ReadOnly, |
Mårten Kongstad | c68c4ea | 2023-05-05 16:20:09 +0200 | [diff] [blame] | 252 | }, |
Mårten Kongstad | 416330b | 2023-05-05 11:10:01 +0200 | [diff] [blame] | 253 | ) |
Mårten Kongstad | 4d2b4b0 | 2023-04-27 16:05:58 +0200 | [diff] [blame] | 254 | .unwrap(); |
Mårten Kongstad | 4d2b4b0 | 2023-04-27 16:05:58 +0200 | [diff] [blame] | 255 | |
Mårten Kongstad | 2f95444 | 2023-05-17 16:51:16 +0200 | [diff] [blame^] | 256 | builder |
Mårten Kongstad | fa23d29 | 2023-05-11 14:47:02 +0200 | [diff] [blame] | 257 | .add_flag_value( |
Mårten Kongstad | 416330b | 2023-05-05 11:10:01 +0200 | [diff] [blame] | 258 | Source::Memory, |
Mårten Kongstad | fa23d29 | 2023-05-11 14:47:02 +0200 | [diff] [blame] | 259 | FlagValue { |
Mårten Kongstad | 3095078 | 2023-05-09 13:31:29 +0200 | [diff] [blame] | 260 | namespace: "ns".to_string(), |
| 261 | name: "foo".to_string(), |
Mårten Kongstad | c68c4ea | 2023-05-05 16:20:09 +0200 | [diff] [blame] | 262 | state: FlagState::Enabled, |
| 263 | permission: Permission::ReadWrite, |
| 264 | }, |
Mårten Kongstad | 416330b | 2023-05-05 11:10:01 +0200 | [diff] [blame] | 265 | ) |
Mårten Kongstad | 4d2b4b0 | 2023-04-27 16:05:58 +0200 | [diff] [blame] | 266 | .unwrap(); |
Mårten Kongstad | 3095078 | 2023-05-09 13:31:29 +0200 | [diff] [blame] | 267 | |
| 268 | // different namespace -> no-op |
Mårten Kongstad | 2f95444 | 2023-05-17 16:51:16 +0200 | [diff] [blame^] | 269 | let error = builder |
Mårten Kongstad | fa23d29 | 2023-05-11 14:47:02 +0200 | [diff] [blame] | 270 | .add_flag_value( |
Mårten Kongstad | 3095078 | 2023-05-09 13:31:29 +0200 | [diff] [blame] | 271 | Source::Memory, |
Mårten Kongstad | fa23d29 | 2023-05-11 14:47:02 +0200 | [diff] [blame] | 272 | FlagValue { |
Mårten Kongstad | 3095078 | 2023-05-09 13:31:29 +0200 | [diff] [blame] | 273 | namespace: "some-other-namespace".to_string(), |
| 274 | name: "foo".to_string(), |
| 275 | state: FlagState::Enabled, |
| 276 | permission: Permission::ReadOnly, |
| 277 | }, |
| 278 | ) |
Mårten Kongstad | fa23d29 | 2023-05-11 14:47:02 +0200 | [diff] [blame] | 279 | .unwrap_err(); |
| 280 | assert_eq!(&format!("{:?}", error), "failed to set values for flag some-other-namespace/foo from <memory>: expected namespace ns"); |
Mårten Kongstad | 2f95444 | 2023-05-17 16:51:16 +0200 | [diff] [blame^] | 281 | |
| 282 | let cache = builder.build(); |
| 283 | let item = cache.iter().find(|&item| item.name == "foo").unwrap(); |
| 284 | assert_eq!(FlagState::Enabled, item.state); |
| 285 | assert_eq!(Permission::ReadWrite, item.permission); |
Mårten Kongstad | 4d2b4b0 | 2023-04-27 16:05:58 +0200 | [diff] [blame] | 286 | } |
Mårten Kongstad | e17ba5f | 2023-05-16 12:52:43 +0200 | [diff] [blame] | 287 | |
| 288 | #[test] |
| 289 | fn test_reject_empty_cache_namespace() { |
Mårten Kongstad | 2f95444 | 2023-05-17 16:51:16 +0200 | [diff] [blame^] | 290 | CacheBuilder::new("".to_string()).unwrap_err(); |
Mårten Kongstad | e17ba5f | 2023-05-16 12:52:43 +0200 | [diff] [blame] | 291 | } |
| 292 | |
| 293 | #[test] |
| 294 | fn test_reject_empty_flag_declaration_fields() { |
Mårten Kongstad | 2f95444 | 2023-05-17 16:51:16 +0200 | [diff] [blame^] | 295 | let mut builder = CacheBuilder::new("ns".to_string()).unwrap(); |
Mårten Kongstad | e17ba5f | 2023-05-16 12:52:43 +0200 | [diff] [blame] | 296 | |
Mårten Kongstad | 2f95444 | 2023-05-17 16:51:16 +0200 | [diff] [blame^] | 297 | let error = builder |
Mårten Kongstad | e17ba5f | 2023-05-16 12:52:43 +0200 | [diff] [blame] | 298 | .add_flag_declaration( |
| 299 | Source::Memory, |
| 300 | FlagDeclaration { name: "".to_string(), description: "Description".to_string() }, |
| 301 | ) |
| 302 | .unwrap_err(); |
| 303 | assert_eq!(&format!("{:?}", error), "empty flag name"); |
| 304 | |
Mårten Kongstad | 2f95444 | 2023-05-17 16:51:16 +0200 | [diff] [blame^] | 305 | let error = builder |
Mårten Kongstad | e17ba5f | 2023-05-16 12:52:43 +0200 | [diff] [blame] | 306 | .add_flag_declaration( |
| 307 | Source::Memory, |
| 308 | FlagDeclaration { name: "foo".to_string(), description: "".to_string() }, |
| 309 | ) |
| 310 | .unwrap_err(); |
| 311 | assert_eq!(&format!("{:?}", error), "empty flag description"); |
| 312 | } |
| 313 | |
| 314 | #[test] |
| 315 | fn test_reject_empty_flag_value_files() { |
Mårten Kongstad | 2f95444 | 2023-05-17 16:51:16 +0200 | [diff] [blame^] | 316 | let mut builder = CacheBuilder::new("ns".to_string()).unwrap(); |
| 317 | builder |
Mårten Kongstad | e17ba5f | 2023-05-16 12:52:43 +0200 | [diff] [blame] | 318 | .add_flag_declaration( |
| 319 | Source::Memory, |
| 320 | FlagDeclaration { name: "foo".to_string(), description: "desc".to_string() }, |
| 321 | ) |
| 322 | .unwrap(); |
| 323 | |
Mårten Kongstad | 2f95444 | 2023-05-17 16:51:16 +0200 | [diff] [blame^] | 324 | let error = builder |
Mårten Kongstad | e17ba5f | 2023-05-16 12:52:43 +0200 | [diff] [blame] | 325 | .add_flag_value( |
| 326 | Source::Memory, |
| 327 | FlagValue { |
| 328 | namespace: "".to_string(), |
| 329 | name: "foo".to_string(), |
| 330 | state: FlagState::Enabled, |
| 331 | permission: Permission::ReadOnly, |
| 332 | }, |
| 333 | ) |
| 334 | .unwrap_err(); |
| 335 | assert_eq!(&format!("{:?}", error), "empty flag namespace"); |
| 336 | |
Mårten Kongstad | 2f95444 | 2023-05-17 16:51:16 +0200 | [diff] [blame^] | 337 | let error = builder |
Mårten Kongstad | e17ba5f | 2023-05-16 12:52:43 +0200 | [diff] [blame] | 338 | .add_flag_value( |
| 339 | Source::Memory, |
| 340 | FlagValue { |
| 341 | namespace: "ns".to_string(), |
| 342 | name: "".to_string(), |
| 343 | state: FlagState::Enabled, |
| 344 | permission: Permission::ReadOnly, |
| 345 | }, |
| 346 | ) |
| 347 | .unwrap_err(); |
| 348 | assert_eq!(&format!("{:?}", error), "empty flag name"); |
| 349 | } |
Mårten Kongstad | 2f95444 | 2023-05-17 16:51:16 +0200 | [diff] [blame^] | 350 | |
| 351 | #[test] |
| 352 | fn test_iter_is_sorted_by_key() { |
| 353 | assert!(iter_is_sorted_by_key(["a", "b", "c"].iter(), |s| s)); |
| 354 | assert!(iter_is_sorted_by_key(Vec::<&str>::new().iter(), |s| s)); |
| 355 | assert!(!iter_is_sorted_by_key(["a", "c", "b"].iter(), |s| s)); |
| 356 | } |
Mårten Kongstad | 4d2b4b0 | 2023-04-27 16:05:58 +0200 | [diff] [blame] | 357 | } |