refactor(test): remove xfail during pytest_generate_tests
before parameterizing
ref: #938
This commit is contained in:
@ -262,6 +262,15 @@ def pytest_generate_tests(metafunc):
|
||||
|
||||
if len(arg_values) > 0:
|
||||
|
||||
# Get the test method
|
||||
test_func = getattr(metafunc.cls, metafunc.definition.name, None)
|
||||
|
||||
# Remove previous xfail mark if present
|
||||
if test_func and hasattr(test_func, 'pytestmark'):
|
||||
test_func.pytestmark = [
|
||||
mark for mark in test_func.pytestmark if mark.name != 'xfail'
|
||||
]
|
||||
|
||||
metafunc.parametrize(
|
||||
argnames = [
|
||||
*fixture_parameters
|
||||
|
@ -148,7 +148,7 @@ class TicketCommentBaseModelTestCases(
|
||||
"is_closed": {
|
||||
'blank': False,
|
||||
'default': False,
|
||||
'field_type': models.fields.IntegerField,
|
||||
'field_type': models.fields.BooleanField,
|
||||
'null': False,
|
||||
'unique': False,
|
||||
},
|
||||
|
@ -54,7 +54,7 @@ class GitGroupModelTestCases(
|
||||
'name': {
|
||||
'blank': False,
|
||||
'default': models.fields.NOT_PROVIDED,
|
||||
'field_type': models.IntegerField,
|
||||
'field_type': models.CharField,
|
||||
'length': 80,
|
||||
'null': False,
|
||||
'unique': False,
|
||||
@ -62,7 +62,7 @@ class GitGroupModelTestCases(
|
||||
'path': {
|
||||
'blank': False,
|
||||
'default': models.fields.NOT_PROVIDED,
|
||||
'field_type': models.IntegerField,
|
||||
'field_type': models.CharField,
|
||||
'length': 80,
|
||||
'null': False,
|
||||
'unique': False,
|
||||
@ -70,7 +70,7 @@ class GitGroupModelTestCases(
|
||||
'description': {
|
||||
'blank': True,
|
||||
'default': models.fields.NOT_PROVIDED,
|
||||
'field_type': models.IntegerField,
|
||||
'field_type': models.TextField,
|
||||
'max_length': 80,
|
||||
'null': True,
|
||||
'unique': False,
|
||||
|
@ -25,7 +25,7 @@ class GitLabRepositoryBaseModelTestCases(
|
||||
'visibility': {
|
||||
'blank': False,
|
||||
'default': models.NOT_PROVIDED,
|
||||
'field_type': models.BooleanField,
|
||||
'field_type': models.IntegerField,
|
||||
'null': False,
|
||||
'unique': False,
|
||||
}
|
||||
|
@ -53,7 +53,7 @@ class SoftwareEnableFeatureFlagModelTestCases(
|
||||
'enabled': {
|
||||
'blank': False,
|
||||
'default': False,
|
||||
'field_type': models.IntegerField,
|
||||
'field_type': models.BooleanField,
|
||||
'null': False,
|
||||
'unique': False,
|
||||
},
|
||||
|
@ -36,14 +36,14 @@ class AppSettingsModelTestCases(
|
||||
'model_notes': {
|
||||
'blank': models.fields.NOT_PROVIDED,
|
||||
'default': models.fields.NOT_PROVIDED,
|
||||
'field_type': models.CharField,
|
||||
'field_type': models.fields.NOT_PROVIDED,
|
||||
'null': models.fields.NOT_PROVIDED,
|
||||
'unique': models.fields.NOT_PROVIDED,
|
||||
},
|
||||
'organization': {
|
||||
'blank': models.fields.NOT_PROVIDED,
|
||||
'default': models.fields.NOT_PROVIDED,
|
||||
'field_type': models.CharField,
|
||||
'field_type': models.fields.NOT_PROVIDED,
|
||||
'null': models.fields.NOT_PROVIDED,
|
||||
'unique': models.fields.NOT_PROVIDED,
|
||||
},
|
||||
|
@ -43,14 +43,14 @@ class UserSettingsModelTestCases(
|
||||
'model_notes': {
|
||||
'blank': models.fields.NOT_PROVIDED,
|
||||
'default': models.fields.NOT_PROVIDED,
|
||||
'field_type': models.CharField,
|
||||
'field_type': models.fields.NOT_PROVIDED,
|
||||
'null': models.fields.NOT_PROVIDED,
|
||||
'unique': models.fields.NOT_PROVIDED,
|
||||
},
|
||||
'organization': {
|
||||
'blank': models.fields.NOT_PROVIDED,
|
||||
'default': models.fields.NOT_PROVIDED,
|
||||
'field_type': models.CharField,
|
||||
'field_type': models.fields.NOT_PROVIDED,
|
||||
'null': models.fields.NOT_PROVIDED,
|
||||
'unique': models.fields.NOT_PROVIDED,
|
||||
},
|
||||
|
Reference in New Issue
Block a user