Need to cover more cases in

RESOLVED FIXED in mozilla32


7 years ago
5 years ago


(Reporter: mdas, Assigned: akruglov)



Firefox Tracking Flags

(Not tracked)


(Whiteboard: [good first bug][lang=py][mentor=automatedtester])


(1 attachment, 1 obsolete attachment)

We don't test for visibility or if the element can be scrolled to. We'll need to fully test out the click() function, and all of its possible exceptions.

I'd like to tackle this one! Could you please point me to guide on how to setup everything for this project?
Follow the steps on to get firefox built.

Make the change as mentioned in comment 1. The tests can be copied across from the Selenium project (linked in comment 1) for running the tests.

Hope that helps!
Thanks, David!
Hm, it seems that tests in that file [0] are already ported to marionette [1], thus, I don't need to copy them one more time. Also, I think that it is a wrong test name here [2]? I looked over and there is nothing about cookies.

So, it is only about porting scrolling tests from [3], right?

Please, review.

I ported tests for scrolling, but three of them are failing. 
At least two of those failures look like valid ones.
Attachment #8414650 - Flags: review?(dburns)
(In reply to akruglov from comment #6)
> Created attachment 8414650 [details] [diff] [review]
> Please, review.
> I ported tests for scrolling, but three of them are failing. 
> At least two of those failures look like valid ones.

Can you please disable the failing tests by putting @skip("Bug number") above the failing tests.

Can you also raise separate bugs for each of the failing tests

marked failed tests with skip
Attachment #8414650 - Attachment is obsolete: true
Attachment #8414650 - Flags: review?(dburns)
Attachment #8415025 - Flags: review?(dburns)
Assignee: nobody → akruglov
pushed to try 


Will check back later on the results
Flags: needinfo?(dburns)
Attachment #8415025 - Flags: review?(dburns) → review+
This has been landed in

Thanks for the hard work on this patch! The next step for this code is that it will be merged, if all goes well, into Mozilla central and this will be resolved fixed.

If you want to work on another Marionette Good First Bug have a look at the bugs in[good%20first%20bug]&bug_status=UNCONFIRMED&bug_status=NEW&bug_status=ASSIGNED&bug_status=REOPENED&component=Marionette&product=Testing&list_id=10013678

If there are none that look appealing the Automation and Tools Team have another list at
Flags: needinfo?(dburns)
Great, thanks!
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla32
You need to log in before you can comment on or make changes to this bug.