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