The default bug view has changed. See this FAQ.

xpcshell: new test_componentsRegistered.js fails

VERIFIED FIXED in seamonkey2.1b1

Status

SeaMonkey
Build Config
--
major
VERIFIED FIXED
7 years ago
7 years ago

People

(Reporter: sgautherie, Assigned: sgautherie)

Tracking

(Blocks: 1 bug)

Trunk
seamonkey2.1b1
Dependency tree / graph
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(2 attachments, 1 obsolete attachment)

(Assignee)

Description

7 years ago
Example:
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1284173593.1284174881.627.gz
Linux comm-central-trunk debug test xpcshell on 2010/09/10 19:53:13
{
TEST-UNEXPECTED-FAIL | .../test_componentsRegistered.js | test failed (with xpcshell return code: 0), see following log:
...
TEST-UNEXPECTED-FAIL | /test_componentsRegistered.js | false == true - See following stack:
...
JS frame :: .../test_componentsRegistered.js :: run_test :: line 3
...
}

Code is
{
2     do_check_true("@mozilla.org/layout-debug/regressiontester;1" in
3                   Components.classes);
}

I would guess it's a build/packaging issue, either in Core or SeaMonkey...
> I would guess it's a build/packaging issue, either in Core or SeaMonkey...

Your right, its our bug.

From package-compare:
+bin/chrome/layoutdebug.jar
+bin/chrome/layoutdebug.manifest

+bin/components/layout_debug.xpt

Lets add it in debug builds *only*.


[I also see some other stuff in that list that might be worth investigating, another bug though, I'll tackle this week]
(Assignee)

Updated

7 years ago
Assignee: nobody → sgautherie.bz
Status: NEW → ASSIGNED
(Assignee)

Updated

7 years ago
Blocks: 534694
Flags: in-testsuite-
(Assignee)

Comment 2

7 years ago
Created attachment 474539 [details] [diff] [review]
(Av1) Package the Layout debug extension
[Checked in: Comment 3]
Attachment #474539 - Flags: review?(bugspam.Callek)
(Assignee)

Updated

7 years ago
Component: Layout → Build Config
Product: Core → SeaMonkey
QA Contact: layout → build-config

Updated

7 years ago
Attachment #474539 - Flags: review?(bugspam.Callek) → review+
(Assignee)

Comment 3

7 years ago
Comment on attachment 474539 [details] [diff] [review]
(Av1) Package the Layout debug extension
[Checked in: Comment 3]

http://hg.mozilla.org/comm-central/rev/8d1b298446ac
Attachment #474539 - Attachment description: (Av1) Package the Layout debug extension → (Av1) Package the Layout debug extension [Checked in: Comment 3]
(Assignee)

Updated

7 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Summary: [SeaMonkey] xpcshell: New test_componentsRegistered.js fails → xpcshell: new test_componentsRegistered.js fails
Target Milestone: --- → seamonkey2.1b1
(Assignee)

Comment 4

7 years ago
(In reply to comment #3)
> http://hg.mozilla.org/comm-central/rev/8d1b298446ac

+
http://hg.mozilla.org/comm-central/rev/1bf06f6d4829
(Bv1) Add needed -DMOZ_DEBUG=1
(Assignee)

Comment 5

7 years ago
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1284329277.1284332814.12645.gz&fulltext=1
Linux comm-central-trunk leak test build on 2010/09/12 15:07:57
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1284325879.1284327429.25584.gz&fulltext=1
OS X 10.5 comm-central-trunk leak test build on 2010/09/12 14:11:19

The packaging part is now fixed :-)
(almost...)

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1284333776.1284335444.22209.gz&fulltext=1
Linux comm-central-trunk debug test xpcshell on 2010/09/12 16:22:56
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1284329979.1284331648.8502.gz&fulltext=1
OS X 10.5 comm-central-trunk debug test xpcshell on 2010/09/12 15:19:39

Yet the failure still happens...
(Assignee)

Comment 6

7 years ago
Created attachment 474605 [details] [diff] [review]
(Cv1) Add needed -DENABLE_TESTS=1, Package @DLL_PREFIX@gkdebug@DLL_SUFFIX@ too

Parts missed in comment 1:

http://mxr.mozilla.org/comm-central/source/mozilla/layout/Makefile.in
78 ifdef ENABLE_TESTS
http://mxr.mozilla.org/comm-central/source/mozilla/layout/tools/layout-debug/src/Makefile.in
46 LIBRARY_NAME    = gkdebug


The latter should fix:

Build:
+bin/components/libgkdebug.so

Xpcshell:
nsNativeModuleLoader::LoadModule(".../components/libgkdebug.so") - load FAILED, rv: 80004005, error:
	.../components/libgkdebug.so: cannot open shared object file: No such file or directory


Do I need any kind of |#ifndef MOZ_STATIC_BUILD|?
Attachment #474605 - Flags: review?(bugspam.Callek)
Comment on attachment 474605 [details] [diff] [review]
(Cv1) Add needed -DENABLE_TESTS=1, Package @DLL_PREFIX@gkdebug@DLL_SUFFIX@ too

I'll r+ this one, but PLLLLEASE one conrete issue per bug. In the future, I'll r- based on that alone, this test is already fixed.

If we don't adhere to that, tracking and release management is harder, by a good deal.
Attachment #474605 - Flags: review?(bugspam.Callek) → review+
(In reply to comment #7)
> I'll r+ this one ...

> ... but PLLLLEASE one conrete issue per bug. In the future,
> I'll r- based on that alone, this test is already fixed.

Err missed c#5, sorry. I was wrong this is needed for this bug :/ (Thats what I get for reviewing while tired)
(Assignee)

Comment 9

7 years ago
Created attachment 474610 [details] [diff] [review]
(Cv1a) Add needed -DENABLE_TESTS=1, Package @DLL_PREFIX@gkdebug@DLL_SUFFIX@ too
[Checked in: Comment 11]

Cv1, with layoutdebug.jar removal update.
Attachment #474605 - Attachment is obsolete: true
Attachment #474610 - Flags: review?(bugspam.Callek)
Comment on attachment 474610 [details] [diff] [review]
(Cv1a) Add needed -DENABLE_TESTS=1, Package @DLL_PREFIX@gkdebug@DLL_SUFFIX@ too
[Checked in: Comment 11]

Not a fan of these removed-files.in changes, but I wont belabour the point
Attachment #474610 - Flags: review?(bugspam.Callek) → review+
(Assignee)

Comment 11

7 years ago
Comment on attachment 474610 [details] [diff] [review]
(Cv1a) Add needed -DENABLE_TESTS=1, Package @DLL_PREFIX@gkdebug@DLL_SUFFIX@ too
[Checked in: Comment 11]

http://hg.mozilla.org/comm-central/rev/9992caf4df07

Fwiw, would any kind of |#ifndef MOZ_STATIC_BUILD| be needed?
Attachment #474610 - Attachment description: (Cv1a) Add needed -DENABLE_TESTS=1, Package @DLL_PREFIX@gkdebug@DLL_SUFFIX@ too → (Cv1a) Add needed -DENABLE_TESTS=1, Package @DLL_PREFIX@gkdebug@DLL_SUFFIX@ too [Checked in: Comment 11]
(In reply to comment #11)
> Fwiw, would any kind of |#ifndef MOZ_STATIC_BUILD| be needed?

I'm not sure, and won't have time to investigate until later this week at earliest.
(Assignee)

Comment 13

7 years ago
V.Fixed, build and xpcshell.
Status: RESOLVED → VERIFIED
(Assignee)

Comment 14

7 years ago
(In reply to comment #11)
> Fwiw, would any kind of |#ifndef MOZ_STATIC_BUILD| be needed?

It probably was, but bug 598644 obsoleted this :->
You need to log in before you can comment on or make changes to this bug.