#78 Migration not working properly - Integrity

Type	To find
responsible:me	tickets assigned to you
tagged:"@high"	tickets tagged @high
milestone:next	tickets in the upcoming milestone
state:invalid	tickets with the state invalid
created:"last week"	tickets created last week
sort:number, importance, updated	tickets sorted by #, importance or updated
Combine keywords for powerful searching.
Use advanced searching »

#78 ✓resolved

Migration not working properly

Reported by Simon Rozet | January 27th, 2009 @ 04:44 PM | in 0.1.9

Steps to reproduce:

Clone master at a3b778400630ee7beb37981a56b6de5f980978d9
rake gemspec
gemi integrity-0.1.5.gem
integrity ~/test-integrity
cd ~/test-integrity && echo ".schema" | sqlite3 integrity.db

BOUM, there is only the migration table

(Thanks to copious on #integrity for the report)

Comments and changes to this ticket

You flagged this item as spam.
Jeremy Hinegardner January 28th, 2009 @ 08:18 AM
Fixed in http://github.com/copiousfreetim...

It boils down to the MigrationRunner being a module and all the migrations that are created are stored in @migrations of the extended class.

In integrity, when the migrations are parsed, the Object instance has the @migrations member is 'main' or Kernel. When the migrate_up! method is invoked, that is being invoked upon the Integrity::Installer which does not have a populated @migrations member variable.
You flagged this item as spam.
ronin-278 (at lighthouseapp) January 28th, 2009 @ 07:19 PM
(from [ac9f0bf499a21dc2d301b813ce9cdd5f1906070d]) failing test for migration not working properly [#78] http://github.com/foca/integrity...
You flagged this item as spam.
ronin-278 (at lighthouseapp) January 28th, 2009 @ 07:19 PM
- State changed from “new” to “resolved”
(from [178ec7c6539d4307bb311270849d6007eb44abc8]) Fix migration not working properly [#78 state:resolved]

Signed-off-by: Simon Rozet simon@rozet.name http://github.com/foca/integrity...
You flagged this item as spam.
ronin-278 (at lighthouseapp) January 29th, 2009 @ 04:47 PM
(from [62db4053cacab2b59c533f93b6d40953b463aa18]) Fix Installer#migrate_db [#78 state:resolved]

NOTE: The test is ugly but at least it's tested

Signed-off-by: Simon Rozet simon@rozet.name http://github.com/foca/integrity...
You flagged this item as spam.
Derek Kastner January 30th, 2009 @ 09:02 PM
I am still getting a database with just the migration table in it. I am grabbing the latest Integrity sources and building the gem manually.
You flagged this item as spam.
Derek Kastner January 30th, 2009 @ 09:08 PM
I am still getting a database with just the migration table in it. I am grabbing the latest Integrity sources and building the 0.1.6 gem manually.
You flagged this item as spam.
Derek Kastner January 30th, 2009 @ 09:11 PM
I am still getting a database with just the migration table in it. I am grabbing the latest Integrity sources and building the 0.1.6 gem manually.
You flagged this item as spam.
Derek Kastner January 30th, 2009 @ 09:12 PM
I am still getting a database with just the migration table in it. I am grabbing the latest Integrity sources and building the 0.1.6 gem manually.
You flagged this item as spam.
Derek Kastner January 30th, 2009 @ 09:14 PM
I am still getting a database with just the migration table in it. I am grabbing the latest Integrity sources and building the 0.1.6 gem manually.
You flagged this item as spam.
Derek Kastner January 30th, 2009 @ 09:17 PM
I am still getting a database with just the migration table in it. I am grabbing the latest Integrity sources and building the 0.1.6 gem manually.
You flagged this item as spam.
Derek Kastner January 30th, 2009 @ 09:31 PM
Sorry I was on the master branch - cherry-picked from background_building
You flagged this item as spam.
Derek Kastner January 30th, 2009 @ 09:34 PM
Sorry I was on the master branch - cherry-picked from background_building
You flagged this item as spam.
ronin-278 (at lighthouseapp) January 31st, 2009 @ 10:53 PM
(from [fe801dfed621348256fc5dfbecf3deb4442fe746]) Fix Installer#migrate_db [#78 state:resolved]

NOTE: The test is ugly but at least it's tested

Signed-off-by: Simon Rozet simon@rozet.name http://github.com/foca/integrity...

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.

Create new ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile »

Automated continuous integration server that doesn't suck.

Shared Ticket Bins (Sort)

↓↑ drag 0 Need Review
↓↑ drag 5 Patches
↓↑ drag 0 Discussions
↓↑ drag 13 Pending

People watching this ticket

Referenced by

78 Migration not working properly (from [ac9f0bf499a21dc2d301b813ce9cdd5f1906070d]) failing...
78 Migration not working properly (from [178ec7c6539d4307bb311270849d6007eb44abc8]) Fix mig...
78 Migration not working properly (from [62db4053cacab2b59c533f93b6d40953b463aa18]) Fix Ins...
78 Migration not working properly (from [fe801dfed621348256fc5dfbecf3deb4442fe746]) Fix Ins...

Integrity Integrity → 0.1.9