Mårten Kongstad | 83a8760 | 2023-06-02 11:20:15 +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 | |
| 17 | #[cfg(test)] |
| 18 | pub mod test_utils { |
Mårten Kongstad | 403658f | 2023-06-14 09:51:56 +0200 | [diff] [blame] | 19 | use crate::commands::Input; |
| 20 | use crate::protos::ProtoParsedFlags; |
Mårten Kongstad | b025507 | 2023-06-08 10:15:43 +0200 | [diff] [blame] | 21 | use itertools; |
Mårten Kongstad | 83a8760 | 2023-06-02 11:20:15 +0200 | [diff] [blame] | 22 | |
Mårten Kongstad | 403658f | 2023-06-14 09:51:56 +0200 | [diff] [blame] | 23 | pub const TEST_PACKAGE: &str = "com.android.aconfig.test"; |
| 24 | |
Mårten Kongstad | 3228b29 | 2023-06-26 10:17:42 +0200 | [diff] [blame] | 25 | pub const TEST_FLAGS_TEXTPROTO: &str = r#" |
| 26 | parsed_flag { |
| 27 | package: "com.android.aconfig.test" |
| 28 | name: "disabled_ro" |
| 29 | namespace: "aconfig_test" |
| 30 | description: "This flag is DISABLED + READ_ONLY" |
| 31 | bug: "123" |
| 32 | state: DISABLED |
| 33 | permission: READ_ONLY |
| 34 | trace { |
| 35 | source: "tests/test.aconfig" |
| 36 | state: DISABLED |
| 37 | permission: READ_WRITE |
| 38 | } |
| 39 | trace { |
| 40 | source: "tests/first.values" |
| 41 | state: DISABLED |
| 42 | permission: READ_ONLY |
| 43 | } |
Zhi Dou | 71f1b35 | 2023-08-21 22:49:46 +0000 | [diff] [blame] | 44 | is_fixed_read_only: false |
Oriol Prieto Gasco | 0b9d289 | 2023-11-20 16:23:51 +0000 | [diff] [blame] | 45 | is_exported: false |
Oriol Prieto Gasco | 7afc7e7 | 2023-11-22 13:26:02 +0000 | [diff] [blame^] | 46 | container: "system" |
Mårten Kongstad | 3228b29 | 2023-06-26 10:17:42 +0200 | [diff] [blame] | 47 | } |
| 48 | parsed_flag { |
| 49 | package: "com.android.aconfig.test" |
| 50 | name: "disabled_rw" |
| 51 | namespace: "aconfig_test" |
| 52 | description: "This flag is DISABLED + READ_WRITE" |
| 53 | bug: "456" |
| 54 | state: DISABLED |
| 55 | permission: READ_WRITE |
| 56 | trace { |
| 57 | source: "tests/test.aconfig" |
| 58 | state: DISABLED |
| 59 | permission: READ_WRITE |
| 60 | } |
Zhi Dou | 71f1b35 | 2023-08-21 22:49:46 +0000 | [diff] [blame] | 61 | is_fixed_read_only: false |
Oriol Prieto Gasco | 0b9d289 | 2023-11-20 16:23:51 +0000 | [diff] [blame] | 62 | is_exported: true |
Oriol Prieto Gasco | 7afc7e7 | 2023-11-22 13:26:02 +0000 | [diff] [blame^] | 63 | container: "system" |
Zhi Dou | 71f1b35 | 2023-08-21 22:49:46 +0000 | [diff] [blame] | 64 | } |
| 65 | parsed_flag { |
| 66 | package: "com.android.aconfig.test" |
Ted Bauer | 4a6af78 | 2023-11-29 15:44:24 +0000 | [diff] [blame] | 67 | name: "disabled_rw_exported" |
| 68 | namespace: "aconfig_test" |
| 69 | description: "This flag is exported" |
| 70 | bug: "111" |
| 71 | state: DISABLED |
| 72 | permission: READ_WRITE |
| 73 | trace { |
| 74 | source: "tests/test.aconfig" |
| 75 | state: DISABLED |
| 76 | permission: READ_WRITE |
| 77 | } |
| 78 | trace { |
| 79 | source: "tests/first.values" |
| 80 | state: DISABLED |
| 81 | permission: READ_WRITE |
| 82 | } |
| 83 | is_fixed_read_only: false |
| 84 | is_exported: true |
Oriol Prieto Gasco | 7afc7e7 | 2023-11-22 13:26:02 +0000 | [diff] [blame^] | 85 | container: "system" |
Ted Bauer | 4a6af78 | 2023-11-29 15:44:24 +0000 | [diff] [blame] | 86 | } |
| 87 | parsed_flag { |
| 88 | package: "com.android.aconfig.test" |
Mårten Kongstad | bc76a3d | 2023-11-17 09:37:06 +0100 | [diff] [blame] | 89 | name: "disabled_rw_in_other_namespace" |
Ted Bauer | c307378 | 2023-11-15 18:04:54 +0000 | [diff] [blame] | 90 | namespace: "other_namespace" |
Mårten Kongstad | bc76a3d | 2023-11-17 09:37:06 +0100 | [diff] [blame] | 91 | description: "This flag is DISABLED + READ_WRITE, and is defined in another namespace" |
Ted Bauer | c307378 | 2023-11-15 18:04:54 +0000 | [diff] [blame] | 92 | bug: "999" |
| 93 | state: DISABLED |
| 94 | permission: READ_WRITE |
| 95 | trace { |
| 96 | source: "tests/test.aconfig" |
| 97 | state: DISABLED |
| 98 | permission: READ_WRITE |
| 99 | } |
| 100 | trace { |
| 101 | source: "tests/first.values" |
| 102 | state: DISABLED |
| 103 | permission: READ_WRITE |
| 104 | } |
| 105 | is_fixed_read_only: false |
Oriol Prieto Gasco | 0b9d289 | 2023-11-20 16:23:51 +0000 | [diff] [blame] | 106 | is_exported: false |
Oriol Prieto Gasco | 7afc7e7 | 2023-11-22 13:26:02 +0000 | [diff] [blame^] | 107 | container: "system" |
Ted Bauer | c307378 | 2023-11-15 18:04:54 +0000 | [diff] [blame] | 108 | } |
| 109 | parsed_flag { |
| 110 | package: "com.android.aconfig.test" |
Zhi Dou | 71f1b35 | 2023-08-21 22:49:46 +0000 | [diff] [blame] | 111 | name: "enabled_fixed_ro" |
| 112 | namespace: "aconfig_test" |
| 113 | description: "This flag is fixed READ_ONLY + ENABLED" |
| 114 | bug: "" |
| 115 | state: ENABLED |
| 116 | permission: READ_ONLY |
| 117 | trace { |
| 118 | source: "tests/test.aconfig" |
| 119 | state: DISABLED |
| 120 | permission: READ_ONLY |
| 121 | } |
| 122 | trace { |
| 123 | source: "tests/first.values" |
| 124 | state: ENABLED |
| 125 | permission: READ_ONLY |
| 126 | } |
| 127 | is_fixed_read_only: true |
Oriol Prieto Gasco | 0b9d289 | 2023-11-20 16:23:51 +0000 | [diff] [blame] | 128 | is_exported: false |
Oriol Prieto Gasco | 7afc7e7 | 2023-11-22 13:26:02 +0000 | [diff] [blame^] | 129 | container: "system" |
Mårten Kongstad | 3228b29 | 2023-06-26 10:17:42 +0200 | [diff] [blame] | 130 | } |
| 131 | parsed_flag { |
| 132 | package: "com.android.aconfig.test" |
| 133 | name: "enabled_ro" |
| 134 | namespace: "aconfig_test" |
| 135 | description: "This flag is ENABLED + READ_ONLY" |
Mårten Kongstad | 3228b29 | 2023-06-26 10:17:42 +0200 | [diff] [blame] | 136 | bug: "abc" |
| 137 | state: ENABLED |
| 138 | permission: READ_ONLY |
| 139 | trace { |
| 140 | source: "tests/test.aconfig" |
| 141 | state: DISABLED |
| 142 | permission: READ_WRITE |
| 143 | } |
| 144 | trace { |
| 145 | source: "tests/first.values" |
| 146 | state: DISABLED |
| 147 | permission: READ_WRITE |
| 148 | } |
| 149 | trace { |
| 150 | source: "tests/second.values" |
| 151 | state: ENABLED |
| 152 | permission: READ_ONLY |
| 153 | } |
Zhi Dou | 71f1b35 | 2023-08-21 22:49:46 +0000 | [diff] [blame] | 154 | is_fixed_read_only: false |
Oriol Prieto Gasco | 0b9d289 | 2023-11-20 16:23:51 +0000 | [diff] [blame] | 155 | is_exported: false |
Oriol Prieto Gasco | 7afc7e7 | 2023-11-22 13:26:02 +0000 | [diff] [blame^] | 156 | container: "system" |
Mårten Kongstad | 3228b29 | 2023-06-26 10:17:42 +0200 | [diff] [blame] | 157 | } |
| 158 | parsed_flag { |
| 159 | package: "com.android.aconfig.test" |
| 160 | name: "enabled_rw" |
| 161 | namespace: "aconfig_test" |
| 162 | description: "This flag is ENABLED + READ_WRITE" |
Mårten Kongstad | 6353c6c | 2023-07-26 13:18:50 +0200 | [diff] [blame] | 163 | bug: "" |
Mårten Kongstad | 3228b29 | 2023-06-26 10:17:42 +0200 | [diff] [blame] | 164 | state: ENABLED |
| 165 | permission: READ_WRITE |
| 166 | trace { |
| 167 | source: "tests/test.aconfig" |
| 168 | state: DISABLED |
| 169 | permission: READ_WRITE |
| 170 | } |
| 171 | trace { |
| 172 | source: "tests/first.values" |
| 173 | state: ENABLED |
| 174 | permission: READ_WRITE |
| 175 | } |
Zhi Dou | 71f1b35 | 2023-08-21 22:49:46 +0000 | [diff] [blame] | 176 | is_fixed_read_only: false |
Oriol Prieto Gasco | 0b9d289 | 2023-11-20 16:23:51 +0000 | [diff] [blame] | 177 | is_exported: false |
Oriol Prieto Gasco | 7afc7e7 | 2023-11-22 13:26:02 +0000 | [diff] [blame^] | 178 | container: "system" |
Mårten Kongstad | 3228b29 | 2023-06-26 10:17:42 +0200 | [diff] [blame] | 179 | } |
| 180 | "#; |
| 181 | |
Mårten Kongstad | 403658f | 2023-06-14 09:51:56 +0200 | [diff] [blame] | 182 | pub fn parse_test_flags() -> ProtoParsedFlags { |
| 183 | let bytes = crate::commands::parse_flags( |
Mårten Kongstad | fbd71e2 | 2023-05-31 13:29:35 +0200 | [diff] [blame] | 184 | "com.android.aconfig.test", |
Oriol Prieto Gasco | 7afc7e7 | 2023-11-22 13:26:02 +0000 | [diff] [blame^] | 185 | Some("system"), |
Mårten Kongstad | 83a8760 | 2023-06-02 11:20:15 +0200 | [diff] [blame] | 186 | vec![Input { |
Mårten Kongstad | 403658f | 2023-06-14 09:51:56 +0200 | [diff] [blame] | 187 | source: "tests/test.aconfig".to_string(), |
Mårten Kongstad | 9c59c31 | 2023-05-30 11:15:02 +0200 | [diff] [blame] | 188 | reader: Box::new(include_bytes!("../tests/test.aconfig").as_slice()), |
Mårten Kongstad | 83a8760 | 2023-06-02 11:20:15 +0200 | [diff] [blame] | 189 | }], |
| 190 | vec![ |
| 191 | Input { |
Mårten Kongstad | 403658f | 2023-06-14 09:51:56 +0200 | [diff] [blame] | 192 | source: "tests/first.values".to_string(), |
Mårten Kongstad | 9c59c31 | 2023-05-30 11:15:02 +0200 | [diff] [blame] | 193 | reader: Box::new(include_bytes!("../tests/first.values").as_slice()), |
Mårten Kongstad | 83a8760 | 2023-06-02 11:20:15 +0200 | [diff] [blame] | 194 | }, |
| 195 | Input { |
Mårten Kongstad | 403658f | 2023-06-14 09:51:56 +0200 | [diff] [blame] | 196 | source: "tests/second.values".to_string(), |
Mårten Kongstad | 9c59c31 | 2023-05-30 11:15:02 +0200 | [diff] [blame] | 197 | reader: Box::new(include_bytes!("../tests/second.values").as_slice()), |
Mårten Kongstad | 83a8760 | 2023-06-02 11:20:15 +0200 | [diff] [blame] | 198 | }, |
| 199 | ], |
Zhi Dou | 24a0b6a | 2023-08-10 21:39:59 +0000 | [diff] [blame] | 200 | crate::commands::DEFAULT_FLAG_PERMISSION, |
Mårten Kongstad | 83a8760 | 2023-06-02 11:20:15 +0200 | [diff] [blame] | 201 | ) |
Mårten Kongstad | 403658f | 2023-06-14 09:51:56 +0200 | [diff] [blame] | 202 | .unwrap(); |
| 203 | crate::protos::parsed_flags::try_from_binary_proto(&bytes).unwrap() |
Mårten Kongstad | 83a8760 | 2023-06-02 11:20:15 +0200 | [diff] [blame] | 204 | } |
Mårten Kongstad | b025507 | 2023-06-08 10:15:43 +0200 | [diff] [blame] | 205 | |
| 206 | pub fn first_significant_code_diff(a: &str, b: &str) -> Option<String> { |
| 207 | let a = a.lines().map(|line| line.trim_start()).filter(|line| !line.is_empty()); |
| 208 | let b = b.lines().map(|line| line.trim_start()).filter(|line| !line.is_empty()); |
| 209 | match itertools::diff_with(a, b, |left, right| left == right) { |
| 210 | Some(itertools::Diff::FirstMismatch(_, mut left, mut right)) => { |
| 211 | Some(format!("'{}' vs '{}'", left.next().unwrap(), right.next().unwrap())) |
| 212 | } |
| 213 | Some(itertools::Diff::Shorter(_, mut left)) => { |
| 214 | Some(format!("LHS trailing data: '{}'", left.next().unwrap())) |
| 215 | } |
| 216 | Some(itertools::Diff::Longer(_, mut right)) => { |
| 217 | Some(format!("RHS trailing data: '{}'", right.next().unwrap())) |
| 218 | } |
| 219 | None => None, |
| 220 | } |
| 221 | } |
| 222 | |
| 223 | #[test] |
| 224 | fn test_first_significant_code_diff() { |
| 225 | assert!(first_significant_code_diff("", "").is_none()); |
| 226 | assert!(first_significant_code_diff(" a", "\n\na\n").is_none()); |
| 227 | let a = r#" |
| 228 | public class A { |
| 229 | private static final String FOO = "FOO"; |
| 230 | public static void main(String[] args) { |
| 231 | System.out.println("FOO=" + FOO); |
| 232 | } |
| 233 | } |
| 234 | "#; |
| 235 | let b = r#" |
| 236 | public class A { |
| 237 | private static final String FOO = "BAR"; |
| 238 | public static void main(String[] args) { |
| 239 | System.out.println("foo=" + FOO); |
| 240 | } |
| 241 | } |
| 242 | "#; |
| 243 | assert_eq!(Some(r#"'private static final String FOO = "FOO";' vs 'private static final String FOO = "BAR";'"#.to_string()), first_significant_code_diff(a, b)); |
| 244 | assert_eq!( |
| 245 | Some("LHS trailing data: 'b'".to_string()), |
| 246 | first_significant_code_diff("a\nb", "a") |
| 247 | ); |
| 248 | assert_eq!( |
| 249 | Some("RHS trailing data: 'b'".to_string()), |
| 250 | first_significant_code_diff("a", "a\nb") |
| 251 | ); |
| 252 | } |
Mårten Kongstad | 83a8760 | 2023-06-02 11:20:15 +0200 | [diff] [blame] | 253 | } |
| 254 | |
| 255 | #[cfg(test)] |
| 256 | pub use test_utils::*; |