diff options
author | Arne H Juul <arnej27959@users.noreply.github.com> | 2019-07-01 13:19:01 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-07-01 13:19:01 +0200 |
commit | dd76185fd1639d22393d600a248b18b44ad534aa (patch) | |
tree | 41c4d5d9248d7fdeb77de0bfceecd88f8aff5b81 /eval/src/tests | |
parent | 0c246c495f47ca037eba8ec33b3d10f10fb5a309 (diff) | |
parent | 733b7c1d374e6cfa4dc6b0f2e39d269448ea85e6 (diff) |
Merge pull request #9853 from vespa-engine/havardpe/remove-error-value-constant
remove error_value constant
Diffstat (limited to 'eval/src/tests')
-rw-r--r-- | eval/src/tests/eval/function/function_test.cpp | 14 |
1 files changed, 4 insertions, 10 deletions
diff --git a/eval/src/tests/eval/function/function_test.cpp b/eval/src/tests/eval/function/function_test.cpp index 6c3839b6cc9..c2561a86d6f 100644 --- a/eval/src/tests/eval/function/function_test.cpp +++ b/eval/src/tests/eval/function/function_test.cpp @@ -14,20 +14,14 @@ std::vector<vespalib::string> params({"x", "y", "z", "w"}); double as_number(const Function &f) { auto number = as<Number>(f.root()); - if (number) { - return number->value(); - } else { - return error_value; - } + ASSERT_TRUE(number); + return number->value(); } vespalib::string as_string(const Function &f) { auto string = as<String>(f.root()); - if (string) { - return string->value(); - } else { - return "<error>"; - } + ASSERT_TRUE(string); + return string->value(); } struct OperatorLayer { |