37959 - Winprofile: -229 error returned in addition to expect results

Status: VERIFIED FIXED

(Core Graveyard :: Installer: XPInstall Engine, defect, P3)

Description

[Jimmy Lee]

Builds: 2000-05-02-09-M16(WIN), 2000-05-02-15-M16(MAC), 2000-05-02-15-M16(LINUX)

1. From http://jimbob/trigger3.html, click Acceptance drop-down menu and choose
   a_winprofile_writestring and click Trigger case button.
2. From http://jimbob/trigger3.html, click Acceptance drop-down menu and choose
   a_winprofile_getstring and click Trigger case button.

RESULT:
The install.log records the expected result.  An error -229 also appears at the 
end of the session.

EXPECTED RESULT:
No -229 appears.

INSTALL.LOG:
-------------------------------------------------------------------------------
http://jimbob/jars/a_winprofile_getstring.xpi  --  05/02/2000 16:46:23
-------------------------------------------------------------------------------

     Acceptance: a_winprofile_getstring
     ----------------------------------

     ** getString should retrieve 'newpathname': newpathname

     Install **FAILED** with error -229
     Finished Installation  05/02/2000 16:46:23

INSTALL SCRIPTS:
////////////////////////////////////////////////////////////////////////////////
// a_winprofile_writestring
// 
// Acceptance test
// 	• Checks that string value can be changed from an .ini file
//        writeString(String section, String key, String value);
// 	• The xpinstall_test.ini file will be created and installed into the 
//        Windows directory;
//	  A section, [Desktop], will be created with "Wallpaper" as a key equal 
//        to the value "newpathname"
//
// ~ XPInstall QA
//   September 2, 1999
//
//   March 17, 2000  jimmylee  Implement initInstall, performInstall, 
//                             cancelInstall
//
////////////////////////////////////////////////////////////////////////////////

initInstall("Acceptance: a_winprofile_writestring", "acceptance", "1.2.3.4", 1); 
f = getFolder("Windows");
ini = getWinProfile(f, "xpinstall_test.ini");
ini.writeString("Desktop", "Wallpaper", "newpathname");

if (0 == getLastError())
	performInstall();
else
	cancelInstall();

////////////////////////////////////////////////////////////////////////////////
// a_winprofile_getstring
// 
// Acceptance test
// 	• Checks that string value can be retrieved from an .ini file
//        getString(String section, String key);
// 	• The install.log will show that getString retrieves the key value, 
//        'newpathname'
// 	• xpinstall_test.ini will be deleted at the end of install script as 
//        part of cleanup
//
// Dependency
// 	• a_winprofile_writestring.xpi needs to be executed first since it 
//        creates the file, xpinstall_test.ini
//
// ~ XPInstall QA
//   December 6, 1999
//
//   March 17, 2000  jimmylee  Implement initInstall, performInstall, 
//                             cancelInstall
//
////////////////////////////////////////////////////////////////////////////////

initInstall("Acceptance: a_winprofile_getstring", "a_winprofile_getstring", 
"1.2.3.4", 0); 
f = getFolder("Windows");
ini = getWinProfile(f, "xpinstall_test.ini");
sname = ini.getString("Desktop", "Wallpaper");
logComment("getString should retrieve 'newpathname': " + sname);

if (0 == getLastError())
	performInstall();
else
	cancelInstall();

Comment 1

[Jimmy Lee]

May be related to Bug 37855

Comment 2

[Cathleen]

acceptance test regression from PR1, must fix in PR2

Comment 3

[Cathleen]

*** This bug has been marked as a duplicate of 37855 ***

Comment 4

[Cathleen]

These are NOT the same, that would be the same as saying all crashes are the 
same bug

Comment 5

[Cathleen]

reassign to dan  :-)

this is a regression

Comment 6

[leger]

Putting on [nsbeta2+] radar for beta2 fix. 

Comment 7

[Daniel Veditz [:dveditz]]

checked in

Comment 8

[Jimmy Lee]

Build: 2000-05-24-09-M16(WIN)

Back to normal.