| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
| | | | |
|
| | | | |
|
|/ / / |
|
|\ \ \
| | | |
| | | | |
Alternative to #23638 log to STDOUT via env var
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
People who deploy to containers or other places where they might have some sort of a log wrapping service use stdout. With this change new rails apps can be configured to output to STDOUT via setting `RAILS_LOG_TO_STDOUT` to any value. This allows container images or services to set the value for all apps without having to modify configuration for each application. If an app wants to opt out, they can either delete from the env hash, or remove that configuration.
cc/ @rafaelfranca
|
|\ \ \ \
| | | | |
| | | | | |
Add `internal` attribute to routes
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This is meant to provide a way for Action Cable, Sprockets, and possibly
other Rack applications to mark themselves as internal, and to exclude
themselves from the routing inspector, and thus `rails routes` / `rake
routes`.
I think this is the only way to have mounted Rack apps be marked as
internal, within AD/Journey. Another option would be to create an array
of regexes for internal apps, and then to iterate over that everytime a
request comes through. Also, I only had the first `add_route` method set
`internal`'s default to false, to avoid littering it all over the
codebase.
|
|/ / / / |
|
| | | |
| | | |
| | | |
| | | |
| | | | |
We should be able to pass options to minitest via TESTOPTS environment
variable
|
| | | |
| | | |
| | | |
| | | | |
Get it? ( ͡° ͜ʖ ͡°)
|
| |/ /
|/| |
| | |
| | |
| | |
| | |
| | | |
The process of converting `Test` to `String` is already defined in minitest.
I think it is better to use that for the consistency of output content.
ref: https://github.com/seattlerb/minitest/blob/master/lib/minitest/test.rb#L261..L267
|
| | |
| | |
| | |
| | |
| | |
| | | |
If you want to use `EventedFileUpdateChecker`, need to specify `listen` to plugin's gemspec.
However, the default is not specified `listen` to plugin's gemspec,
`EventedFileUpdateChecker` should be disabled.
|
|\ \ \
| |/ /
|/| | |
modify to `error` also abort when specify fail fast option
|
| | | |
|
|/ /
| |
| |
| | |
`app_templates_dir` is unused since dc53fe9.
|
| | |
|
|\ \
| | |
| | | |
generate mailer views in Rails API
|
| | |
| | |
| | |
| | |
| | | |
View files is necessary to send mail, it should be generated even Rails API.
Fixes #23617
|
| | |
| | |
| | |
| | |
| | | |
This is a follow-up to ea70c29 to bring back the assertion error
location information provided by minitest.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This reverts commit 0db310586ac3e15be95d2ef27ff40cfa24c05c10.
Closes #23686.
Conflicts:
railties/test/application/test_runner_test.rb
It's possible that the `result.location` returned by minitest is outside
the test file itself. For example in the case of mocha. This resulted in
bad rerun snipptets:
```
bin/rails test app/models/deliveries/delivery.rb:103
```
Let's always use the first line of the failed test-case in our rerun
snippet. We can display the line number of the assertion error elsewhere.
|
|\ \ \
| | | |
| | | | |
mark as #:nodoc: [ci skip]
|
| | | |
| | | |
| | | | |
can’t be instantiate directly or may be no need for doc.
|
|\ \ \ \
| |/ / /
|/| | | |
remove `active_support.test_order` from environment file
|
| | | |
| | | |
| | | |
| | | |
| | | | |
The default test order has been changed to `:random` in 5f777e4b5ee2e3e8e6fd0e2a208ec2a4d25a960d.
Therefore, it is no more need to be specified in the environment file.
|
|\ \ \ \
| |/ / /
|/| | | |
Inject Rails related configuration through Railtie
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
We were explicitly referencing Rails.root in ActionCable::Server::Configuration.initialize,
thereby coupling ourselves to Rails.
Instead add `app/channels` to Rails' app paths and assign the existent files
to `channel_paths`.
Users can still append to those load paths with `<<` and `push` in `config/application.rb`.
This means we can remove the custom `Dir` lookup in `channel_paths` and the Rails
and root definitions in the tests.
|
| | | | |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
s/config\/production/config\/environments\/production/
[ci skip]
|
| |/ /
|/| |
| | |
| | |
| | |
| | | |
- Fixed related documentation and usage all around
Fixes #23561
|
|\ \ \
| | | |
| | | | |
Set database poolsize via rails_max_threads
|
| |/ / |
|
|\ \ \
| | | |
| | | | |
set association name to generated fixtures if attribute is reference
|
| |/ /
| | |
| | |
| | |
| | |
| | |
| | | |
It has been changed to require `belongs_to` by default in Rails 5.
Therefore in order to pass the controller test, have association of set to fixtures.
Fixes #23384
|
| | | |
|
| | | |
|
| | | |
|
|/ / |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Previous commit accidentally broke mixing line filters with string -n filter.
Fix by checking if it is a string and returning it.
We also need to ensure the -n filter carry forward into any other composite filters.
Fix by letting the named filter be extractable, so we'll keep this for the next runnable's
run.
|
| |
| |
| |
| |
| |
| |
| |
| | |
on sprockets environment
- Remove jquery-rails if --skip-sprockets is true
Fixes #23431
|
|\ \
| | |
| | | |
Pass api only option to rails rails:update task to update only based on api Apps
|
| |/
| |
| |
| | |
Fixes #23470
|
| |
| |
| |
| | |
All grown up and shit!
|
| |
| |
| |
| | |
So no need to have it on by default in the Gemfile.
|
| |
| |
| |
| | |
So it can properly show stats for an app with 1,000,000+ LOC
|
| |
| |
| |
| |
| |
| | |
This reverts commit 293bd95c3e77275193130bc14c986348aae8b0e2.
This broke the header :<
|
|\ \
| | |
| | | |
Active Record supports MySQL >= 5.0
|
| | |
| | |
| | |
| | |
| | | |
Currently some features uses `information_schema` (e.g. foreign key
support). `information_schema` introduced since MySQL 5.0.
|
|\ \ \
| |/ /
|/| | |
Fix line filters running tests from multiple runnables.
|
| |/
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
`derive_regexp` was written with the assumption that we were run from a
blank slate — that if the filter didn't match we might as well return it
because it was nil.
This isn't the case because minitest calls `run` on every runnable. Which
is any subclass of Minitest::Runnable, such as ActiveSupport::TestCase,
ActionDispatch::IntegrationTest as well as any inheriting from those.
Thus after the first `run` we'd have put in a composite filter in
`options[:filter]` making the next `run` create a linked list when it
failed to match the regexp and put the composite filter as the head.
Every runnable would accumulate more and more of the same filters,
which effectively acted like an expanding whitelist and we ran tests
from other runnables.
Clog the accumulation by returning nil if there's no filter to derive
a regexp from.
Note: we pass a seed in the tests because Minitest shuffles the runnables
to ensure the whitelist is expanded enough that the failure is triggered.
|
|/
|
|
|
|
|
|
| |
Previously Safari would try to load these files when you visit
localhost:3000. That created two exceptions in the log. It also caused
the exception notifier to send them out.
In response to #23427
|