Skip to content
Snippets Groups Projects

Issue #3215110: Add automated test coverage

1 unresolved thread

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
94 'append_member_page' => 'append_member_page',
95 'member_page_as_link' => 'member_page_as_link',
96 'remove_home' => 'remove_home',
97 'add_home' => 'add_home',
98 'front_title' => 'front_title',
99 'exclude_empty_url' => 'exclude_empty_url',
100 'exclude_disabled_menu_items' => 'exclude_disabled_menu_items',
101 'derived_active_trail' => 'derived_active_trail',
102 ];
103 $config = $this->config('menu_breadcrumb.settings');
104 foreach ($mapFieldsToConfig as $key => $value) {
105 $this->assertSession()->fieldValueEquals(
106 $key,
107 $config->get($value),
108 );
109 }
  • Author Maintainer

    This feels more complex than it might be, because the keys and values are the same. I was expecting some values to need massaging but that wasn't the case for any of these. There's an additional sequence value for the menus which could be tested somehow also?

  • Please register or sign in to reply
  • Chris Burgess added 6 commits

    added 6 commits

    Compare with previous version

  • Chris Burgess added 4 commits

    added 4 commits

    Compare with previous version

  • Chris Burgess added 5 commits

    added 5 commits

    Compare with previous version

  • Chris Burgess added 1 commit

    added 1 commit

    Compare with previous version

  • Chris Burgess added 7 commits

    added 7 commits

    Compare with previous version

  • Chris Burgess added 1 commit

    added 1 commit

    Compare with previous version

  • Chris Burgess added 1 commit

    added 1 commit

    • f2eb99cf - Fix merge conflict accidentally committed to README

    Compare with previous version

  • Chris Burgess added 1 commit

    added 1 commit

    • 38dd8c19 - Issue #3215110: New parameter added to \Drupal\Core\Form\ConfigFormBase::__construct

    Compare with previous version

  • Chris Burgess added 1 commit

    added 1 commit

    Compare with previous version

  • Chris Burgess added 3 commits

    added 3 commits

    Compare with previous version

  • Chris Burgess added 1 commit

    added 1 commit

    Compare with previous version

  • Chris Burgess added 1 commit

    added 1 commit

    Compare with previous version

  • Chris Burgess added 1 commit

    added 1 commit

    Compare with previous version

  • Chris Burgess added 2 commits

    added 2 commits

    Compare with previous version

  • Chris Burgess added 1 commit

    added 1 commit

    Compare with previous version

  • Chris Burgess added 1 commit

    added 1 commit

    Compare with previous version

  • Chris Burgess started a merge train

    started a merge train

  • merged

  • Please register or sign in to reply
    Loading