Skip to content

Commit

Permalink
tests: only run excel cellerrors test on non-macOS platforms
Browse files Browse the repository at this point in the history
- as its flaky on macOS
  • Loading branch information
jqnatividad committed Dec 1, 2024
1 parent 8c7d711 commit f098f7a
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions tests/test_excel.rs
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,8 @@ fn excel_cellerrors() {
}

#[test]
#[ignore = "skip this for now as its flaky retrieving the formula defn"]
// #[ignore = "skip this for now as its flaky retrieving the formula defn"]
#[cfg(not(target_os = "macos"))]
fn excel_cellerrors_formula() {
let wrk = Workdir::new("excel_cellerrors_formula");

Expand Down Expand Up @@ -77,7 +78,8 @@ fn excel_cellerrors_formula() {
}

#[test]
#[ignore = "skip this for now as its flaky retrieving the formula defn"]
// #[ignore = "skip this for now as its flaky retrieving the formula defn"]
#[cfg(not(target_os = "macos"))]
fn excel_cellerrors_both() {
let wrk = Workdir::new("excel_cellerrors_both");

Expand Down

0 comments on commit f098f7a

Please sign in to comment.