Open Bug 1150091 Opened 9 years ago Updated 2 years ago

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

Categories

(Testing :: Mozbase, defect)

Product:
Testing
Component:
Mozbase
Type:
defect

Tracking

(Not tracked)

Status:
UNCONFIRMED

People

(Reporter: alessarik, Unassigned)

References

Details

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.
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.
Blocks: 1000870
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.