Skip to content

Commit

Permalink
fix unused test functions
Browse files Browse the repository at this point in the history
  • Loading branch information
andrewjstone committed Mar 20, 2024
1 parent b7c1d67 commit c51bf3e
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion sled-agent/src/instance_manager.rs
Original file line number Diff line number Diff line change
Expand Up @@ -750,7 +750,7 @@ impl InstanceTicket {
InstanceTicket { id, terminate_tx: Some(terminate_tx) }
}

#[cfg(test)]
#[cfg(all(test, target_os = "illumos"))]
pub(crate) fn new_without_manager_for_test(id: Uuid) -> Self {
Self { id, terminate_tx: None }
}
Expand Down
2 changes: 1 addition & 1 deletion sled-agent/src/vmm_reservoir.rs
Original file line number Diff line number Diff line change
Expand Up @@ -121,7 +121,7 @@ impl VmmReservoirManagerHandle {
}

/// TODO: We should be able run to tests in VMs that can use the real VmmReservoir
#[cfg(test)]
#[cfg(all(test, target_os = "illumos"))]
pub fn stub_for_test() -> Self {
let (tx, _) = flume::bounded(1);
let (size_updated_tx, _) = broadcast::channel(1);
Expand Down

0 comments on commit c51bf3e

Please sign in to comment.