45188 - getFolder: file:/// is unexpectedly returning null

Status: VERIFIED FIXED

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

Description

[Jimmy Lee]

Build: 2000-07-11-09-M17(WIN)

1. From http://jimbob/trigger3.html, click drop-down for Acceptance and select
   a_getfolder_win_fileurl
2. Click Trigger case button
3. Click Install button from confirmation dialog

RESULT:
File is not installed to temp directory.  Install.log reports that null is 
returned from getFolder.

EXPECTED RESULT:
Path to temp directory is returned from getFolder.  File is installed to temp 
directory.

NOTE:
Bug 41165 results in a crash on Linux.  There is no crash with this bug.  A 
leading "/" for the second parameter of getFolder still fails in this test 
unlike what is reported for Bug 41165.

INSTALL.LOG:
-------------------------------------------------------------------------------
http://jimbob/jars/a_getfolder_win_fileurl.xpi  --  07/11/2000 15:13:33
-------------------------------------------------------------------------------

     Acceptance: a_getfolder_win_fileurl
     -----------------------------------

     ** getFolder file:/// returns = null
     ** addFile returns = -208

     Install cancelled by script
     Finished Installation  07/11/2000 15:13:33

Install Script:
////////////////////////////////////////////////////////////////////////////////
// a_getfolder_win_fileurl
// 
// Acceptance test
// 	• Checks getFolder with "File URL" (file:///) as target
// 	• jarSrc is installed to C:\temp\
//
// ~ XPInstall QA
//   August 13, 1999
//
//   March 16, 2000  jimmylee  Implement initInstall, performInstall, 
//                             cancelInstall
//   July 11, 2000   jimmylee  Add logComments for debugging purposes
//
////////////////////////////////////////////////////////////////////////////////

regName = "a_getfolder_win_fileurl";
jarSrc = "smrtupdt.txt";
var vi = "1.0.20.21";

initInstall("Acceptance: a_getfolder_win_fileurl", regName, vi, 0);
f = getFolder("file:///", "C|/temp/");
logComment("getFolder file:/// returns = " + f);
err = addFile(regName, jarSrc, f, jarSrc);
logComment("addFile returns = " + err);

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

Comment 1

[Daniel Veditz [:dveditz]]

This one is a result of the underlying brokenness that's also causing bug 
41165. I'm not going to mark this a duplicate because you can't exactly test 
the real bug so its various manifestations need to be double-checked.

Comment 2

[David Epstein]

this is fixed. can't dup this anymore.

Comment 3

[Daniel Veditz [:dveditz]]

fixed

Comment 4

[Jimmy Lee]

Build: 2000-08-13-09-M18(WIN), 2000-08-14-04-M18(MAC), 2000-08-14-08-M18(LINUX)

Marking Verified!