Code

Add tests for the head component#923

Code

Merged

caendesilva merged 25 commits intomasterhydephp/develop:masterfrom update-head-componenthydephp/develop:update-head-componentCopy head branch name to clipboardFeb 5, 2023

+79

Commits

Commits on Feb 5, 2023

• Create HeadComponentViewTest.php

  caendesilva

  committed
  fba5f48

• Create render testing helper

  caendesilva

  committed
  0491b88

• Test component can be rendered

  caendesilva

  committed
  c00dc1a

• Change snake_case test method name to camelCase

  caendesilva

  committed
  4c0a276

• Refactor mock control

  caendesilva

  committed
  67b9374

• Test rendered title element uses the page's htmlTitle

  caendesilva

  committed
  da0564a

• Change test to mock method as actual composition is irrelevant to scope of test

  caendesilva

  committed
  b460bcf

• Fix property assignment call

  caendesilva

  committed
  c24ec46

• Reset value after use

  caendesilva

  committed
  c4d7605

• Introduce local variable

  caendesilva

  committed
  f661e74

• Wrap code to single line

  caendesilva

  committed
  d510963

• Mock includes

  caendesilva

  committed
  ea2180b

• Replace code comment with extracted helper method

  caendesilva

  committed
  7954cec

• Remove site name from expectation

  caendesilva

  committed
  a31e74e

• Call on mocked property directly

  caendesilva

  committed
  5b7ebbb

• Inline local test variables

  caendesilva

  committed
  d005a58

• Simplify mocked page handling

  caendesilva

  committed
  2a4d914

• Test the favicon link

  caendesilva

  committed
  9b092c5

• Test component inclusion

  caendesilva

  committed
  0f901b2

• Update includes mocker to escape includes instead

  caendesilva

  committed
  ac88a76

• Rename helper method for refactored usage

  caendesilva

  committed
  cbb7041

• Split out test with multiple assertions to two tests

  caendesilva

  committed
  665c05b

• Put repeated code in setup method

  caendesilva

  committed
  ad37bf2

• Apply fixes from StyleCI

  StyleCIBot

  committed
  e1e2bce

• Merge branch 'master' into update-head-component

  caendesilva

  authored
  e98b44c