Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Fix exception when logging SQL w/ nil binary value. | James Coleman | 2014-05-02 | 1 | -0/+7 |
| | |||||
* | Don't skip tests if we don't need to. | Rafael Mendonça França | 2013-11-08 | 1 | -6/+6 |
| | | | | | | | We can conditional define the tests depending on the adapter or connection. Lets keep the skip for fail tests that need to be fixed. | ||||
* | log the sql that is actually sent to the database | Neeraj Singh | 2013-06-19 | 1 | -0/+7 |
| | | | | | | | | | | | | If I have a query that produces sql `WHERE "users"."name" = 'a b'` then in the log all the whitespace is being squeezed. So the sql that is printed in the log is `WHERE "users"."name" = 'a b'`. This can be confusing. This commit fixes it by ensuring that whitespace is not squeezed. fixes #10982 | ||||
* | Ignore binds payload with nil column in AR log subscriber | Carlos Antonio da Silva | 2013-01-08 | 1 | -16/+26 |
| | | | | | | | | | | | | | Some tests were raising the following error: Could not log "sql.active_record" event. NoMethodError: undefined method `type' for nil:NilClass` Due to the way binds were being logged, the column info was considered always present, but that is not true for some of the tests listed in the issue. Closes #8806. | ||||
* | Address test_binary_data_is_not_logged with Oracle database | Yasuo Honda | 2012-12-19 | 1 | -2/+1 |
| | | | | | | The number of sql statement logged depends on each database adapter implementation. Also, this test does not depends on how many sql statement executed. | ||||
* | Skip binary data with binds test for mysql2, fix build | Carlos Antonio da Silva | 2012-12-14 | 1 | -0/+2 |
| | | | | | | | Mysql2 doesn't support binds, which means no binds payload is set when logging, so the logic to render binary data differently here doesn't work. Introduced in 99d142a9375f9ba1960863b3cc745265aa9a14df. | ||||
* | Do not log the binding values for binary columns. | Matthew M. Boedicker | 2012-12-14 | 1 | -0/+8 |
| | | | | They tend to be large and not very useful in the log. | ||||
* | Remove ActiveRecord::Base.to_a | Jon Leighton | 2012-08-03 | 1 | -6/+6 |
| | | | | | On reflection, it seems like a bit of a weird method to have on ActiveRecord::Base, and it shouldn't be needed most of the time anyway. | ||||
* | ActiveRecord::Base.all returns a Relation. | Jon Leighton | 2012-07-27 | 1 | -6/+6 |
| | | | | | | | | | | | Previously it returned an Array. If you want an array, call e.g. `Post.to_a` rather than `Post.all`. This is more explicit. In most cases this should not break existing code, since Relations use method_missing to delegate unknown methods to #to_a anyway. | ||||
* | Remove IdentityMap | Carlos Antonio da Silva | 2012-03-13 | 1 | -12/+0 |
| | |||||
* | remove more uses of deprecated logger methods | Aaron Patterson | 2011-12-19 | 1 | -1/+1 |
| | |||||
* | Fix broken test_exists_query_logging from the table_exists? change | Jon Leighton | 2011-12-08 | 1 | -0/+1 |
| | |||||
* | Log instrumentation name for exists? queries | Jon Guymon | 2011-06-22 | 1 | -0/+8 |
| | |||||
* | please use ruby -I lib:test path/to/test.rb, or export RUBY_OPT | Aaron Patterson | 2011-06-06 | 1 | -1/+1 |
| | |||||
* | Refactor Active Record test connection setup. Please see the ↵ | Jon Leighton | 2011-06-04 | 1 | -1/+1 |
| | | | | RUNNING_UNIT_TESTS file for details, but essentially you can now configure things in test/config.yml. You can also run tests directly via the command line, e.g. ruby path/to/test.rb (no rake needed, uses default db connection from test/config.yml). This will help us fix the CI by enabling us to isolate the different Rails versions to different databases. | ||||
* | Fix failing AR test. | José Valim | 2011-05-02 | 1 | -0/+12 |
| | |||||
* | Disable IdentityMap in log tests, it's not important and when running tests ↵ | Emilio Tagua | 2011-04-05 | 1 | -0/+3 |
| | | | | | | rake task it logs more messages in the tested buffer. Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | sql logger ignores schema statements | Aaron Patterson | 2011-03-28 | 1 | -0/+27 |
| | |||||
* | Move runtime back to the Thread.current | José Valim | 2010-07-26 | 1 | -0/+4 |
| | |||||
* | Move runtime back to connection. | José Valim | 2010-07-25 | 1 | -4/+0 |
| | |||||
* | Revert the previous three commits. | José Valim | 2010-07-25 | 1 | -0/+4 |
| | | | | | | * AS::Notifications#instrument should not measure anything, it is not its responsibility; * Adding another argument to AS::Notifications#instrument API needs to be properly discussed; | ||||
* | use a hash to collect optional statistics about the instrumentation | Aaron Patterson | 2010-07-25 | 1 | -4/+0 |
| | |||||
* | runtime should be properly initialized. | José Valim | 2010-07-25 | 1 | -0/+4 |
| | |||||
* | Make use of severity levels | Santiago Pastorino | 2010-07-20 | 1 | -2/+3 |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Fix a failing test in Railtie and properly define all severity levels in ↵ | José Valim | 2010-07-20 | 1 | -4/+2 |
| | | | | MockLogger for LogSubscriber. | ||||
* | Test added, we shouldn't log sql calls when logger is not on debug? mode | Santiago Pastorino | 2010-07-20 | 1 | -0/+17 |
| | |||||
* | mocking out debing? call in the MockLogger | Aaron Patterson | 2010-07-19 | 1 | -0/+2 |
| | |||||
* | activerecord tests should inherit from ActiveRecord::TestCase | Aaron Patterson | 2010-07-14 | 1 | -1/+1 |
| | |||||
* | Move Rails::LogSubscriber to ActiveSupport::LogSubscriber, allowing ↵ | José Valim | 2010-06-24 | 1 | -5/+4 |
| | | | | frameworks like ActiveRecord and ActiveResource to log outsude Rails::Application [#4816 state:resolved] | ||||
* | assert log output match in case insensitive mode to avoid failure when ↵ | Raimonds Simanovskis | 2010-06-04 | 1 | -2/+2 |
| | | | | quoted table name is in uppercase (when using Oracle) | ||||
* | cleaning up a bunch of parse time warnings in AR [#4186 state:resolved] | Aaron Patterson | 2010-03-15 | 1 | -5/+5 |
| | | | | Signed-off-by: wycats <wycats@gmail.com> | ||||
* | Somehow this fixes the AR tests when ran with rake. I should probably figure ↵ | Carl Lerche | 2010-03-04 | 1 | -1/+1 |
| | | | | out why. | ||||
* | Fix active_records' log tests | Carl Lerche | 2010-03-02 | 1 | -2/+3 |
| | |||||
* | Revert "Fix test load paths for those not using bundler" | Carlhuda | 2010-02-23 | 1 | -4/+1 |
| | | | | | | This reverts commit eec2d301d4ce9df9c71c1a5aa63053eb970b6818. This commit broke tests. You cannot have a file called "bundler" on the load path. | ||||
* | Merge master. | José Valim | 2010-02-17 | 1 | -1/+4 |
| | |||||
* | Rename Rails::Subscriber to Rails::LogSubscriber | Prem Sichanugrist | 2010-02-16 | 1 | -0/+42 |