Skip to content

Commit

Permalink
fix flakiness in test_artifact_access_time
Browse files Browse the repository at this point in the history
Summary: Test is reliable on CI but it's flaky on devservers, fix it.

Reviewed By: stepancheg

Differential Revision: D50852868

fbshipit-source-id: 47da11b73212bcf28705dd10f978888ad9686d51
  • Loading branch information
christolliday authored and facebook-github-bot committed Oct 31, 2023
1 parent e6a582b commit 26a94f2
Showing 1 changed file with 3 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -116,8 +116,10 @@ struct Iterate {
sender: UnboundedSender<(ProjectRelativePathBuf, Box<dyn DeferredMaterializerEntry>)>,
}

impl<T> ExtensionCommand<T> for Iterate {
impl<T: IoHandler> ExtensionCommand<T> for Iterate {
fn execute(self: Box<Self>, processor: &mut DeferredMaterializerCommandProcessor<T>) {
// Ensure up to date access times
processor.flush_access_times(0);
for (path, data) in processor.tree.iter_with_paths() {
let stage = match &data.stage {
ArtifactMaterializationStage::Declared { method, .. } => {
Expand Down

0 comments on commit 26a94f2

Please sign in to comment.