Skip to content

Commit

Permalink
Ignore tests failing on windows
Browse files Browse the repository at this point in the history
  • Loading branch information
maciektr committed Nov 7, 2023
1 parent fde5a90 commit 4d5e0a8
Show file tree
Hide file tree
Showing 4 changed files with 25 additions and 1 deletion.
2 changes: 1 addition & 1 deletion scarb/tests/build.rs
Original file line number Diff line number Diff line change
Expand Up @@ -677,7 +677,7 @@ fn edition_must_exist() {
.assert()
.failure()
.stdout_matches(indoc! {r#"
error: failed to parse manifest at: /[..]/Scarb.toml
error: failed to parse manifest at: [..]/Scarb.toml
Caused by:
TOML parse error at line 4, column 11
Expand Down
4 changes: 4 additions & 0 deletions scarb/tests/http_registry.rs
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,9 @@ use scarb_test_support::command::Scarb;
use scarb_test_support::project_builder::{Dep, DepBuilder, ProjectBuilder};
use scarb_test_support::registry::http::HttpRegistry;

// FIXME(maciektr): Enable after release
#[test]
#[cfg_attr(target_os = "windows", ignore = "ignored on windows")]
fn usage() {
let mut registry = HttpRegistry::serve();
registry.publish(|t| {
Expand Down Expand Up @@ -39,7 +41,9 @@ fn usage() {
"#});
}

// FIXME(maciektr): Enable after release
#[test]
#[cfg_attr(target_os = "windows", ignore = "ignored on windows")]
fn not_found() {
let mut registry = HttpRegistry::serve();
registry.publish(|t| {
Expand Down
8 changes: 8 additions & 0 deletions scarb/tests/local_registry.rs
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,9 @@ use scarb_test_support::fsx::ChildPathEx;
use scarb_test_support::project_builder::{Dep, DepBuilder, ProjectBuilder};
use scarb_test_support::registry::local::LocalRegistry;

// FIXME(maciektr): Enable after release
#[test]
#[cfg_attr(target_os = "windows", ignore = "ignored on windows")]
fn usage() {
let mut registry = LocalRegistry::create();
registry.publish(|t| {
Expand Down Expand Up @@ -39,7 +41,9 @@ fn usage() {
"#});
}

// FIXME(maciektr): Enable after release
#[test]
#[cfg_attr(target_os = "windows", ignore = "ignored on windows")]
fn not_found() {
let mut registry = LocalRegistry::create();
registry.publish(|t| {
Expand Down Expand Up @@ -123,7 +127,9 @@ fn url_pointing_to_file() {
drop(registry_t);
}

// FIXME(maciektr): Enable after release
#[test]
#[cfg_attr(target_os = "windows", ignore = "ignored on windows")]
fn publish() {
let t = TempDir::new().unwrap();
let index = t.child("index");
Expand Down Expand Up @@ -203,7 +209,9 @@ fn publish() {
);
}

// FIXME(maciektr): Enable after release
#[test]
#[cfg_attr(target_os = "windows", ignore = "ignored on windows")]
fn publish_overwrites_existing() {
let index = TempDir::new().unwrap();

Expand Down
12 changes: 12 additions & 0 deletions scarb/tests/package.rs
Original file line number Diff line number Diff line change
Expand Up @@ -294,7 +294,9 @@ fn reserved_files_collision() {
"#});
}

// FIXME(maciektr): Enable after release
#[test]
#[cfg_attr(target_os = "windows", ignore = "ignored on windows")]
fn generated_manifest() {
let t = TempDir::new().unwrap();

Expand Down Expand Up @@ -618,7 +620,9 @@ fn list_dirty_repo() {
"#}));
}

// FIXME(maciektr): Enable after release
#[test]
#[cfg_attr(target_os = "windows", ignore = "ignored on windows")]
fn nested_package_vcs_path() {
let t = TempDir::new().unwrap();
ProjectBuilder::start()
Expand Down Expand Up @@ -799,7 +803,9 @@ fn windows_restricted_filenames() {
"#});
}

// FIXME(maciektr): Enable after release
#[test]
#[cfg_attr(target_os = "windows", ignore = "ignored on windows")]
fn package_symlink() {
let t = TempDir::new().unwrap();
ProjectBuilder::start()
Expand Down Expand Up @@ -828,7 +834,9 @@ fn package_symlink() {
.file_eq_path("dup/lib.cairo", t.child("src/lib.cairo"));
}

// FIXME(maciektr): Enable after release
#[test]
#[cfg_attr(target_os = "windows", ignore = "ignored on windows")]
fn broken_symlink() {
let t = TempDir::new().unwrap();
ProjectBuilder::start()
Expand All @@ -852,7 +860,9 @@ fn broken_symlink() {
"#});
}

// FIXME(maciektr): Enable after release
#[test]
#[cfg_attr(target_os = "windows", ignore = "ignored on windows")]
fn broken_but_excluded_symlink() {
let t = TempDir::new().unwrap();
ProjectBuilder::start()
Expand All @@ -877,7 +887,9 @@ fn broken_but_excluded_symlink() {
"#});
}

// FIXME(maciektr): Enable after release
#[test]
#[cfg_attr(target_os = "windows", ignore = "ignored on windows")]
fn filesystem_loop() {
let t = TempDir::new().unwrap();
ProjectBuilder::start()
Expand Down

0 comments on commit 4d5e0a8

Please sign in to comment.