Closed Bug 927240 Opened 11 years ago Closed 7 years ago

Support ANDed filtering in MozTrap

Categories

(Mozilla QA Graveyard :: MozTrap, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: sbai, Unassigned)

References

Details

Attachments

(1 file, 1 obsolete file)

42 bytes, text/x-github-pull-request
camd
: review-
Details | Review
+++ This bug was initially created as a clone of Bug #900678 +++

I'm going to release an experimental patch to support ANDed tags filtering.
If possible, ANDed product version, creator etc. will be supported later.

Currently in MozTrap, query foramt in GET method is like filter-tag=246&filter-tag=2114&filter-productversion=103.
And filter conditions in the same group will be merged into a list and perform a SELECT IN SQL query, e.g. SELECT ... WHERE tags in (246, 2114), thus the result is ORed.

My approach:
Try to support new kinds of tag formats in URL GET method, like filter-and-tag, filter-and-product, etc.
With these tags specified in url, the ANDed filtering will be performed in the corresponding filter group instead.
Attached file pr (obsolete) —
https://github.com/mozilla/moztrap/pull/38

Can you help to review the patch?
Thanks
Attachment #817678 - Flags: review?(cdawson)
Usage:
First, copy the shraing link of your query.
Then append &filter-and-<key> (e.g. &filter-and-tag) to the link.
Use the modified link to open the ANDed result page.

Support keys: tag, product, productversion, creator, suite, run, envelement...
Hi Matt,
Could you please help review?

Thanks
Flags: needinfo?(mbrandt)
https://github.com/mozilla/moztrap/pull/40

I resend a pull request since I move this bug patch to a new branch
Attachment #817678 - Attachment is obsolete: true
Attachment #817678 - Flags: review?(cdawson)
Attachment #819629 - Flags: review?(cdawson)
Hi shiro - thanks for the pull request. I'll take a look at it, but it needs other review. The rest of the team is on vacation, so please be patient with us.
Flags: needinfo?(mbrandt)
Here's my comment from github in the PR:

@sh1r0 : I am actually not able to get this to work for me, locally. Is this what you had in mind for the url?
http://localhost:8000/manage/cases/?filter-and-tag=8&filter-and-tag=7

I was expecting it to show me only cases with BOTH of those tags. But it seems as though it's not doing any filtering at all. Would you double-check this still works for you? Thanks!
Comment on attachment 819629 [details] [review]
support ANDed filtering

Shiro decided to drop this feature
Attachment #819629 - Flags: review?(cdawson) → review-
Mass-closing remaining MozTrap bugs as WONTFIX, due to 1) the Mozilla-hosted instance being decommissioned (see https://wiki.mozilla.org/TestEngineering/Testrail), and, for now, 2) the still-up code archived at its GitHub page: https://github.com/mozilla/moztrap (we'll decide what's next for that, in the near future).

See also the history and more-detailed discussion which led us here, at https://groups.google.com/forum/#!topic/mozilla.dev.quality/Sa75hV8Ywvk

(If you'd like, you should be able to filter these notification emails using at least the unique string of "Sa75hV8Ywvk" in the message body.

Thanks!
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
Product: Mozilla QA → Mozilla QA Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: