Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Remove the leading \n added by textarea on assert_select | Santiago Pastorino | 2012-03-30 | 1 | -0/+1 |
| | |||||
* | Merge pull request #5456 from brianmario/redirect-sanitization | Aaron Patterson | 2012-03-15 | 1 | -1/+1 |
| | | | | Strip null bytes from Location header | ||||
* | fix assert_select_email to work on non-multipart emails as well as ↵ | Andrew Kaspick | 2011-08-11 | 1 | -2/+2 |
| | | | | converting the Mail::Body to a string to prevent errors. | ||||
* | Remove blank line | Santiago Pastorino | 2011-07-26 | 1 | -1/+0 |
| | |||||
* | refactored 'assert_redirected_to': local call to validate_request! will be ↵ | thoefer | 2011-07-26 | 1 | -5/+4 |
| | | | | called in assert_response already. changed names of local variables in order to recognize the semantics a bit easier. | ||||
* | TODO fix explicitly loading exceptations, autoload removed | Vishnu Atrai | 2011-07-11 | 1 | -0/+1 |
| | |||||
* | Corrected some typos and American vs. Queen's English issues | Lee Reilly | 2011-05-29 | 1 | -4/+4 |
| | |||||
* | Merge branch 'master' of git://github.com/lifo/docrails | Xavier Noria | 2011-05-25 | 1 | -9/+9 |
|\ | | | | | | | | | | | Conflicts: actionmailer/lib/action_mailer/base.rb activesupport/lib/active_support/core_ext/kernel/requires.rb | ||||
| * | Remove extra white spaces on ActionPack docs. | Sebastian Martinez | 2011-05-23 | 1 | -9/+9 |
| | | |||||
* | | Restructure TemplateAssertions-related code to eliminate circular requires. | wycats | 2011-05-22 | 1 | -7/+0 |
|/ | | | Also, no need to include dependencies in AS::Concerns inside included blocks. | ||||
* | Follow code conventions on docs | Sebastian Martinez | 2011-05-10 | 1 | -1/+1 |
| | |||||
* | Flunk makes a lot more sense, doesnt it (hat tip @tenderlove) | David Heinemeier Hansson | 2011-05-02 | 1 | -1/+1 |
| | |||||
* | We cant use assert_block because its buggy in MiniTest and wont actually ↵ | David Heinemeier Hansson | 2011-04-22 | 1 | -1/+1 |
| | | | | show you the failure message you provide -- instead you just always get a "Expected block to return true" | ||||
* | Remove `#among?` from Active Support | Prem Sichanugrist | 2011-04-13 | 1 | -1/+1 |
| | | | | | | After a long list of discussion about the performance problem from using varargs and the reason that we can't find a great pair for it, it would be best to remove support for it for now. It will come back if we can find a good pair for it. For now, Bon Voyage, `#among?`. | ||||
* | renames response_from_page_or_rjs -> response_from_page, and extracts the ↵ | Xavier Noria | 2011-04-13 | 1 | -58/+5 |
| | | | | RJS in it | ||||
* | removes assert_select_rjs | Xavier Noria | 2011-04-13 | 1 | -139/+1 |
| | |||||
* | Change Object#either? to Object#among? -- thanks to @jamesarosen for the ↵ | David Heinemeier Hansson | 2011-04-12 | 2 | -2/+2 |
| | | | | suggestion! | ||||
* | Using Object#in? and Object#either? in various places | Prem Sichanugrist | 2011-04-11 | 2 | -2/+5 |
| | | | | There're a lot of places in Rails source code which make a lot of sense to switching to Object#in? or Object#either? instead of using [].include?. | ||||
* | use assert_equal so we get normal error messages along with our custom ↵ | Aaron Patterson | 2011-03-30 | 1 | -3/+3 |
| | | | | failure message | ||||
* | Remove incorrect assert_recognizes example | Andrew White | 2011-02-13 | 1 | -3/+0 |
| | |||||
* | Fix assert_recognizes with block constraints [#5805 state:resolved] | Andrew White | 2011-02-13 | 1 | -1/+2 |
| | |||||
* | corrected the location of status code symbols | misfo | 2011-01-29 | 1 | -1/+1 |
| | |||||
* | _ is not a valid scheme name character, \w includes it and also is redundant ↵ | Emilio Tagua | 2010-09-29 | 1 | -1/+1 |
| | | | | | | with \d. 'The scheme name consists of a letter followed by any combination of letters, digits, and the plus ("+"), period ("."), or hyphen ("-") characters; and is terminated by a colon (":").' | ||||
* | Remove redundant conditional. | Emilio Tagua | 2010-09-29 | 1 | -5/+1 |
| | |||||
* | Remove more warnings on AP. | Emilio Tagua | 2010-09-28 | 1 | -1/+2 |
| | |||||
* | Avoid more uninitialized variable warnings. | Emilio Tagua | 2010-09-28 | 1 | -1/+2 |
| | |||||
* | @selected may not be defined here, avoid warning. | Emilio Tagua | 2010-09-28 | 1 | -1/+1 |
| | |||||
* | Remove more warnings shadowing outer local variable. | Emilio Tagua | 2010-09-27 | 1 | -2/+2 |
| | | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com> | ||||
* | _routes must be inside @controller conditional. | Emilio Tagua | 2010-09-27 | 1 | -1/+1 |
| | |||||
* | Refactor method to avoid warnings and not run unnecessary code. | Emilio Tagua | 2010-09-27 | 1 | -11/+11 |
| | |||||
* | Remove more warnings shadowing outer local variable. | Emilio Tagua | 2010-09-27 | 1 | -2/+2 |
| | |||||
* | Ask is @controller is defined to avoid warning. | Emilio Tagua | 2010-09-27 | 1 | -1/+1 |
| | |||||
* | The call-seq directive has no useful effect on our documentation and is not ↵ | Joost Baaij | 2010-08-28 | 1 | -19/+0 |
| | | | | | | | | | | | used anywhere else. Worse still, its use breaks some rdoc, replacing method names with a curly brace. Having just one call-seq directive doesn't add anything since this is covered by regular rdoc. Having multiple might make sense, but these are already documented with examples. This partly re-reverts 60de0e56b7f57f0052d7e4f43bd2ef9e363c6a1a but does not touch the vendorized code in html-scanner. | ||||
* | Revert "call-seq directive makes rdoc cry and replaces method names with a ↵ | Xavier Noria | 2010-08-27 | 1 | -9/+28 |
| | | | | | | | | curly brace; besides it doesn't add anything when you only list one call example since this is obviously documented already" Reason: Part of this commit touches vendorized stuff, the rest is fine, could you please repatch? This reverts commit 60de0e56b7f57f0052d7e4f43bd2ef9e363c6a1a. | ||||
* | escape constants that should not be linked to | Joost Baaij | 2010-08-27 | 3 | -8/+8 |
| | |||||
* | call-seq directive makes rdoc cry and replaces method names with a curly ↵ | Joost Baaij | 2010-08-27 | 1 | -28/+9 |
| | | | | brace; besides it doesn't add anything when you only list one call example since this is obviously documented already | ||||
* | Support routing constraints in functional tests | Andrew White | 2010-08-20 | 1 | -14/+37 |
| | | | | | | | | | | Extend assert_recognizes and assert_generates to support passing full urls as the path argument. This allows testing of routing constraints such as subdomain and host within functional tests. [#5005 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Remove assert_valid. It was already deprecated on Rails 2.3. | José Valim | 2010-07-19 | 1 | -19/+0 |
| | |||||
* | Remove unused variable on assert_recognizes [#4912 state:resolved] | Paco Guzman | 2010-06-22 | 1 | -1/+0 |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | removed 'unless const_defined?' code smell | Josh Kalderimis | 2010-06-21 | 1 | -23/+19 |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Fix a bunch of minor spelling mistakes | Evgeniy Dolzhenko | 2010-06-11 | 1 | -1/+1 |
| | |||||
* | regular expressions are usually ASCII-encoded, so force_encoding the content ↵ | wycats | 2010-06-07 | 1 | -2/+0 |
| | | | | of a Node to the encoding of the regular expression is wrong. | ||||
* | fix assert_select messages to its declaration behaviour | Paco Guzman | 2010-05-15 | 1 | -5/+11 |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Some require indifferent_access added | Santiago Pastorino | 2010-04-21 | 1 | -0/+1 |
| | | | | Signed-off-by: Xavier Noria <fxn@ubuntu.(none)> | ||||
* | Consistent routing language | Joshua Peek | 2010-03-30 | 1 | -10/+10 |
| | |||||
* | Fix stack overflow bug in integration test router helpers | Joshua Peek | 2010-03-30 | 1 | -1/+1 |
| | |||||
* | Each controller class has it's own view context subclass. This removes the ↵ | Carlhuda | 2010-03-18 | 1 | -3/+4 |
| | | | | need for ActionView::Base.for_controller | ||||
* | All tests pass without memoizing view_context | Carlhuda | 2010-03-18 | 1 | -3/+2 |
| | |||||
* | Modify assert_template to use instrumentation | Carlhuda | 2010-03-17 | 1 | -52/+0 |
| | |||||
* | Do not always include the named URL helpers into AC::Base and AV::Base. | Carl Lerche | 2010-03-16 | 1 | -2/+16 |
| |