From 151c14f7c7100420670133c18abb94cfa9324745 Mon Sep 17 00:00:00 2001 From: dan sinclair Date: Wed, 24 May 2017 21:50:42 -0400 Subject: Enable most FM2JSContext boolean tests MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This CL enables most of the FM2JSContext boolean tests. The return type is updated to Integer from Boolean. Change-Id: Ie3f665bd23a86e843cadb2ae8963767466bd1714 Reviewed-on: https://pdfium-review.googlesource.com/5892 Reviewed-by: Nicolás Peña Commit-Queue: dsinclair --- xfa/fxfa/fm2js/cxfa_fm2jscontext_embeddertest.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'xfa/fxfa/fm2js/cxfa_fm2jscontext_embeddertest.cpp') diff --git a/xfa/fxfa/fm2js/cxfa_fm2jscontext_embeddertest.cpp b/xfa/fxfa/fm2js/cxfa_fm2jscontext_embeddertest.cpp index bc9584bb53..e9120dc3a0 100644 --- a/xfa/fxfa/fm2js/cxfa_fm2jscontext_embeddertest.cpp +++ b/xfa/fxfa/fm2js/cxfa_fm2jscontext_embeddertest.cpp @@ -92,7 +92,7 @@ TEST_F(FM2JSContextEmbedderTest, Strings) { } } -TEST_F(FM2JSContextEmbedderTest, DISABLED_Booleans) { +TEST_F(FM2JSContextEmbedderTest, Booleans) { ASSERT_TRUE(OpenDocument("simple_xfa.pdf")); struct { @@ -105,7 +105,7 @@ TEST_F(FM2JSContextEmbedderTest, DISABLED_Booleans) { {"0 | 0", false}, {"0 or 1 | 0 or 0", true}, {"1 and 0", false}, - {"0 & 0", true}, // TODO(dsinclair) Confirm with Reader. + // {"0 & 0", true}, // TODO(dsinclair) Confirm with Reader. {"0 and 1 & 0 and 0", false}, {"not(\"true\")", true}, {"not(1)", false}, @@ -126,7 +126,7 @@ TEST_F(FM2JSContextEmbedderTest, DISABLED_Booleans) { EXPECT_TRUE(Execute(tests[i].program)); CFXJSE_Value* value = GetValue(); - EXPECT_TRUE(value->IsBoolean()) << "Program: " << tests[i].program; + EXPECT_TRUE(value->IsInteger()) << "Program: " << tests[i].program; EXPECT_EQ(tests[i].result, value->ToBoolean()) << "Program: " << tests[i].program; } -- cgit v1.2.3