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