Commit Graph

19 Commits

Author SHA1 Message Date
Andrew Halberstadt
2faf25828e Bug 1392391 - [reftest] Pre-parse the manifests in a separate Firefox instance, r=jmaher
Instead of parsing the manifests and running the tests all in one go, this will
spawn an extra Firefox instance at the beginning that does nothing but parse the
manifest and dump them to a file.

This will allow the python harness to load and manipulate the test objects, before
sending them back to the JS harness as a list of tests to run. The main motivation
for this change is to implement run-by-manifest, a mode where we restart the
browser in between every test manifest. But there are other benefits as well, like
sharing the chunking logic used by other harnesses and the ability for the python
harness to stuff arbitrary metadata into the test objects.

For now, Android will continue to parse the manifests and run the tests all in one
go. Converting Android to this new mechanism will be left to a follow-up bug.


MozReview-Commit-ID: AfUBmQpx3Zz

--HG--
extra : rebase_source : 82e611e8795150daf01d791ea16517ec1ffb2896
2018-02-01 14:18:00 -05:00
Gurzau Raul
9c39f70c0f Backed out 5 changesets (bug 1392391) for frequently failing jsreftests on Android. a=backout
Backed out changeset 50df56a0cebf (bug 1392391)
Backed out changeset 406806a088d5 (bug 1392391)
Backed out changeset 6bd3abc55ea8 (bug 1392391)
Backed out changeset 846d50ea8403 (bug 1392391)
Backed out changeset b9e9374af9a1 (bug 1392391)
2018-02-07 02:20:38 +02:00
Andrew Halberstadt
00015fe0a8 Bug 1392391 - [reftest] Pre-parse the manifests in a separate Firefox instance, r=jmaher
Instead of parsing the manifests and running the tests all in one go, this will
spawn an extra Firefox instance at the beginning that does nothing but parse the
manifest and dump them to a file.

This will allow the python harness to load and manipulate the test objects, before
sending them back to the JS harness as a list of tests to run. The main motivation
for this change is to implement run-by-manifest, a mode where we restart the
browser in between every test manifest. But there are other benefits as well, like
sharing the chunking logic used by other harnesses and the ability for the python
harness to stuff arbitrary metadata into the test objects.

For now, Android will continue to parse the manifests and run the tests all in one
go. Converting Android to this new mechanism will be left to a follow-up bug.


MozReview-Commit-ID: AfUBmQpx3Zz

--HG--
extra : rebase_source : 955966c07bb650946c7c0e5706856f028335e850
2018-02-01 14:18:00 -05:00
Andrew Halberstadt
5b6ee32372 Bug 1421799 - [reftest] Log 'GECKO(<pid>)' instead of 'ProcessReader' for process_output log actions r=gbrown
MozReview-Commit-ID: FsKg5Gtwx9Z

--HG--
extra : rebase_source : 9b29b44fcec93ade12bc7147a04226a234f1d730
2017-12-06 14:11:55 -05:00
Sebastian Hengst
e6dbbae0aa Backed out 5 changesets (bug 1421799) for failing firefox ui functional tests. r=backout on a CLOSED TREE
Backed out changeset f9f32474fdaf (bug 1421799)
Backed out changeset 18b4fa241936 (bug 1421799)
Backed out changeset 88dca5c8cfac (bug 1421799)
Backed out changeset 8c1f1ab754a5 (bug 1421799)
Backed out changeset e028f677bc80 (bug 1421799)
2018-01-03 20:21:28 +02:00
Andrew Halberstadt
0b44c941df Bug 1421799 - [reftest] Log 'GECKO(<pid>)' instead of 'ProcessReader' for process_output log actions r=gbrown
MozReview-Commit-ID: FsKg5Gtwx9Z

--HG--
extra : rebase_source : 0b012271e1673b641c4514714f0ab68b42d2bd0d
2017-12-06 14:11:55 -05:00
Dorel Luca
61983e9ada Backed out 5 changesets (bug 1421799) for failing browser-chrome on Linux and OSX and for failing autophone-mochitest on Android r=backout on a CLOSED TREE
Backed out changeset 7fb20bced076 (bug 1421799)
Backed out changeset 629e467a07be (bug 1421799)
Backed out changeset 2de335c0287a (bug 1421799)
Backed out changeset afe14ec646ab (bug 1421799)
Backed out changeset bbe4d2292f86 (bug 1421799)
2017-12-14 15:41:22 +02:00
Andrew Halberstadt
b59244590b Bug 1421799 - [reftest] Log 'GECKO(<pid>)' instead of 'ProcessReader' for process_output log actions r=gbrown
MozReview-Commit-ID: FsKg5Gtwx9Z

--HG--
extra : rebase_source : 3bb118c510be08bb8716d553beeae09c7d60d82b
2017-12-06 14:11:55 -05:00
Andrew Halberstadt
db3fb1fa34 Bug 1403983 - Fix reftest regression where screenshots aren't being output and add a test, r=jmaher
MozReview-Commit-ID: 8ZqmukDtWpO

--HG--
extra : rebase_source : 86544bda6e79e5e35444b4083e76d2cf8642c408
2017-09-28 12:26:51 -04:00
Andrew Halberstadt
9ca2dfe204 Bug 1373745 - Fix structured logging problems in reftest, r=jmaher
Currently reftest.jsm uses logger.testEnd(..) to log all kinds of different
potential failures, from proper test failures to harness issues. This means
there are all kinds of edge cases that would cause multiple testEnd messages to
get logged, something the structured log protocol is supposed to prevent.

This modifies the reftest harness to instead use testStatus for everything. The
testEnd call will always be logged with status "OK". This required some changes
to the reftest formatter and reftest selftests.

MozReview-Commit-ID: 8RxsmHW49oy

--HG--
extra : rebase_source : 2f53e63f00bfc5e0e751b0f2bb3dc67477dc07eb
2017-06-16 15:08:45 -04:00
Geoff Brown
78c5013b6a Bug 1359292 - Handle non-ascii characters in reftest logs; r=jmaher
Non-ascii characters in error messages can cause exceptions when processing
structured log messages. The mochitest MessageLogger already handles such
cases; this is a simpler implementation for the reftest OutputHandler.
2017-05-08 07:49:05 -06:00
L. David Baron
e7546e008c Bug 1342252 - Fix printing of reftest images for failed != tests. r=jgraham
This makes the code work the way it looks like it was intended to work
based on what was already in output.py.

MozReview-Commit-ID: 8LJHxlfiQQ8

--HG--
extra : rebase_source : 46b2435ddbd3e8485536806d9fc7987599227f16
2017-02-23 14:34:27 -08:00
Deepa
31c56b8f1f Bug 1280572 - Add layout/tools/reftest to flake8 linter r=ahal
MozReview-Commit-ID: Cqllz9BeV7X

--HG--
extra : rebase_source : 862709bfe8600a0761467cc79cfa9dadb4e66e62
2017-02-15 01:20:56 +05:30
Andrew Halberstadt
79791a2cff Bug 1330772 - Remove all b2g related code from reftests, r=jmaher
MozReview-Commit-ID: HY1XbcliTNA

--HG--
extra : rebase_source : f02093222f1cc619e328c877382361950f83329d
2017-01-12 16:37:37 -05:00
James Graham
2cbfbc0ec6 Bug 1265582 - Fix reftest structured logs to match wpt, r=ahal
This allows the logs to work with the structured reftest viewer.

MozReview-Commit-ID: CY71vSdDjLP

--HG--
extra : rebase_source : 6b83d98aff1c5e73ac0a802b5a83b8be95adf59a
2016-09-27 22:54:50 +01:00
James Graham
8bdb6331c6 Bug 1293250 - Make reftest log generation less magical, r=ahal
Ensure that the first line of the log for failing tests is identical to
one that would be produced by the tbplformatter from mozlog. This means
that treeherder will be able to sucessfully cross-match error lines.

Also make the data structures used for storing screenshots identical
between reftests and web-platform-tests so that we can start to move
toward removing the reftest-specific codepath here.

MozReview-Commit-ID: FZQXLjj9Ejv

--HG--
extra : rebase_source : 3e9bf70ff7915817b8c02032c8da0d0a659e4661
2016-08-08 17:48:39 +01:00
Andrew Halberstadt
f94d7e71d8 Bug 1034290 - Use structured log output for test results in reftest, r=jmaher
Structured logs bring many benefits. We can stop parsing the logs for magic strings, we
can modify the format without breaking things, and we can stream results into systems like
ActiveData. The structured logs originate primarily in reftest.js. StructuredLog.jsm is
used to generate the JSON-based log stream. Finally OutputHandler in the python harness
reads structured output from stdout, and formats it into human readable form.

MozReview-Commit-ID: G3ZLkMRl6p7

--HG--
extra : commitid : J3ui9XYWR3Q
extra : rebase_source : 77ed0ba842cc8e557141fb3494212b06868c728a
extra : amend_source : 735d48225a2e627e0fe45fc11b50b6c49a885a4b
extra : source : d1779fe421c3c7cd8e3d191816776390dc104f37
2016-02-05 15:44:20 -05:00
Phil Ringnalda
66e6da78e6 Back out d1779fe421c3 (bug 1034290) for UnicodeDecodeErrors in things run by the reftest harness
CLOSED TREE
2016-02-05 08:18:47 -08:00
Andrew Halberstadt
8a462fd84b Bug 1034290 - Use structured log output for test results in reftest, r=jmaher
Structured logs bring many benefits. We can stop parsing the logs for magic strings, we
can modify the format without breaking things, and we can stream results into systems like
ActiveData. The structured logs originate primarily in reftest.js. StructuredLog.jsm is
used to generate the JSON-based log stream. Finally OutputHandler in the python harness
reads structured output from stdout, and formats it into human readable form.

--HG--
extra : commitid : J3ui9XYWR3Q
extra : rebase_source : 6bae978126dbd5beddc39332c7cbce0c1354cd87
extra : amend_source : 735d48225a2e627e0fe45fc11b50b6c49a885a4b
2015-10-29 15:01:44 -04:00