Skip to content
Snippets Groups Projects

Resolve #2795567 "Allow Symfony dump() to work in kernel and browser tests"

Open Resolve #2795567 "Allow Symfony dump() to work in kernel and browser tests"
9 unresolved threads
9 unresolved threads

Closes #2795567

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
1847 1849 vampirize
1848 1850 vancode
1849 1851 varchar
1852 VarDumper's
  • Words in the dictionary should be lowercased by convention. See the spellcheck:make-drupal-dict command in core/package.json, tr '[:upper:]' '[:lower:] converted all words to lowercase.

    "spellcheck:make-drupal-dict": "rm -f misc/cspell/dictionary.txt && yarn -s spellcheck:core --unique --wordsOnly | tr '[:upper:]' '[:lower:]' | tr -d \\\\\\\\ | LC_ALL=C sort -u -o misc/cspell/dictionary.txt",

  • changed this line in version 6 of the diff

  • Please register or sign in to reply
  • Qiangjun Ran
    Qiangjun Ran @jungle started a thread on an outdated change in commit 1f699ca3
  • 43 45 */
    44 46 protected function setUp() {
    45 47 parent::setUp();
    48
    49 VarDumper::setHandler(TestVarDumper::class . '::cliHandler');
  • 1256 1257 proxied
    1257 1258 proxying
    1258 1259 pseudotag
    1260 PSFS_FEED_ME
  • Joachim Noreiko added 24 commits

    added 24 commits

    • ab8f25f1...77480bee - 11 commits from branch project:9.2.x
    • 508a1e06 - Patch from comment #72.
    • 6baaa5cb - Added capturing and testing of dump() output in testVarDump() kernel test.
    • 2c48a6a6 - Fixed dump() not working in test code of browser tests.
    • 337ceb67 - Added testing of dump() in test code of browser tests.
    • a6a322db - Moved StreamCapturer class now it’s used by both kernel and functional tests.
    • 941c91e7 - Added documentation about the use of dump().
    • 7c9bee5e - Added words used by StreamCapturer and in documentation to dictionary.
    • 4c962824 - Fixed dump() not working in unit tests run in a separate process.
    • e46e2144 - Added tests for dump() in unit tests.
    • bd443a12 - Fixed wrapping.
    • 526dd3da - Added documentation about the use of dump().
    • f4810ac6 - Fixed case of words added to cspell dictionary.
    • c76d0cf4 - Moved setting up of VarDumper to setUpBeforeClass().

    Compare with previous version

  • added 1 commit

    • 7bb3acb4 - Fixed word added to dictionary.

    Compare with previous version

  • mondrake added 63 commits

    added 63 commits

    • 7bb3acb4...a4095b9b - 62 commits from branch project:9.2.x
    • 96825ae2 - Merge branch '9.2.x' into 2795567-symfony-dump-kernel-browser-tests

    Compare with previous version

  • 346 346 data's
    347 347 databasefilename
    348 348 databasename
    349 datalen
  • 1847 1849 vampirize
    1848 1850 vancode
    1849 1851 varchar
    1852 vardumper's
  • 64 66 * KernelTestBase::installEntitySchema(). Alternately, tests which need modules
    65 67 * to be fully installed could inherit from \Drupal\Tests\BrowserTestBase.
    66 68 *
    69 * Using Symfony VarDumper's dump() in Kernel tests will produce output on the
  • 36 38 * translation functionality. For example, avoid wrapping test text with t()
    37 39 * or TranslatableMarkup().
    38 40 *
    41 * Using Symfony VarDumper's dump() in functional test test code will produce
  • 10 10 use Drupal\Core\StringTranslation\TranslatableMarkup;
    11 11 use Drupal\Core\StringTranslation\PluralTranslatableMarkup;
    12 12 use Drupal\Tests\Traits\PhpUnitWarnings;
    13 use Drupal\TestTools\TestVarDumper;
    13 14 use PHPUnit\Framework\TestCase;
    15 use Symfony\Component\VarDumper\VarDumper;
    14 16 use Symfony\Bridge\PhpUnit\ExpectDeprecationTrait;
    15 17
    16 18 /**
    17 19 * Provides a base class and helpers for Drupal unit tests.
    18 20 *
    21 * Using Symfony VarDumper's dump() in Unit tests will produce output on the
  • 39 44 protected $root;
    40 45
    46 /**
    47 * {@inheritdoc}
    48 */
    49 public static function setUpBeforeClass() {
    50 parent::setUpBeforeClass();
    51 VarDumper::setHandler(TestVarDumper::class . '::cliHandler');
    52 }
    53
    41 54 /**
    42 55 * {@inheritdoc}
    43 56 */
    44 57 protected function setUp() {
    45 58 parent::setUp();
    59
  • Joachim Noreiko added 3 commits

    added 3 commits

    • bb966f42 - Reverted unrelated whitespace change.
    • c73fba5b - Fixed ambiguous phrase in docs.
    • a6ce06f0 - Removed words from scpell; added disable comment.

    Compare with previous version

  • Joachim Noreiko added 34 commits

    added 34 commits

    • a6ce06f0...cc41d94d - 17 commits from branch project:9.2.x
    • ba96f2a5 - Patch from comment #72.
    • 5d24f036 - Added capturing and testing of dump() output in testVarDump() kernel test.
    • 94d89f29 - Fixed dump() not working in test code of browser tests.
    • 8ca08170 - Added testing of dump() in test code of browser tests.
    • 742bf36f - Moved StreamCapturer class now it’s used by both kernel and functional tests.
    • d32a64a3 - Added documentation about the use of dump().
    • b7db10e1 - Added words used by StreamCapturer and in documentation to dictionary.
    • 77532739 - Fixed dump() not working in unit tests run in a separate process.
    • ed6860ea - Added tests for dump() in unit tests.
    • 5a19d902 - Fixed wrapping.
    • e6287dac - Added documentation about the use of dump().
    • 6615d414 - Fixed case of words added to cspell dictionary.
    • 9be4cbf6 - Moved setting up of VarDumper to setUpBeforeClass().
    • c61714c0 - Fixed word added to dictionary.
    • 5ce15bb0 - Reverted unrelated whitespace change.
    • ef282dca - Fixed ambiguous phrase in docs.
    • 7bd13328 - Removed words from scpell; added disable comment.

    Compare with previous version

  • Please register or sign in to reply
    Loading