[1LP][RFR] New Test: Test tag delete using automate method #9616
Conversation
ganeshhubale
force-pushed
the
tag_delete
branch
from
0d8a04a to
928b7a6
Compare
- Testing deletion of tag using automate method Signed-off-by: Ganesh Hubale <[email protected]>
ganeshhubale
force-pushed
the
tag_delete
branch
from
928b7a6 to
6cdb903
Compare
ganeshhubale
changed the title
digitronik
changed the title
| @pytest.mark.tier(2) | ||
| @pytest.mark.meta(blockers=[BZ(1744514)], automates=[1744514]) | ||
| @pytest.mark.customer_scenario | ||
| def test_delete_tag_from_category(custom_instance): |
There was a problem hiding this comment.
Suggestion:
Whole test is around CRUD via Automate method, test case name should be something like: test_automate_method_delete_tag_from_category or test_automate_delete_tag_from_category to have consistency around it.
There was a problem hiding this comment.
I don't think we need to change test name. It is already under automate related files. hence keeping as it is.
There was a problem hiding this comment.
Yes, @ganeshhubale I agree with you about the test is present in the same file, but looking all of your tests in the same file have suggested name, implementation is up to you 😄
There was a problem hiding this comment.
As long as there isn't duplication of the test case names across test modules, I think the naming here is fine, as some context is given by the module.
mshriver
added
test-automation
Purpose or Intent
tag_deletemethod introduced for deletion of tags under categorytest_create,tag_existsetctest_archived_instance_statusandtest_orphaned_instance_statusare already automated in test casetest_power_options_on_archived_instance_all_pagePRT Run
{{ pytest: cfme/tests/automate/test_method.py::test_delete_tag_from_category --use-template-cache -qsvvv }}