Closed
Bug 566507
Opened 14 years ago
Closed 12 years ago
Review log out testcase
Categories
(Testing Graveyard :: WebQA, defect)
Testing Graveyard
WebQA
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: krupa.mozbugs, Unassigned)
Details
Attachments
(5 files)
Simple testcase to check Log out functionality.Uses AMOfunctions
Attachment #445859 -
Flags: review?(vish.mozilla)
Attachment #445859 -
Flags: review?(trumanwsmith)
Attachment #445859 -
Flags: review?(stephen.donner)
Attachment #445859 -
Flags: review?(mozbugs.retornam)
Reporter | ||
Comment 1•14 years ago
|
||
Updated•14 years ago
|
Attachment #445859 -
Attachment mime type: text/html → text/x-python-script
Comment 2•14 years ago
|
||
the top most 'if' statement needs an else too. if ret == amo_fcns.functionPass: 'some code' else: unittest.fail('some msg')
Reporter | ||
Comment 3•14 years ago
|
||
I made the changes you suggested.Thanks
Attachment #446041 -
Flags: review?(vish.mozilla)
Updated•14 years ago
|
Attachment #446041 -
Attachment mime type: text/html → text/x-python-script
Comment 4•14 years ago
|
||
My suggested code was pseudo-code (sorry for not making that clear). The real code should look like else: unittest.TestCase.fail('some msg')
Reporter | ||
Comment 5•14 years ago
|
||
Reporter | ||
Updated•14 years ago
|
Attachment #446083 -
Flags: review?(vish.mozilla)
Updated•14 years ago
|
Attachment #445859 -
Attachment mime type: text/x-python-script → text/plain
Updated•14 years ago
|
Attachment #445859 -
Flags: review?(vish.mozilla) → review+
Reporter | ||
Comment 6•14 years ago
|
||
Updated the formatting to match pep rules
Attachment #446632 -
Flags: review?(vish.mozilla)
Comment 7•14 years ago
|
||
looks good. commit it.
Updated•14 years ago
|
Attachment #446632 -
Flags: review?(vish.mozilla) → review+
Updated•12 years ago
|
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•12 years ago
|
Attachment #445859 -
Attachment filename: verifyUserLogout.py → verifyUserLogout.py
Attachment #445859 -
Flags: review?(trumanwsmith)
Attachment #445859 -
Flags: review?(stephen.donner)
Attachment #445859 -
Flags: review?(mozbugs.retornam)
Updated•6 years ago
|
Product: Testing → Testing Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•