Skip to content

Commit 312d33a

Browse files
committed
refactor: rename as_overrides to as_product_config_overrides
1 parent 8abea6e commit 312d33a

3 files changed

Lines changed: 5 additions & 5 deletions

File tree

crates/stackable-operator/src/config_overrides.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ impl KeyValueConfigOverrides {
5858
///
5959
/// This is useful when implementing [`KeyValueOverridesProvider`] for a
6060
/// typed override struct that contains [`KeyValueConfigOverrides`] fields.
61-
pub fn as_overrides(&self) -> BTreeMap<String, Option<String>> {
61+
pub fn as_product_config_overrides(&self) -> BTreeMap<String, Option<String>> {
6262
self.overrides
6363
.iter()
6464
.map(|(k, v)| (k.clone(), Some(v.clone())))
@@ -269,13 +269,13 @@ mod tests {
269269
}
270270

271271
#[test]
272-
fn key_value_config_overrides_as_overrides() {
272+
fn key_value_config_overrides_as_product_config_overrides() {
273273
let mut overrides = BTreeMap::new();
274274
overrides.insert("key1".to_owned(), "value1".to_owned());
275275
overrides.insert("key2".to_owned(), "value2".to_owned());
276276

277277
let kv = KeyValueConfigOverrides { overrides };
278-
let result = kv.as_overrides();
278+
let result = kv.as_product_config_overrides();
279279

280280
assert_eq!(result.len(), 2);
281281
assert_eq!(result.get("key1"), Some(&Some("value1".to_owned())));

crates/stackable-operator/src/product_config_utils.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -711,7 +711,7 @@ mod tests {
711711
fn get_key_value_overrides(&self, file: &str) -> BTreeMap<String, Option<String>> {
712712
self.files
713713
.get(file)
714-
.map(|kv| kv.as_overrides())
714+
.map(|kv| kv.as_product_config_overrides())
715715
.unwrap_or_default()
716716
}
717717
}

crates/xtask/src/crd/dummy.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ impl KeyValueOverridesProvider for DummyConfigOverrides {
5656
"dummy.properties" => self
5757
.dummy_properties
5858
.as_ref()
59-
.map(|o| o.as_overrides())
59+
.map(|o| o.as_product_config_overrides())
6060
.unwrap_or_default(),
6161
_ => BTreeMap::new(),
6262
}

0 commit comments

Comments
 (0)