1184415 - [Bugzilla Lite] Testing fill new Bug in Bugzilla Lite

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: Gaia::UI Tests, defect)

Description

[Manel Rhaiem [:manel]]

This Bug is intended to test filling new bug with Bugzilla Lite App, for that you need to :
Open this link with the browser on your device : bzlite-staging.herokuapp.com/install.html
Install this version of Bugzilla Lite because it's the staging version point on : https://bugzilla-dev.allizom.org/ .

Comment 1

[GitHub Autolander Bot]

Attachment: [gaia] mermi:testFillBug > mozilla-b2g:master

Comment 2

[Manel Rhaiem [:manel]]

Attachment: fill new bug

Comment 3

[GitHub Autolander Bot]

Attachment: [gaia] mermi:newFillBug > mozilla-b2g:master

Comment 4

[Manel Rhaiem [:manel]]

Attachment: Testing Fill new Bug

Comment 5

[John Dorlus [:Silne30]]

There were a bunch of things that needed to be fixed. I am going to r- for now until the fixes are made.

Comment 6

[Manel Rhaiem [:manel]]

(In reply to John Dorlus [:Silne30] from comment #5)
> There were a bunch of things that needed to be fixed. I am going to r- for
> now until the fixes are made.

fixed John.

Comment 7

[Martijn Wargers (dead)]

Manel, when you've fixed the issues, then you should re-ask review to John, so he will see it in his bugzilla queue and he will get a mail about it.

Comment 8

[Johan Lorenzo [:jlorenzo]]

Comment on attachment 8650642 [details] [review]
Testing Fill new Bug

This is a good start. One change introduced here, might break the other tests (uninstalling the actual BZ lite app). I'd also split the create new bug view into a page region (smaller is better :) ) and create the test variables inside the test itself. The manifest[1] needs to be updated if we want the test to run.

Thanks for your help!

[1] https://github.com/mermi/gaia/blob/newFillBug/tests/python/gaia-ui-tests/gaiatest/tests/functional/bzlite/manifest.ini

Comment 9

[Martijn Wargers (dead)]

Comment on attachment 8650573 [details] [review]
[gaia] mermi:newFillBug > mozilla-b2g:master

I added some comments in the pull request.

Comment 10

[Johan Lorenzo [:jlorenzo]]

Comment on attachment 8650573 [details] [review]
[gaia] mermi:newFillBug > mozilla-b2g:master

The breakdown in many small functions is on the right track. We can continue to make things smaller (and then, easier to read). More details in the PR.

Comment 11

[Martijn Wargers (dead)]

Comment on attachment 8650573 [details] [review]
[gaia] mermi:newFillBug > mozilla-b2g:master

I added comments in the pull request. After you've addressed those (and from the other reviewers), you can ask me again for review and I'll test it out on the device.

Comment 12

[Martijn Wargers (dead)]

Oh, the more I think of it, we should do a little bit more with adding the install method in gaia_test.py. I filed a new bug for this, bug 1212843.

Comment 13

[Manel Rhaiem [:manel]]

(In reply to Martijn Wargers [:mwargers] (QA) from comment #12)
> Oh, the more I think of it, we should do a little bit more with adding the
> install method in gaia_test.py. I filed a new bug for this, bug 1212843.

Yes, I think too we should add it I'll take work on it now

Comment 14

[Johan Lorenzo [:jlorenzo]]

Comment on attachment 8650573 [details] [review]
[gaia] mermi:newFillBug > mozilla-b2g:master

Cool, this looks nicer :)

The part about navigate_filed_bug() is a bit odd, I proposed a way to improve it. Also, some time.sleeps can be replaced with Waits(). I put more details in the PR.

Comment 15

[BMO Automation]

Firefox OS is not being worked on