Problem/Motivation

No test coverage for a context with the http status code condition.

Proposed resolution

Needs a Kernel test.

Remaining tasks

1. Write Kernel test.
2. Review and commit/merge issue #3188463: Error: Call to undefined method Drupal\Core\Form\EnforcedResponseException::getStatusCode, that solves the failing test bit.
3. Check/test that automated test passes after #2 was merged.
4. Review and commit/merge the issue.

Issue fork context-3190409

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

DeaOm created an issue. See original summary.

deaom’s picture

deaom commented
Title: Add test for context http sattus code » Add test for context http satus code

DeaOm opened merge request !3

deaom’s picture

deaom commented
Issue summary: View changes
Status: Active » Needs work
Related issues: +#3188463: Error: Call to undefined method Drupal\Core\Form\EnforcedResponseException::getStatusCode
deaom’s picture

deaom commented
Parent issue: » #3176668: [META] Add tests for context module

Pooja Ganjage made their first commit to this issue’s fork.

nmatja made their first commit to this issue’s fork.

paulocs made their first commit to this issue’s fork.

paulocs’s picture

paulocs
Primary language Portuguese, Brazil
Location Belo Horizonte
commented

Merged dev branch into 3190409-add-test-http-status because #3188463: Error: Call to undefined method Drupal\Core\Form\EnforcedResponseException::getStatusCode was committed.

paulocs’s picture

paulocs
Primary language Portuguese, Brazil
Location Belo Horizonte
commented
Status: Needs work » Reviewed & tested by the community

Looks good to me!
Thanks!

  • paulocs committed dd033a1 on 8.x-4.x authored by DeaOm 
    Issue #3190409 by Pooja Ganjage, DeaOm, nmatja, paulocs: Add test for...
paulocs’s picture

paulocs
Primary language Portuguese, Brazil
Location Belo Horizonte
commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.