Closed
Bug 912343
Opened 11 years ago
Closed 4 years ago
Extend testBookmarkKeyword to check for search term
Categories
(Firefox for Android Graveyard :: Awesomescreen, defect)
Tracking
(Not tracked)
RESOLVED
INCOMPLETE
People
(Reporter: Margaret, Unassigned, Mentored)
References
Details
(Whiteboard: [lang=java])
Bug 908344 introduced testBookmarkKeyword, which adds a bookmark with a keyword to the DB, then checks to make sure entering that keyword in the urlbar opens the correct page. We should expand this to also have bookmarks keyword with a "%s" parameter in it, to test that we correctly load the right search url.
Comment 1•11 years ago
|
||
Hi my partner and I were interested in fixing this bug. Can we get assigned to it?
Reporter | ||
Comment 2•11 years ago
|
||
(In reply to Erica Hansen from comment #1)
> Hi my partner and I were interested in fixing this bug. Can we get assigned
> to it?
Hi Erica! Sure, I can help you work on this bug. However, I noticed that you also expressed interest in working on bug 825399, and we usually recommend that new contributors just focus on one bug at a time.
First things first, though - do you have a build environment set up? You'll want to follow the directions here to get that set up:
https://wiki.mozilla.org/Mobile/Fennec/Android
For this bug in particular, you'll need to be able to run robocop tests locally, which you can read about here:
https://wiki.mozilla.org/Auto-tools/Projects/Robocop
You should also refer to our "Get Involved" page for more information about getting started contributing to Firefox for Android:
https://wiki.mozilla.org/Mobile/Get_Involved
Feel free to join us in #mobile on irc.mozilla.org to ask questions!
Reporter | ||
Updated•11 years ago
|
Assignee: nobody → ss
Comment 3•11 years ago
|
||
Hey Margaret,
Was there still an issue with this bug? We had a brief conversation in #mobile about me taking on this bug and there was some sort of issue (I admittedly forget what the problem was, but it was related to another bug).
If there is still an issue, I'd gladly take on 904217 instead. Again, this was mentioned in #mobile.
If there aren't any issues, I'll start this one. I already have Firefox and Fennec built successfully and can run the Robocop tests locally.
--ss
Flags: needinfo?(margaret.leibovic)
Reporter | ||
Comment 4•11 years ago
|
||
This issue is that testBookmarkKeyword is currently disabled because of bug 915350.
So, yes, let's not work on this bug until that issue is resolved.
Assignee | ||
Updated•11 years ago
|
Mentor: margaret.leibovic
Whiteboard: [mentor=margaret][lang=java] → [lang=java]
Comment 5•10 years ago
|
||
Can I work on this? Do I just have to make a new keyword variable with "%s" added in it and update the bookmark with it? Sorry if I did not understand what needs to be done. :) I'm kinda new.
Comment 6•10 years ago
|
||
Oh, I also have my build environment set and I also know how to run robocop tests locally.
Comment 7•4 years ago
|
||
We have completed our launch of our new Firefox on Android. The development of the new versions use GitHub for issue tracking. If the bug report still reproduces in a current version of [Firefox on Android nightly](https://play.google.com/store/apps/details?id=org.mozilla.fenix) an issue can be reported at the [Fenix GitHub project](https://github.com/mozilla-mobile/fenix/). If you want to discuss your report please use [Mozilla's chat](https://wiki.mozilla.org/Matrix#Connect_to_Matrix) server https://chat.mozilla.org and join the [#fenix](https://chat.mozilla.org/#/room/#fenix:mozilla.org) channel.
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → INCOMPLETE
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•