Switch tests to using moz.build and manifests

RESOLVED FIXED in seamonkey2.33

Status

SeaMonkey
Build Config
RESOLVED FIXED
4 years ago
3 years ago

People

(Reporter: neil@parkwaycc.co.uk, Assigned: ewong)

Tracking

Trunk
seamonkey2.33

SeaMonkey Tracking Flags

(seamonkey2.30 wontfix, seamonkey2.31 wontfix, seamonkey2.32 wontfix, seamonkey2.33 fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

4 years ago
Currently we appear to install tests using custom Makefile.in rules. We should switch to using a test manifest which is registered in moz.build e.g. replace suite/smile/test/Makefile.in with moz.build and browser.ini as follows:

suite/smile/test/moz.build:
> BROWSER_CHROME_MANIFESTS += ['browser.ini']

suite/smile/test/browser.ini:
> [DEFAULT]
> support-files =
>   ContentA.html
>   ContentB.html
>   ContentWithFrames.html
> 
> [browser_Applications.js]
> [browser_ApplicationPrefs.js]
> [browser_ApplicationQuitting.js]
> [browser_ApplicationStorage.js]
> [browser_Bookmarks.js]
> [browser_Browser.js]

Note: I don't know whether we pick up enough of the build system for this to work or whether parts of it still need to be ported.
(Assignee)

Updated

4 years ago
Status: NEW → ASSIGNED
(Assignee)

Comment 1

4 years ago
Created attachment 813023 [details] [diff] [review]
bug_922076.diff
(Assignee)

Updated

4 years ago
Attachment #813023 - Flags: review?(bugspam.Callek)
Comment on attachment 813023 [details] [diff] [review]
bug_922076.diff

Review of attachment 813023 [details] [diff] [review]:
-----------------------------------------------------------------

I'm assuming this has bitrotted in the ~year its taken me to get to it, is this still needed/valid. If so can you redo and possibly see if jcranmer can grab it (if not I can try and squeeze in)
Attachment #813023 - Flags: review?(bugspam.Callek)
(Assignee)

Comment 3

3 years ago
Created attachment 8517854 [details] [diff] [review]
bug_922076.diff
Attachment #813023 - Attachment is obsolete: true
Attachment #8517854 - Flags: review?(Pidgeot18)
Comment on attachment 8517854 [details] [diff] [review]
bug_922076.diff

Review of attachment 8517854 [details] [diff] [review]:
-----------------------------------------------------------------

I'll admit that mochitests are not my forté, but I offhand don't see any problems in this conversion.
Attachment #8517854 - Flags: review?(Pidgeot18) → review+
(Assignee)

Comment 5

3 years ago
Comment on attachment 8517854 [details] [diff] [review]
bug_922076.diff

Pushed to comm-central:
https://hg.mozilla.org/comm-central/rev/e4dbeb8f01a8
(Assignee)

Updated

3 years ago
status-seamonkey2.30: --- → wontfix
status-seamonkey2.31: --- → wontfix
status-seamonkey2.32: --- → wontfix
status-seamonkey2.33: --- → affected
(Assignee)

Updated

3 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.33

Updated

3 years ago
status-seamonkey2.33: affected → fixed
You need to log in before you can comment on or make changes to this bug.