fix tests

precompiles/oracle/src/lib.rs

@@ -52,7 +52,7 @@ where
         keys: Vec<U256>,
         values: Vec<U256>,
     ) -> EvmResult {
-        handle.record_cost(RuntimeHelper::<Runtime>::db_write_gas_cost())?;
+        handle.record_cost(RuntimeHelper::<Runtime>::db_write_gas_cost() * keys.len())?;
 
         let origin = Runtime::AddressMapping::into_account_id(handle.context().caller);
 
@@ -83,9 +83,9 @@ where
 
         let key = Self::u256_to_key(key)?;
 
-        // Get the value from the oracle
+        // Get the value from the oracle, return (0, 0) if not found
         let TimestampedValue { value, timestamp } = pallet_oracle::Pallet::<Runtime>::get(key)
-            .ok_or_else(|| revert("No value found for key"))?;
+            .unwrap_or(TimestampedValue { value: 0, timestamp: 0 });
 
         Ok((value.into(), timestamp.into()))
     }

precompiles/oracle/src/tests.rs

@@ -72,14 +72,18 @@ fn get_value_works() {
 }
 
 #[test]
-fn get_value_fails_for_non_existent_key() {
+fn get_value_returns_zero_for_non_existent_key() {
     ExtBuilder::default().build().execute_with(|| {
         let mut tester = PrecompileTester::new(precompiles(), H160::repeat_byte(0x01), PRECOMPILE_ADDRESS);
 
         // Try to read a non-existent key
-        tester
+        let (value, timestamp): (U256, U256) = tester
             .call("getValue(uint256)", U256::from(999u32))
             .expect_no_logs()
-            .execute_reverts(|output| output == b"No value found for key");
+            .execute_returns();
+
+        // Should return zero values for both value and timestamp
+        assert_eq!(value, U256::zero());
+        assert_eq!(timestamp, U256::zero());
     });
 }