566507 - Review log out testcase

Status: RESOLVED FIXED

(Testing Graveyard :: WebQA, defect)

Description

[krupa raj[:krupa]]

Attachment: verify User logout

Simple testcase to check Log out functionality.Uses AMOfunctions

Comment 1

[krupa raj[:krupa]]

Attachment: AMOfunctions

Comment 2

[Vishal Kamdar [:vish_moz]]

the top most 'if' statement needs an else too.
if ret  == amo_fcns.functionPass:
    'some code'
else:
    unittest.fail('some msg')

Comment 3

[krupa raj[:krupa]]

Attachment: Updated testcase

I made the changes you suggested.Thanks

Comment 4

[Vishal Kamdar [:vish_moz]]

My suggested code was pseudo-code (sorry for not making that clear).
The real code should look like
else:
    unittest.TestCase.fail('some msg')

Comment 5

[krupa raj[:krupa]]

Attachment: Third time is the charm...

Comment 6

[krupa raj[:krupa]]

Attachment: Logout tc-05/20

Updated the formatting to match pep rules

Comment 7

[Vishal Kamdar [:vish_moz]]

looks good. commit it.