treat platform-switching similarly to benchmarks

This commit is contained in:
Bryce Miller 2022-09-02 10:44:29 -04:00
parent 780945061f
commit a1543b5cc4
No known key found for this signature in database
GPG key ID: 46B3D9909ED7FB90

View file

@ -822,26 +822,35 @@ mod cli_run {
let example_dir_name = entry.file_name().into_string().unwrap();
// TODO: Improve this with a more-dynamic approach. (Read all subdirectories?)
// Some platform-switching examples live in nested directories
// Some platform-switching examples live in nested directories, and
// the rocLovesFoo.roc examples live directly under the platform-switching directory
if example_dir_name == "platform-switching" {
for sub_dir in [
// We exclude the C platforming switching example
// because the main platform switching example runs the c platform.
// If we don't a race condition leads to test flakiness.
// "c-platform",
"rust-platform",
"swift-platform",
"web-assembly-platform",
"zig-platform",
] {
all_examples.remove(format!("{}/{}", example_dir_name, sub_dir).as_str()).unwrap_or_else(|| {
panic!("The example directory {}/{}/{} does not have any corresponding tests in cli_run. Please add one, so if it ever stops working, we'll know about it right away!", examples_dir, example_dir_name, sub_dir);
});
}
// The subdirectory checks broke when we moved the rodLovesFoo.roc examples
// Treating the platform-switching directory similarly to the benchmarks directory for now
all_examples.remove(example_dir_name.as_str()).unwrap_or_else(|| {
panic!("The example directory {}/{} does not have any corresponding tests in cli_run. Please add one, so if it ever stops working, we'll know about it right away!", examples_dir, example_dir_name);
});
// for sub_dir in [
// // We exclude the C platforming switching example
// // because the main platform switching example runs the c platform.
// // If we don't a race condition leads to test flakiness.
// // "c-platform",
// "rust-platform",
// "swift-platform",
// "web-assembly-platform",
// "zig-platform",
// ] {
// all_examples.remove(format!("{}/{}", example_dir_name, sub_dir).as_str()).unwrap_or_else(|| {
// panic!("The example directory {}/{}/{} does not have any corresponding tests in cli_run. Please add one, so if it ever stops working, we'll know about it right away!", examples_dir, example_dir_name, sub_dir);
// });
// }
}
// We test benchmarks separately
if example_dir_name != "benchmarks" {
// platform-switching examples are handled above
if example_dir_name != "benchmarks" && example_dir_name != "platform-switching" {
all_examples.remove(example_dir_name.as_str()).unwrap_or_else(|| {
panic!("The example directory {}/{} does not have any corresponding tests in cli_run. Please add one, so if it ever stops working, we'll know about it right away!", examples_dir, example_dir_name);
});