[manifestparser] Adding support-files under tests overwrites DEFAULT instead of appending

UNCONFIRMED
Unassigned

Status

Testing
Mozbase
UNCONFIRMED
3 years ago
3 years ago

People

(Reporter: Maksim Lebedev, Unassigned)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

3 years ago
User Agent: Mozilla/5.0 (Windows NT 6.2; WOW64; rv:36.0) Gecko/20100101 Firefox/36.0
Build ID: 20150320202338

Steps to reproduce:

I create mochitest.ini file with such content:

[DEFAULT]
support-files = file1.js
[test_file2.html]
  support-files = file3.html
[test_file4.html]
  support-files = file5.html


Actual results:

As result in target folder I can see:

test_file2.html
file3.html
test_file4.html
file5.html


Expected results:

I expect that file1.js should be in target folder. But it was missing. This is unobvious.
(Reporter)

Updated

3 years ago
OS: Windows 8 → All
Hardware: x86_64 → All
Component: Untriaged → Mozbase
Product: Firefox → Testing
Summary: Unobvious behavior mochitest manifest files → [manifestparser] Adding support-files under tests overwrites DEFAULT instead of appending
To expand comment 0 a bit, the problem happens when every test in a manifest includes a support-files attribute. So no tests ever get assigned DEFAULT and the files listed there never get copied.

Possibly the best way to fix this is just not allow tests to overwrite it, to avoid confusion.
(Reporter)

Updated

3 years ago
Blocks: 1000870
You need to log in before you can comment on or make changes to this bug.