Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

This adds an assertion to a rawtest to catch an intermittent #2775

Closed
wants to merge 1 commit into from
Closed
Changes from all commits
Commits
File filter...
Filter file types
Jump to…
Jump to file
Failed to load files.

Always

Just for now

This adds an assertion to a rawtest to catch an intermittent

It seems like this test sometimes fails. This should move the crash
earlier.
  • Loading branch information
jrmuizel committed May 24, 2018
commit 485c8961b2ae3a99ab8a7d55e7a267a6cdbc1a2a
@@ -253,7 +253,8 @@ impl<'a> RawtestHarness<'a> {
let called_inner = Arc::clone(&called);

self.wrench.callbacks.lock().unwrap().request = Box::new(move |_| {
called_inner.fetch_add(1, Ordering::SeqCst);
// we want to ensure this is only ever called once
assert_eq!(called_inner.fetch_add(1, Ordering::SeqCst), 0);
});

let pixels_first = self.render_and_get_pixels(window_rect);
ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.