summaryrefslogtreecommitdiffstats
path: root/tests
diff options
context:
space:
mode:
authorTao Bao <tbao@google.com>2016-10-13 19:30:14 +0200
committerandroid-build-merger <android-build-merger@google.com>2016-10-13 19:30:14 +0200
commit039e9dc5ca7c949cf66b7bd7ecd34ae1880dbc9e (patch)
treeb7c6d625baefc3521fe46c794d8d82fe99535229 /tests
parentMerge "updater: Kill the duplicate PrintSha1() in install.cpp." (diff)
parentMerge "edify: Some clean-ups to libedify." (diff)
downloadandroid_bootable_recovery-039e9dc5ca7c949cf66b7bd7ecd34ae1880dbc9e.tar
android_bootable_recovery-039e9dc5ca7c949cf66b7bd7ecd34ae1880dbc9e.tar.gz
android_bootable_recovery-039e9dc5ca7c949cf66b7bd7ecd34ae1880dbc9e.tar.bz2
android_bootable_recovery-039e9dc5ca7c949cf66b7bd7ecd34ae1880dbc9e.tar.lz
android_bootable_recovery-039e9dc5ca7c949cf66b7bd7ecd34ae1880dbc9e.tar.xz
android_bootable_recovery-039e9dc5ca7c949cf66b7bd7ecd34ae1880dbc9e.tar.zst
android_bootable_recovery-039e9dc5ca7c949cf66b7bd7ecd34ae1880dbc9e.zip
Diffstat (limited to 'tests')
-rw-r--r--tests/component/edify_test.cpp28
-rw-r--r--tests/component/updater_test.cpp1
2 files changed, 23 insertions, 6 deletions
diff --git a/tests/component/edify_test.cpp b/tests/component/edify_test.cpp
index ede2ecb0a..a4dbb9fbe 100644
--- a/tests/component/edify_test.cpp
+++ b/tests/component/edify_test.cpp
@@ -22,17 +22,18 @@
static void expect(const char* expr_str, const char* expected) {
Expr* e;
- int error_count;
- EXPECT_EQ(parse_string(expr_str, &e, &error_count), 0);
+ int error_count = 0;
+ EXPECT_EQ(0, parse_string(expr_str, &e, &error_count));
+ EXPECT_EQ(0, error_count);
State state(expr_str, nullptr);
char* result = Evaluate(&state, e);
if (expected == nullptr) {
- EXPECT_EQ(result, nullptr);
+ EXPECT_EQ(nullptr, result);
} else {
- EXPECT_STREQ(result, expected);
+ EXPECT_STREQ(expected, result);
}
free(result);
@@ -42,7 +43,6 @@ class EdifyTest : public ::testing::Test {
protected:
virtual void SetUp() {
RegisterBuiltins();
- FinishRegistration();
}
};
@@ -149,3 +149,21 @@ TEST_F(EdifyTest, big_string) {
expect(std::string(8192, 's').c_str(), std::string(8192, 's').c_str());
}
+TEST_F(EdifyTest, unknown_function) {
+ // unknown function
+ const char* script1 = "unknown_function()";
+ Expr* expr;
+ int error_count = 0;
+ EXPECT_EQ(1, parse_string(script1, &expr, &error_count));
+ EXPECT_EQ(1, error_count);
+
+ const char* script2 = "abc; unknown_function()";
+ error_count = 0;
+ EXPECT_EQ(1, parse_string(script2, &expr, &error_count));
+ EXPECT_EQ(1, error_count);
+
+ const char* script3 = "unknown_function1() || yes";
+ error_count = 0;
+ EXPECT_EQ(1, parse_string(script3, &expr, &error_count));
+ EXPECT_EQ(1, error_count);
+}
diff --git a/tests/component/updater_test.cpp b/tests/component/updater_test.cpp
index bd6534b3b..64a6b37ce 100644
--- a/tests/component/updater_test.cpp
+++ b/tests/component/updater_test.cpp
@@ -54,7 +54,6 @@ class UpdaterTest : public ::testing::Test {
virtual void SetUp() {
RegisterBuiltins();
RegisterInstallFunctions();
- FinishRegistration();
}
};