- Reverted the incorrectly replaced resize and fixed some flawed initializations.

- Add Shape unittests.

Author: ragmani

runtime/compute/cker/include/cker/Shape.h

@@ -60,7 +60,7 @@ class Shape
   }
 
   // Constructor that creates a shape from an array of dimension data.
-  Shape(int dimensions_count, const int32_t *dims_data) : _size(0)
+  Shape(int dimensions_count, const int32_t *dims_data) : _size(dimensions_count)
   {
     initStorage(dimensions_count);
     ReplaceWith(dimensions_count, dims_data);
@@ -205,6 +205,7 @@ class Shape
   // Replaces the current shape with a new one defined by dimensions_count and dims_data.
   inline void ReplaceWith(int dimensions_count, const int32_t *dims_data)
   {
+    assert(dims_data != nullptr);
     Resize(dimensions_count);
     std::memcpy(DimsData(), dims_data, dimensions_count * sizeof(int32_t));
   }
@@ -266,6 +267,7 @@ class Shape
   // Helper function: initialize dims_ storage based on the number of dimensions.
   inline void initStorage(int dimensions_count)
   {
+    assert(dimensions_count > 0);
     if (dimensions_count <= kMaxSmallSize)
       dims_ = std::array<int32_t, kMaxSmallSize>{};
     else
@@ -275,19 +277,18 @@ class Shape
   // For use only by ExtendedShape(), written to guarantee (return-value) copy
   // elision in C++17.
   // This creates a shape padded to the desired size with the specified value.
-  Shape(int new_shape_size, const Shape &shape, int pad_value) : _size(0)
+  Shape(int new_shape_size, const Shape &shape, int pad_value) : _size(new_shape_size)
   {
     assert(new_shape_size >= shape.DimensionsCount());
     assert(new_shape_size <= kMaxSmallSize);
-    initStorage(new_shape_size);
+    Resize(new_shape_size);
     const int size_increase = new_shape_size - shape.DimensionsCount();
     for (int i = 0; i < size_increase; ++i)
     {
       SetDim(i, pad_value);
     }
     std::memcpy(DimsData() + size_increase, shape.DimsData(),
                 sizeof(int32_t) * shape.DimensionsCount());
-    _size = new_shape_size;
   }
 
   int32_t _size;

runtime/compute/cker/src/DeathTestMacro.h

@@ -0,0 +1,32 @@
+/*
+ * Copyright (c) 2025 Samsung Electronics Co., Ltd. All Rights Reserved
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+#ifndef __DEATH_TEST_MACROS_H__
+#define __DEATH_TEST_MACROS_H__
+
+#include <gtest/gtest.h>
+
+// In release mode, assertions might not trigger abort() via assert(),
+// so we use EXPECT_EXIT and require that the statement exits with EXIT_FAILURE.
+// In debug mode, we use EXPECT_DEATH as usual.
+#ifdef NDEBUG
+#define EXPECT_DEATH_DEBUG_ONLY(statement, regex)
+#else
+#define EXPECT_DEATH_DEBUG_ONLY(statement, regex) \
+  EXPECT_EXIT(statement, ::testing::KilledBySignal(SIGABRT), regex)
+#endif
+
+#endif // __DEATH_TEST_MACROS_H__