diff --git a/scripts/testng-junit/src/testng2junit5.py b/scripts/testng-junit/src/testng2junit5.py index 6870b9b587..6b66f40b54 100644 --- a/scripts/testng-junit/src/testng2junit5.py +++ b/scripts/testng-junit/src/testng2junit5.py @@ -114,8 +114,6 @@ def migrate_imports(content): # refer to migrate_guice_annotation if '@Guice' in content_new and '@BeforeAll' not in content_new: imports.append('import org.junit.jupiter.api.BeforeAll;') - imports.append('import org.junit.jupiter.api.extension.ExtendWith;') - imports.append('import com.addepar.infra.library.testing.extention.GuiceInjectionExtension;') content_new = re.sub('org.junit.jupiter.api.Test;', '\n'.join(imports), content_new) @@ -417,7 +415,6 @@ def migrate_guice_annotation(content): or 'public abstract class' in line): left_spaces = ' ' * (len(line) - len(line.lstrip())) new_content.append(left_spaces + '@TestInstance(Lifecycle.PER_CLASS)') - new_content.append(left_spaces + '@ExtendWith(GuiceInjectionExtension.class)') new_content.append(line) if '{' in line: diff --git a/scripts/testng-junit/src/tests/test_migrate_guice_annotation.py b/scripts/testng-junit/src/tests/test_migrate_guice_annotation.py index dd68710870..409458931c 100644 --- a/scripts/testng-junit/src/tests/test_migrate_guice_annotation.py +++ b/scripts/testng-junit/src/tests/test_migrate_guice_annotation.py @@ -18,7 +18,6 @@ expected_1 = """ @TestInstance(Lifecycle.PER_CLASS) - @ExtendWith(GuiceInjectionExtension.class) public class SomeTest { private final Injector injector = Guice.createInjector(new SomeModule()); @@ -46,7 +45,6 @@ expected_2 = """ @TestInstance(Lifecycle.PER_CLASS) - @ExtendWith(GuiceInjectionExtension.class) public class SomeTest { private final Injector injector = Guice.createInjector(); @@ -75,7 +73,6 @@ expected_3 = """ @TestInstance(Lifecycle.PER_CLASS) - @ExtendWith(GuiceInjectionExtension.class) public class SomeTest { private final Injector injector = Guice.createInjector(new TestModuleA(), new Test.ModuleB());