DataStore mochitests should use mochitest.ini file

RESOLVED FIXED in mozilla27

Status

()

Core
DOM
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: baku, Assigned: baku)

Tracking

Trunk
mozilla27
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Comment hidden (empty)
(Assignee)

Updated

5 years ago
Blocks: 916089
OS: Linux → All
Hardware: x86_64 → All
(Assignee)

Comment 1

5 years ago
Created attachment 813713 [details] [diff] [review]
dataStore_mochitest.patch
Attachment #813713 - Flags: review?(Ms2ger)
Comment on attachment 813713 [details] [diff] [review]
dataStore_mochitest.patch

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

::: dom/datastore/tests/moz.build
@@ +3,5 @@
>  # This Source Code Form is subject to the terms of the Mozilla Public
>  # License, v. 2.0. If a copy of the MPL was not distributed with this
>  # file, You can obtain one at http://mozilla.org/MPL/2.0/.
> +
> +MOCHITEST_MANIFESTS += ['mochitest.ini']

It's preferred to move this into the parent dir, remove this dir from DIRS there, and remove this moz.build file.
(Assignee)

Comment 3

5 years ago
Created attachment 813753 [details] [diff] [review]
dataStore_mochitest.patch
Attachment #813713 - Attachment is obsolete: true
Attachment #813713 - Flags: review?(Ms2ger)
Attachment #813753 - Flags: review?(Ms2ger)
Comment on attachment 813753 [details] [diff] [review]
dataStore_mochitest.patch

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

Looks good, thanks!
Attachment #813753 - Flags: review?(Ms2ger) → review+
https://hg.mozilla.org/mozilla-central/rev/f239497c4d05
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla27
You need to log in before you can comment on or make changes to this bug.