Skip to content

Commit

Permalink
rust: kunit: add KUnit case and suite macros
Browse files Browse the repository at this point in the history
Add a couple of Rust macros to allow to develop KUnit tests without
relying on generated C code:

 - The `kunit_test_suite!` Rust macro is similar to the
   `kunit_test_suite` C macro.
 - The `kunit_case!` Rust macro is similar to the `KUNIT_CASE` C macro.
   It can be used with parameters to generate a test case or without
   parameters to be used as delimiter in `kunit_test_suite!`.

While these macros can be used on their own, a future patch will
introduce another macro to create KUnit tests using a user-space like
syntax.

Co-developed-by: David Gow <davidgow@google.com>
Signed-off-by: José Expósito <jose.exposito89@gmail.com>
  • Loading branch information
JoseExposito committed Jan 14, 2023
1 parent 84b0fc2 commit 2dc7ae0
Showing 1 changed file with 86 additions and 0 deletions.
86 changes: 86 additions & 0 deletions rust/kernel/kunit.rs
Original file line number Diff line number Diff line change
Expand Up @@ -88,3 +88,89 @@ macro_rules! kunit_assert_eq {
$crate::kunit_assert!($test, $left == $right);
}};
}

/// Represents an individual test case.
///
/// The test case should have the signature
/// `unsafe extern "C" fn test_case(test: *mut crate::bindings::kunit)`.
///
/// The `kunit_test_suite!` macro expects a NULL-terminated list of test cases. This macro can be
/// invoked without parameters to generate the delimiter.
#[macro_export]
macro_rules! kunit_case {
() => {
$crate::bindings::kunit_case {
run_case: None,
name: core::ptr::null_mut(),
generate_params: None,
status: $crate::bindings::kunit_status_KUNIT_SUCCESS,
log: core::ptr::null_mut(),
}
};
($name:ident, $run_case:ident) => {
$crate::bindings::kunit_case {
run_case: Some($run_case),
name: $crate::c_str!(core::stringify!($name)).as_char_ptr(),
generate_params: None,
status: $crate::bindings::kunit_status_KUNIT_SUCCESS,
log: core::ptr::null_mut(),
}
};
}

/// Registers a KUnit test suite.
///
/// # Examples
///
/// ```ignore
/// unsafe extern "C" fn test_fn(_test: *mut crate::bindings::kunit) {
/// let actual = 1 + 1;
/// let expected = 2;
/// assert_eq!(actual, expected);
/// }
///
/// static mut KUNIT_TEST_CASE: crate::bindings::kunit_case = crate::kunit_case!(test_name, test_fn);
/// static mut KUNIT_NULL_CASE: crate::bindings::kunit_case = crate::kunit_case!();
/// static mut KUNIT_TEST_CASES: &mut[crate::bindings::kunit_case] = unsafe {
/// &mut[KUNIT_TEST_CASE, KUNIT_NULL_CASE]
/// };
/// crate::kunit_test_suite!(suite_name, KUNIT_TEST_CASES);
/// ```
#[macro_export]
macro_rules! kunit_test_suite {
($name:ident, $test_cases:ident) => {
static KUNIT_TEST_SUITE_NAME: [i8; 256] = {
let name_u8 = core::stringify!($name).as_bytes();
let mut ret = [0; 256];

let mut i = 0;
while i < name_u8.len() {
ret[i] = name_u8[i] as i8;
i += 1;
}

ret
};

// SAFETY: `test_cases` is valid as it should be static.
static mut KUNIT_TEST_SUITE: $crate::bindings::kunit_suite =
$crate::bindings::kunit_suite {
name: KUNIT_TEST_SUITE_NAME,
test_cases: unsafe { $test_cases.as_mut_ptr() },
suite_init: None,
suite_exit: None,
init: None,
exit: None,
status_comment: [0; 256usize],
debugfs: core::ptr::null_mut(),
log: core::ptr::null_mut(),
suite_init_err: 0,
};

// SAFETY: `KUNIT_TEST_SUITE` is static.
#[used]
#[link_section = ".kunit_test_suites"]
static mut KUNIT_TEST_SUITE_ENTRY: *const $crate::bindings::kunit_suite =
unsafe { &KUNIT_TEST_SUITE };
};
}

0 comments on commit 2dc7ae0

Please sign in to comment.