Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1522)

Issue 20780043: Make searching for a bundle's basket name work.

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 6 months ago by bac
Modified:
10 years, 6 months ago
Reviewers:
rharding, benji, mp+193593
Visibility:
Public.

Description

Make searching for a bundle's basket name work. The search field was incorrectly listed as 'basket' not 'basket_name'. Also when creating a new test to search for the basket name using the old test data 'wordpress-basket' the test inexplicably passed even when the search code was broken. Upon reflection it is due to the way the search term is tokenized. The test was updated to use a unique term for the basket name which solved the false positive problem. https://code.launchpad.net/~bac/charmworld/bug-1246459/+merge/193593 (do not edit description out of merge proposal)

Patch Set 1 #

Total comments: 3
Unified diffs Side-by-side diffs Delta from patch set Stats (+18 lines, -3 lines) Patch
A [revision details] View 1 chunk +2 lines, -0 lines 0 comments Download
M charmworld/search.py View 1 chunk +1 line, -1 line 1 comment Download
M charmworld/tests/test_search.py View 2 chunks +15 lines, -2 lines 2 comments Download

Messages

Total messages: 4
bac
Please take a look.
10 years, 6 months ago (2013-11-01 13:01:27 UTC) #1
bac
I forgot to mention, for QA purposes I have created a bundle on LP with ...
10 years, 6 months ago (2013-11-01 13:09:26 UTC) #2
rharding
code looks good with one scoring comment. Doing QA https://codereview.appspot.com/20780043/diff/1/charmworld/search.py File charmworld/search.py (right): https://codereview.appspot.com/20780043/diff/1/charmworld/search.py#newcode46 charmworld/search.py:46: ...
10 years, 6 months ago (2013-11-01 14:18:28 UTC) #3
benji
10 years, 6 months ago (2013-11-01 15:55:00 UTC) #4
QA LGTM
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b