diff --git a/test/unittest/capi/modifiers/BUILD.gn b/test/unittest/capi/modifiers/BUILD.gn index 8484b33d4846958987ac288f04ab62d745af2654..120c94511d4fcc46df0693b9d70365466aae314e 100644 --- a/test/unittest/capi/modifiers/BUILD.gn +++ b/test/unittest/capi/modifiers/BUILD.gn @@ -52,18 +52,18 @@ ace_unittest("capi_all_modifiers_test") { #"common_method_modifier_test7.cpp", #"common_method_modifier_test8.cpp", #"common_method_modifier_test9.cpp", - #"common_modifier_test.cpp", - #"common_shape_method_modifier_resources_test.cpp", - #"common_shape_method_modifier_test.cpp", - #"counter_modifier_test.cpp", - #"data_panel_modifier_test.cpp", - #"date_picker_modifier_test.cpp", - #"divider_modifier_test.cpp", - #"effect_component_modifier_test.cpp", - #"ellipse_modifier_test.cpp", - #"flex_modifier_test.cpp", - #"folder_stack_modifier_test.cpp", - #"form_component_modifier_test.cpp", + "common_modifier_test.cpp", + "common_shape_method_modifier_resources_test.cpp", + "common_shape_method_modifier_test.cpp", + "counter_modifier_test.cpp", + "data_panel_modifier_test.cpp", + "date_picker_modifier_test.cpp", + "divider_modifier_test.cpp", + "effect_component_modifier_test.cpp", + "ellipse_modifier_test.cpp", + "flex_modifier_test.cpp", + "folder_stack_modifier_test.cpp", + "form_component_modifier_test.cpp", #"form_link_modifier_test.cpp", #"gauge_modifier_test.cpp", #"grid_col_modifier_test.cpp", diff --git a/test/unittest/capi/modifiers/common_shape_method_modifier_resources_test.cpp b/test/unittest/capi/modifiers/common_shape_method_modifier_resources_test.cpp index 03b1c20ef3406a3a11ca20bc22dd183a6af5d46b..7b21acf1b4d556a3e6e7d38b56eb5922261b90ca 100644 --- a/test/unittest/capi/modifiers/common_shape_method_modifier_resources_test.cpp +++ b/test/unittest/capi/modifiers/common_shape_method_modifier_resources_test.cpp @@ -84,9 +84,9 @@ HWTEST_F(CommonShapeMethodModifierResourcesTest, setStrokeWidthTestResources, Te std::unique_ptr jsonValue; double result; - typedef std::pair OneTestStep; + typedef std::pair OneTestStep; const std::vector testPlan = { - { Converter::ArkValue(FAKE_RES_ID), 10 } + { Converter::ArkValue(FAKE_RES_ID), 10 } }; for (const auto &[arkLength, expected]: testPlan) { @@ -105,8 +105,10 @@ HWTEST_F(CommonShapeMethodModifierResourcesTest, setStrokeWidthTestResources, Te HWTEST_F(CommonShapeMethodModifierResourcesTest, setFillOpacityTestResources, TestSize.Level1) { std::unique_ptr jsonValue; + for (const auto &[value, expectVal]: UNION_NUM_STR_RES_RESOURECES_TEST_PLAN) { - modifier_->setFillOpacity(node_, &value); + auto res = Converter::ArkValue(value); + modifier_->setFillOpacity(node_, &res); jsonValue = GetJsonValue(node_); auto checkVal = GetAttrValue(jsonValue, ATTRIBUTE_FILL_OPACITY_NAME); EXPECT_EQ(checkVal, expectVal); @@ -121,8 +123,10 @@ HWTEST_F(CommonShapeMethodModifierResourcesTest, setFillOpacityTestResources, Te HWTEST_F(CommonShapeMethodModifierResourcesTest, setStrokeOpacityTestResources, TestSize.Level1) { std::unique_ptr jsonValue; + Opt_Union_Number_String_Resource res; for (const auto &[value, expectVal]: UNION_NUM_STR_RES_RESOURECES_TEST_PLAN) { - modifier_->setStrokeOpacity(node_, &value); + res = Converter::ArkValue(value); + modifier_->setStrokeOpacity(node_, &res); jsonValue = GetJsonValue(node_); auto checkVal = GetAttrValue(jsonValue, ATTRIBUTE_STROKE_OPACITY_NAME); EXPECT_EQ(checkVal, expectVal); diff --git a/test/unittest/capi/modifiers/common_shape_method_modifier_test.cpp b/test/unittest/capi/modifiers/common_shape_method_modifier_test.cpp index 41d8bb8c805d72e8ac262e25b8a24c0b7fb45ab3..6fcb13092c8336e75cdecd25f10803a7540e156a 100644 --- a/test/unittest/capi/modifiers/common_shape_method_modifier_test.cpp +++ b/test/unittest/capi/modifiers/common_shape_method_modifier_test.cpp @@ -44,8 +44,8 @@ const auto ATTRIBUTE_FILL_OPACITY_DEFAULT_VALUE = "1.000000"; const auto ATTRIBUTE_STROKE_WIDTH_DEFAULT_VALUE = 0.0f; // global types -using OneTestColorStep = std::pair; -using OneUnionNumStrResStep = std::pair; +using OneTestColorStep = std::pair; +using OneUnionNumStrResStep = std::pair; // global test plans const auto RES_NAME = NamedResourceId{"aa.bb.cc", Converter::ResourceType::COLOR}; @@ -53,13 +53,13 @@ const auto RES_ID = IntResourceId{11111, Converter::ResourceType::COLOR}; static const std::string EXPECTED_RESOURCE_COLOR = Color::RED.ToString(); // Color::RED is result of ThemeConstants::GetColorXxxx stubs static const std::vector TEST_COLOR_PLAN = { - { ArkUnion(ARK_COLOR_WHITE), "#FFFFFFFF" }, - { ArkUnion(0x123456), "#FF123456" }, - { ArkUnion(0.5f), "#00000000" }, - { ArkUnion("#11223344"), "#11223344" }, - { ArkUnion("65535"), "#FF00FFFF" }, - { CreateResourceUnion(RES_NAME), EXPECTED_RESOURCE_COLOR }, - { CreateResourceUnion(RES_ID), EXPECTED_RESOURCE_COLOR }, + { ArkUnion(ARK_COLOR_WHITE), "#FFFFFFFF" }, + { ArkUnion(0x123456), "#FF123456" }, + { ArkUnion(0.5f), "#00000000" }, + { ArkUnion("#11223344"), "#11223344" }, + { ArkUnion("65535"), "#FF00FFFF" }, + { CreateResourceUnion(RES_NAME), EXPECTED_RESOURCE_COLOR }, + { CreateResourceUnion(RES_ID), EXPECTED_RESOURCE_COLOR }, }; } // namespace @@ -136,13 +136,13 @@ HWTEST_F(CommonShapeMethodModifierTest, setStrokeLineCapTestValidValues, TestSiz { std::unique_ptr jsonValue; std::string resultStr; - Ark_LineCapStyle inputValueStrokeLineCap; // Initial setup - inputValueStrokeLineCap = ARK_LINE_CAP_STYLE_BUTT; + auto inputValueStrokeLineCap = Converter::ArkValue(ARK_LINE_CAP_STYLE_BUTT); // Test - modifier_->setStrokeLineCap(node_, inputValueStrokeLineCap); + modifier_->setStrokeLineCap(node_, &inputValueStrokeLineCap); + // Initial verification jsonValue = GetJsonValue(node_); @@ -150,14 +150,15 @@ HWTEST_F(CommonShapeMethodModifierTest, setStrokeLineCapTestValidValues, TestSiz EXPECT_EQ(resultStr, "LineCapStyle.Butt"); // Verifying attribute's other values - inputValueStrokeLineCap = ARK_LINE_CAP_STYLE_ROUND; - modifier_->setStrokeLineCap(node_, inputValueStrokeLineCap); + inputValueStrokeLineCap = Converter::ArkValue(ARK_LINE_CAP_STYLE_ROUND); + modifier_->setStrokeLineCap(node_, &inputValueStrokeLineCap); jsonValue = GetJsonValue(node_); resultStr = GetAttrValue(jsonValue, ATTRIBUTE_STROKE_LINE_CAP_NAME); EXPECT_EQ(resultStr, "LineCapStyle.Round"); - inputValueStrokeLineCap = ARK_LINE_CAP_STYLE_SQUARE; - modifier_->setStrokeLineCap(node_, inputValueStrokeLineCap); + inputValueStrokeLineCap = Converter::ArkValue(ARK_LINE_CAP_STYLE_SQUARE); + modifier_->setStrokeLineCap(node_, &inputValueStrokeLineCap); + jsonValue = GetJsonValue(node_); resultStr = GetAttrValue(jsonValue, ATTRIBUTE_STROKE_LINE_CAP_NAME); EXPECT_EQ(resultStr, "LineCapStyle.Square"); @@ -172,13 +173,12 @@ HWTEST_F(CommonShapeMethodModifierTest, setStrokeLineCapTestInvalidValues, TestS { std::unique_ptr jsonValue; std::string resultStr; - Ark_LineCapStyle inputValueStrokeLineCap; // Initial setup - inputValueStrokeLineCap = static_cast(-1); + auto inputValueStrokeLineCap = Converter::ArkValue(static_cast(-1)); // Test - modifier_->setStrokeLineCap(node_, inputValueStrokeLineCap); + modifier_->setStrokeLineCap(node_, &inputValueStrokeLineCap); // Initial verification jsonValue = GetJsonValue(node_); @@ -209,13 +209,12 @@ HWTEST_F(CommonShapeMethodModifierTest, setStrokeLineJoinTestValidValues, TestSi { std::unique_ptr jsonValue; std::string resultStr; - Ark_LineJoinStyle inputValueStrokeLineJoin; // Initial setup - inputValueStrokeLineJoin = ARK_LINE_JOIN_STYLE_MITER; + auto inputValueStrokeLineJoin = Converter::ArkValue(ARK_LINE_JOIN_STYLE_MITER); // Test - modifier_->setStrokeLineJoin(node_, inputValueStrokeLineJoin); + modifier_->setStrokeLineJoin(node_, &inputValueStrokeLineJoin); // Initial verification jsonValue = GetJsonValue(node_); @@ -223,14 +222,14 @@ HWTEST_F(CommonShapeMethodModifierTest, setStrokeLineJoinTestValidValues, TestSi EXPECT_EQ(resultStr, "LineJoinStyle.Miter"); // Verifying attribute's other values - inputValueStrokeLineJoin = ARK_LINE_JOIN_STYLE_ROUND; - modifier_->setStrokeLineJoin(node_, inputValueStrokeLineJoin); + inputValueStrokeLineJoin = Converter::ArkValue(ARK_LINE_JOIN_STYLE_ROUND); + modifier_->setStrokeLineJoin(node_, &inputValueStrokeLineJoin); jsonValue = GetJsonValue(node_); resultStr = GetAttrValue(jsonValue, ATTRIBUTE_STROKE_LINE_JOIN_NAME); EXPECT_EQ(resultStr, "LineJoinStyle.Round"); - inputValueStrokeLineJoin = ARK_LINE_JOIN_STYLE_BEVEL; - modifier_->setStrokeLineJoin(node_, inputValueStrokeLineJoin); + inputValueStrokeLineJoin = Converter::ArkValue(ARK_LINE_JOIN_STYLE_BEVEL); + modifier_->setStrokeLineJoin(node_, &inputValueStrokeLineJoin); jsonValue = GetJsonValue(node_); resultStr = GetAttrValue(jsonValue, ATTRIBUTE_STROKE_LINE_JOIN_NAME); EXPECT_EQ(resultStr, "LineJoinStyle.Bevel"); @@ -245,13 +244,12 @@ HWTEST_F(CommonShapeMethodModifierTest, setStrokeLineJoinTestInvalidValues, Test { std::unique_ptr jsonValue; std::string resultStr; - Ark_LineJoinStyle inputValueStrokeLineJoin; // Initial setup - inputValueStrokeLineJoin = static_cast(-1); + auto inputValueStrokeLineJoin = Converter::ArkValue(static_cast(-1)); // Test - modifier_->setStrokeLineJoin(node_, inputValueStrokeLineJoin); + modifier_->setStrokeLineJoin(node_, &inputValueStrokeLineJoin); // Initial verification jsonValue = GetJsonValue(node_); @@ -279,20 +277,34 @@ HWTEST_F(CommonShapeMethodModifierTest, setStrokeOpacityTestDefault, TestSize.Le HWTEST_F(CommonShapeMethodModifierTest, setStrokeOpacityTest, TestSize.Level1) { static const std::vector UNION_NUM_STR_RES_TEST_PLAN = { - { ArkUnion(ArkValue(123)), "1.000000" }, - { ArkUnion(ArkValue(-123)), "0.000000" }, - { ArkUnion(ArkValue("5.6vp")), "1.000000" }, - { ArkUnion(ArkValue("-5.6vp")), "1.000000" }, - { ArkUnion(ArkValue("45px")), "1.000000" }, - { ArkUnion(ArkValue("-45px")), "1.000000" }, - { ArkUnion(ArkValue(0.23f)), "0.230000" }, - { ArkUnion(ArkValue(-0.23f)), "0.000000" }, - { ArkUnion(ArkValue("10%")), "0.100000" }, - { ArkUnion(ArkValue("45dp")), "1.000000" }, - { ArkUnion(ArkValue("undefVal")), "1.000000" }, - { ArkUnion(ArkValue("")), "1.000000" }, - { ArkUnion(ArkValue("qw111vp")), "1.000000" }, - { ArkUnion(ArkValue("qw111")), "1.000000" }, + { Converter::ArkValue( + ArkUnion(ArkValue(123))), "1.000000" }, + { Converter::ArkValue( + ArkUnion(ArkValue(-123))), "0.000000" }, + { Converter::ArkValue( + ArkUnion(ArkValue("5.6vp"))), "1.000000" }, + { Converter::ArkValue( + ArkUnion(ArkValue("-5.6vp"))), "1.000000" }, + { Converter::ArkValue( + ArkUnion(ArkValue("45px"))), "1.000000" }, + { Converter::ArkValue( + ArkUnion(ArkValue("-45px"))), "1.000000" }, + { Converter::ArkValue( + ArkUnion(ArkValue(0.23f))), "0.230000" }, + { Converter::ArkValue( + ArkUnion(ArkValue(-0.23f))), "0.000000" }, + { Converter::ArkValue( + ArkUnion(ArkValue("10%"))), "0.100000" }, + { Converter::ArkValue( + ArkUnion(ArkValue("45dp"))), "1.000000" }, + { Converter::ArkValue( + ArkUnion(ArkValue("undefVal"))), "1.000000" }, + { Converter::ArkValue( + ArkUnion(ArkValue(""))), "1.000000" }, + { Converter::ArkValue( + ArkUnion(ArkValue("qw111vp"))), "1.000000" }, + { Converter::ArkValue( + ArkUnion(ArkValue("qw111"))), "1.000000" }, }; for (const auto &[value, expectVal]: UNION_NUM_STR_RES_TEST_PLAN) { @@ -322,20 +334,34 @@ HWTEST_F(CommonShapeMethodModifierTest, setFillOpacityTestDefault, TestSize.Leve HWTEST_F(CommonShapeMethodModifierTest, setFillOpacityTest, TestSize.Level1) { static const std::vector UNION_NUM_STR_RES_TEST_PLAN = { - { ArkUnion(ArkValue(123)), "1.000000" }, - { ArkUnion(ArkValue(-123)), "0.000000" }, - { ArkUnion(ArkValue("5.6vp")), "1.000000" }, - { ArkUnion(ArkValue("-5.6vp")), "1.000000" }, - { ArkUnion(ArkValue("45px")), "1.000000" }, - { ArkUnion(ArkValue("-45px")), "1.000000" }, - { ArkUnion(ArkValue(0.23f)), "0.230000" }, - { ArkUnion(ArkValue(-0.23f)), "0.000000" }, - { ArkUnion(ArkValue("10%")), "0.100000" }, - { ArkUnion(ArkValue("45dp")), "1.000000" }, - { ArkUnion(ArkValue("undefVal")), "1.000000" }, - { ArkUnion(ArkValue("")), "1.000000" }, - { ArkUnion(ArkValue("qw111vp")), "1.000000" }, - { ArkUnion(ArkValue("qw111")), "1.000000" }, + { Converter::ArkValue( + ArkUnion(ArkValue(123))), "1.000000" }, + { Converter::ArkValue( + ArkUnion(ArkValue(-123))), "0.000000" }, + { Converter::ArkValue( + ArkUnion(ArkValue("5.6vp"))), "1.000000" }, + { Converter::ArkValue( + ArkUnion(ArkValue("-5.6vp"))), "1.000000" }, + { Converter::ArkValue( + ArkUnion(ArkValue("45px"))), "1.000000" }, + { Converter::ArkValue( + ArkUnion(ArkValue("-45px"))), "1.000000" }, + { Converter::ArkValue( + ArkUnion(ArkValue(0.23f))), "0.230000" }, + { Converter::ArkValue( + ArkUnion(ArkValue(-0.23f))), "0.000000" }, + { Converter::ArkValue( + ArkUnion(ArkValue("10%"))), "0.100000" }, + { Converter::ArkValue( + ArkUnion(ArkValue("45dp"))), "1.000000" }, + { Converter::ArkValue( + ArkUnion(ArkValue("undefVal"))), "1.000000" }, + { Converter::ArkValue( + ArkUnion(ArkValue(""))), "1.000000" }, + { Converter::ArkValue( + ArkUnion(ArkValue("qw111vp"))), "1.000000" }, + { Converter::ArkValue( + ArkUnion(ArkValue("qw111"))), "1.000000" }, }; for (const auto &[value, expectVal]: UNION_NUM_STR_RES_TEST_PLAN) { @@ -369,19 +395,19 @@ HWTEST_F(CommonShapeMethodModifierTest, setStrokeWidthTestValidValues, TestSize. std::unique_ptr jsonValue; double result; - typedef std::pair OneTestStep; + typedef std::pair OneTestStep; static const std::vector testPlan = { - { ArkValue(1.0f), 1.0f }, - { ArkValue(2.45f), 2.45f }, - { ArkValue(5.0_px), 5.0f }, - { ArkValue(22.35_px), 22.35f }, - { ArkValue("23.00%"), 0.0f }, - { ArkValue(7.0_vp), 7.0f }, - { ArkValue(1.65_vp), 1.65f }, - { ArkValue(-0.1f), 0.0f }, - { ArkValue(65.0_fp), 65.0f }, - { ArkValue(4.3_fp), 4.30f }, - { ArkValue(-5.0_px), 0.0f }, + { ArkValue(1.0f), 1.0f }, + { ArkValue(2.45f), 2.45f }, + { ArkValue(5.0_px), 5.0f }, + { ArkValue(22.35_px), 22.35f }, + { ArkValue("23.00%"), 0.0f }, + { ArkValue(7.0_vp), 7.0f }, + { ArkValue(1.65_vp), 1.65f }, + { ArkValue(-0.1f), 0.0f }, + { ArkValue(65.0_fp), 65.0f }, + { ArkValue(4.3_fp), 4.30f }, + { ArkValue(-5.0_px), 0.0f }, }; for (const auto &[arkLength, expected]: testPlan) { @@ -414,24 +440,36 @@ HWTEST_F(CommonShapeMethodModifierTest, setStrokeDashOffsetTestDefault, TestSize */ HWTEST_F(CommonShapeMethodModifierTest, setStrokeDashOffsetTest, TestSize.Level1) { - using OneTestStep = std::pair; + using OneTestStep = std::pair; static const std::vector testPlan = { - { ArkUnion(1), "1.00vp" }, - { ArkUnion(0), "0.00px" }, - { ArkUnion(2.45f), "2.45vp" }, - { ArkUnion(-2.45f), "0.00px" }, - { ArkUnion("5px"), "5.00px" }, - { ArkUnion("22.35px"), "22.35px" }, - { ArkUnion("7vp"), "7.00vp" }, - { ArkUnion("1.65vp"), "1.65vp" }, - { ArkUnion("65fp"), "65.00fp" }, - { ArkUnion("4.3fp"), "4.30fp" }, - { ArkUnion("11lpx"), "11.00lpx" }, - { ArkUnion("0.5lpx"), "0.50lpx" }, - { ArkUnion("3"), "3.00fp" }, - { ArkUnion(""), "0.00px" }, - { ArkUnion("10.65"), "10.65fp" }, - { ArkUnion("23%"), "0.00px" }, + { Converter::ArkValue(ArkUnion(1)), "1.00vp" }, + { Converter::ArkValue(ArkUnion(0)), "0.00px" }, + { Converter::ArkValue(ArkUnion(2.45f)), + "2.45vp" }, + { Converter::ArkValue(ArkUnion(-2.45f)), + "0.00px" }, + { Converter::ArkValue(ArkUnion("5px")), + "5.00px" }, + { Converter::ArkValue(ArkUnion("22.35px")), + "22.35px" }, + { Converter::ArkValue(ArkUnion("7vp")), + "7.00vp" }, + { Converter::ArkValue(ArkUnion("1.65vp")), + "1.65vp" }, + { Converter::ArkValue(ArkUnion("65fp")), + "65.00fp" }, + { Converter::ArkValue(ArkUnion("4.3fp")), + "4.30fp" }, + { Converter::ArkValue(ArkUnion("11lpx")), + "11.00lpx" }, + { Converter::ArkValue(ArkUnion("0.5lpx")), + "0.50lpx" }, + { Converter::ArkValue(ArkUnion("3")), "3.00fp" }, + { Converter::ArkValue(ArkUnion("")), "0.00px" }, + { Converter::ArkValue(ArkUnion("10.65")), + "10.65fp" }, + { Converter::ArkValue(ArkUnion("23%")), + "0.00px" }, }; std::unique_ptr jsonValue; @@ -468,24 +506,38 @@ HWTEST_F(CommonShapeMethodModifierTest, setStrokeMiterLimitTestDefaultValue, Tes */ HWTEST_F(CommonShapeMethodModifierTest, setStrokeMiterLimitTest, TestSize.Level1) { - using OneTestStep = std::pair; + using OneTestStep = std::pair; static const std::vector testPlan = { - { ArkUnion(1), "1.000000" }, - { ArkUnion(0), "1.000000" }, - { ArkUnion(2.45f), "2.450000" }, - { ArkUnion("5px"), "4.000000" }, - { ArkUnion("22.35px"), "4.000000" }, - { ArkUnion("7vp"), "4.000000" }, - { ArkUnion("1.65vp"), "4.000000" }, - { ArkUnion("65fp"), "4.000000" }, - { ArkUnion("4.3fp"), "4.000000" }, - { ArkUnion("11lpx"), "4.000000" }, - { ArkUnion("0.5lpx"), "4.000000" }, - { ArkUnion("3"), "3.000000" }, - { ArkUnion("-3"), "1.000000" }, - { ArkUnion(""), "4.000000" }, - { ArkUnion("10.65"), "10.650000" }, - { ArkUnion("23%"), "1.000000" }, + { Converter::ArkValue(ArkUnion(1)), "1.000000" }, + { Converter::ArkValue(ArkUnion(0)), "1.000000" }, + { Converter::ArkValue(ArkUnion(2.45f)), + "2.450000" }, + { Converter::ArkValue(ArkUnion("5px")), + "4.000000" }, + { Converter::ArkValue(ArkUnion("22.35px")), + "4.000000" }, + { Converter::ArkValue(ArkUnion("7vp")), + "4.000000" }, + { Converter::ArkValue(ArkUnion("1.65vp")), + "4.000000" }, + { Converter::ArkValue(ArkUnion("65fp")), + "4.000000" }, + { Converter::ArkValue(ArkUnion("4.3fp")), + "4.000000" }, + { Converter::ArkValue(ArkUnion("11lpx")), + "4.000000" }, + { Converter::ArkValue(ArkUnion("0.5lpx")), + "4.000000" }, + { Converter::ArkValue(ArkUnion("3")), + "3.000000" }, + { Converter::ArkValue(ArkUnion("-3")), + "1.000000" }, + { Converter::ArkValue(ArkUnion("")), + "4.000000" }, + { Converter::ArkValue(ArkUnion("10.65")), + "10.650000" }, + { Converter::ArkValue(ArkUnion("23%")), + "1.000000" }, }; std::unique_ptr jsonValue; @@ -509,22 +561,22 @@ HWTEST_F(CommonShapeMethodModifierTest, setAntiAliasTest, TestSize.Level1) { static const std::string PROP_NAME("antiAlias"); ASSERT_NE(modifier_->setAntiAlias, nullptr); - using OneBoolStep = std::tuple; + using OneBoolStep = std::tuple; const std::vector BOOL_TEST_PLAN = { - { false, "false" }, - { true, "true" }, - { 0, "false" }, - { -25, "true" }, - { 0, "false" }, - { 25, "true" }, - { false, "false" } + { Converter::ArkValue(false), "false" }, + { Converter::ArkValue(true), "true" }, + { Converter::ArkValue(0), "false" }, + { Converter::ArkValue(-25), "true" }, + { Converter::ArkValue(0), "false" }, + { Converter::ArkValue(25), "true" }, + { Converter::ArkValue(false), "false" } }; auto fullJson = GetJsonValue(node_); auto checkVal = GetAttrValue(fullJson, PROP_NAME); EXPECT_EQ(checkVal, "true"); for (const auto& [value, expectVal] : BOOL_TEST_PLAN) { - modifier_->setAntiAlias(node_, value); + modifier_->setAntiAlias(node_, &value); auto fullJson = GetJsonValue(node_); checkVal = GetAttrValue(fullJson, PROP_NAME); EXPECT_EQ(checkVal, expectVal); @@ -540,16 +592,16 @@ HWTEST_F(CommonShapeMethodModifierTest, setStrokeDashArrayTest, TestSize.Level1) { static const std::string propName("strokeDashArray"); ASSERT_NE(modifier_->setStrokeDashArray, nullptr); - using OneArrayLengthStep = std::tuple>; + using OneArrayLengthStep = std::tuple>; std::vector vec1 = {3._px, 6._px}; std::vector vec2 = {5.4f}; std::vector vec3 = {8.43_vp}; std::vector vec4 = {0.1_pct, 0.55_pct}; const std::vector TEST_PLAN = { - { Converter::ArkValue(vec1, Converter::FC), {"3.00px", "6.00px"} }, - { Converter::ArkValue(vec2, Converter::FC), {"5.40vp", "5.40vp"} }, - { Converter::ArkValue(vec3, Converter::FC), {"8.43vp", "8.43vp"} }, - { Converter::ArkValue(vec4, Converter::FC), {"10.00%", "55.00%"} }, + { Converter::ArkValue(vec1, Converter::FC), {"3.00px", "6.00px"} }, + { Converter::ArkValue(vec2, Converter::FC), {"5.40vp", "5.40vp"} }, + { Converter::ArkValue(vec3, Converter::FC), {"8.43vp", "8.43vp"} }, + { Converter::ArkValue(vec4, Converter::FC), {"10.00%", "55.00%"} }, }; auto fullJson = GetJsonValue(node_); auto checkVal = GetAttrValue>(fullJson, propName); @@ -574,7 +626,7 @@ HWTEST_F(CommonShapeMethodModifierTest, setStrokeDashArrayTest, TestSize.Level1) * @tc.desc: Check the invalid cases for setStrokeDashArray * @tc.type: FUNC */ -HWTEST_F(CommonShapeMethodModifierTest, setStrokeDashArrayInavlidTest, TestSize.Level1) +HWTEST_F(CommonShapeMethodModifierTest, DISABLED_setStrokeDashArrayInavlidTest, TestSize.Level1) { static const std::string propName("strokeDashArray"); ASSERT_NE(modifier_->setStrokeDashArray, nullptr); diff --git a/test/unittest/capi/modifiers/counter_modifier_test.cpp b/test/unittest/capi/modifiers/counter_modifier_test.cpp index d75e47ea20482ade634c1b17f494fe74496f8bf3..5bcf1b8a7b32439995bfc085632df4cfbc19c127 100644 --- a/test/unittest/capi/modifiers/counter_modifier_test.cpp +++ b/test/unittest/capi/modifiers/counter_modifier_test.cpp @@ -73,15 +73,14 @@ HWTEST_F(CounterModifierTest, setEnableDecTestDefaultValues, TestSize.Level1) HWTEST_F(CounterModifierTest, setEnableDecTestValidValues, TestSize.Level1) { std::string strResult; - Ark_Boolean inputValue; - inputValue = Converter::ArkValue(false); - modifier_->setEnableDec(node_, inputValue); + auto inputValue = Converter::ArkValue(false); + modifier_->setEnableDec(node_, &inputValue); strResult = GetStringAttribute(node_, ATTRIBUTE_ENABLE_DEC_NAME); EXPECT_EQ(strResult, "false"); - inputValue = Converter::ArkValue(true); - modifier_->setEnableDec(node_, inputValue); + inputValue = Converter::ArkValue(true); + modifier_->setEnableDec(node_, &inputValue); strResult = GetStringAttribute(node_, ATTRIBUTE_ENABLE_DEC_NAME); EXPECT_EQ(strResult, "true"); } @@ -107,15 +106,14 @@ HWTEST_F(CounterModifierTest, setEnableIncTestDefaultValues, TestSize.Level1) HWTEST_F(CounterModifierTest, setEnableIncTestValidValues, TestSize.Level1) { std::string strResult; - Ark_Boolean inputValue; - inputValue = Converter::ArkValue(false); - modifier_->setEnableInc(node_, inputValue); + auto inputValue = Converter::ArkValue(false); + modifier_->setEnableInc(node_, &inputValue); strResult = GetStringAttribute(node_, ATTRIBUTE_ENABLE_INC_NAME); EXPECT_EQ(strResult, "false"); - inputValue = Converter::ArkValue(true); - modifier_->setEnableInc(node_, inputValue); + inputValue = Converter::ArkValue(true); + modifier_->setEnableInc(node_, &inputValue); strResult = GetStringAttribute(node_, ATTRIBUTE_ENABLE_INC_NAME); EXPECT_EQ(strResult, "true"); } @@ -148,8 +146,9 @@ HWTEST_F(CounterModifierTest, setOnIncTest, TestSize.Level1) }; VoidCallback arkCallback = Converter::ArkValue(checkCallback, contextId); + auto optCallback = Converter::ArkValue(arkCallback); - modifier_->setOnInc(node_, &arkCallback); + modifier_->setOnInc(node_, &optCallback); EXPECT_EQ(checkEvent.has_value(), false); subGestureHub->ActClick(); @@ -186,8 +185,9 @@ HWTEST_F(CounterModifierTest, setOnDecTest, TestSize.Level1) }; VoidCallback arkCallback = Converter::ArkValue(checkCallback, contextId); + auto optCallback = Converter::ArkValue(arkCallback); - modifier_->setOnDec(node_, &arkCallback); + modifier_->setOnDec(node_, &optCallback); EXPECT_EQ(checkEvent.has_value(), false); subGestureHub->ActClick(); @@ -203,20 +203,23 @@ HWTEST_F(CounterModifierTest, setOnDecTest, TestSize.Level1) */ HWTEST_F(CounterModifierTest, setBackgroundColorTest, TestSize.Level1) { - using OneTestStep = std::pair; + using OneTestStep = std::pair; static const std::string PROP_NAME("backgroundColor"); static auto resName = NamedResourceId("aa.bb.cc", Converter::ResourceType::COLOR); static auto resId = IntResourceId(1234, Converter::ResourceType::COLOR); static const std::string EXPECTED_RESOURCE_COLOR = Color::RED.ToString(); // Color::RED is result of ThemeConstants::GetColorXxxx stubs static const std::vector testPlan = { - { ArkUnion(ARK_COLOR_WHITE), "#FFFFFFFF" }, - { ArkUnion(0x123456), "#FF123456" }, - { ArkUnion(0.5f), "#00000000" }, - { ArkUnion("#11223344"), "#11223344" }, - { ArkUnion("65535"), "#FF00FFFF" }, - { CreateResourceUnion(resName), EXPECTED_RESOURCE_COLOR }, - { CreateResourceUnion(resId), EXPECTED_RESOURCE_COLOR }, + { Converter::ArkValue(ArkUnion(ARK_COLOR_WHITE)), + "#FFFFFFFF" }, + { Converter::ArkValue(ArkUnion(0x123456)), "#FF123456" }, + { Converter::ArkValue(ArkUnion(0.5f)), "#00000000" }, + { Converter::ArkValue(ArkUnion("#11223344")), "#11223344" }, + { Converter::ArkValue(ArkUnion("65535")), "#FF00FFFF" }, + { Converter::ArkValue(CreateResourceUnion(resName)), + EXPECTED_RESOURCE_COLOR }, + { Converter::ArkValue(CreateResourceUnion(resId)), + EXPECTED_RESOURCE_COLOR }, }; ASSERT_NE(commonModifier_->setBackgroundColor0, nullptr); @@ -249,22 +252,21 @@ HWTEST_F(CounterModifierTest, setWidthTestDefaultValues, TestSize.Level1) */ HWTEST_F(CounterModifierTest, setWidth0TestValidValues, TestSize.Level1) { - typedef std::pair OneTestStep; + typedef std::pair OneTestStep; static const std::vector testPlan = { - { Converter::ArkValue(2.45f), "2.45vp" }, - { Converter::ArkValue(5.0_px), "5.00px" }, - { Converter::ArkValue(22.35_px), "22.35px" }, - { Converter::ArkValue(7.0_vp), "7.00vp" }, - { Converter::ArkValue(1.65_vp), "1.65vp" }, - { Converter::ArkValue(65.0_fp), "65.00fp" }, - { Converter::ArkValue(4.3_fp), "4.30fp" }, - { Converter::ArkValue("12.00%"), "12.00%" }, - { RES_ARK_LENGTH, "10.00px" }, + { Converter::ArkValue(2.45f), "2.45vp" }, + { Converter::ArkValue(5.0_px), "5.00px" }, + { Converter::ArkValue(22.35_px), "22.35px" }, + { Converter::ArkValue(7.0_vp), "7.00vp" }, + { Converter::ArkValue(1.65_vp), "1.65vp" }, + { Converter::ArkValue(65.0_fp), "65.00fp" }, + { Converter::ArkValue(4.3_fp), "4.30fp" }, + { Converter::ArkValue("12.00%"), "12.00%" }, + { Converter::ArkValue(RES_ARK_LENGTH), "10.00px" }, }; for (const auto &[arkLength, expected]: testPlan) { - Ark_Length inputValue = Converter::ArkValue(arkLength); - commonModifier_->setWidth0(node_, &inputValue); + commonModifier_->setWidth0(node_, &arkLength); auto strResult = GetStringAttribute(node_, ATTRIBUTE_WIDTH_NAME); EXPECT_EQ(strResult, expected); } @@ -275,21 +277,20 @@ HWTEST_F(CounterModifierTest, setWidth0TestValidValues, TestSize.Level1) * @tc.desc: * @tc.type: FUNC */ -HWTEST_F(CounterModifierTest, setWidth0TestInvalidValues, TestSize.Level1) +HWTEST_F(CounterModifierTest, DISABLED_setWidth0TestInvalidValues, TestSize.Level1) { - static const std::vector testPlan = { - Converter::ArkValue(-2.45f), - Converter::ArkValue(-5.0_px), - Converter::ArkValue(-22.35_px), - Converter::ArkValue(-7.0_vp), - Converter::ArkValue(-1.65_vp), - Converter::ArkValue(-65.0_fp), - Converter::ArkValue(-4.3_fp), + static const std::vector testPlan = { + Converter::ArkValue(-2.45f), + Converter::ArkValue(-5.0_px), + Converter::ArkValue(-22.35_px), + Converter::ArkValue(-7.0_vp), + Converter::ArkValue(-1.65_vp), + Converter::ArkValue(-65.0_fp), + Converter::ArkValue(-4.3_fp), }; for (const auto &arkLength : testPlan) { - Ark_Length inputValue = Converter::ArkValue(arkLength); - commonModifier_->setWidth0(node_, &inputValue); + commonModifier_->setWidth0(node_, &arkLength); auto strResult = GetStringAttribute(node_, ATTRIBUTE_WIDTH_NAME); EXPECT_EQ(strResult, ATTRIBUTE_WIDTH_DEFAULT_VALUE); } @@ -317,22 +318,21 @@ HWTEST_F(CounterModifierTest, setHeightTestDefaultValues, TestSize.Level1) */ HWTEST_F(CounterModifierTest, setHeight0TestValidValues, TestSize.Level1) { - typedef std::pair OneTestStep; + typedef std::pair OneTestStep; static const std::vector testPlan = { - { Converter::ArkValue(2.45f), "2.45vp" }, - { Converter::ArkValue(5.0_px), "5.00px" }, - { Converter::ArkValue(22.35_px), "22.35px" }, - { Converter::ArkValue(7.0_vp), "7.00vp" }, - { Converter::ArkValue(1.65_vp), "1.65vp" }, - { Converter::ArkValue(65.0_fp), "65.00fp" }, - { Converter::ArkValue(4.3_fp), "4.30fp" }, - { Converter::ArkValue("12.00%"), "12.00%" }, - { RES_ARK_LENGTH, "10.00px" }, + { Converter::ArkValue(2.45f), "2.45vp" }, + { Converter::ArkValue(5.0_px), "5.00px" }, + { Converter::ArkValue(22.35_px), "22.35px" }, + { Converter::ArkValue(7.0_vp), "7.00vp" }, + { Converter::ArkValue(1.65_vp), "1.65vp" }, + { Converter::ArkValue(65.0_fp), "65.00fp" }, + { Converter::ArkValue(4.3_fp), "4.30fp" }, + { Converter::ArkValue("12.00%"), "12.00%" }, + { Converter::ArkValue(RES_ARK_LENGTH), "10.00px" }, }; for (const auto &[arkLength, expected]: testPlan) { - Ark_Length inputValue = Converter::ArkValue(arkLength); - commonModifier_->setHeight0(node_, &inputValue); + commonModifier_->setHeight0(node_, &arkLength); auto strResult = GetStringAttribute(node_, ATTRIBUTE_HEIGHT_NAME); EXPECT_EQ(strResult, expected); } @@ -343,21 +343,20 @@ HWTEST_F(CounterModifierTest, setHeight0TestValidValues, TestSize.Level1) * @tc.desc: * @tc.type: FUNC */ -HWTEST_F(CounterModifierTest, setHeight0TestInvalidValues, TestSize.Level1) +HWTEST_F(CounterModifierTest, DISABLED_setHeight0TestInvalidValues, TestSize.Level1) { - static const std::vector testPlan = { - Converter::ArkValue(-2.45f), - Converter::ArkValue(-5.0_px), - Converter::ArkValue(-22.35_px), - Converter::ArkValue(-7.0_vp), - Converter::ArkValue(-1.65_vp), - Converter::ArkValue(-65.0_fp), - Converter::ArkValue(-4.3_fp), + static const std::vector testPlan = { + Converter::ArkValue(-2.45f), + Converter::ArkValue(-5.0_px), + Converter::ArkValue(-22.35_px), + Converter::ArkValue(-7.0_vp), + Converter::ArkValue(-1.65_vp), + Converter::ArkValue(-65.0_fp), + Converter::ArkValue(-4.3_fp), }; for (const auto &arkLength : testPlan) { - Ark_Length inputValue = Converter::ArkValue(arkLength); - commonModifier_->setHeight0(node_, &inputValue); + commonModifier_->setHeight0(node_, &arkLength); auto strResult = GetStringAttribute(node_, ATTRIBUTE_HEIGHT_NAME); EXPECT_EQ(strResult, ATTRIBUTE_HEIGHT_DEFAULT_VALUE); } @@ -391,7 +390,8 @@ HWTEST_F(CounterModifierTest, setSizeTestValidHeightValues, TestSize.Level1) Ark_SizeOptions inputValue; inputValue.height = Converter::ArkValue(arkLength); inputValue.width = Converter::ArkValue(Ark_Empty()); - commonModifier_->setSize(node_, &inputValue); + auto optValue = Converter::ArkValue(inputValue); + commonModifier_->setSize(node_, &optValue); auto strResult = GetStringAttribute(node_, ATTRIBUTE_HEIGHT_NAME); EXPECT_EQ(strResult, expected); strResult = GetStringAttribute(node_, ATTRIBUTE_WIDTH_NAME); @@ -423,7 +423,8 @@ HWTEST_F(CounterModifierTest, setSizeTestValidWidthValues, TestSize.Level1) Ark_SizeOptions inputValue; inputValue.width = Converter::ArkValue(arkLength); inputValue.height = Converter::ArkValue(Ark_Empty()); - commonModifier_->setSize(node_, &inputValue); + auto optValue = Converter::ArkValue(inputValue); + commonModifier_->setSize(node_, &optValue); auto strResult = GetStringAttribute(node_, ATTRIBUTE_WIDTH_NAME); EXPECT_EQ(strResult, expected); strResult = GetStringAttribute(node_, ATTRIBUTE_HEIGHT_NAME); @@ -453,7 +454,8 @@ HWTEST_F(CounterModifierTest, setSizeTestInvalidValues, TestSize.Level1) for (const auto &arkLength : testPlan) { inputValue.width = Converter::ArkValue(arkLength); inputValue.height = Converter::ArkValue(Ark_Empty()); - commonModifier_->setSize(node_, &inputValue); + auto optValue = Converter::ArkValue(inputValue); + commonModifier_->setSize(node_, &optValue); auto strResult = GetStringAttribute(node_, ATTRIBUTE_WIDTH_NAME); EXPECT_EQ(strResult, ATTRIBUTE_WIDTH_DEFAULT_VALUE); strResult = GetStringAttribute(node_, ATTRIBUTE_HEIGHT_NAME); @@ -463,7 +465,8 @@ HWTEST_F(CounterModifierTest, setSizeTestInvalidValues, TestSize.Level1) for (const auto &arkLength : testPlan) { inputValue.height = Converter::ArkValue(arkLength); inputValue.width = Converter::ArkValue(Ark_Empty()); - commonModifier_->setSize(node_, &inputValue); + auto optValue = Converter::ArkValue(inputValue); + commonModifier_->setSize(node_, &optValue); auto strResult = GetStringAttribute(node_, ATTRIBUTE_WIDTH_NAME); EXPECT_EQ(strResult, ATTRIBUTE_WIDTH_DEFAULT_VALUE); strResult = GetStringAttribute(node_, ATTRIBUTE_HEIGHT_NAME); diff --git a/test/unittest/capi/modifiers/data_panel_modifier_test.cpp b/test/unittest/capi/modifiers/data_panel_modifier_test.cpp index 4a914c9a592f88168f690ce701697ee833e3a2f7..3d9a0dcfe6d3b9a42b77c70f4a4440f5bc2f26f6 100644 --- a/test/unittest/capi/modifiers/data_panel_modifier_test.cpp +++ b/test/unittest/capi/modifiers/data_panel_modifier_test.cpp @@ -293,7 +293,8 @@ HWTEST_F(DataPanelModifierTest, setCloseEffectTestValidValues, TestSize.Level1) inputValueCloseEffect = initValueCloseEffect; for (auto&& value: closeEffectCloseEffectValidValues) { inputValueCloseEffect = std::get<1>(value); - modifier_->setCloseEffect(node_, inputValueCloseEffect); + auto optValue = Converter::ArkValue(inputValueCloseEffect); + modifier_->setCloseEffect(node_, &optValue); jsonValue = GetJsonValue(node_); resultStr = GetAttrValue(jsonValue, ATTRIBUTE_CLOSE_EFFECT_NAME); expectedStr = std::get<2>(value); @@ -340,7 +341,8 @@ HWTEST_F(DataPanelModifierTest, setStrokeWidthTestValidValues, TestSize.Level1) inputValueStrokeWidth = initValueStrokeWidth; for (auto&& value: strokeWidthStrokeWidthValidValues) { inputValueStrokeWidth = std::get<1>(value); - modifier_->setStrokeWidth(node_, &inputValueStrokeWidth); + auto optValue = Converter::ArkValue(inputValueStrokeWidth); + modifier_->setStrokeWidth(node_, &optValue); jsonValue = GetJsonValue(node_); resultStr = GetAttrValue(jsonValue, ATTRIBUTE_STROKE_WIDTH_NAME); expectedStr = std::get<2>(value); @@ -372,7 +374,8 @@ HWTEST_F(DataPanelModifierTest, setStrokeWidthTestResourceValues, TestSize.Level inputValueStrokeWidth = initValueStrokeWidth; for (auto&& value: strokeWidthStrokeWidthResourceValues) { inputValueStrokeWidth = std::get<1>(value); - modifier_->setStrokeWidth(node_, &inputValueStrokeWidth); + auto optValue = Converter::ArkValue(inputValueStrokeWidth); + modifier_->setStrokeWidth(node_, &optValue); jsonValue = GetJsonValue(node_); resultStr = GetAttrValue(jsonValue, ATTRIBUTE_STROKE_WIDTH_NAME); expectedStr = std::get<2>(value); @@ -404,7 +407,8 @@ HWTEST_F(DataPanelModifierTest, setStrokeWidthTestInvalidValues, TestSize.Level1 inputValueStrokeWidth = initValueStrokeWidth; for (auto&& value: strokeWidthStrokeWidthInvalidValues) { inputValueStrokeWidth = std::get<1>(value); - modifier_->setStrokeWidth(node_, &inputValueStrokeWidth); + auto optValue = Converter::ArkValue(inputValueStrokeWidth); + modifier_->setStrokeWidth(node_, &optValue); jsonValue = GetJsonValue(node_); resultStr = GetAttrValue(jsonValue, ATTRIBUTE_STROKE_WIDTH_NAME); expectedStr = std::get<2>(value); @@ -457,7 +461,8 @@ HWTEST_F(DataPanelModifierTest, setTrackBackgroundColorTestValidValues, TestSize inputValueTrackBackgroundColor = initValueTrackBackgroundColor; for (auto&& value: trackBackgroundColorValidValues) { inputValueTrackBackgroundColor = std::get<1>(value); - modifier_->setTrackBackgroundColor(node_, &inputValueTrackBackgroundColor); + auto optValue = Converter::ArkValue(inputValueTrackBackgroundColor); + modifier_->setTrackBackgroundColor(node_, &optValue); jsonValue = GetJsonValue(node_); resultStr = GetAttrValue(jsonValue, ATTRIBUTE_TRACK_BACKGROUND_COLOR_NAME); expectedStr = std::get<2>(value); @@ -492,7 +497,8 @@ HWTEST_F(DataPanelModifierTest, setTrackBackgroundColorTestValidResourceValues, inputValueTrackBackgroundColor = initValueTrackBackgroundColor; for (auto&& value: trackBackgroundColorValidResourceValues) { inputValueTrackBackgroundColor = std::get<1>(value); - modifier_->setTrackBackgroundColor(node_, &inputValueTrackBackgroundColor); + auto optValue = Converter::ArkValue(inputValueTrackBackgroundColor); + modifier_->setTrackBackgroundColor(node_, &optValue); jsonValue = GetJsonValue(node_); resultStr = GetAttrValue(jsonValue, ATTRIBUTE_TRACK_BACKGROUND_COLOR_NAME); expectedStr = std::get<2>(value); @@ -529,7 +535,8 @@ HWTEST_F(DataPanelModifierTest, setTrackBackgroundColorTestInvalidValues, TestSi inputValueTrackBackgroundColor = initValueTrackBackgroundColor; for (auto&& value: trackBackgroundColorInvalidValues) { inputValueTrackBackgroundColor = std::get<1>(value); - modifier_->setTrackBackgroundColor(node_, &inputValueTrackBackgroundColor); + auto optValue = Converter::ArkValue(inputValueTrackBackgroundColor); + modifier_->setTrackBackgroundColor(node_, &optValue); jsonValue = GetJsonValue(node_); resultStr = GetAttrValue(jsonValue, ATTRIBUTE_TRACK_BACKGROUND_COLOR_NAME); expectedStr = std::get<2>(value); @@ -581,8 +588,8 @@ HWTEST_F(DataPanelModifierTest, setValueColorsTestStringValidValues, TestSize.Le Converter::ArkArrayHolder colorArrayHolder(colorArray); auto arkColorArray = colorArrayHolder.ArkValue(); - - modifier_->setValueColors(node_, &arkColorArray); + auto optValue = Converter::ArkValue(arkColorArray); + modifier_->setValueColors(node_, &optValue); std::unique_ptr jsonValue = GetJsonValue(node_); auto jsonArray = GetAttrValue>(jsonValue, ATTRIBUTE_VALUE_COLORS_NAME); @@ -622,8 +629,8 @@ HWTEST_F(DataPanelModifierTest, setValueColorsTestStringInvalidValues, TestSize. Converter::ArkArrayHolder colorArrayHolder(colorArray); auto arkColorArray = colorArrayHolder.ArkValue(); - - modifier_->setValueColors(node_, &arkColorArray); + auto optValue = Converter::ArkValue(arkColorArray); + modifier_->setValueColors(node_, &optValue); std::unique_ptr jsonValue = GetJsonValue(node_); auto jsonArray = GetAttrValue>(jsonValue, ATTRIBUTE_VALUE_COLORS_NAME); @@ -663,8 +670,8 @@ HWTEST_F(DataPanelModifierTest, setValueColorsTestNumberValidValues, TestSize.Le Converter::ArkArrayHolder colorArrayHolder(colorArray); auto arkColorArray = colorArrayHolder.ArkValue(); - - modifier_->setValueColors(node_, &arkColorArray); + auto optValue = Converter::ArkValue(arkColorArray); + modifier_->setValueColors(node_, &optValue); std::unique_ptr jsonValue = GetJsonValue(node_); auto jsonArray = GetAttrValue>(jsonValue, ATTRIBUTE_VALUE_COLORS_NAME); @@ -704,8 +711,8 @@ HWTEST_F(DataPanelModifierTest, setValueColorsTestEnumValidValues, TestSize.Leve Converter::ArkArrayHolder colorArrayHolder(colorArray); auto arkColorArray = colorArrayHolder.ArkValue(); - - modifier_->setValueColors(node_, &arkColorArray); + auto optValue = Converter::ArkValue(arkColorArray); + modifier_->setValueColors(node_, &optValue); std::unique_ptr jsonValue = GetJsonValue(node_); auto jsonArray = GetAttrValue>(jsonValue, ATTRIBUTE_VALUE_COLORS_NAME); @@ -745,8 +752,8 @@ HWTEST_F(DataPanelModifierTest, setValueColorsTestEnumInvalidValues, TestSize.Le Converter::ArkArrayHolder colorArrayHolder(colorArray); auto arkColorArray = colorArrayHolder.ArkValue(); - - modifier_->setValueColors(node_, &arkColorArray); + auto optValue = Converter::ArkValue(arkColorArray); + modifier_->setValueColors(node_, &optValue); std::unique_ptr jsonValue = GetJsonValue(node_); auto jsonArray = GetAttrValue>(jsonValue, ATTRIBUTE_VALUE_COLORS_NAME); @@ -786,8 +793,8 @@ HWTEST_F(DataPanelModifierTest, setValueColorsTestResourceValidValues, TestSize. Converter::ArkArrayHolder colorArrayHolder(colorArray); auto arkColorArray = colorArrayHolder.ArkValue(); - - modifier_->setValueColors(node_, &arkColorArray); + auto optValue = Converter::ArkValue(arkColorArray); + modifier_->setValueColors(node_, &optValue); std::unique_ptr jsonValue = GetJsonValue(node_); auto jsonArray = GetAttrValue>(jsonValue, ATTRIBUTE_VALUE_COLORS_NAME); @@ -841,8 +848,8 @@ HWTEST_F(DataPanelModifierTest, setValueColorsTestLinearGradientValidValues, Tes Converter::ArkArrayHolder colorArrayHolder(colorArray); auto arkColorArray = colorArrayHolder.ArkValue(); - - modifier_->setValueColors(node_, &arkColorArray); + auto optValue = Converter::ArkValue(arkColorArray); + modifier_->setValueColors(node_, &optValue); std::unique_ptr jsonValue = GetJsonValue(node_); auto jsonArray = GetAttrValue>(jsonValue, ATTRIBUTE_VALUE_COLORS_NAME); @@ -878,8 +885,8 @@ HWTEST_F(DataPanelModifierTest, setValueColorsTestLinearGradientInvalidValues, T Converter::ArkArrayHolder colorArrayHolder(colorArray); auto arkColorArray = colorArrayHolder.ArkValue(); - - modifier_->setValueColors(node_, &arkColorArray); + auto optValue = Converter::ArkValue(arkColorArray); + modifier_->setValueColors(node_, &optValue); std::unique_ptr jsonValue = GetJsonValue(node_); auto jsonArray = GetAttrValue>(jsonValue, ATTRIBUTE_VALUE_COLORS_NAME); @@ -1024,7 +1031,8 @@ HWTEST_F(DataPanelModifierTest, setTrackShadowTestRadiusValidValues, TestSize.Le inputValueTrackShadow = initValueTrackShadow; for (auto&& value: trackShadowRadiusValidValues) { inputValueTrackShadow.radius = std::get<1>(value); - modifier_->setTrackShadow(node_, &inputValueTrackShadow); + auto optValue = Converter::ArkValue(inputValueTrackShadow); + modifier_->setTrackShadow(node_, &optValue); jsonValue = GetJsonValue(node_); resultTrackShadow = GetAttrValue>(jsonValue, ATTRIBUTE_TRACK_SHADOW_NAME); resultStr = GetAttrValue(resultTrackShadow, ATTRIBUTE_TRACK_SHADOW_RADIUS_NAME); @@ -1057,7 +1065,8 @@ HWTEST_F(DataPanelModifierTest, DISABLED_setTrackShadowTestRadiusInvalidValues, inputValueTrackShadow = initValueTrackShadow; for (auto&& value: trackShadowRadiusInvalidValues) { inputValueTrackShadow.radius = std::get<1>(value); - modifier_->setTrackShadow(node_, &inputValueTrackShadow); + auto optValue = Converter::ArkValue(inputValueTrackShadow); + modifier_->setTrackShadow(node_, &optValue); jsonValue = GetJsonValue(node_); resultTrackShadow = GetAttrValue>(jsonValue, ATTRIBUTE_TRACK_SHADOW_NAME); resultStr = GetAttrValue(resultTrackShadow, ATTRIBUTE_TRACK_SHADOW_RADIUS_NAME); @@ -1090,7 +1099,8 @@ HWTEST_F(DataPanelModifierTest, setTrackShadowTestOffsetXValidValues, TestSize.L inputValueTrackShadow = initValueTrackShadow; for (auto&& value: trackShadowOffsetXValidValues) { inputValueTrackShadow.offsetX = std::get<1>(value); - modifier_->setTrackShadow(node_, &inputValueTrackShadow); + auto optValue = Converter::ArkValue(inputValueTrackShadow); + modifier_->setTrackShadow(node_, &optValue); jsonValue = GetJsonValue(node_); resultTrackShadow = GetAttrValue>(jsonValue, ATTRIBUTE_TRACK_SHADOW_NAME); resultStr = GetAttrValue(resultTrackShadow, ATTRIBUTE_TRACK_SHADOW_OFFSET_X_NAME); @@ -1123,7 +1133,8 @@ HWTEST_F(DataPanelModifierTest, DISABLED_setTrackShadowTestOffsetXInvalidValues, inputValueTrackShadow = initValueTrackShadow; for (auto&& value: trackShadowOffsetXInvalidValues) { inputValueTrackShadow.offsetX = std::get<1>(value); - modifier_->setTrackShadow(node_, &inputValueTrackShadow); + auto optValue = Converter::ArkValue(inputValueTrackShadow); + modifier_->setTrackShadow(node_, &optValue); jsonValue = GetJsonValue(node_); resultTrackShadow = GetAttrValue>(jsonValue, ATTRIBUTE_TRACK_SHADOW_NAME); resultStr = GetAttrValue(resultTrackShadow, ATTRIBUTE_TRACK_SHADOW_OFFSET_X_NAME); @@ -1156,7 +1167,8 @@ HWTEST_F(DataPanelModifierTest, setTrackShadowTestOffsetYValidValues, TestSize.L inputValueTrackShadow = initValueTrackShadow; for (auto&& value: trackShadowOffsetYValidValues) { inputValueTrackShadow.offsetY = std::get<1>(value); - modifier_->setTrackShadow(node_, &inputValueTrackShadow); + auto optValue = Converter::ArkValue(inputValueTrackShadow); + modifier_->setTrackShadow(node_, &optValue); jsonValue = GetJsonValue(node_); resultTrackShadow = GetAttrValue>(jsonValue, ATTRIBUTE_TRACK_SHADOW_NAME); resultStr = GetAttrValue(resultTrackShadow, ATTRIBUTE_TRACK_SHADOW_OFFSET_Y_NAME); @@ -1189,7 +1201,8 @@ HWTEST_F(DataPanelModifierTest, DISABLED_setTrackShadowTestOffsetYInvalidValues, inputValueTrackShadow = initValueTrackShadow; for (auto&& value: trackShadowOffsetYInvalidValues) { inputValueTrackShadow.offsetY = std::get<1>(value); - modifier_->setTrackShadow(node_, &inputValueTrackShadow); + auto optValue = Converter::ArkValue(inputValueTrackShadow); + modifier_->setTrackShadow(node_, &optValue); jsonValue = GetJsonValue(node_); resultTrackShadow = GetAttrValue>(jsonValue, ATTRIBUTE_TRACK_SHADOW_NAME); resultStr = GetAttrValue(resultTrackShadow, ATTRIBUTE_TRACK_SHADOW_OFFSET_Y_NAME); @@ -1229,8 +1242,8 @@ HWTEST_F(DataPanelModifierTest, setTrackShadowTestColorStringValidValues, TestSi initValueTrackShadow.offsetX = Converter::ArkValue(Ark_Empty()); initValueTrackShadow.offsetY = Converter::ArkValue(Ark_Empty()); initValueTrackShadow.colors = Converter::ArkValue(arkColorArray); - - modifier_->setTrackShadow(node_, &initValueTrackShadow); + auto optValue = Converter::ArkValue(initValueTrackShadow); + modifier_->setTrackShadow(node_, &optValue); jsonValue = GetJsonValue(node_); jsonTrackShadow = GetAttrValue>(jsonValue, ATTRIBUTE_TRACK_SHADOW_NAME); @@ -1285,8 +1298,8 @@ HWTEST_F(DataPanelModifierTest, setTrackShadowTestColorStringInvalidValues, Test initValueTrackShadow.offsetX = Converter::ArkValue(Ark_Empty()); initValueTrackShadow.offsetY = Converter::ArkValue(Ark_Empty()); initValueTrackShadow.colors = Converter::ArkValue(arkColorArray); - - modifier_->setTrackShadow(node_, &initValueTrackShadow); + auto optValue = Converter::ArkValue(initValueTrackShadow); + modifier_->setTrackShadow(node_, &optValue); jsonValue = GetJsonValue(node_); jsonTrackShadow = GetAttrValue>(jsonValue, ATTRIBUTE_TRACK_SHADOW_NAME); @@ -1341,8 +1354,8 @@ HWTEST_F(DataPanelModifierTest, setTrackShadowTestColorNumberValidValues, TestSi initValueTrackShadow.offsetX = Converter::ArkValue(Ark_Empty()); initValueTrackShadow.offsetY = Converter::ArkValue(Ark_Empty()); initValueTrackShadow.colors = Converter::ArkValue(arkColorArray); - - modifier_->setTrackShadow(node_, &initValueTrackShadow); + auto optValue = Converter::ArkValue(initValueTrackShadow); + modifier_->setTrackShadow(node_, &optValue); jsonValue = GetJsonValue(node_); jsonTrackShadow = GetAttrValue>(jsonValue, ATTRIBUTE_TRACK_SHADOW_NAME); @@ -1397,8 +1410,8 @@ HWTEST_F(DataPanelModifierTest, setTrackShadowTestColorEnumValidValues, TestSize initValueTrackShadow.offsetX = Converter::ArkValue(Ark_Empty()); initValueTrackShadow.offsetY = Converter::ArkValue(Ark_Empty()); initValueTrackShadow.colors = Converter::ArkValue(arkColorArray); - - modifier_->setTrackShadow(node_, &initValueTrackShadow); + auto optValue = Converter::ArkValue(initValueTrackShadow); + modifier_->setTrackShadow(node_, &optValue); jsonValue = GetJsonValue(node_); jsonTrackShadow = GetAttrValue>(jsonValue, ATTRIBUTE_TRACK_SHADOW_NAME); @@ -1453,8 +1466,8 @@ HWTEST_F(DataPanelModifierTest, setTrackShadowTestColorEnumInvalidValues, TestSi initValueTrackShadow.offsetX = Converter::ArkValue(Ark_Empty()); initValueTrackShadow.offsetY = Converter::ArkValue(Ark_Empty()); initValueTrackShadow.colors = Converter::ArkValue(arkColorArray); - - modifier_->setTrackShadow(node_, &initValueTrackShadow); + auto optValue = Converter::ArkValue(initValueTrackShadow); + modifier_->setTrackShadow(node_, &optValue); jsonValue = GetJsonValue(node_); jsonTrackShadow = GetAttrValue>(jsonValue, ATTRIBUTE_TRACK_SHADOW_NAME); @@ -1509,8 +1522,8 @@ HWTEST_F(DataPanelModifierTest, setTrackShadowTestColorResourceValidValues, Test initValueTrackShadow.offsetX = Converter::ArkValue(Ark_Empty()); initValueTrackShadow.offsetY = Converter::ArkValue(Ark_Empty()); initValueTrackShadow.colors = Converter::ArkValue(arkColorArray); - - modifier_->setTrackShadow(node_, &initValueTrackShadow); + auto optValue = Converter::ArkValue(initValueTrackShadow); + modifier_->setTrackShadow(node_, &optValue); jsonValue = GetJsonValue(node_); jsonTrackShadow = GetAttrValue>(jsonValue, ATTRIBUTE_TRACK_SHADOW_NAME); @@ -1565,8 +1578,8 @@ HWTEST_F(DataPanelModifierTest, setTrackShadowTestColorLinearGradientValidValues initValueTrackShadow.offsetX = Converter::ArkValue(Ark_Empty()); initValueTrackShadow.offsetY = Converter::ArkValue(Ark_Empty()); initValueTrackShadow.colors = Converter::ArkValue(arkColorArray); - - modifier_->setTrackShadow(node_, &initValueTrackShadow); + auto optValue = Converter::ArkValue(initValueTrackShadow); + modifier_->setTrackShadow(node_, &optValue); jsonValue = GetJsonValue(node_); jsonTrackShadow = GetAttrValue>(jsonValue, ATTRIBUTE_TRACK_SHADOW_NAME); @@ -1617,8 +1630,8 @@ HWTEST_F(DataPanelModifierTest, setTrackShadowTestColorLinearGradientInvalidValu initValueTrackShadow.offsetX = Converter::ArkValue(Ark_Empty()); initValueTrackShadow.offsetY = Converter::ArkValue(Ark_Empty()); initValueTrackShadow.colors = Converter::ArkValue(arkColorArray); - - modifier_->setTrackShadow(node_, &initValueTrackShadow); + auto optValue = Converter::ArkValue(initValueTrackShadow); + modifier_->setTrackShadow(node_, &optValue); jsonValue = GetJsonValue(node_); jsonTrackShadow = GetAttrValue>(jsonValue, ATTRIBUTE_TRACK_SHADOW_NAME); diff --git a/test/unittest/capi/modifiers/date_picker_modifier_test.cpp b/test/unittest/capi/modifiers/date_picker_modifier_test.cpp index 23f6315e82504cc050095fa90431344bb3c86a78..e5c457ba53372bbc1d67464cc466a763f97eb659 100644 --- a/test/unittest/capi/modifiers/date_picker_modifier_test.cpp +++ b/test/unittest/capi/modifiers/date_picker_modifier_test.cpp @@ -316,8 +316,8 @@ HWTEST_F(DatePickerModifierTest, setLunarTest, TestSize.Level1) EXPECT_EQ(initialValue, ATTRIBUTE_LUNAR_DEFAULT_VALUE); for (auto lunar : BOOL_TEST_PLAN) { - Ark_Boolean inputValue = Converter::ArkValue(lunar.first); - modifier_->setLunar0(node_, inputValue); + auto inputValue = Converter::ArkValue(lunar.first); + modifier_->setLunar0(node_, &inputValue); auto checkValue = GetAttrValue(node_, ATTRIBUTE_LUNAR_NAME); EXPECT_EQ(checkValue, lunar.second); } @@ -372,7 +372,8 @@ HWTEST_F(DatePickerModifierTest, setDisappearTexFontStyle, TestSize.Level1) for (auto style : FONT_STYLE_TEST_PLAN) { font.style = style.first; pickerStyle.font.value = font; - modifier_->setDisappearTextStyle0(node_, &pickerStyle); + auto optValue = Converter::ArkValue(pickerStyle); + modifier_->setDisappearTextStyle0(node_, &optValue); auto fullJson = GetJsonValue(node_); auto styleObject = GetAttrValue>(fullJson, ATTRIBUTE_DISAPPEAR_TEXT_STYLE_NAME); auto fontObject = GetAttrValue(styleObject, ATTRIBUTE_FONT_NAME); @@ -412,7 +413,8 @@ HWTEST_F(DatePickerModifierTest, setDisappearTexFontWeight, TestSize.Level1) for (auto weight : FONT_WEIGHT_TEST_PLAN) { font.weight = weight.first; pickerStyle.font.value = font; - modifier_->setDisappearTextStyle0(node_, &pickerStyle); + auto optValue = Converter::ArkValue(pickerStyle); + modifier_->setDisappearTextStyle0(node_, &optValue); auto fullJson = GetJsonValue(node_); auto styleObject = GetAttrValue>(fullJson, ATTRIBUTE_DISAPPEAR_TEXT_STYLE_NAME); auto fontObject = GetAttrValue(styleObject, ATTRIBUTE_FONT_NAME); @@ -429,7 +431,8 @@ HWTEST_F(DatePickerModifierTest, setDisappearTexFontWeight, TestSize.Level1) for (auto weight : FONT_WEIGHT_TEST_PLAN2) { font.weight = weight.first; pickerStyle.font.value = font; - modifier_->setDisappearTextStyle0(node_, &pickerStyle); + auto optValue = Converter::ArkValue(pickerStyle); + modifier_->setDisappearTextStyle0(node_, &optValue); auto fullJson = GetJsonValue(node_); auto styleObject = GetAttrValue>(fullJson, ATTRIBUTE_DISAPPEAR_TEXT_STYLE_NAME); auto fontObject = GetAttrValue(styleObject, ATTRIBUTE_FONT_NAME); @@ -469,7 +472,8 @@ HWTEST_F(DatePickerModifierTest, setDisappearTextFontFamily, TestSize.Level1) for (auto family : FONT_FAMILY_TEST_PLAN) { font.family = family.first; pickerStyle.font.value = font; - modifier_->setDisappearTextStyle0(node_, &pickerStyle); + auto optValue = Converter::ArkValue(pickerStyle); + modifier_->setDisappearTextStyle0(node_, &optValue); auto fullJson = GetJsonValue(node_); auto styleObject = GetAttrValue>(fullJson, ATTRIBUTE_DISAPPEAR_TEXT_STYLE_NAME); auto fontObject = GetAttrValue(styleObject, ATTRIBUTE_FONT_NAME); @@ -509,7 +513,8 @@ HWTEST_F(DatePickerModifierTest, setDisappearTextFontSize, TestSize.Level1) for (auto size : FONT_SIZE_TEST_PLAN) { font.size = size.first; pickerStyle.font.value = font; - modifier_->setDisappearTextStyle0(node_, &pickerStyle); + auto optValue = Converter::ArkValue(pickerStyle); + modifier_->setDisappearTextStyle0(node_, &optValue); auto fullJson = GetJsonValue(node_); auto styleObject = GetAttrValue>(fullJson, ATTRIBUTE_DISAPPEAR_TEXT_STYLE_NAME); auto fontObject = GetAttrValue(styleObject, ATTRIBUTE_FONT_NAME); @@ -540,7 +545,8 @@ HWTEST_F(DatePickerModifierTest, setDisappearTextColor, TestSize.Level1) for (const auto& [value, expectVal] : COLOR_TEST_PLAN) { pickerStyle.color = value; - modifier_->setDisappearTextStyle0(node_, &pickerStyle); + auto optValue = Converter::ArkValue(pickerStyle); + modifier_->setDisappearTextStyle0(node_, &optValue); auto fullJson = GetJsonValue(node_); auto styleObject = GetAttrValue>(fullJson, ATTRIBUTE_DISAPPEAR_TEXT_STYLE_NAME); checkVal = GetAttrValue(styleObject, ATTRIBUTE_COLOR_NAME); @@ -597,7 +603,8 @@ HWTEST_F(DatePickerModifierTest, setTexFontStyle, TestSize.Level1) for (auto style : FONT_STYLE_TEST_PLAN) { font.style = style.first; pickerStyle.font.value = font; - modifier_->setTextStyle0(node_, &pickerStyle); + auto optValue = Converter::ArkValue(pickerStyle); + modifier_->setTextStyle0(node_, &optValue); auto fullJson = GetJsonValue(node_); auto styleObject = GetAttrValue>(fullJson, ATTRIBUTE_TEXT_STYLE_NAME); auto fontObject = GetAttrValue(styleObject, ATTRIBUTE_FONT_NAME); @@ -637,7 +644,8 @@ HWTEST_F(DatePickerModifierTest, setTexFontWeight, TestSize.Level1) for (auto weight : FONT_WEIGHT_TEST_PLAN) { font.weight = weight.first; pickerStyle.font.value = font; - modifier_->setTextStyle0(node_, &pickerStyle); + auto optValue = Converter::ArkValue(pickerStyle); + modifier_->setTextStyle0(node_, &optValue); auto fullJson = GetJsonValue(node_); auto styleObject = GetAttrValue>(fullJson, ATTRIBUTE_TEXT_STYLE_NAME); auto fontObject = GetAttrValue(styleObject, ATTRIBUTE_FONT_NAME); @@ -654,7 +662,8 @@ HWTEST_F(DatePickerModifierTest, setTexFontWeight, TestSize.Level1) for (auto weight : FONT_WEIGHT_TEST_PLAN2) { font.weight = weight.first; pickerStyle.font.value = font; - modifier_->setTextStyle0(node_, &pickerStyle); + auto optValue = Converter::ArkValue(pickerStyle); + modifier_->setTextStyle0(node_, &optValue); auto fullJson = GetJsonValue(node_); auto styleObject = GetAttrValue>(fullJson, ATTRIBUTE_TEXT_STYLE_NAME); auto fontObject = GetAttrValue(styleObject, ATTRIBUTE_FONT_NAME); @@ -694,7 +703,8 @@ HWTEST_F(DatePickerModifierTest, setTextFontFamily, TestSize.Level1) for (auto family : FONT_FAMILY_TEST_PLAN) { font.family = family.first; pickerStyle.font.value = font; - modifier_->setTextStyle0(node_, &pickerStyle); + auto optValue = Converter::ArkValue(pickerStyle); + modifier_->setTextStyle0(node_, &optValue); auto fullJson = GetJsonValue(node_); auto styleObject = GetAttrValue>(fullJson, ATTRIBUTE_TEXT_STYLE_NAME); auto fontObject = GetAttrValue(styleObject, ATTRIBUTE_FONT_NAME); @@ -734,7 +744,8 @@ HWTEST_F(DatePickerModifierTest, setTextFontSize, TestSize.Level1) for (auto size : FONT_SIZE_TEST_PLAN) { font.size = size.first; pickerStyle.font.value = font; - modifier_->setTextStyle0(node_, &pickerStyle); + auto optValue = Converter::ArkValue(pickerStyle); + modifier_->setTextStyle0(node_, &optValue); auto fullJson = GetJsonValue(node_); auto styleObject = GetAttrValue>(fullJson, ATTRIBUTE_TEXT_STYLE_NAME); auto fontObject = GetAttrValue(styleObject, ATTRIBUTE_FONT_NAME); @@ -765,7 +776,8 @@ HWTEST_F(DatePickerModifierTest, setTextColor, TestSize.Level1) for (const auto& [value, expectVal] : COLOR_TEST_PLAN) { pickerStyle.color = value; - modifier_->setTextStyle0(node_, &pickerStyle); + auto optValue = Converter::ArkValue(pickerStyle); + modifier_->setTextStyle0(node_, &optValue); auto fullJson = GetJsonValue(node_); auto styleObject = GetAttrValue>(fullJson, ATTRIBUTE_TEXT_STYLE_NAME); checkVal = GetAttrValue(styleObject, ATTRIBUTE_COLOR_NAME); @@ -822,7 +834,8 @@ HWTEST_F(DatePickerModifierTest, setSelectedTexFontStyle, TestSize.Level1) for (auto style : FONT_STYLE_TEST_PLAN) { font.style = style.first; pickerStyle.font.value = font; - modifier_->setSelectedTextStyle0(node_, &pickerStyle); + auto optValue = Converter::ArkValue(pickerStyle); + modifier_->setSelectedTextStyle0(node_, &optValue); auto fullJson = GetJsonValue(node_); auto styleObject = GetAttrValue>(fullJson, ATTRIBUTE_SELECTED_TEXT_STYLE_NAME); auto fontObject = GetAttrValue(styleObject, ATTRIBUTE_FONT_NAME); @@ -862,7 +875,8 @@ HWTEST_F(DatePickerModifierTest, setSelectedTexFontWeight, TestSize.Level1) for (auto weight : FONT_WEIGHT_TEST_PLAN) { font.weight = weight.first; pickerStyle.font.value = font; - modifier_->setSelectedTextStyle0(node_, &pickerStyle); + auto optValue = Converter::ArkValue(pickerStyle); + modifier_->setSelectedTextStyle0(node_, &optValue); auto fullJson = GetJsonValue(node_); auto styleObject = GetAttrValue>(fullJson, ATTRIBUTE_SELECTED_TEXT_STYLE_NAME); auto fontObject = GetAttrValue(styleObject, ATTRIBUTE_FONT_NAME); @@ -879,7 +893,8 @@ HWTEST_F(DatePickerModifierTest, setSelectedTexFontWeight, TestSize.Level1) for (auto weight : FONT_WEIGHT_TEST_PLAN2) { font.weight = weight.first; pickerStyle.font.value = font; - modifier_->setSelectedTextStyle0(node_, &pickerStyle); + auto optValue = Converter::ArkValue(pickerStyle); + modifier_->setSelectedTextStyle0(node_, &optValue); auto fullJson = GetJsonValue(node_); auto styleObject = GetAttrValue>(fullJson, ATTRIBUTE_SELECTED_TEXT_STYLE_NAME); auto fontObject = GetAttrValue(styleObject, ATTRIBUTE_FONT_NAME); @@ -919,7 +934,8 @@ HWTEST_F(DatePickerModifierTest, setSelectedTextFontFamily, TestSize.Level1) for (auto family : FONT_FAMILY_TEST_PLAN) { font.family = family.first; pickerStyle.font.value = font; - modifier_->setSelectedTextStyle0(node_, &pickerStyle); + auto optValue = Converter::ArkValue(pickerStyle); + modifier_->setSelectedTextStyle0(node_, &optValue); auto fullJson = GetJsonValue(node_); auto styleObject = GetAttrValue>(fullJson, ATTRIBUTE_SELECTED_TEXT_STYLE_NAME); auto fontObject = GetAttrValue(styleObject, ATTRIBUTE_FONT_NAME); @@ -959,7 +975,8 @@ HWTEST_F(DatePickerModifierTest, setSelectedTextFontSize, TestSize.Level1) for (auto size : FONT_SIZE_TEST_PLAN) { font.size = size.first; pickerStyle.font.value = font; - modifier_->setSelectedTextStyle0(node_, &pickerStyle); + auto optValue = Converter::ArkValue(pickerStyle); + modifier_->setSelectedTextStyle0(node_, &optValue); auto fullJson = GetJsonValue(node_); auto styleObject = GetAttrValue>(fullJson, ATTRIBUTE_SELECTED_TEXT_STYLE_NAME); auto fontObject = GetAttrValue(styleObject, ATTRIBUTE_FONT_NAME); @@ -990,7 +1007,8 @@ HWTEST_F(DatePickerModifierTest, setSelectedTextColor, TestSize.Level1) for (const auto& [value, expectVal] : COLOR_TEST_PLAN) { pickerStyle.color = value; - modifier_->setSelectedTextStyle0(node_, &pickerStyle); + auto optValue = Converter::ArkValue(pickerStyle); + modifier_->setSelectedTextStyle0(node_, &optValue); auto fullJson = GetJsonValue(node_); auto styleObject = GetAttrValue>(fullJson, ATTRIBUTE_SELECTED_TEXT_STYLE_NAME); checkVal = GetAttrValue(styleObject, ATTRIBUTE_COLOR_NAME); @@ -1033,7 +1051,8 @@ HWTEST_F(DatePickerModifierTest, setOnChangeTest, TestSize.Level1) }; }; auto arkCallback = Converter::ArkValue(onChange, frameNode->GetId()); - modifier_->setOnChange(node_, &arkCallback); + auto optCallback = Converter::ArkValue(arkCallback); + modifier_->setOnChange(node_, &optCallback); for (const auto& testValue : CHANGE_EVENT_TEST_PLAN) { DatePickerChangeEvent event(testValue.first.ToString(true)); @@ -1123,8 +1142,9 @@ HWTEST_F(DatePickerModifierTest, setOnDateChangeTest, TestSize.Level1) }, .call = onDateChange }; + auto optFunc = Converter::ArkValue(func); - modifier_->setOnDateChange0(node_, &func); + modifier_->setOnDateChange0(node_, &optFunc); for (const auto& testValue : CHANGE_EVENT_TEST_PLAN) { DatePickerChangeEvent event(testValue.first.ToString(true)); diff --git a/test/unittest/capi/modifiers/divider_modifier_test.cpp b/test/unittest/capi/modifiers/divider_modifier_test.cpp index b44364c4d55c2d1f324408c831422fcd83d33140..83c0de0f7ca3966322d703f8b0e7523c497ef7e3 100644 --- a/test/unittest/capi/modifiers/divider_modifier_test.cpp +++ b/test/unittest/capi/modifiers/divider_modifier_test.cpp @@ -42,68 +42,50 @@ public: } }; +using OneColorStep = std::tuple; +const std::vector COLOR_TEST_PLAN = { + { Converter::ArkUnion(ARK_COLOR_WHITE), "#FFFFFFFF" }, + { Converter::ArkUnion(Converter::ArkValue(0x123456)), "#FF123456" }, + { Converter::ArkUnion(Converter::ArkValue(0.5f)), "#00000000" }, + { Converter::ArkUnion(Converter::ArkValue("#11223344")), "#11223344" }, + { Converter::ArkUnion(Converter::ArkValue("65535")), "#FF00FFFF" }, + { CreateResourceUnion(NamedResourceId{"aa.bb.cc", Converter::ResourceType::COLOR}), + "#FFFF0000" }, + { CreateResourceUnion(IntResourceId{1234, Converter::ResourceType::COLOR}), "#FFFF0000" }, + { CreateResourceUnion(NamedResourceId{"incorrect_color", Converter::ResourceType::STRING}), + "#FFFF0000" }, +}; + /** * @tc.name: DividerModifierTest001 * @tc.desc: Check the functionality of DividerModifier.setColor * @tc.type: FUNC */ -HWTEST_F(DividerModifierTest, DISABLED_DividerModifierTest001, TestSize.Level1) +HWTEST_F(DividerModifierTest, DividerModifierTest001, TestSize.Level1) { static const std::string PROP_NAME("color"); ASSERT_NE(modifier_->setColor, nullptr); - auto checkVal1 = GetStringAttribute(node_, PROP_NAME); - EXPECT_EQ(checkVal1, "#FF000000"); - Ark_ResourceColor color = Converter::ArkUnion(ARK_COLOR_WHITE); - modifier_->setColor(node_, &color); - auto checkVal2 = GetStringAttribute(node_, PROP_NAME); - EXPECT_EQ(checkVal2, "#FFFFFFFF"); - - Ark_ResourceColor numberInt = { .selector = 1, .value1 = Converter::ArkValue(0x123456) }; - modifier_->setColor(node_, &numberInt); - auto checkVal3 = GetStringAttribute(node_, PROP_NAME); - EXPECT_EQ(checkVal3, "#FF123456"); - - Ark_ResourceColor numberFlt = { .selector = 1, .value1 = Converter::ArkValue(0.5f) }; - modifier_->setColor(node_, &numberFlt); - auto checkVal4 = GetStringAttribute(node_, PROP_NAME); - EXPECT_EQ(checkVal4, "#00000000"); - - Ark_ResourceColor strColor = { .selector = 2, .value2 = Converter::ArkValue("#11223344") }; - modifier_->setColor(node_, &strColor); - auto checkVal5 = GetStringAttribute(node_, PROP_NAME); - EXPECT_EQ(checkVal5, "#11223344"); + auto checkVal = GetStringAttribute(node_, PROP_NAME); + EXPECT_EQ(checkVal, "#FF000000"); - Ark_ResourceColor strNumber = { .selector = 2, .value2 = Converter::ArkValue("65535") }; - modifier_->setColor(node_, &strNumber); - auto checkVal6 = GetStringAttribute(node_, PROP_NAME); - EXPECT_EQ(checkVal6, "#FF00FFFF"); + for (const auto& [value, expectVal] : COLOR_TEST_PLAN) { + auto color = Converter::ArkValue(value); + modifier_->setColor(node_, &color); + auto fullJson = GetJsonValue(node_); + checkVal = GetStringAttribute(node_, PROP_NAME); + EXPECT_EQ(checkVal, expectVal); + } - auto resNameColor = CreateResourceUnion( - NamedResourceId{"aa.bb.cc", Converter::ResourceType::COLOR}); - modifier_->setColor(node_, &resNameColor); - auto checkVal7 = GetStringAttribute(node_, PROP_NAME); - EXPECT_EQ(checkVal7, "#FFFF0000"); // Color::RED is result of mocked ThemeConstants::GetColorByName - - auto resIdColor = CreateResourceUnion(IntResourceId{1234, Converter::ResourceType::COLOR}); - modifier_->setColor(node_, &resIdColor); - auto checkVal8 = GetStringAttribute(node_, PROP_NAME); - EXPECT_EQ(checkVal8, "#FFFF0000"); // Color::RED is result of mocked ThemeConstants::GetColor(int) - - resNameColor = CreateResourceUnion( - NamedResourceId{"incorrect_color", Converter::ResourceType::STRING}); - modifier_->setColor(node_, &resNameColor); - auto checkVal9 = GetStringAttribute(node_, PROP_NAME); - EXPECT_EQ(checkVal9, "#FF000000"); // Should be Color::RED, but converter from Resource works incorrect now. - // So modifier pass Color::BLACK to divider component int this case - - strNumber = { .selector = 2, .value2 = Converter::ArkValue("incorrect_color") }; - modifier_->setColor(node_, &strNumber); + Ark_ResourceColor strNumber = { .selector = 2, .value2 = Converter::ArkValue("incorrect_color") }; + auto optValue = Converter::ArkValue(strNumber); + modifier_->setColor(node_, &optValue); auto checkVal10 = GetStringAttribute(node_, PROP_NAME); EXPECT_EQ(checkVal10, "#FF000000"); strNumber = { .selector = 2, .value2 = Converter::ArkValue("") }; - modifier_->setColor(node_, &strNumber); + optValue = Converter::ArkValue(strNumber); + modifier_->setColor(node_, &optValue); auto checkVal11 = GetStringAttribute(node_, PROP_NAME); EXPECT_EQ(checkVal11, "#FF000000"); } @@ -121,27 +103,28 @@ HWTEST_F(DividerModifierTest, DividerModifierTest002, TestSize.Level1) auto checkVal1 = GetStringAttribute(node_, PROP_NAME); EXPECT_EQ(checkVal1, "true"); - modifier_->setVertical(node_, false); + auto optValue = Converter::ArkValue(false); + modifier_->setVertical(node_, &optValue); auto checkVal2 = GetStringAttribute(node_, PROP_NAME); EXPECT_EQ(checkVal2, "false"); - modifier_->setVertical(node_, true); + optValue = Converter::ArkValue(true); + modifier_->setVertical(node_, &optValue); auto checkVal3 = GetStringAttribute(node_, PROP_NAME); EXPECT_EQ(checkVal3, "true"); - modifier_->setVertical(node_, 0); - auto checkVal4 = GetStringAttribute(node_, PROP_NAME); - EXPECT_EQ(checkVal4, "false"); - - modifier_->setVertical(node_, -25); + optValue = Converter::ArkValue(-25); + modifier_->setVertical(node_, &optValue); auto checkVal5 = GetStringAttribute(node_, PROP_NAME); EXPECT_EQ(checkVal5, "true"); - modifier_->setVertical(node_, 0); + optValue = Converter::ArkValue(0); + modifier_->setVertical(node_, &optValue); auto checkVal6 = GetStringAttribute(node_, PROP_NAME); EXPECT_EQ(checkVal6, "false"); - modifier_->setVertical(node_, 25); + optValue = Converter::ArkValue(25); + modifier_->setVertical(node_, &optValue); auto checkVal7 = GetStringAttribute(node_, PROP_NAME); EXPECT_EQ(checkVal7, "true"); } @@ -151,27 +134,31 @@ HWTEST_F(DividerModifierTest, DividerModifierTest002, TestSize.Level1) * @tc.desc: Check the functionality of DividerModifier.setLineCap * @tc.type: FUNC */ -HWTEST_F(DividerModifierTest, DISABLED_DividerModifierTest003, TestSize.Level1) +HWTEST_F(DividerModifierTest, DividerModifierTest003, TestSize.Level1) { static const std::string PROP_NAME("lineCap"); - ASSERT_NE(modifier_->setVertical, nullptr); + ASSERT_NE(modifier_->setLineCap, nullptr); auto checkVal1 = GetStringAttribute(node_, PROP_NAME); EXPECT_EQ(checkVal1, LINE_CUP_BUTT); - modifier_->setLineCap(node_, ARK_LINE_CAP_STYLE_SQUARE); + auto optValue = Converter::ArkValue(ARK_LINE_CAP_STYLE_SQUARE); + modifier_->setLineCap(node_, &optValue); auto checkVal2 = GetStringAttribute(node_, PROP_NAME); EXPECT_EQ(checkVal2, LINE_CUP_SQUARE); - modifier_->setLineCap(node_, ARK_LINE_CAP_STYLE_BUTT); + optValue = Converter::ArkValue(ARK_LINE_CAP_STYLE_BUTT); + modifier_->setLineCap(node_, &optValue); auto checkVal3 = GetStringAttribute(node_, PROP_NAME); EXPECT_EQ(checkVal3, LINE_CUP_BUTT); - modifier_->setLineCap(node_, ARK_LINE_CAP_STYLE_ROUND); + optValue = Converter::ArkValue(ARK_LINE_CAP_STYLE_ROUND); + modifier_->setLineCap(node_, &optValue); auto checkVal4 = GetStringAttribute(node_, PROP_NAME); EXPECT_EQ(checkVal4, LINE_CUP_ROUND); - modifier_->setLineCap(node_, static_cast(-25)); + optValue = Converter::ArkValue(static_cast(-25)); + modifier_->setLineCap(node_, &optValue); auto checkVal5 = GetStringAttribute(node_, PROP_NAME); EXPECT_EQ(checkVal5, LINE_CUP_BUTT); } @@ -190,31 +177,51 @@ HWTEST_F(DividerModifierTest, DividerModifierTest004, TestSize.Level1) EXPECT_EQ(checkVal1, DEFAULT_STROKE_WIDTH); Ark_Union_Number_String intVal = { .selector = 0, .value0 = Converter::ArkValue(123) }; - modifier_->setStrokeWidth(node_, &intVal); + auto optValue = Converter::ArkValue(intVal); + modifier_->setStrokeWidth(node_, &optValue); auto checkVal2 = GetStringAttribute(node_, PROP_NAME); EXPECT_EQ(checkVal2, "123.00vp"); Ark_Union_Number_String floatVal = { .selector = 0, .value0 = Converter::ArkValue(1.23f) }; - modifier_->setStrokeWidth(node_, &floatVal); + optValue = Converter::ArkValue(floatVal); + modifier_->setStrokeWidth(node_, &optValue); auto checkVal3 = GetStringAttribute(node_, PROP_NAME); EXPECT_EQ(checkVal3, "1.23vp"); Ark_Union_Number_String pxVal = { .selector = 1, .value1 = Converter::ArkValue("45px") }; - modifier_->setStrokeWidth(node_, &pxVal); + optValue = Converter::ArkValue(pxVal); + modifier_->setStrokeWidth(node_, &optValue); auto checkVal4 = GetStringAttribute(node_, PROP_NAME); EXPECT_EQ(checkVal4, "45.00px"); Ark_Union_Number_String vpVal = { .selector = 1, .value1 = Converter::ArkValue("5.6vp") }; - modifier_->setStrokeWidth(node_, &vpVal); + optValue = Converter::ArkValue(vpVal); + modifier_->setStrokeWidth(node_, &optValue); auto checkVal5 = GetStringAttribute(node_, PROP_NAME); EXPECT_EQ(checkVal5, "5.60vp"); Ark_Union_Number_String percentVal = { .selector = 1, .value1 = Converter::ArkValue("10%") }; - modifier_->setStrokeWidth(node_, &percentVal); + optValue = Converter::ArkValue(percentVal); + modifier_->setStrokeWidth(node_, &optValue); auto checkVal6 = GetStringAttribute(node_, PROP_NAME); EXPECT_EQ(checkVal6, DEFAULT_STROKE_WIDTH); } +using OneWidthStep = std::tuple; +const std::vector WIDTH_TEST_PLAN = { + { Converter::ArkUnion(Converter::ArkValue(-123)), "-123.00vp" }, + { Converter::ArkUnion(Converter::ArkValue(-1.23f)), "-1.23vp" }, + { Converter::ArkUnion(Converter::ArkValue("-4.5px")), "-4.50px" }, + { Converter::ArkUnion(Converter::ArkValue("-56vp")), "-56.00vp" }, + { Converter::ArkUnion(Converter::ArkValue("undefVal")), + "0.00fp" }, + { Converter::ArkUnion(Converter::ArkValue("-10%")), + DEFAULT_STROKE_WIDTH }, + { Converter::ArkUnion(Converter::ArkValue("")), "0.00fp" }, + { Converter::ArkUnion(Converter::ArkValue("qw111vp")), "0.00vp" }, + { Converter::ArkUnion(Converter::ArkValue("qw111")), "0.00fp" }, +}; + /** * @tc.name: DividerModifierTest005 * @tc.desc: Check the functionality of DividerModifier.setStrokeWidth for negative values @@ -225,52 +232,15 @@ HWTEST_F(DividerModifierTest, DividerModifierTest005, TestSize.Level1) static const std::string PROP_NAME("strokeWidth"); ASSERT_NE(modifier_->setStrokeWidth, nullptr); - auto checkVal1 = GetStringAttribute(node_, PROP_NAME); - EXPECT_EQ(checkVal1, DEFAULT_STROKE_WIDTH); - - Ark_Union_Number_String intNegVal = { .selector = 0, .value0 = Converter::ArkValue(-123) }; - modifier_->setStrokeWidth(node_, &intNegVal); - auto checkVal2 = GetStringAttribute(node_, PROP_NAME); - EXPECT_EQ(checkVal2, "-123.00vp"); - - Ark_Union_Number_String floatNegVal = { .selector = 0, .value0 = Converter::ArkValue(-1.23f) }; - modifier_->setStrokeWidth(node_, &floatNegVal); - auto checkVal3 = GetStringAttribute(node_, PROP_NAME); - EXPECT_EQ(checkVal3, "-1.23vp"); - - Ark_Union_Number_String pxNegVal = { .selector = 1, .value1 = Converter::ArkValue("-4.5px") }; - modifier_->setStrokeWidth(node_, &pxNegVal); - auto checkVal4 = GetStringAttribute(node_, PROP_NAME); - EXPECT_EQ(checkVal4, "-4.50px"); - - Ark_Union_Number_String vpNegVal = { .selector = 1, .value1 = Converter::ArkValue("-56vp") }; - modifier_->setStrokeWidth(node_, &vpNegVal); - auto checkVal5 = GetStringAttribute(node_, PROP_NAME); - EXPECT_EQ(checkVal5, "-56.00vp"); - - Ark_Union_Number_String undefVal = { .selector = 1, .value1 = Converter::ArkValue("undefVal") }; - modifier_->setStrokeWidth(node_, &undefVal); - auto checkVal6 = GetStringAttribute(node_, PROP_NAME); - EXPECT_EQ(checkVal6, "0.00fp"); - - Ark_Union_Number_String percentNegVal = { .selector = 1, .value1 = Converter::ArkValue("-10%") }; - modifier_->setStrokeWidth(node_, &percentNegVal); - auto checkVal7 = GetStringAttribute(node_, PROP_NAME); - EXPECT_EQ(checkVal7, DEFAULT_STROKE_WIDTH); + auto checkVal = GetStringAttribute(node_, PROP_NAME); + EXPECT_EQ(checkVal, DEFAULT_STROKE_WIDTH); - Ark_Union_Number_String emptyVal = { .selector = 1, .value1 = Converter::ArkValue("") }; - modifier_->setStrokeWidth(node_, &emptyVal); - auto checkVal8 = GetStringAttribute(node_, PROP_NAME); - EXPECT_EQ(checkVal8, "0.00fp"); - - Ark_Union_Number_String invalidVal = { .selector = 1, .value1 = Converter::ArkValue("qw111vp") }; - modifier_->setStrokeWidth(node_, &invalidVal); - auto checkVal9 = GetStringAttribute(node_, PROP_NAME); - EXPECT_EQ(checkVal9, "0.00vp"); - - Ark_Union_Number_String invalidVal2 = { .selector = 1, .value1 = Converter::ArkValue("qw111") }; - modifier_->setStrokeWidth(node_, &invalidVal2); - auto checkVal10 = GetStringAttribute(node_, PROP_NAME); - EXPECT_EQ(checkVal10, "0.00fp"); + for (const auto& [value, expectVal] : WIDTH_TEST_PLAN) { + auto width = Converter::ArkValue(value); + modifier_->setStrokeWidth(node_, &width); + auto fullJson = GetJsonValue(node_); + checkVal = GetStringAttribute(node_, PROP_NAME); + EXPECT_EQ(checkVal, expectVal); + } } } // namespace OHOS::Ace::NG diff --git a/test/unittest/capi/modifiers/flex_modifier_test.cpp b/test/unittest/capi/modifiers/flex_modifier_test.cpp index 89e3a71aa09dad121d43bd39c4a5252b897054d7..5fbe52f353cc59b79944201c622f4a971f6b0bac 100755 --- a/test/unittest/capi/modifiers/flex_modifier_test.cpp +++ b/test/unittest/capi/modifiers/flex_modifier_test.cpp @@ -269,7 +269,8 @@ HWTEST_F(FlexModifierTest, setPointLightTestPointLightLightSourcePositionXValidV Ark_PointLightStyle inputValuePointLight = initValuePointLight; WriteTo(inputValuePointLight.lightSource).positionX = value; - modifier_->setPointLight(node_, &inputValuePointLight); + auto optValue = Converter::ArkValue(inputValuePointLight); + modifier_->setPointLight(node_, &optValue); auto jsonValue = GetJsonValue(node_); auto resultLightSource = GetAttrValue>(jsonValue, ATTRIBUTE_POINT_LIGHT_I_LIGHT_SOURCE_I_POSITION_NAME); @@ -312,7 +313,8 @@ HWTEST_F(FlexModifierTest, setPointLightTestPointLightLightSourcePositionYValidV Ark_PointLightStyle inputValuePointLight = initValuePointLight; WriteTo(inputValuePointLight.lightSource).positionY = value; - modifier_->setPointLight(node_, &inputValuePointLight); + auto optValue = Converter::ArkValue(inputValuePointLight); + modifier_->setPointLight(node_, &optValue); auto jsonValue = GetJsonValue(node_); auto resultLightSource = GetAttrValue>(jsonValue, ATTRIBUTE_POINT_LIGHT_I_LIGHT_SOURCE_I_POSITION_NAME); @@ -355,7 +357,8 @@ HWTEST_F(FlexModifierTest, setPointLightTestPointLightLightSourcePositionZValidV Ark_PointLightStyle inputValuePointLight = initValuePointLight; WriteTo(inputValuePointLight.lightSource).positionZ = value; - modifier_->setPointLight(node_, &inputValuePointLight); + auto optValue = Converter::ArkValue(inputValuePointLight); + modifier_->setPointLight(node_, &optValue); auto jsonValue = GetJsonValue(node_); auto resultLightSource = GetAttrValue>(jsonValue, ATTRIBUTE_POINT_LIGHT_I_LIGHT_SOURCE_I_POSITION_NAME); @@ -398,7 +401,8 @@ HWTEST_F(FlexModifierTest, setPointLightTestPointLightLightSourceIntensity, Test Ark_PointLightStyle inputValuePointLight = initValuePointLight; WriteTo(inputValuePointLight.lightSource).intensity = value; - modifier_->setPointLight(node_, &inputValuePointLight); + auto optValue = Converter::ArkValue(inputValuePointLight); + modifier_->setPointLight(node_, &optValue); auto jsonValue = GetJsonValue(node_); auto resultLightSource = GetAttrValue>(jsonValue, ATTRIBUTE_POINT_LIGHT_NAME); @@ -446,7 +450,8 @@ HWTEST_F(FlexModifierTest, setPointLightTestPointLightLightSourceColorValidValue Ark_PointLightStyle inputValuePointLight = initValuePointLight; WriteTo(inputValuePointLight.lightSource).color = value; - modifier_->setPointLight(node_, &inputValuePointLight); + auto optValue = Converter::ArkValue(inputValuePointLight); + modifier_->setPointLight(node_, &optValue); auto jsonValue = GetJsonValue(node_); auto resultPointLight = GetAttrValue>(jsonValue, ATTRIBUTE_POINT_LIGHT_NAME); @@ -497,10 +502,11 @@ HWTEST_F(FlexModifierTest, setPointLightTestPointLightLightSourceColorInvalidVal auto checkValue = [this, &initValuePointLight](const std::string& input, const Opt_ResourceColor& value) { Ark_PointLightStyle inputValuePointLight = initValuePointLight; - - modifier_->setPointLight(node_, &inputValuePointLight); + auto optValue = Converter::ArkValue(inputValuePointLight); + modifier_->setPointLight(node_, &optValue); WriteTo(inputValuePointLight.lightSource).color = value; - modifier_->setPointLight(node_, &inputValuePointLight); + optValue = Converter::ArkValue(inputValuePointLight); + modifier_->setPointLight(node_, &optValue); auto jsonValue = GetJsonValue(node_); auto resultPointLight = GetAttrValue>(jsonValue, ATTRIBUTE_POINT_LIGHT_NAME); @@ -549,7 +555,8 @@ HWTEST_F(FlexModifierTest, setPointLightTestPointLightIlluminatedValidValues, Te Ark_PointLightStyle inputValuePointLight = initValuePointLight; inputValuePointLight.illuminated = value; - modifier_->setPointLight(node_, &inputValuePointLight); + auto optValue = Converter::ArkValue(inputValuePointLight); + modifier_->setPointLight(node_, &optValue); auto jsonValue = GetJsonValue(node_); auto resultPointLight = GetAttrValue>(jsonValue, ATTRIBUTE_POINT_LIGHT_NAME); auto resultStr = GetAttrValue(resultPointLight, ATTRIBUTE_POINT_LIGHT_I_ILLUMINATED_NAME); @@ -588,7 +595,8 @@ HWTEST_F(FlexModifierTest, setPointLightTestPointLightIlluminatedInvalidValues, auto checkValue = [this, &initValuePointLight](const std::string& input, const Opt_IlluminatedType& value) { Ark_PointLightStyle inputValuePointLight = initValuePointLight; inputValuePointLight.illuminated = value; - modifier_->setPointLight(node_, &inputValuePointLight); + auto optValue = Converter::ArkValue(inputValuePointLight); + modifier_->setPointLight(node_, &optValue); auto jsonValue = GetJsonValue(node_); auto resultPointLight = GetAttrValue>(jsonValue, ATTRIBUTE_POINT_LIGHT_NAME); auto resultStr = GetAttrValue(resultPointLight, ATTRIBUTE_POINT_LIGHT_I_ILLUMINATED_NAME); @@ -629,7 +637,8 @@ HWTEST_F(FlexModifierTest, setPointLightTestPointLightBloomValidValues, TestSize Ark_PointLightStyle inputValuePointLight = initValuePointLight; inputValuePointLight.bloom = value; - modifier_->setPointLight(node_, &inputValuePointLight); + auto optValue = Converter::ArkValue(inputValuePointLight); + modifier_->setPointLight(node_, &optValue); auto jsonValue = GetJsonValue(node_); auto resultPointLight = GetAttrValue>(jsonValue, ATTRIBUTE_POINT_LIGHT_NAME); @@ -672,10 +681,11 @@ HWTEST_F(FlexModifierTest, setPointLightTestPointLightBloomInvalidValues, TestSi auto checkValue = [this, &initValuePointLight](const std::string& input, const Opt_Number& value) { Ark_PointLightStyle inputValuePointLight = initValuePointLight; - - modifier_->setPointLight(node_, &inputValuePointLight); + auto optValue = Converter::ArkValue(inputValuePointLight); + modifier_->setPointLight(node_, &optValue); inputValuePointLight.bloom = value; - modifier_->setPointLight(node_, &inputValuePointLight); + optValue = Converter::ArkValue(inputValuePointLight); + modifier_->setPointLight(node_, &optValue); auto jsonValue = GetJsonValue(node_); auto resultPointLight = GetAttrValue>(jsonValue, ATTRIBUTE_POINT_LIGHT_NAME); diff --git a/test/unittest/capi/modifiers/folder_stack_modifier_test.cpp b/test/unittest/capi/modifiers/folder_stack_modifier_test.cpp index a71e30b679935503c16c84ce057a4960100844a5..b73445e92d132c8e0b9d4190354ed53a349099d2 100644 --- a/test/unittest/capi/modifiers/folder_stack_modifier_test.cpp +++ b/test/unittest/capi/modifiers/folder_stack_modifier_test.cpp @@ -103,7 +103,8 @@ HWTEST_F(FolderStackModifierTest, setAlignContentTestValidValues, TestSize.Level inputValueAlignContent = initValueAlignContent; for (auto&& value: alignContentAlignContentValidValues) { inputValueAlignContent = std::get<1>(value); - modifier_->setAlignContent(node_, inputValueAlignContent); + auto optValue = Converter::ArkValue(inputValueAlignContent); + modifier_->setAlignContent(node_, &optValue); jsonValue = GetJsonValue(node_); resultStr = GetAttrValue(jsonValue, ATTRIBUTE_ALIGN_CONTENT_NAME); expectedStr = std::get<2>(value); @@ -135,9 +136,11 @@ HWTEST_F(FolderStackModifierTest, setAlignContentTestInvalidValues, TestSize.Lev // Verifying attribute's values inputValueAlignContent = initValueAlignContent; - modifier_->setAlignContent(node_, inputValueAlignContent); + auto optValue = Converter::ArkValue(inputValueAlignContent); + modifier_->setAlignContent(node_, &optValue); inputValueAlignContent = std::get<1>(alignContentAlignContentInvalidValue); - modifier_->setAlignContent(node_, inputValueAlignContent); + optValue = Converter::ArkValue(inputValueAlignContent); + modifier_->setAlignContent(node_, &optValue); jsonValue = GetJsonValue(node_); resultStr = GetAttrValue(jsonValue, ATTRIBUTE_ALIGN_CONTENT_NAME); expectedStr = ATTRIBUTE_ALIGN_CONTENT_DEFAULT_VALUE; @@ -163,7 +166,8 @@ HWTEST_F(FolderStackModifierTest, setOnFolderStateChangeTest, TestSize.Level1) ASSERT_NE(frameNode, nullptr); auto context = MockPipelineContext::GetCurrent(); frameNode->AttachToMainTree(true, Referenced::RawPtr(context)); - modifier_->setOnFolderStateChange(node_, &func); + auto optFunc = Converter::ArkValue(func); + modifier_->setOnFolderStateChange(node_, &optFunc); auto folderStackNode = AceType::DynamicCast(frameNode); ASSERT_NE(folderStackNode, nullptr); @@ -194,7 +198,8 @@ HWTEST_F(FolderStackModifierTest, setOnHoverStatusChangeTest, TestSize.Level1) ASSERT_NE(frameNode, nullptr); auto context = MockPipelineContext::GetCurrent(); frameNode->AttachToMainTree(true, Referenced::RawPtr(context)); - modifier_->setOnHoverStatusChange(node_, &func); + auto optFunc = Converter::ArkValue(func); + modifier_->setOnHoverStatusChange(node_, &optFunc); auto folderStackNode = AceType::DynamicCast(frameNode); ASSERT_NE(folderStackNode, nullptr); @@ -246,7 +251,8 @@ HWTEST_F(FolderStackModifierTest, setEnableAnimationTestValidValues, TestSize.Le inputValueEnableAnimation = initValueEnableAnimation; for (auto&& value: enableAnimationEnableAnimationValidValues) { inputValueEnableAnimation = std::get<1>(value); - modifier_->setEnableAnimation(node_, inputValueEnableAnimation); + auto optValue = Converter::ArkValue(inputValueEnableAnimation); + modifier_->setEnableAnimation(node_, &optValue); jsonValue = GetJsonValue(node_); resultStr = GetAttrValue(jsonValue, ATTRIBUTE_ENABLE_ANIMATION_NAME); expectedStr = std::get<2>(value); @@ -294,7 +300,8 @@ HWTEST_F(FolderStackModifierTest, setAutoHalfFoldTestValidValues, TestSize.Level inputValueAutoHalfFold = initValueAutoHalfFold; for (auto&& value: autoHalfFoldAutoHalfFoldValidValues) { inputValueAutoHalfFold = std::get<1>(value); - modifier_->setAutoHalfFold(node_, inputValueAutoHalfFold); + auto optValue = Converter::ArkValue(inputValueAutoHalfFold); + modifier_->setAutoHalfFold(node_, &optValue); jsonValue = GetJsonValue(node_); resultStr = GetAttrValue(jsonValue, ATTRIBUTE_AUTO_HALF_FOLD_NAME); expectedStr = std::get<2>(value); diff --git a/test/unittest/capi/modifiers/form_component_modifier_test.cpp b/test/unittest/capi/modifiers/form_component_modifier_test.cpp index 911fd75e5c9a5288385670aa9419bc7b6150c858..f7137b666c1660acb06831a703f486cb6ba3f035 100644 --- a/test/unittest/capi/modifiers/form_component_modifier_test.cpp +++ b/test/unittest/capi/modifiers/form_component_modifier_test.cpp @@ -182,7 +182,8 @@ HWTEST_F(FormComponentModifierTest, DISABLED_setSizeTestSizeWidthValidValues, Te const std::string& input, const Opt_Length& value, const std::string& expectedStr) { Ark_SizeOptions inputValueSize = initValueSize; inputValueSize.width = value; - modifier_->setSize(node_, &inputValueSize); + auto optValue = Converter::ArkValue(inputValueSize); + modifier_->setSize(node_, &optValue); auto jsonValue = GetJsonValue(node_); auto resultConstructor = GetAttrValue>(jsonValue, ATTRIBUTE_CONSTRUCTOR_NAME); auto resultSize = GetAttrValue>(resultConstructor, ATTRIBUTE_SIZE_NAME); @@ -210,7 +211,8 @@ HWTEST_F(FormComponentModifierTest, DISABLED_setSizeTestSizeHeightValidValues, T const std::string& input, const Opt_Length& value, const std::string& expectedStr) { Ark_SizeOptions inputValueSize = initValueSize; inputValueSize.height = value; - modifier_->setSize(node_, &inputValueSize); + auto optValue = Converter::ArkValue(inputValueSize); + modifier_->setSize(node_, &optValue); auto jsonValue = GetJsonValue(node_); auto resultConstructor = GetAttrValue>(jsonValue, ATTRIBUTE_CONSTRUCTOR_NAME); auto resultSize = GetAttrValue>(resultConstructor, ATTRIBUTE_SIZE_NAME); @@ -253,7 +255,8 @@ HWTEST_F(FormComponentModifierTest, setModuleNameTestModuleNameValidValues, Test const std::string& input, const Ark_String& value, const std::string& expectedStr) { Ark_String inputValueModuleName = initValueModuleName; inputValueModuleName = value; - modifier_->setModuleName(node_, &inputValueModuleName); + auto optValue = Converter::ArkValue(inputValueModuleName); + modifier_->setModuleName(node_, &optValue); auto jsonValue = GetJsonValue(node_); auto resultConstructor = GetAttrValue>(jsonValue, ATTRIBUTE_CONSTRUCTOR_NAME); auto resultStr = GetAttrValue(resultConstructor, ATTRIBUTE_MODULE_NAME_NAME); @@ -294,7 +297,8 @@ HWTEST_F(FormComponentModifierTest, setAllowUpdateTestAllowUpdateValidValues, Te const std::string& input, const Ark_Boolean& value, const std::string& expectedStr) { Ark_Boolean inputValueAllowUpdate = initValueAllowUpdate; inputValueAllowUpdate = value; - modifier_->setAllowUpdate(node_, inputValueAllowUpdate); + auto optValue = Converter::ArkValue(inputValueAllowUpdate); + modifier_->setAllowUpdate(node_, &optValue); auto jsonValue = GetJsonValue(node_); auto resultConstructor = GetAttrValue>(jsonValue, ATTRIBUTE_CONSTRUCTOR_NAME); auto resultStr = GetAttrValue(resultConstructor, ATTRIBUTE_ALLOW_UPDATE_NAME); @@ -335,8 +339,8 @@ HWTEST_F(FormComponentModifierTest, setDimensionTestDimensionValidValues, TestSi const std::string& input, const Ark_FormDimension& value, const std::string& expectedStr) { Ark_FormDimension inputValueDimension = initValueDimension; inputValueDimension = value; - - modifier_->setDimension(node_, inputValueDimension); + auto optValue = Converter::ArkValue(inputValueDimension); + modifier_->setDimension(node_, &optValue); auto jsonValue = GetJsonValue(node_); auto resultConstructor = GetAttrValue>(jsonValue, ATTRIBUTE_CONSTRUCTOR_NAME); auto resultStr = GetAttrValue(resultConstructor, ATTRIBUTE_DIMENSION_NAME); @@ -364,7 +368,8 @@ HWTEST_F(FormComponentModifierTest, setDimensionTestDimensionInvalidValues, Test Ark_FormDimension inputValueDimension = initValueDimension; inputValueDimension = value; - modifier_->setDimension(node_, inputValueDimension); + auto optValue = Converter::ArkValue(inputValueDimension); + modifier_->setDimension(node_, &optValue); auto jsonValue = GetJsonValue(node_); auto resultConstructor = GetAttrValue>(jsonValue, ATTRIBUTE_CONSTRUCTOR_NAME); auto resultStr = GetAttrValue(resultConstructor, ATTRIBUTE_DIMENSION_NAME); @@ -405,8 +410,8 @@ HWTEST_F(FormComponentModifierTest, setVisibilityTestVisibilityValidValues, Test const std::string& input, const Ark_Visibility& value, const std::string& expectedStr) { Ark_Visibility inputValueVisibility = initValueVisibility; inputValueVisibility = value; - - modifier_->setVisibility(node_, inputValueVisibility); + auto optValue = Converter::ArkValue(inputValueVisibility); + modifier_->setVisibility(node_, &optValue); auto jsonValue = GetJsonValue(node_); auto resultStr = GetAttrValue(jsonValue, ATTRIBUTE_VISIBILITY_NAME); EXPECT_EQ(resultStr, expectedStr) << @@ -433,7 +438,8 @@ HWTEST_F(FormComponentModifierTest, setVisibilityTestVisibilityInvalidValues, Te Ark_Visibility inputValueVisibility = initValueVisibility; inputValueVisibility = value; - modifier_->setVisibility(node_, inputValueVisibility); + auto optValue = Converter::ArkValue(inputValueVisibility); + modifier_->setVisibility(node_, &optValue); auto jsonValue = GetJsonValue(node_); auto resultStr = GetAttrValue(jsonValue, ATTRIBUTE_VISIBILITY_NAME); EXPECT_EQ(resultStr, ATTRIBUTE_VISIBILITY_DEFAULT_VALUE) << @@ -473,7 +479,8 @@ HWTEST_F(FormComponentModifierTest, setOnAcquiredTest, TestSize.Level1) }, .call = onAcquired }; - modifier_->setOnAcquired(node_, &func); + auto optFunc = Converter::ArkValue(func); + modifier_->setOnAcquired(node_, &optFunc); for (const auto& [actual, expectedNum, expectedStr] : testFixtureFormOnAcquiredCallbackTestValues) { formInfo = std::nullopt; auto testValue = ToJson(actual); @@ -512,7 +519,8 @@ HWTEST_F(FormComponentModifierTest, setOnErrorTest, TestSize.Level1) }, .call = onError }; - modifier_->setOnError(node_, &func); + auto optFunc = Converter::ArkValue(func); + modifier_->setOnError(node_, &optFunc); for (const auto& [code, msg, expected] : testFixtureFormOnErrorCallbackTestValues) { formInfo = std::nullopt; auto testValue = ToJson(code, msg); @@ -553,7 +561,8 @@ HWTEST_F(FormComponentModifierTest, setOnUninstallTest, TestSize.Level1) }, .call = onUninstall }; - modifier_->setOnUninstall(node_, &func); + auto optFunc = Converter::ArkValue(func); + modifier_->setOnUninstall(node_, &optFunc); for (const auto& [actual, expectedNum, expectedStr] : testFixtureFormOnAcquiredCallbackTestValues) { formInfo = std::nullopt; auto testValue = ToJson(actual); @@ -592,7 +601,8 @@ HWTEST_F(FormComponentModifierTest, setOnLoadTest, TestSize.Level1) }, .call = onLoad }; - modifier_->setOnLoad(node_, &func); + auto optFunc = Converter::ArkValue(func); + modifier_->setOnLoad(node_, &optFunc); formInfo = std::nullopt; eventHub->FireOnLoad(FORM_EMPTY_STRING); EXPECT_TRUE(formInfo.has_value());