aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Fix formatting of `ActiveJob::TestHelper` api docsbogdanvlviv2018-08-201-4/+4
|
* Allow `assert_performed_with` to be called without a block.bogdanvlviv2018-08-203-14/+129
| | | | | | | | | | | | | | | Example: ``` def test_assert_performed_with MyJob.perform_later(1,2,3) perform_enqueued_jobs assert_performed_with(job: MyJob, args: [1,2,3], queue: 'high') end ``` Follow up #33626.
* Fix `assert_performed_jobs` and `assert_no_performed_jobs`bogdanvlviv2018-08-203-24/+289
| | | | | Execution of `assert_performed_jobs`, and `assert_no_performed_jobs` without a block should respect passed `:except`, `:only`, and `:queue` options.
* Add changelog entry about adding `:queue` option to job assertions and helpersbogdanvlviv2018-08-201-13/+4
| | | | | Note that it removes changelog entry of #33265 since the entry in this commits includes that too.
* Allow `:queue` option to `assert_no_performed_jobs`.bogdanvlviv2018-08-202-2/+65
| | | | | | | | | | | | | | If the `:queue` option is specified, then only the job(s) enqueued to a specific queue will not be performed. Example: ``` def test_assert_no_performed_jobs_with_queue_option assert_no_performed_jobs queue: :some_queue do HelloJob.set(queue: :other_queue).perform_later("jeremy") end end ```
* Allow `:queue` option to `assert_performed_jobs`.bogdanvlviv2018-08-202-2/+70
| | | | | | | | | | | | | | | If the `:queue` option is specified, then only the job(s) enqueued to a specific queue will be performed. Example: ``` def test_assert_performed_jobs_with_queue_option assert_performed_jobs 1, queue: :some_queue do HelloJob.set(queue: :some_queue).perform_later("jeremy") HelloJob.set(queue: :other_queue).perform_later("bogdan") end end ```
* Fix `perform_enqueued_jobs`bogdanvlviv2018-08-201-1/+4
| | | | | | | | | | | | | Set ```` queue_adapter.perform_enqueued_jobs = true queue_adapter.perform_enqueued_at_jobs = true queue_adapter.filter = only queue_adapter.reject = except queue_adapter.queue = queue ``` if block given. Execution of `flush_enqueued_jobs` doesn't require that.
* Allow `:queue` option to `perform_enqueued_jobs`.bogdanvlviv2018-08-203-12/+168
| | | | | | | | | | | | | | | | | | | | If the `:queue` option is specified, then only the job(s) enqueued to a specific queue will be performed. Example: ``` def test_perform_enqueued_jobs_with_queue perform_enqueued_jobs queue: :some_queue do MyJob.set(queue: :some_queue).perform_later(1, 2, 3) # will be performed HelloJob.set(queue: :other_queue).perform_later(1, 2, 3) # will not be performed end assert_performed_jobs 1 end ``` Follow up #33265 [bogdanvlviv & Jeremy Daer]
* Don't put up with not saying it.Kasper Timm Hansen2018-08-191-1/+1
| | | | Commands generally prefer say to puts.
* Merge pull request #33657 from jychen7/fix-test-association-enumRyuta Kamizono2018-08-201-0/+3
|\ | | | | Improve test case to test enum correctly
| * Improve test case to test enum correctlyRich2018-08-201-0/+3
| | | | | | | | | | | | without define the enum in class “SpecialBook”, any string status will be casted to integer 0. Then the test have no meaning. [Rich Chen]
* | Merge pull request #33631 from anniecodes/initializers-commandKasper Timm Hansen2018-08-195-4/+100
|\ \ | | | | | | Move the initializers rake task to Rails::Command
| * | Update 'rake initializers' to use Rails::Command under the hoodAnnie-Claude Côté2018-08-162-4/+48
| | | | | | | | | | | | | | | | | | * Invoke Rails::Command within the rake task * Adds test for calling initializers with 'bin/rake' * Adds deprecation warning to the rake task
| * | Move 'rails initializers' command to Rails::CommandAnnie-Claude Côté2018-08-162-0/+48
| | |
| * | [ci skip] Add deprecration of to CHANGELOGAnnie-Claude Côté2018-08-161-0/+4
| | |
* | | Merge pull request #33656 from bogdanvlviv/follow-up-33413Kasper Timm Hansen2018-08-191-4/+4
|\ \ \ | | | | | | | | activestorage: yarn build
| * | | activestorage: yarn buildbogdanvlviv2018-08-191-4/+4
| | | | | | | | | | | | | | | | Context https://github.com/rails/rails/pull/33413#issuecomment-414137587
* | | | SQLite3: Fix rename reference column not to lose foreign key constraintRyuta Kamizono2018-08-202-11/+31
| |_|/ |/| | | | | | | | Fixes #33520.
* | | Merge pull request #33655 from bogdanvlviv/follow-up-33653Ryuta Kamizono2018-08-201-16/+0
|\ \ \ | |/ / |/| | DRY in Active Record Query Interface [ci skip]
| * | DRY in Active Record Query Interface [ci skip]bogdanvlviv2018-08-191-16/+0
|/ / | | | | | | | | | | | | | | | | | | | | The sentence "This is exactly the same as defining a class method ..." is not true, so #33653 fixed it, but added changes repeat what is explained a few lines below. We can remove this part since a user is able to get info about the difference between scopes and class methods below. Context https://github.com/rails/rails/pull/33653#discussion_r211105969. Reverts #33653, 97feb4996b1c88f770101dfce6d4d3a6baf6bb33.
* | Fix syntax error in the doc [ci skip]Ryuta Kamizono2018-08-191-1/+1
| |
* | Merge pull request #33653 from LemonAndroid/patch-1Richard Schneeman2018-08-191-3/+9
|\ \ | | | | | | Explained difference between scope & class method
| * | Explained difference between scope & class methodLemonAndroid2018-08-191-3/+9
| | |
* | | Update `google-cloud-storage` gem version [ci skip]yuuji.yaginuma2018-08-191-1/+1
| | | | | | | | | | | | Now requires version 1.11 or newer. Ref: bf5f41d948b6f3f27db7fdc2b70897aec991065f
* | | Retry ActiveStorage::AnalyzeJobs on download integrity check failuresGeorge Claghorn2018-08-191-0/+2
| | |
* | | Remove superfluous testGeorge Claghorn2018-08-191-10/+0
| | | | | | | | | | | | This test no longer covers the behavior of ActiveStorage::PurgeJob. Attached blobs are ignored by ActiveStorage::Blob#purge as of 934fccd, which includes an equivalent model test.
* | | Address intermittent CI failure due to non-determined sort orderRyuta Kamizono2018-08-191-1/+1
|/ / | | | | | | https://travis-ci.org/rails/rails/jobs/417783260#L1173-L1179
* | Ensure `foreign_keys` assertions after alter tableRyuta Kamizono2018-08-191-1/+5
| | | | | | | | | | If `foreign_keys` is fetched before alter table, it couldn't notice whether foreign keys are broken or not after alter table.
* | Enable Style/ParenthesesAroundCondition copRyuta Kamizono2018-08-196-7/+10
| | | | | | | | To prevent style check in review like https://github.com/rails/rails/pull/33608#discussion_r211087605.
* | Merge pull request #33413 from ↵Javan Makhmali2018-08-182-2/+22
|\ \ | | | | | | | | | | | | cseelus/active-storage-direct-uploads-multiple-submit-buttons Support multiple submit buttons in Active Storage forms
| * | Enable multiple submit buttons for Active Storage formsChris Seelus2018-08-182-2/+22
|/ /
* | Fix unclosed tags [ci skip]yuuji.yaginuma2018-08-182-2/+2
| |
* | Increment execution count before deserialize argumentsyuuji.yaginuma2018-08-183-3/+12
| | | | | | | | | | | | | | | | Currently, the execution count increments after deserializes arguments. Therefore, if an error occurs with deserialize, it retries indefinitely. In order to prevent this, the count is moved before deserialize. Fixes #33344.
* | Merge pull request #33645 from arye-dov-eidelman/masterRichard Schneeman2018-08-171-1/+5
|\ \ | | | | | | [skip ci] Fix overlapping text in the guide
| * | [skip ci] Fix overlapping text in the guideArye Dov Eidelman2018-08-171-1/+5
|/ / | | | | | | Fix a layout issue in the rails guides, where the navigation covers the main text, if the page is between 800 and 960 pixels wide. (issue #33406)
* | Remove unused requiresyuuji.yaginuma2018-08-171-3/+0
| |
* | Merge pull request #33537 from ↵Ryuta Kamizono2018-08-171-0/+15
|\ \ | | | | | | | | | | | | | | | ZASMan/update_action_mailer_docs_custom_view_paths Add note for custom mailer view paths in action mailer guide. [ci skip]
| * | Add note for custom mailer view paths in action mailer guide. [ci skip]Zane2018-08-161-0/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds stuff Fixes a typo Integrates changes Adds link to append_view_path in actionmailer guide.
* | | Merge pull request #33636 from wjordan/normalize_encode_params_require_hashYuji Yaginuma2018-08-171-0/+2
|\ \ \ | | | | | | | | add missing indifferent_access require for #normalize_encode_params
| * | | add missing indifferent_access require for #normalize_encode_paramsWill Jordan2018-08-161-0/+2
|/ / / | | | | | | | | | Fixes #33634.
* | | Merge pull request #33633 from utilum/update_image_processing_to_latest_versionRyuta Kamizono2018-08-171-4/+4
|\ \ \ | |_|/ |/| | Update image_processing to the latest version
| * | Update image_processing to the latest versionutilum2018-08-161-4/+4
|/ / | | | | | | | | | | Version 1.6.0 includes https://github.com/janko-m/image_processing/commit/3a958bc419854703860fae606423bca3667a4127 which saves us a warning: shadowing outer local variable - options. I see it locally, but for some reason not on the CI.
* | Merge pull request #33576 from aried3r/patch-5Richard Schneeman2018-08-161-7/+7
|\ \ | | | | | | Harmonize shell commands in dev guide [ci skip]
| * | Harmonize shell commands in dev guide [ci skip]Anton Rieder2018-08-161-7/+7
| | |
* | | Merge pull request #33585 from yahonda/diag33520Ryuta Kamizono2018-08-163-1/+26
|\ \ \ | | | | | | | | | | | | SQLite3 adapter `alter_table` method restores foreign keys
| * | | SQLite3 adapter `alter_table` method restores foreign keysYasuo Honda2018-08-113-1/+26
| | | | | | | | | | | | | | | | Related to #33520
* | | | Merge pull request #33603 from kamipo/fix_numericality_validatorRyuta Kamizono2018-08-164-2/+28
|\ \ \ \ | |_|/ / |/| | | Fix numericality validator not to be affected by custom getter
| * | | Fix numericality validator not to be affected by custom getterRyuta Kamizono2018-08-134-2/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since fe9547b6, numericality validator would parse raw value only when a value came from user to work type casting to a value from database. But that was caused a regression that the validator would work against getter value instead of parsed raw value, a getter is sometimes customized by people. #33550 There we never guarantees that the value before type cast was going to the used in this validation (actually here is only place that getter value might not be used), but we should not change the behavior unless there is some particular reason. The purpose of fe9547b6 is to work type casting to a value from database. We could achieve the purpose by using `read_attribute`, without using getter value. Fixes #33550.
* | | | Merge pull request #33629 from bogdanvlviv/follow-33626Ryuta Kamizono2018-08-161-0/+8
|\ \ \ \ | | | | | | | | | | Add example `perform_enqueued_jobs` without block to api docs [ci skip]
| * | | | Add example `perform_enqueued_jobs` without block to api docs [ci skip]bogdanvlviv2018-08-161-0/+8
|/ / / / | | | | | | | | | | | | Follow up #33626
generated by cgit v1.2.3 (git 2.25.1) at 2024-11-16 21:56:02 +0000