Skip to content
This repository has been archived by the owner on Jun 5, 2024. It is now read-only.

Commit

Permalink
Update check_render_snapshot to avoid using wgpu in CI
Browse files Browse the repository at this point in the history
  • Loading branch information
PoignardAzur committed Apr 19, 2024
1 parent 41e28d9 commit 9e56c30
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions src/testing/harness.rs
Original file line number Diff line number Diff line change
Expand Up @@ -501,16 +501,20 @@ impl TestHarness {
test_module_path: &str,
test_name: &str,
) {
let new_image = self.render();

if option_env!("SKIP_RENDER_SNAPSHOTS").is_some() {
// FIXME - This is a terrible, awful hack.
// We need a way to skip render snapshots on CI and locally
// until we can make sure the snapshots render the same on
// different platforms.

// We still redraw to get some coverage in the paint code.
let _ = self.render_root.redraw();

return;
}

let new_image = self.render();

let workspace_path = get_cargo_workspace(manifest_dir);
let test_file_path_abs = workspace_path.join(test_file_path);
let folder_path = test_file_path_abs.parent().unwrap();
Expand Down

0 comments on commit 9e56c30

Please sign in to comment.