Khronos Public Bugzilla

Bug 367

Summary: WFCT_Utils_testElementBooleanAttrib: incorrect checking of the return code
Product: OpenWF Reporter: Cyril PROTAT <cyril.protat-nonst>
Component: Conformance TestsAssignee: Robert Palmer <robert.palmer>
Status: RESOLVED INVALID QA Contact: Composition work group email alias <openwf>
Severity: major    
Priority: P3 CC: lars.f.persson
Version: unspecified   
Target Milestone: ---   
Hardware: All   
OS: All   

Description Cyril PROTAT 2010-10-22 04:44:11 PDT
In the wfc_test_utils.c file, the function "WFCT_Utils_testElementBooleanAttrib" tests the setting of an incorrect parameter : 

[ wfc_test_utils.c, line 224]

"/* Set bad handle (float) */"
wfcSetElementAttribf(appCtx->dev, WFC_INVALID_HANDLE, attrib, expectedf);
WFCT_CHECK_ERROR(WFC_ERROR_BAD_ATTRIBUTE, "Raised wrong or no error for bad handle (float setter).");
valuef = wfcGetElementAttribf(appCtx->dev, WFC_INVALID_HANDLE, attrib);
WFCT_CHECK_ERROR(WFC_ERROR_BAD_ATTRIBUTE, "Raised wrong or no error for bad handle (float getter).");

The return code for such an invalid attribute (WFC_INVALID_HANDLE) should be (and actually is) WFC_ERROR_BAD_HANDLE, not WFC_ERROR_BAD_ATTRIBUTE as in the above code.
Comment 1 lars.f.persson 2010-11-09 06:06:30 PST
Moved this bug to internal Bugzilla.
Comment 2 lars.f.persson 2010-11-09 06:07:21 PST
*** Bug 368 has been marked as a duplicate of this bug. ***
Comment 3 lars.f.persson 2010-11-09 06:08:42 PST
*** Bug 369 has been marked as a duplicate of this bug. ***