Fair enough, I'll prepare another PR then
ᐧ


On Wed, Jun 4, 2014 at 4:16 PM, Mick Hansen <notifications@github.com>
wrote:

> In test/migrator.test.js:
>
> > @@ -156,16 +156,20 @@ describe(Support.getTestDialectTeaser("Migrator"), function() {
> >
> >        it("executes migration #20111117063700 correctly up (createTable) and downwards (dropTable)", function(done) {
> >          var self = this
> > -
> > -        this.sequelize.getQueryInterface().showAllTables().success(function(tableNames) {
> > -          tableNames = tableNames.filter(function(e){ return e != 'SequelizeMeta' })
> > -          expect(tableNames).to.eql([ 'Person' ])
> > -
> > -          self.migrator.migrate({ method: 'down' }).success(function() {
> > -            self.sequelize.getQueryInterface().showAllTables().success(function(tableNames) {
> > -              tableNames = tableNames.filter(function(e){ return e != 'SequelizeMeta' })
> > -              expect(tableNames).to.eql([])
> > -              done()
> > +        this.init({
>
> I see, but shouldn't that be a seperate test then. This test tests on
> actual migrations, it shouldn't be replaced.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sequelize/sequelize/pull/1857/files#r13387947>.
>