Skip to content

Commit

Permalink
Merge pull request #228 from Screenly/disable-flaky-tests
Browse files Browse the repository at this point in the history
Temporarily disable tests overriding env. They are very unstable on macos.
  • Loading branch information
sergey-borovkov authored Nov 1, 2024
2 parents 901f57f + 818ca13 commit b9feb6e
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 3 deletions.
2 changes: 1 addition & 1 deletion src/cli.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1216,7 +1216,7 @@ mod tests {
}

#[test]
#[ignore]
#[cfg_attr(target_os = "macos", ignore)]
fn test_transform_edge_app_path_to_manifest_without_path_should_return_correct_path() {
let dir = tempdir().unwrap();
let dir_path = dir.path();
Expand Down
7 changes: 5 additions & 2 deletions src/commands/edge_app/utils.rs
Original file line number Diff line number Diff line change
Expand Up @@ -979,6 +979,7 @@ mod tests {
}

#[test]
#[cfg_attr(target_os = "macos", ignore)]
fn test_transform_edge_app_instance_path_to_instance_manifest_should_return_current_dir_with_()
{
let dir = tempdir().unwrap();
Expand All @@ -991,7 +992,7 @@ mod tests {
}

#[test]
#[ignore]
#[cfg_attr(target_os = "macos", ignore)]
fn test_transform_edge_app_instance_path_to_instance_manifest_when_path_provided_should_return_path_with_instance_manifest(
) {
let dir = tempdir().unwrap();
Expand Down Expand Up @@ -1025,6 +1026,7 @@ mod tests {
}

#[test]
#[cfg_attr(target_os = "macos", ignore)]
fn test_transform_edge_app_instance_path_to_instance_manifest_with_env_instance_override_should_return_overrided_manifest_path(
) {
let dir = tempdir().unwrap();
Expand Down Expand Up @@ -1052,6 +1054,7 @@ mod tests {
}

#[test]
#[cfg_attr(target_os = "macos", ignore)]
fn test_transform_edge_app_path_to_manifest_should_return_current_dir_with_() {
let dir = tempdir().unwrap();
let dir_path = dir.path();
Expand Down Expand Up @@ -1093,7 +1096,7 @@ mod tests {
}

#[test]
#[ignore]
#[cfg_attr(target_os = "macos", ignore)]
fn test_transform_edge_app_path_to_manifest_with_env_instance_override_should_return_overrided_manifest_path(
) {
let dir = tempdir().unwrap();
Expand Down

0 comments on commit b9feb6e

Please sign in to comment.