From aaaa49a65ff39387ec94178b8a272d2c6ae304f2 Mon Sep 17 00:00:00 2001 From: "Anna.Kozlova" Date: Thu, 6 Jul 2017 18:07:37 +0200 Subject: [PATCH] ensure non-abstract producer don't expose ConfigurationType in constructor plugin compatibility: Application component which is at the same time ConfigurationType would produce errors --- .../AbstractTestNGInClassConfigurationProducer.java | 8 +------- .../configuration/TestNGInClassConfigurationProducer.java | 6 +----- 2 files changed, 2 insertions(+), 12 deletions(-) diff --git a/plugins/testng/src/com/theoryinpractice/testng/configuration/AbstractTestNGInClassConfigurationProducer.java b/plugins/testng/src/com/theoryinpractice/testng/configuration/AbstractTestNGInClassConfigurationProducer.java index a41412ca4a6b..0465506fb6d4 100644 --- a/plugins/testng/src/com/theoryinpractice/testng/configuration/AbstractTestNGInClassConfigurationProducer.java +++ b/plugins/testng/src/com/theoryinpractice/testng/configuration/AbstractTestNGInClassConfigurationProducer.java @@ -15,17 +15,11 @@ */ package com.theoryinpractice.testng.configuration; -import com.intellij.execution.configurations.ConfigurationType; //to be delete in 2018 @Deprecated -public class AbstractTestNGInClassConfigurationProducer extends TestNGInClassConfigurationProducer { +public abstract class AbstractTestNGInClassConfigurationProducer extends TestNGInClassConfigurationProducer { @Deprecated public AbstractTestNGInClassConfigurationProducer() { } - - @Deprecated - public AbstractTestNGInClassConfigurationProducer(ConfigurationType configurationType) { - super(configurationType); - } } diff --git a/plugins/testng/src/com/theoryinpractice/testng/configuration/TestNGInClassConfigurationProducer.java b/plugins/testng/src/com/theoryinpractice/testng/configuration/TestNGInClassConfigurationProducer.java index d54f1a69d128..16a785f99aff 100644 --- a/plugins/testng/src/com/theoryinpractice/testng/configuration/TestNGInClassConfigurationProducer.java +++ b/plugins/testng/src/com/theoryinpractice/testng/configuration/TestNGInClassConfigurationProducer.java @@ -26,11 +26,7 @@ import org.jetbrains.annotations.NotNull; public class TestNGInClassConfigurationProducer extends TestNGConfigurationProducer { private TestNGInClassConfigurationProducerDelegate myDelegate = new TestNGInClassConfigurationProducerDelegate(TestNGConfigurationType.getInstance()); protected TestNGInClassConfigurationProducer() { - this(TestNGConfigurationType.getInstance()); - } - - protected TestNGInClassConfigurationProducer(final ConfigurationType configurationType) { - super(configurationType); + super(TestNGConfigurationType.getInstance()); } @Override