Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 364217

Summary: Exception x.signum is not a function running mathLib.modf
Product: z_Archived Reporter: broy2
Component: EDTAssignee: Kathy Carroll <carrollk>
Status: CLOSED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: hjiyong, mayunf, mheitz, svihovec
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
fix for problem 1 lasher: iplog+

Description broy2 CLA 2011-11-19 08:50:28 EST
This is an example in the EDT Help for mathLib.modf.
Use EUnit to run:

library modfRBD 
    y int; 
    x, result smallFloat;  
    function modfTest1(){@Test}
        x = 23.5678;
        try
            result = mathLib.modf(x, y);
        onException(oops AnyException)
            LogResult.failed("Exception");
            exit;
        end
        LogResult.assertFloatEqual("modf", 0.5678005218505859, result);
    end
    function modfTest2(){@Test}
        LogResult.assertFloatEqual("modf", 23, y);
    end
end 

eglx.javascript.JavaScriptObjectException x.signum is not a function

modfRBD::modfTest1: FAILED - Exception
modfRBD::modfTest2: FAILED - modf - Failed: Expected value = '23' Actual value = '0'
Comment 1 Jing Qian CLA 2011-11-21 20:15:29 EST
deferred
Comment 2 Jing Qian CLA 2011-11-21 20:16:22 EST
Brenda, if this is not in eunit.test project in cvs, please add to it and commit the changes, thanks, so it will be regression tested everytime we run eunit.test
Comment 3 broy2 CLA 2011-11-22 15:45:40 EST
Done.
Comment 4 Huang Ji Yong CLA 2011-12-29 00:55:29 EST
Created attachment 208838 [details]
fix for problem 1
Comment 5 Huang Ji Yong CLA 2011-12-29 00:56:34 EST
The exception problem is fixed.
The second problem is the same as bug 364216. Waiting for the discussion.
Comment 6 Matt Heitz CLA 2012-01-13 17:17:12 EST
Ji Yong is right.  The remaining problem is the same as 364216.  We ought to take care of that problem using 364216, and close this one since the exception has been fixed.
Comment 7 Huang Ji Yong CLA 2012-01-15 20:34:31 EST
Route to Kathy to change the testcase.
If any problems, reopen it for me, thanks.
Comment 8 Kathy Carroll CLA 2012-02-24 10:53:38 EST
The testcases using MODF have been modified and released to CVS.  The modified variations are in project org.eclipse.edt.eunit.test: api.SystemTypes.mathLibFunctions and fvt.librarySupport.TestMathLib2Lib.

The exception issue has been resolved.
Comment 9 Kathy Carroll CLA 2012-02-24 10:54:22 EST
Verified and closed with 02/22 build