From e188078dfd9febe9ed845a0f945dce61dfb89b73 Mon Sep 17 00:00:00 2001 From: Dmitry Batkovich Date: Tue, 25 Sep 2018 08:45:16 +0300 Subject: [PATCH] "name" attribute can be null --- .../DuplicatedDataProviderNamesInspection.java | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/plugins/testng/src/com/theoryinpractice/testng/inspection/DuplicatedDataProviderNamesInspection.java b/plugins/testng/src/com/theoryinpractice/testng/inspection/DuplicatedDataProviderNamesInspection.java index a5cf8f258753..7397ce2b3b6c 100644 --- a/plugins/testng/src/com/theoryinpractice/testng/inspection/DuplicatedDataProviderNamesInspection.java +++ b/plugins/testng/src/com/theoryinpractice/testng/inspection/DuplicatedDataProviderNamesInspection.java @@ -34,12 +34,13 @@ public class DuplicatedDataProviderNamesInspection extends AbstractBaseJavaLocal final PsiAnnotation annotation = AnnotationUtil.findAnnotation(method, dataProviderFqn); if (annotation != null) { final PsiAnnotationMemberValue value = annotation.findAttributeValue(NAME_ATTRIBUTE); - LOG.assertTrue(value != null); - final String dataProviderName = PsiTreeUtil.isAncestor(annotation, value, false) - ? AnnotationUtil.getStringAttributeValue(annotation, NAME_ATTRIBUTE) - : method.getName(); - if (dataProviderName != null) { - dataProvidersByName.putValue(dataProviderName, method); + if (value != null) { + final String dataProviderName = PsiTreeUtil.isAncestor(annotation, value, false) + ? AnnotationUtil.getStringAttributeValue(annotation, NAME_ATTRIBUTE) + : method.getName(); + if (dataProviderName != null) { + dataProvidersByName.putValue(dataProviderName, method); + } } } }