Closed Bug 756445 Opened 13 years ago Closed 13 years ago

redefinition of test method in test_AUS.py

Categories

(Release Engineering :: General, defect, P2)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: bhearsum, Assigned: nthomas)

References

Details

Attachments

(1 file)

I was running Balrog through pyflakes this afternoon and it said this: auslib/test/test_AUS.py:123: redefinition of function 'testSpecialQueryParamForced' from line 107 And indeed, we have two methods named the same in that class: https://github.com/mozilla/balrog/blob/master/auslib/test/test_AUS.py#L107 https://github.com/mozilla/balrog/blob/master/auslib/test/test_AUS.py#L123
Assignee: nobody → nrthomas
Priority: -- → P2
Attached patch Fix tests (deleted) — Splinter Review
Looks like a copy-and-paste-o for the forcing test for a non-bouncer download site. The other change here is to resolve this message while running tests: .../balrog/auslib/web/views/forms.py:31: DeprecationWarning: BaseException.message has been deprecated as of Python 2.6 self.process_errors.append(e.message) I picked up usage in wtforms/fields.py where it's also catching ValueError.
Attachment #625865 - Flags: review?(bhearsum)
... and all the tests PASS afterwards, but that's otherwise untested.
Attachment #625865 - Flags: review?(bhearsum) → review+
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering
Component: General Automation → General
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: