Fix relayer test

modules/src/core/ics02_client/handler/update_client.rs

@@ -66,7 +66,7 @@ pub fn process(
         .ok_or_else(|| {
             Error::invalid_consensus_state_timestamp(
                 latest_consensus_state.timestamp(),
-                header.timestamp(),
+                Timestamp::from(now),
             )
         })?;
 

modules/src/mock/context.rs

@@ -227,7 +227,13 @@ impl MockContext {
     pub fn tick(&mut self) -> Time {
         let mut current_time = self.current_time.write().unwrap();
         let time = *current_time;
-        *current_time = (time + Duration::from_millis(100)).unwrap();
+
+        // The time increment has to be at least a second, or the "current" time of the
+        // mock chain will fall behind the relayer's real time clock, causing
+        // relayer tests to fail.
+        // Possible fix: Add a `current_time` method in `ChainEndpoint` so that it can
+        // be mocked
+        *current_time = (time + Duration::from_millis(1000)).unwrap();
         time
     }
 

relayer/src/foreign_client.rs

@@ -1606,8 +1606,6 @@ mod test {
 
         // Update each client
         for _i in 1..num_iterations {
-            std::thread::sleep(core::time::Duration::from_secs(1));
-
             let res = client_on_a.update();
             assert!(res.is_ok(), "Client update for chain a failed {:?}", res);