fix(backend): rename modules after annotations are processed (#1024)

* fix(backend): rename modules after annotations are processed

* test: update failing tests

Author: lars-reimann

api-editor/backend/src/main/kotlin/com/larsreimann/apiEditor/transformation/TransformationPlan.kt

@@ -6,18 +6,17 @@ import com.larsreimann.apiEditor.mutableModel.PythonPackage
  * Processes all annotations and updates the AST to create adapters.
  */
 fun PythonPackage.transform(newPackageName: String = "new_package") {
-    preprocess(newPackageName)
+    preprocess()
     processAnnotations()
-    postprocess()
+    postprocess(newPackageName)
 }
 
 /**
  * Transformation steps that have to be run before annotations can be processed.
  */
-private fun PythonPackage.preprocess(newPackageName: String) {
+private fun PythonPackage.preprocess() {
     removePrivateDeclarations()
     addOriginalDeclarations()
-    changeModulePrefix(newPackageName)
     replaceClassMethodsWithStaticMethods()
     updateParameterAssignment()
     normalizeNamesOfImplicitParameters()
@@ -43,8 +42,9 @@ private fun PythonPackage.processAnnotations() {
 /**
  * Transformation steps that have to be run after annotations were processed.
  */
-private fun PythonPackage.postprocess() {
+private fun PythonPackage.postprocess(newPackageName: String) {
     removeEmptyModules()
+    changeModulePrefix(newPackageName)
     reorderParameters()
     extractConstructors()
     createAttributesForParametersOfConstructor()

api-editor/backend/src/test/kotlin/com/larsreimann/apiEditor/codegen/ConstructorPythonCodeGeneratorFullPipelineTest.kt

@@ -710,10 +710,10 @@ class ConstructorPythonCodeGeneratorFullPipelineTest {
             RenameAnnotation("renamedTestClass"),
         )
         testClass.annotations.add(
-            MoveAnnotation("movedTestModule"),
+            MoveAnnotation("testModule.movedTestModule"),
         )
         // when
-        testPackage.transform()
+        testPackage.transform("newTestModule")
         val moduleContent = testPackage.modules[0].toPythonCode()
 
         // then
@@ -753,6 +753,6 @@ class ConstructorPythonCodeGeneratorFullPipelineTest {
             """.trimMargin()
 
         moduleContent shouldBe expectedModuleContent
-        testPackage.modules[0].name shouldBe "movedTestModule"
+        testPackage.modules[0].name shouldBe "newTestModule.movedTestModule"
     }
 }

api-editor/backend/src/test/kotlin/com/larsreimann/apiEditor/codegen/FunctionPythonCodeGeneratorFullPipelineTest.kt

@@ -638,10 +638,10 @@ class FunctionPythonCodeGeneratorFullPipelineTest {
             ),
         )
         testFunction.annotations.add(
-            MoveAnnotation("movedTestModule"),
+            MoveAnnotation("testModule.movedTestModule"),
         )
         // when
-        testPackage.transform()
+        testPackage.transform("newTestModule")
         val moduleContent = testPackage.modules[0].toPythonCode()
 
         // then
@@ -677,7 +677,7 @@ class FunctionPythonCodeGeneratorFullPipelineTest {
             """.trimMargin()
 
         moduleContent shouldBe expectedModuleContent
-        testPackage.modules[0].name shouldBe "movedTestModule"
+        testPackage.modules[0].name shouldBe "newTestModule.movedTestModule"
     }
 
     @Test