fix(install): fix upgrading from older versions (#245)

* fix(install): fix upgrading from older versions

Resolves #243

* refactor: remove migrations that will never be run

* test: add tests

* test: add unhappy flow test case

* refactor: move account update to pdk migration

* test: increase coverage

myparcelnl.php

@@ -142,20 +142,20 @@ public function uninstall(): bool
     /**
      * @return string
      */
-    private function getBaseUrl(): string
+    protected function getVersionFromComposer(): string
     {
-        return $this->context->link->getAdminLink('AdminModules');
+        $filename     = __DIR__ . '/composer.json';
+        $composerData = json_decode(file_get_contents($filename), true);
+
+        return $composerData['version'];
     }
 
     /**
      * @return string
      */
-    private function getVersionFromComposer(): string
+    private function getBaseUrl(): string
     {
-        $filename     = __DIR__ . '/composer.json';
-        $composerData = json_decode(file_get_contents($filename), true);
-
-        return $composerData['version'];
+        return $this->context->link->getAdminLink('AdminModules');
     }
 
     /**

src/Facade/MyParcelModule.php

@@ -4,13 +4,15 @@
 
 namespace MyParcelNL\PrestaShop\Facade;
 
+use Module;
 use MyParcelNL;
 use MyParcelNL\Pdk\Base\Facade;
 use MyParcelNL\PrestaShop\Service\ModuleService;
 
 /**
  * @method static MyParcelNL getInstance()
  * @method static void registerHooks()
+ * @method static bool install(Module $module)
  * @see \MyParcelNL\PrestaShop\Service\ModuleService
  */
 final class MyParcelModule extends Facade

src/Migration/AbstractPsMigration.php

@@ -13,6 +13,11 @@
 
 abstract class AbstractPsMigration implements MigrationInterface
 {
+    public const LEGACY_TABLE_ORDER_LABEL           = 'myparcelnl_order_label';
+    public const LEGACY_TABLE_CARRIER_CONFIGURATION = 'myparcelnl_carrier_configuration';
+    public const LEGACY_TABLE_DELIVERY_SETTINGS     = 'myparcelnl_delivery_settings';
+    public const LEGACY_TABLE_PRODUCT_CONFIGURATION = 'myparcelnl_product_configuration';
+
     /**
      * @var \Db
      */