added invalid numeric width (#8947)

* added invalid numeric width

* updated unit test

Author: jwoo-msft

source/shared/cpp/AdaptiveCardsSharedModel/AdaptiveCardsSharedModelUnitTest/ExplicitDimensionTest.cpp

@@ -455,6 +455,12 @@ namespace AdaptiveCardsSharedModelUnitTest
                     {\
                         \"type\":\"ColumnSet\",\
                         \"columns\": [\
+                            {\
+                                \"type\": \"Column\",\
+                                \"width\": \"20\",\
+                                \"items\": [\
+                                ]\
+                            },\
                             {\
                                 \"type\": \"Column\",\
                                 \"width\": \"20\",\
@@ -531,7 +537,7 @@ namespace AdaptiveCardsSharedModelUnitTest
             std::shared_ptr<ColumnSet> columnSet = std::static_pointer_cast<ColumnSet>(element);
             std::shared_ptr<Column> column = columnSet->GetColumns().front();
             Assert::AreEqual<int>(0, column->GetPixelWidth());
-            Assert::AreEqual<std::string>("20px20", column->GetWidth());
+            Assert::AreEqual<std::string>("stretch", column->GetWidth());
         }
 
         TEST_METHOD(ExplicitWidthMalformedValueTest)

source/shared/cpp/ObjectModel/ColumnSet.cpp

@@ -53,6 +53,37 @@ void ColumnSet::DeserializeChildren(ParseContext& context, const Json::Value& va
         AdaptiveCardSchemaKey::Columns,
         false,                                             // isRequired
         CardElementTypeToString(CardElementType::Column)); // impliedType
+
+    auto counts = 0;
+    std::shared_ptr<Column> cur = nullptr;
+
+    // If there is one column with numeric width,
+    // set it to stretch since the ratio requires at least two columns
+    // with numeric width.
+    for (auto const& column : m_columns)
+    {
+        const auto width {column->GetWidth()};
+        if (width.empty())
+        {
+            continue;
+        }
+        const auto ch = width.at(0);
+        if (column->GetPixelWidth() == 0 &&
+            (ch != 'a' && ch != 's') &&
+            // The check of the numeric width is purposely
+            // left at minimum. The stronger check should
+            // be placed at the column.
+             std::isdigit(ch))
+        {
+            counts++;
+            cur = column;
+        }
+    }
+
+    if (counts == 1 && cur)
+    {
+        cur->SetWidth("stretch");
+    }
 }
 
 std::optional<HorizontalAlignment> AdaptiveCards::ColumnSet::GetHorizontalAlignment() const