address comments

Author: cctdaniel

program/rust/src/tests/test_ema.rs

@@ -126,6 +126,8 @@ fn test_ema_multiple_publishers_same_slot() -> Result<(), Box<dyn std::error::Er
         let price_data = load_checked::<PriceAccount>(&price_account, PC_VERSION).unwrap();
         assert_eq!(price_data.prev_twap_.val_, 10);
         assert_eq!(price_data.prev_twac_.val_, 1);
+        // The EMA value decreases to 12 despite an increase in the aggregate price due to the higher confidence associated with the new price.
+        // The EMA calculation considers the weight of 1/confidence for the new price, leading to a lower weighted average when the confidence is high.
         assert_eq!(price_data.twap_.val_, 12);
         assert_eq!(price_data.twac_.val_, 1);
     }

program/rust/src/tests/test_upd_price.rs

@@ -348,6 +348,9 @@ fn test_upd_price() {
         assert_eq!(price_data.prev_price_, -100);
         assert_eq!(price_data.prev_conf_, 1);
         assert_eq!(price_data.prev_timestamp_, 4);
+        assert_eq!(price_data.twap_.numer_, 1677311098);
+        assert_eq!(price_data.twap_.denom_, 1279419481);
+        assert_eq!(price_data.price_cumulative.price, 86);
     }
 
     // reset twap_.denom_ to 0 to simulate program upgrade in the same slot and make sure agg_.price_ is not updated again
@@ -381,6 +384,9 @@ fn test_upd_price() {
         assert_eq!(price_data.prev_price_, -100);
         assert_eq!(price_data.prev_conf_, 1);
         assert_eq!(price_data.prev_timestamp_, 4);
+        assert_eq!(price_data.twap_.numer_, 1677311098); // twap_.numer_ should not be updated
+        assert_eq!(price_data.twap_.denom_, 1279419481); // twap_.denom_ should not be updated
+        assert_eq!(price_data.price_cumulative.price, 86); // price_cumulative should not be updated
     }
 }