Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fixes not seeing changes to the type of a field #60

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

burnt43
Copy link
Contributor

@burnt43 burnt43 commented Nov 14, 2015

closes #59.

@DAddYE
Copy link
Owner

DAddYE commented Nov 16, 2015

Can you fix the tests and make a regression test?

Thanks a lot

@burnt43
Copy link
Contributor Author

burnt43 commented Nov 16, 2015

I'll correct the tests when I get a chance.

@burnt43
Copy link
Contributor Author

burnt43 commented Nov 23, 2015

The psql and sqlite3 tests are passing, but the mysql and mysql2 ones don't fully pass. I don't really understand what about my change messed it up. Maybe some of these errors are just because of my environment. I will further into it.

@burnt43 burnt43 force-pushed the master branch 2 times, most recently from e340ae4 to 685fed2 Compare November 23, 2015 06:20
@burnt43
Copy link
Contributor Author

burnt43 commented Nov 23, 2015

@DAddYE, when I look at the output of the build from Travis CI, it seems something might be messed up with bundler on whatever server those tests run on. It is not even running the tests. Its failing at bundler install. I actually reverted my changes in this PR so the Travis CI would attempt to run them again and even with my change removed it still failed at installing mini_record -v 0.4.6.

I can run the tests fine myself as you can see from below. I think maybe bundler needs to be updated on the test server.

[jcarson@burnt43 mini_record]$ rake test
/usr/local/bin/ruby -I"lib:test" -I"/usr/local/lib/ruby/1.9.1" "/usr/local/lib/ruby/1.9.1/rake/rake_test_loader.rb" "test/test_mini_record.rb" 
Testing with DB=sqlite
Run options: --seed 53860
# Running:
.........S...............SSSSSSS..S.S.S..SS..SS..
Finished in 4.313712s, 11.3591 runs/s, 59.8093 assertions/s.
49 runs, 258 assertions, 0 failures, 0 errors, 15 skips
You have skipped tests. Run with --verbose for details.

Also to note, I noticed from successful passed Travis CI builds that it only runs the sqlite3 tests. I ran the test from the other 3 adapters and found that psql passes, mysql fails on some, and mysql2 fails on some, but I guess this was not caught because it is only running sqlite3 test

@ghost
Copy link

ghost commented Feb 19, 2016

@burnt43, CI fixed, please resubmit your PR

@burnt43
Copy link
Contributor Author

burnt43 commented Feb 19, 2016

@sleewoo, done deal

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Auto changing column type don't work on PostgreSQL
2 participants