fix: ObjValue is_empty should not account hidden fields

crates/jrsonnet-evaluator/src/obj.rs

@@ -669,17 +669,18 @@ impl ObjectLike for OopObject {
 	}
 
 	fn len(&self) -> usize {
+		// Maybe it will be better to not compute sort key here?
 		self.fields_visibility()
 			.into_iter()
 			.filter(|(_, (visible, _))| *visible)
 			.count()
 	}
 
+	/// Returns false only if there is any visible entry.
+	///
+	/// Note that object with hidden fields `{a:: 1}` will be reported as empty here.
 	fn is_empty(&self) -> bool {
-		if !self.this_entries.is_empty() {
-			return false;
-		}
-		self.sup.as_ref().map_or(true, ObjValue::is_empty)
+		self.len() != 0
 	}
 
 	/// Run callback for every field found in object

crates/jrsonnet-stdlib/src/strings.rs

@@ -28,9 +28,9 @@ pub fn builtin_str_replace(str: String, from: IStr, to: IStr) -> String {
 }
 
 #[builtin]
-pub fn builtin_escape_string_bash(str: String) -> String {
+pub fn builtin_escape_string_bash(str_: String) -> String {
 	const QUOTE: char = '\'';
-	let mut out = str.replace(QUOTE, "'\"'\"'");
+	let mut out = str_.replace(QUOTE, "'\"'\"'");
 	out.insert(0, QUOTE);
 	out.push(QUOTE);
 	out