Clean up LanguageSpecificCheckValid
- removed redundant checks: (Recursive calls, List<interface>)
- moved a check to CheckValid: (interface[])
Bug: n/a
Test: aidl_unittests
Change-Id: Id72ba2f16861ab6919421180e470e791724997c7
diff --git a/aidl_unittest.cpp b/aidl_unittest.cpp
index 5e0f473..ec9b7a8 100644
--- a/aidl_unittest.cpp
+++ b/aidl_unittest.cpp
@@ -1986,13 +1986,10 @@
}
TEST_P(AidlTest, RejectsListArray) {
- const string expected_stderr = "ERROR: a/Foo.aidl:2.1-7: List[] is not supported.\n";
- const string list_array_parcelable =
- "package a; parcelable Foo {\n"
- " List[] lists; }";
+ const string input = "package a; parcelable Foo { List<String>[] lists; }";
CaptureStderr();
- EXPECT_EQ(nullptr, Parse("a/Foo.aidl", list_array_parcelable, typenames_, GetLanguage()));
- EXPECT_EQ(expected_stderr, GetCapturedStderr());
+ EXPECT_EQ(nullptr, Parse("a/Foo.aidl", input, typenames_, GetLanguage()));
+ EXPECT_THAT(GetCapturedStderr(), HasSubstr("Arrays of List are not supported"));
}
TEST_P(AidlTest, RejectsPrimitiveListInStableAidl) {