Skip to content
This repository has been archived by the owner on Apr 7, 2022. It is now read-only.

[1LP][RFR] Fixing jenkins failure #10012

Merged

Conversation

mnadeem92
Copy link
Contributor

Signed-off-by: mnadeem92 [email protected]

{{ pytest: cfme/tests/v2v/test_csv_import.py -k "test_csv_security_group_flavor" --use-provider osp13-ims --use-provider vsphere67-ims --provider-limit 2 -v }}

@dajoRH dajoRH changed the title [RFR] Fixing jenkins failure [WIP] Fixing jenkins failure Mar 24, 2020
@mnadeem92 mnadeem92 force-pushed the fix_test_csv_security_group_flavor_test branch from 04416e0 to de9cbb9 Compare March 24, 2020 10:59
@mnadeem92 mnadeem92 changed the title [WIP] Fixing jenkins failure [RFR] Fixing jenkins failure Mar 24, 2020
@dajoRH dajoRH added lint-ok and removed needs-lint labels Mar 24, 2020
cfme/tests/v2v/test_csv_import.py Outdated Show resolved Hide resolved
cfme/tests/v2v/test_csv_import.py Outdated Show resolved Hide resolved
table_hover=True, security_group=True)
assert not DeepDiff(actual_attributes, expected_attributes)

soft_assert(expected_attributes["security_group"] == security_group)
Copy link
Contributor

Choose a reason for hiding this comment

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

Why are you using soft_assert here instead of assert?
From what I understand, you only use soft_assert when you have to assert more than 1 statement in a single test to pass and you don't want your test to break in the middle if any one of the assert statements fails.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

down the line we have one more assert which is why i have used soft_assert above.

@valaparthvi valaparthvi added test-cleanup Test removal, collection changes, re-organization fix-test labels Mar 24, 2020
Copy link
Contributor

@sshveta sshveta left a comment

Choose a reason for hiding this comment

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

Looks good 👍 , Only implement Parthvi's comments

@mnadeem92 mnadeem92 force-pushed the fix_test_csv_security_group_flavor_test branch from de9cbb9 to 0ac71c4 Compare March 25, 2020 07:28
@valaparthvi valaparthvi changed the title [RFR] Fixing jenkins failure [1LP][RFR] Fixing jenkins failure Mar 25, 2020
@digitronik digitronik self-assigned this Mar 25, 2020
@digitronik digitronik merged commit 4fda656 into ManageIQ:master Mar 25, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
fix-test lint-ok test-cleanup Test removal, collection changes, re-organization
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants