Skip to content

Commit

Permalink
mobile: Fix the thread priority expectation in InternalEngineTest (#3…
Browse files Browse the repository at this point in the history
…6390)

Signed-off-by: Ali Beyad <[email protected]>
  • Loading branch information
abeyad committed Sep 30, 2024
1 parent f7bde8c commit e3528a7
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions mobile/test/common/internal_engine_test.cc
Original file line number Diff line number Diff line change
Expand Up @@ -503,8 +503,8 @@ TEST_F(ThreadPriorityInternalEngineTest, SetThreadPriority) {
}

TEST_F(ThreadPriorityInternalEngineTest, SetOutOfRangeThreadPriority) {
// 42 is outside the range of acceptable thread priorities.
const int expected_thread_priority = 42;
// 102 is outside the range of acceptable thread priorities on all platforms.
const int expected_thread_priority = 102;
const int actual_thread_priority = startEngineWithPriority(expected_thread_priority);
// The `setpriority` system call doesn't define what happens when the thread priority is out of
// range, and the behavior could be system dependent. On Linux, if the supplied priority value
Expand Down

0 comments on commit e3528a7

Please sign in to comment.