Fix: Ensure scoped migration created in subdirectory

Anson Tsao · Anson Tsao · commit cd128e2e0484 · 2019-11-08T16:35:06.000-08:00
db-migrate create someScope/aMigration --sql-file, the following is created:

./migrations/someScope/xxx-someScope.js
./migrations/sqls/xxx-someScope-up.sql
./migrations/sqls/xxx-someScope-down.sql
Instead, I expect:

./migrations/someScope/xxx-aMigration.js
./migrations/someScope/sqls/xxx-aMigration-up.sql
./migrations/someScope/sqls/xxx-aMigration-down.sql

Signed-off-by: Anson Tsao &lt;atsao@goaddy.com&gt;
diff --git a/lib/commands/create-migration.js b/lib/commands/create-migration.js
@@ -51,7 +51,7 @@ async function executeCreateMigration (internals, config) {
   internals.argv.title = internals.argv._.shift();
   folder = internals.argv.title.split('/');
 
-  internals.argv.title = folder[folder.length - 2] || folder[0];
+  internals.argv.title = folder[folder.length - 1] || folder[0];
   path = migrationsDir;
 
   if (folder.length > 1) {
@@ -61,6 +61,7 @@ async function executeCreateMigration (internals, config) {
       path += folder[i] + '/';
     }
   }
+  internals.argv['migrations-dir'] = path
 
   let templateType = Migration.TemplateType.DEFAULT_JS;
   if (

Original file line number	Diff line number	Diff line change
`@@ -51,7 +51,7 @@ async function executeCreateMigration (internals, config) {`
`51`	`51`	`internals.argv.title = internals.argv._.shift();`
`52`	`52`	`folder = internals.argv.title.split('/');`
`53`	`53`
`54`		`- internals.argv.title = folder[folder.length - 2] \|\| folder[0];`
	`54`	`+ internals.argv.title = folder[folder.length - 1] \|\| folder[0];`
`55`	`55`	`path = migrationsDir;`
`56`	`56`
`57`	`57`	`if (folder.length > 1) {`
`@@ -61,6 +61,7 @@ async function executeCreateMigration (internals, config) {`
`61`	`61`	`path += folder[i] + '/';`
`62`	`62`	`}`
`63`	`63`	`}`
	`64`	`+ internals.argv['migrations-dir'] = path`
`64`	`65`
`65`	`66`	`let templateType = Migration.TemplateType.DEFAULT_JS;`
`66`	`67`	`if (`