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

Commit

Permalink
Mark a bunch of threading tests that are relient on timing as flaky
Browse files Browse the repository at this point in the history
  • Loading branch information
miscco committed Mar 6, 2023
1 parent e80886c commit 5ae45e2
Show file tree
Hide file tree
Showing 24 changed files with 47 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@
//
// UNSUPPORTED: libcpp-has-no-threads

// FLAKY_TEST.

// This test uses the POSIX header <sys/time.h> which Windows doesn't provide
// UNSUPPORTED: windows

Expand Down
2 changes: 2 additions & 0 deletions libcxx/test/std/thread/futures/futures.async/async.pass.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,8 @@
// UNSUPPORTED: libcpp-has-no-threads
// UNSUPPORTED: c++98, c++03

// FLAKY_TEST.

// <future>

// template <class F, class... Args>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,8 @@
// UNSUPPORTED: libcpp-has-no-threads
// UNSUPPORTED: c++98, c++03

// FLAKY_TEST.

// <future>

// class shared_future<R>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,8 @@
// UNSUPPORTED: libcpp-has-no-threads
// UNSUPPORTED: c++98, c++03

// FLAKY_TEST.

// <future>

// class future<R>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@
//
// UNSUPPORTED: libcpp-has-no-threads

// FLAKY_TEST.

// <condition_variable>

// class condition_variable;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@
//
// UNSUPPORTED: libcpp-has-no-threads

// FLAKY_TEST.

// <condition_variable>

// class condition_variable;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@
//
// UNSUPPORTED: libcpp-has-no-threads

// FLAKY_TEST.

// <condition_variable>

// class condition_variable;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@
//
// UNSUPPORTED: libcpp-has-no-threads

// FLAKY_TEST.

// <condition_variable>

// class condition_variable;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@
//
// UNSUPPORTED: libcpp-has-no-threads

// FLAKY_TEST.

// <condition_variable>

// class condition_variable_any;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@
//
// UNSUPPORTED: libcpp-has-no-threads

// FLAKY_TEST.

// <condition_variable>

// class condition_variable_any;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@
//
// UNSUPPORTED: libcpp-has-no-threads

// FLAKY_TEST.

// <condition_variable>

// class condition_variable_any;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@
//
// UNSUPPORTED: libcpp-has-no-threads

// FLAKY_TEST.

// <mutex>

// template <class Mutex> class unique_lock;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@
//
// UNSUPPORTED: libcpp-has-no-threads

// FLAKY_TEST.

// <mutex>

// class mutex;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@
//
// UNSUPPORTED: libcpp-has-no-threads

// FLAKY_TEST.

// <mutex>

// class recursive_mutex;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@
//
// UNSUPPORTED: libcpp-has-no-threads

// FLAKY_TEST.

// <mutex>

// class recursive_mutex;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,8 @@
// UNSUPPORTED: libcpp-has-no-threads
// UNSUPPORTED: c++98, c++03, c++11

// FLAKY_TEST.

// <shared_mutex>

// class shared_timed_mutex;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@
//
// UNSUPPORTED: libcpp-has-no-threads

// FLAKY_TEST.

// <mutex>

// class timed_mutex;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@
//
// UNSUPPORTED: libcpp-has-no-threads

// FLAKY_TEST.

// <mutex>

// class timed_mutex;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@
//
// UNSUPPORTED: libcpp-has-no-threads

// FLAKY_TEST.

// <mutex>

// class timed_mutex;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@
//
// UNSUPPORTED: libcpp-has-no-threads

// FLAKY_TEST.

// <mutex>

// class timed_mutex;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@
//
// UNSUPPORTED: libcpp-has-no-threads

// FLAKY_TEST.

// <mutex>

// class recursive_timed_mutex;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@
//
// UNSUPPORTED: libcpp-has-no-threads

// FLAKY_TEST.

// <mutex>

// class recursive_timed_mutex;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@
//
// UNSUPPORTED: libcpp-has-no-threads

// FLAKY_TEST.

// <mutex>

// class recursive_timed_mutex;
Expand Down
2 changes: 1 addition & 1 deletion libcxx/utils/libcxx/test/format.py
Original file line number Diff line number Diff line change
Expand Up @@ -200,7 +200,7 @@ def _evaluate_pass_test(self, test, tmpBase, lit_config,
data_files = [os.path.join(local_cwd, f)
for f in os.listdir(local_cwd) if f.endswith('.dat')]
is_flaky = self._get_parser('FLAKY_TEST.', parsers).getValue()
max_retry = 3 if is_flaky else 1
max_retry = 5 if is_flaky else 1
for retry_count in range(max_retry):
cmd, out, err, rc = self.executor.run(exec_path, [exec_path],
local_cwd, data_files,
Expand Down

0 comments on commit 5ae45e2

Please sign in to comment.