Merge pull request #1138 from nasa/integration-candidate

osal Integration candidate: 2021-08-31
nasa · Sep 1, 2021 · 7ebb463 · 7ebb463
2 parents 2cd118e + 8939836
commit 7ebb463
Show file tree

Hide file tree

Showing 4 changed files with 46 additions and 9 deletions.
diff --git a/README.md b/README.md
@@ -11,6 +11,11 @@ The autogenerated OSAL user's guide can be viewed at <https://github.com/nasa/cF
 
 ## Version History
 
+### Development Build: v5.1.0-rc1+dev598
+
+- Add UTAssert macros that help test bit field storage
+- See <https://github.com/nasa/osal/pull/1138> and <https://github.com/nasa/cFS/pull/348>
+
 ### Development Build: v5.1.0-rc1+dev594
 
 - Add test case types similar to NA

diff --git a/src/os/inc/osapi-version.h b/src/os/inc/osapi-version.h
@@ -36,7 +36,7 @@
 /*
  * Development Build Macro Definitions
  */
-#define OS_BUILD_NUMBER   594
+#define OS_BUILD_NUMBER   598
 #define OS_BUILD_BASELINE "v5.1.0-rc1"
 
 /*

diff --git a/ut_assert/inc/utassert.h b/ut_assert/inc/utassert.h
@@ -81,14 +81,16 @@ typedef enum
  */
 typedef enum
 {
-    UtAssert_Compare_NONE, /**< invalid/not used, always false */
-    UtAssert_Compare_EQ,   /**< actual equals reference value */
-    UtAssert_Compare_NEQ,  /**< actual does not non equal reference value */
-    UtAssert_Compare_LT,   /**< actual less than reference (exclusive) */
-    UtAssert_Compare_GT,   /**< actual greater than reference (exclusive)  */
-    UtAssert_Compare_LTEQ, /**< actual less than or equal to reference (inclusive) */
-    UtAssert_Compare_GTEQ, /**< actual greater than reference (inclusive) */
-    UtAssert_Compare_MAX   /**< placeholder, not used */
+    UtAssert_Compare_NONE,          /**< invalid/not used, always false */
+    UtAssert_Compare_EQ,            /**< actual equals reference value */
+    UtAssert_Compare_NEQ,           /**< actual does not non equal reference value */
+    UtAssert_Compare_LT,            /**< actual less than reference (exclusive) */
+    UtAssert_Compare_GT,            /**< actual greater than reference (exclusive)  */
+    UtAssert_Compare_LTEQ,          /**< actual less than or equal to reference (inclusive) */
+    UtAssert_Compare_GTEQ,          /**< actual greater than reference (inclusive) */
+    UtAssert_Compare_BITMASK_SET,   /**< actual equals reference value */
+    UtAssert_Compare_BITMASK_UNSET, /**< actual equals reference value */
+    UtAssert_Compare_MAX            /**< placeholder, not used */
 } UtAssert_Compare_t;
 
 /**
@@ -404,6 +406,24 @@ typedef struct
     UtAssert_GenericUnsignedCompare((uint32)(expr), UtAssert_Compare_GT, (uint32)(ref), UtAssert_Radix_DECIMAL, \
                                     __FILE__, __LINE__, "", #expr, #ref)
 
+/**
+ * \brief Macro for checking that bits in a bit field are set
+ *
+ * Test Passes if all the bits specified in "mask" are set in "rawval"
+ */
+#define UtAssert_BITMASK_SET(rawval, mask)                                                          \
+    UtAssert_GenericUnsignedCompare((uint32)(rawval), UtAssert_Compare_BITMASK_SET, (uint32)(mask), \
+                                    UtAssert_Radix_HEX, __FILE__, __LINE__, "", #rawval, #mask)
+
+/**
+ * \brief Macro for checking that bits in a bit field are unset
+ *
+ * Test Passes if none of the bits specified in "mask" are set in "rawval"
+ */
+#define UtAssert_BITMASK_UNSET(rawval, mask)                                                          \
+    UtAssert_GenericUnsignedCompare((uint32)(rawval), UtAssert_Compare_BITMASK_UNSET, (uint32)(mask), \
+                                    UtAssert_Radix_HEX, __FILE__, __LINE__, "", #rawval, #mask)
+
 /**
  * \brief Macro for logging calls to a "void" function
  *

diff --git a/ut_assert/src/utassert.c b/ut_assert/src/utassert.c
@@ -336,6 +336,12 @@ const char *UtAssert_GetOpText(UtAssert_Compare_t CompareType)
         case UtAssert_Compare_GTEQ: /* actual greater than reference (inclusive) */
             OpText = ">=";
             break;
+        case UtAssert_Compare_BITMASK_SET: /* bit(s) in reference are set in actual */
+            OpText = "&";
+            break;
+        case UtAssert_Compare_BITMASK_UNSET: /* bit(s) in reference are not set in actual */
+            OpText = "&~";
+            break;
         default: /* should never happen */
             OpText = "??";
             break;
@@ -371,6 +377,12 @@ bool UtAssert_GenericUnsignedCompare(unsigned long ActualValue, UtAssert_Compare
         case UtAssert_Compare_GTEQ: /* actual greater than reference (inclusive) */
             Result = (ActualValue >= ReferenceValue);
             break;
+        case UtAssert_Compare_BITMASK_SET: /* bit(s) in reference are set in actual */
+            Result = (ActualValue & ReferenceValue) == ReferenceValue;
+            break;
+        case UtAssert_Compare_BITMASK_UNSET: /* bit(s) in reference are not set in actual */
+            Result = (ActualValue & ReferenceValue) == 0;
+            break;
         default: /* should never happen */
             Result = false;
             break;