All users were logged out of Bugzilla on October 13th, 2018

getcomponentfolder test case fails

VERIFIED FIXED in mozilla1.4alpha

Status

VERIFIED FIXED
16 years ago
3 years ago

People

(Reporter: jimmykenlee, Assigned: ssu0262)

Tracking

({regression})

Trunk
mozilla1.4alpha
regression
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [adt3])

Attachments

(1 obsolete attachment)

(Reporter)

Description

16 years ago
Build: 2003-01-03-08-trunk(WIN), 2003-01-03-08-trunk(LINUX),
2003-01-03-08-trunk(MAC OS X)

1. From http://jimbob/trigger3.html, select a_getcomponentfolder from the
   Acceptance drop-down and click Trigger case button
2. Click Install button from Software Installation dialog

RESULT:
Dialog returns "script error".

Install.log returns:
-------------------------------------------------------------------------------
http://jimbob/jars/a_getcomponentfolder.xpi  --  01/06/2003 16:06:59
-------------------------------------------------------------------------------

     Acceptance: a_getcomponentfolder
     --------------------------------

     [1/1]	Installing: C:\Program Files\Netscape\122302\a_getcomponentfolder.txt

     Install **FAILED** with error -229  --  01/06/2003 16:06:59

Installscript:
regName = "a_getcomponentfolder";
jarSrc = "smrtupdt.txt";
var vi = "1.0.1.15";

initInstall("Acceptance: a_getcomponentfolder", "acceptance", vi, 0);
f = getFolder("Program");
addFile(regName, vi, jarSrc, f, "a_getcomponentfolder.txt", true);
if (0 == getLastError())
	performInstall();
else
	cancelInstall();
compFolder = getComponentFolder("acceptance" + "/" + regName);
logComment("GetComponentFolder should return path to Program directory: " +
compFolder);

EXPECTED RESULT:
Installation is successful.  It appears the error occurs at:
compFolder = getComponentFolder("acceptance" + "/" + regName);
(Reporter)

Comment 1

16 years ago
Nominating for beta.  This used to work, and now it doesn't.  By the way, this
broke in 1.2.1.  I don't know how important this API is, but it is a regression.
Keywords: nsbeta1
Installer triage team: nsbeta1+/adt3
Assignee: dveditz → ssu
Keywords: nsbeta1 → nsbeta1+
Whiteboard: [adt3]
(Assignee)

Updated

16 years ago
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla1.1alpha
(Assignee)

Updated

16 years ago
Target Milestone: mozilla1.1alpha → mozilla1.4alpha
(Assignee)

Updated

16 years ago
Blocks: 197105
(Assignee)

Comment 3

16 years ago
Created attachment 118345 [details] [diff] [review]
patch v1.0

this fixes the problem, but I'm not sure if it's the right thing to do. 
checking with dveditz....
This patch throws away the subdirectory. The right fix is contained in the
125106 uber-patch which contained general nsIFile/intl cleanups. You can't
really separate out just the fix for this one spot though.
Depends on: 125106
Attachment #118345 - Attachment is obsolete: true
Attachment #118345 - Flags: review-
No longer depends on: 125106
Depends on: 125106

Updated

16 years ago
QA Contact: jimmylee → gbush

Comment 5

16 years ago
fwiw, I am seeing these tests show 'success' now vs cancelled as noted here ,
however they show 'null' as  result
(Assignee)

Comment 6

16 years ago
problem was fixed along with bug 125106.

both 1 and 2 param flavors of the function should be working now.
Status: ASSIGNED → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED

Comment 7

16 years ago
verified, test cases returning 'success'
Status: RESOLVED → VERIFIED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.