Fix CI: add CUDA EP external data guard, fix TreeEnsemble test dims

- Add HasExternalData check to CUDA EP non-plugin LabelEncoder path
  (GetAttrOrTensor and TryGetScalarTensorAttribute) so error messages
  are consistent across all EPs.
- Remove CUDA EP exclusion from LabelEncoder tests since all EPs now
  reject external data uniformly.
- Fix TreeEnsemble opset 5 test: nodes_splits dims must match the
  number of elements in nodes_featureids and other node attributes
  (changed from 3 to 1).

Author: yuslepukhin

onnxruntime/core/providers/cuda/ml/label_encoder.cc

@@ -71,6 +71,8 @@ static bool TryGetScalarTensorAttribute(const OpKernelInfo& info, const std::str
   auto* attr_tensor_proto = GetTensorProto(attr_tensor_holder);
   auto result = info.GetAttr(tensor_name, attr_tensor_proto);
   if (result.IsOK() && utils::HasDataType(*attr_tensor_proto)) {
+    ORT_ENFORCE(!utils::HasExternalData(*attr_tensor_proto),
+                "Tensor attribute ", tensor_name, " with external data is not supported.");
     const auto [raw_data, raw_data_len] = GetRawData(*attr_tensor_proto);
     result = utils::UnpackTensor<T>(*attr_tensor_proto, raw_data, raw_data_len, &value, 1);
     ORT_ENFORCE(result.IsOK(), "LabelEncoder could not unpack tensor attribute ", attr_name);
@@ -117,6 +119,8 @@ static std::vector<T> GetAttrOrTensor(const OpKernelInfo& info, const std::strin
   } else {
     ORT_ENFORCE(result.IsOK(), "LabelEncoder is missing attribute ", tensor_name, " or ", name);
   }
+  ORT_ENFORCE(!utils::HasExternalData(*attr_tensor_proto),
+              "Tensor attribute ", tensor_name, " with external data is not supported.");
   SafeInt<int64_t> element_count(1);
   for (auto dim : attr_tensor_proto->dims()) {
     element_count *= dim;

onnxruntime/test/providers/cpu/ml/label_encoder_test.cc

@@ -819,9 +819,7 @@ TEST(LabelEncoder, RejectsExternalDataInKeysTensorOpset4) {
   test.AddInput<int64_t>("X", {1, 2}, {1, 2});
   test.AddOutput<int64_t>("Y", {1, 2}, {10, 20});
 
-  // CUDA EP uses a different code path that doesn't hit this issue, exclude it.
-  test.Run(OpTester::ExpectResult::kExpectFailure, "external data is not supported",
-           {kCudaExecutionProvider});
+  test.Run(OpTester::ExpectResult::kExpectFailure, "external data is not supported");
 }
 
 TEST(LabelEncoder, RejectsExternalDataInDefaultTensorOpset4) {
@@ -837,9 +835,7 @@ TEST(LabelEncoder, RejectsExternalDataInDefaultTensorOpset4) {
   test.AddInput<int64_t>("X", {1, 2}, {1, 3});
   test.AddOutput<int64_t>("Y", {1, 2}, {10, 0});
 
-  // CUDA EP uses a different code path that doesn't hit this issue, exclude it.
-  test.Run(OpTester::ExpectResult::kExpectFailure, "external data is not supported",
-           {kCudaExecutionProvider});
+  test.Run(OpTester::ExpectResult::kExpectFailure, "external data is not supported");
 }
 
 TEST(LabelEncoder, RejectsExternalDataInValuesTensorOpset4) {
@@ -868,9 +864,7 @@ TEST(LabelEncoder, RejectsExternalDataInValuesTensorOpset4) {
   test.AddInput<int64_t>("X", {1, 2}, {1, 2});
   test.AddOutput<int64_t>("Y", {1, 2}, {10, 20});
 
-  // CUDA EP uses a different code path that doesn't hit this issue, exclude it.
-  test.Run(OpTester::ExpectResult::kExpectFailure, "external data is not supported",
-           {kCudaExecutionProvider});
+  test.Run(OpTester::ExpectResult::kExpectFailure, "external data is not supported");
 }
 
 #endif  // !defined(ORT_NO_EXCEPTIONS)

onnxruntime/test/providers/cpu/ml/tree_ensembler_test.cc

@@ -462,19 +462,19 @@ TEST(MLOpTest, TreeEnsembleRejectsExternalDataInTensorAttribute) {
   PathString filename(ORT_TSTR("ext_data_XXXXXX"));
   FILE* fp = nullptr;
   CreateTestFile(fp, filename);
-  std::vector<char> data(12, 0);  // 3 x float
+  std::vector<char> data(4, 0);  // 1 x float
   fwrite(data.data(), 1, data.size(), fp);
   fclose(fp);
   ScopedFileDeleter ext_deleter(filename);
   std::string ext_path = ToUTF8String(filename);
 
   OpTester test("TreeEnsemble", 5, onnxruntime::kMLDomain);
 
-  // nodes_splits with external data location
+  // nodes_splits with external data location (1 node)
   ONNX_NAMESPACE::TensorProto splits_proto;
   splits_proto.set_name("nodes_splits");
   splits_proto.set_data_type(ONNX_NAMESPACE::TensorProto_DataType_FLOAT);
-  splits_proto.add_dims(3);
+  splits_proto.add_dims(1);
   splits_proto.set_data_location(ONNX_NAMESPACE::TensorProto_DataLocation_EXTERNAL);
   auto* loc = splits_proto.add_external_data();
   loc->set_key("location");
@@ -484,7 +484,7 @@ TEST(MLOpTest, TreeEnsembleRejectsExternalDataInTensorAttribute) {
   offset->set_value("0");
   auto* length = splits_proto.add_external_data();
   length->set_key("length");
-  length->set_value("12");
+  length->set_value("4");
   test.AddAttribute("nodes_splits", splits_proto);
 
   // Minimal valid structure for remaining attributes