Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wrote test cases for Sprint 1 #51

Closed
wants to merge 15 commits into from
Closed

Conversation

harshini53
Copy link
Collaborator

No description provided.

Copy link

github-actions bot commented Jan 15, 2024

Test Results

0 tests  ±0   0 ✅ ±0   0s ⏱️ ±0s
0 suites ±0   0 💤 ±0 
0 files   ±0   0 ❌ ±0 

Results for commit 5232810. ± Comparison against base commit 5edc3cb.

♻️ This comment has been updated with latest results.

Copy link
Owner

@LWJ-Nicholas LWJ-Nicholas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The test cases should not be continuous. Because if one test case fails, then the test cases following it will also fail accordingly. Each test case should be separate (able to run independently) and not be dependent on the previous test case.

Also since we are reusing most of the elements (eg. Cancel, Create, etc) it is better to store all of them in variables. And remove all the old code that was commented out to make it less confusing.

create_new_entry.robot Show resolved Hide resolved
create_new_account.robot Outdated Show resolved Hide resolved
create_new_account.robot Outdated Show resolved Hide resolved
create_new_account.robot Outdated Show resolved Hide resolved
create_new_account.robot Outdated Show resolved Hide resolved
Input Text id=username ${login_user_username}
Input Text id=password 123456789
Click Element xpath=//button[contains(text(),'Login')]
Wait Until Element Is Visible css=.alert.alert-danger
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Put this into a variable (Error message)
ensure that the relevant error message is prompted.

# Input Text id=username
# Input Text id=password
Click Element xpath=//button[contains(text(),'Login')]
Wait Until Element Is Visible css=.alert.alert-danger
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Put this into a variable (Error message)
ensure that the relevant error message is prompted.

Entering Login Details - Fail Blank for both input fields
# Input Text id=username
# Input Text id=password
Click Element xpath=//button[contains(text(),'Login')]
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

reuse login variable

Entering Login Details - Fail Diff Username and password
Input Text id=username ${login_admin_username}
Input Text id=password ${login_user_password}
Click Element xpath=//button[contains(text(),'Login')]
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

reuse login variable

Input Text id=username ${login_admin_username}
Input Text id=password ${login_user_password}
Click Element xpath=//button[contains(text(),'Login')]
Wait Until Element Is Visible css=.alert.alert-danger
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Put this into a variable (Error message)
ensure that the relevant error message is prompted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants