Get rid of unnecessary condition in queryInto() (idaholab#32781)

Author: nmnobre

framework/include/base/TheWarehouse.h

@@ -493,10 +493,9 @@ class TheWarehouse
     {
       mooseAssert(obj, "Null object");
       auto cast_obj = dynamic_cast<T *>(obj);
-      if (obj)
-        mooseAssert(cast_obj,
-                    "Queried object " + obj->typeAndName() + " has incompatible c++ type with " +
-                        MooseUtils::prettyCppType<T>());
+      mooseAssert(cast_obj,
+                  "Queried object " + obj->typeAndName() + " has incompatible c++ type with " +
+                      MooseUtils::prettyCppType<T>());
       mooseAssert(std::find(results.begin(), results.end(), cast_obj) == results.end(),
                   "Duplicate object");
       if (show_all || obj->enabled())