diff mbox series

rust: re-enable the unit tests that are previously ignored

Message ID 20250513093208.2326563-1-Deepesh.Varatharajan@windriver.com
State New
Headers show
Series rust: re-enable the unit tests that are previously ignored | expand

Commit Message

Deepesh Varatharajan May 13, 2025, 9:32 a.m. UTC
From: Deepesh Varatharajan <Deepesh.Varatharajan@windriver.com>

Some unit tests were previously marked as ignored due to failures in earlier
versions of Rust. With the upgrade to Rust 1.85.1, these tests are now passing
consistently. They've been re-enabled and verified to run successfully on the
latest version.

Signed-off-by: Deepesh Varatharajan <Deepesh.Varatharajan@windriver.com>
---
 .../rust/files/rust-oe-selftest.patch         | 44 -------------------
 1 file changed, 44 deletions(-)

Comments

Deepesh Varatharajan May 13, 2025, 9:37 a.m. UTC | #1
Hi All,

Rust oe-selftest ran with these unit tests were re-enabled and executed 
successfully on x86_64 , x86 , arm, arm64 and ppc architectures.

Regards,
Deepesh

On 13-05-2025 15:02, Deepesh.Varatharajan@windriver.com wrote:
> From: Deepesh Varatharajan <Deepesh.Varatharajan@windriver.com>
>
> Some unit tests were previously marked as ignored due to failures in earlier
> versions of Rust. With the upgrade to Rust 1.85.1, these tests are now passing
> consistently. They've been re-enabled and verified to run successfully on the
> latest version.
>
> Signed-off-by: Deepesh Varatharajan <Deepesh.Varatharajan@windriver.com>
> ---
>   .../rust/files/rust-oe-selftest.patch         | 44 -------------------
>   1 file changed, 44 deletions(-)
>
> diff --git a/meta/recipes-devtools/rust/files/rust-oe-selftest.patch b/meta/recipes-devtools/rust/files/rust-oe-selftest.patch
> index 909bc971db..048fc3f10b 100644
> --- a/meta/recipes-devtools/rust/files/rust-oe-selftest.patch
> +++ b/meta/recipes-devtools/rust/files/rust-oe-selftest.patch
> @@ -103,50 +103,6 @@ index 1a9d3d3f12f..0a9cfc48806 100644
>    fn test_get_mut_poison() {
>        let mut m = new_poisoned_rwlock(NonCopy(10));
>   
> -diff --git a/library/std/src/thread/tests.rs b/library/std/src/thread/tests.rs
> -index 5d6b9e94ee9..a5aacb2eb87 100644
> ---- a/library/std/src/thread/tests.rs
> -+++ b/library/std/src/thread/tests.rs
> -@@ -116,6 +116,7 @@ fn test_is_finished() {
> - }
> -
> - #[test]
> -+#[ignore]
> - fn test_join_panic() {
> -     match thread::spawn(move || panic!()).join() {
> -         result::Result::Err(_) => (),
> -@@ -218,6 +219,7 @@ fn test_simple_newsched_spawn() {
> - }
> -
> - #[test]
> -+#[ignore]
> - fn test_try_panic_message_string_literal() {
> -     match thread::spawn(move || {
> -         panic!("static string");
> -@@ -234,6 +236,7 @@ fn test_try_panic_message_string_literal() {
> - }
> -
> - #[test]
> -+#[ignore]
> - fn test_try_panic_any_message_owned_str() {
> -     match thread::spawn(move || {
> -         panic_any("owned string".to_string());
> -@@ -250,6 +253,7 @@ fn test_try_panic_any_message_owned_str() {
> - }
> -
> - #[test]
> -+#[ignore]
> - fn test_try_panic_any_message_any() {
> -     match thread::spawn(move || {
> -         panic_any(Box::new(413u16) as Box<dyn Any + Send>);
> -@@ -268,6 +272,7 @@ fn test_try_panic_any_message_any() {
> - }
> -
> - #[test]
> -+#[ignore]
> - fn test_try_panic_any_message_unit_struct() {
> -     struct Juju;
> -
>   diff --git a/library/test/src/tests.rs b/library/test/src/tests.rs
>   --- a/library/test/src/tests.rs
>   +++ b/library/test/src/tests.rs
diff mbox series

Patch

diff --git a/meta/recipes-devtools/rust/files/rust-oe-selftest.patch b/meta/recipes-devtools/rust/files/rust-oe-selftest.patch
index 909bc971db..048fc3f10b 100644
--- a/meta/recipes-devtools/rust/files/rust-oe-selftest.patch
+++ b/meta/recipes-devtools/rust/files/rust-oe-selftest.patch
@@ -103,50 +103,6 @@  index 1a9d3d3f12f..0a9cfc48806 100644
  fn test_get_mut_poison() {
      let mut m = new_poisoned_rwlock(NonCopy(10));
 
-diff --git a/library/std/src/thread/tests.rs b/library/std/src/thread/tests.rs
-index 5d6b9e94ee9..a5aacb2eb87 100644
---- a/library/std/src/thread/tests.rs
-+++ b/library/std/src/thread/tests.rs
-@@ -116,6 +116,7 @@ fn test_is_finished() {
- }
-
- #[test]
-+#[ignore]
- fn test_join_panic() {
-     match thread::spawn(move || panic!()).join() {
-         result::Result::Err(_) => (),
-@@ -218,6 +219,7 @@ fn test_simple_newsched_spawn() {
- }
-
- #[test]
-+#[ignore]
- fn test_try_panic_message_string_literal() {
-     match thread::spawn(move || {
-         panic!("static string");
-@@ -234,6 +236,7 @@ fn test_try_panic_message_string_literal() {
- }
-
- #[test]
-+#[ignore]
- fn test_try_panic_any_message_owned_str() {
-     match thread::spawn(move || {
-         panic_any("owned string".to_string());
-@@ -250,6 +253,7 @@ fn test_try_panic_any_message_owned_str() {
- }
-
- #[test]
-+#[ignore]
- fn test_try_panic_any_message_any() {
-     match thread::spawn(move || {
-         panic_any(Box::new(413u16) as Box<dyn Any + Send>);
-@@ -268,6 +272,7 @@ fn test_try_panic_any_message_any() {
- }
-
- #[test]
-+#[ignore]
- fn test_try_panic_any_message_unit_struct() {
-     struct Juju;
-
 diff --git a/library/test/src/tests.rs b/library/test/src/tests.rs
 --- a/library/test/src/tests.rs
 +++ b/library/test/src/tests.rs