Closed
Bug 1441433
Opened 7 years ago
Closed 7 years ago
Move test_caret_*.html files into layout subdirectory
Categories
(Remote Protocol :: Marionette, enhancement, P3)
Tracking
(firefox61 fixed)
RESOLVED
FIXED
mozilla61
Tracking | Status | |
---|---|---|
firefox61 | --- | fixed |
People
(Reporter: whimboo, Assigned: akk5597, Mentored)
References
Details
(Keywords: good-first-bug, Whiteboard: [lang=py])
User Story
For steps in how to get started please consult our documentation for new users: https://firefox-source-docs.mozilla.org/testing/marionette/marionette/NewContributors.html In the case of questions you can find us on IRC in the #ateam channel.
Attachments
(1 file)
5.28 KB,
patch
|
ato
:
review+
|
Details | Diff | Splinter Review |
All the testcase HTML files starting with test_caret_*.html in the following directory seem to be used by layout tests, which basically should not run as part of our unit tests. Given that there is currently no easy way to specify a different location for those tests, all testcase HTML files got added to our base `www` folder: https://dxr.mozilla.org/mozilla-central/rev/default/testing/marionette/harness/marionette_harness/www/ We should move those files to a subfolder with the name layout, and update the Marionette tests under the following location: https://dxr.mozilla.org/mozilla-central/source/layout/base/tests/marionette It could be that some of the files are not needed anymore. If that is the case they have to be removed.
Comment 1•7 years ago
|
||
@whimboo, I'd like to work on this. I'm going to first read about what exactly are layout tests and how they differ from unit tests, and then, I'll familiarize with the flow of control among the files involved.
Comment 2•7 years ago
|
||
Hello Henrik, I would like to work on this bug if Pooja is not working on this. If give a chance,this would be first contribution and I am really excited. Thanks.
Reporter | ||
Comment 3•7 years ago
|
||
Adithya, thank you for your interest. Given that we haven't heard from Pooja for nearly a month feel free to get started on this bug. I will assign it to you once a patch has been uploaded. Thanks.
Comment 4•7 years ago
|
||
Henrik, thank you so much. I will work on this bug.
Comment 5•7 years ago
|
||
Hello Henrik, I have a query regarding this bug. Could you please verify if this is the right location to clone from ? hg clone http://hg.mozilla.org/mozilla-central/ I did clone from the above location but I am unable to find '/mozilla-central/rev/default/testing/marionette/harness/marionette_harness/www/'. Could you please help me out. Looks like I am missing something here. Apologies for troubling you. Thanks, Adithya
Reporter | ||
Comment 6•7 years ago
|
||
Yes, you can clone from that repository, and the files should be present: https://hg.mozilla.org/mozilla-central/file/tip/testing/marionette/harness/marionette_harness/www So please check again.
Assignee | ||
Comment 7•7 years ago
|
||
Hey! Is anyone working on this bug? If not, I would like to work on this one. As I understand, the test_carets_*.html files need to be added in a layout directory and corresponding tests need to point to the correct files.
Reporter | ||
Comment 8•7 years ago
|
||
We haven't heard back from the person in comment 5 for about a month. So yes, please feel free to get started. And yes your assumption is correct.
Assignee | ||
Comment 9•7 years ago
|
||
Assignee | ||
Comment 10•7 years ago
|
||
Thanks! This is my first attempt at contributing. So I created the subdirectory and moved the files there and also changed the appropriate lines in the test files. Then I used hg bzexport and the patch has been attached automatically. How do I ask for a review?
Flags: needinfo?(hskupin)
Comment 11•7 years ago
|
||
try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=95e89cdbeaffccd5d422204018dffa9e2f34082e
Flags: needinfo?(hskupin)
Updated•7 years ago
|
Attachment #8972032 -
Flags: review?(ato)
Updated•7 years ago
|
Attachment #8972032 -
Flags: review?(ato) → review+
Comment 12•7 years ago
|
||
Pushed by atolfsen@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/1a64f46e7fe5 Move test_carets_* files into layout subdirectory. r=ato
Comment 13•7 years ago
|
||
Thanks for the patch! Provided no tests fail, it should land on central within a few hours. Aditya: For your reference I’ve updated the author name in your commit as you had not set one. I don’t know how to change it in Mercurial, but if you submit a new patch please make sure you set the author name to your full name.
Assignee | ||
Comment 14•7 years ago
|
||
Thanks a lot! I'm going to follow the mercurial setup guidelines to ensure that I have the required setup.
Comment 15•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/1a64f46e7fe5
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox61:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
Updated•7 years ago
|
Assignee: nobody → akk5597
Updated•2 years ago
|
Product: Testing → Remote Protocol
You need to log in
before you can comment on or make changes to this bug.
Description
•