Skip to content

Conversation

@christianlupus
Copy link
Collaborator

The class JsonService did not check if a @context was given. This should cover that case.

@christianlupus christianlupus added the Backend Issue or PR related to the backend code label Jan 23, 2021
Signed-off-by: Christian Wolf <[email protected]>
@codecov
Copy link

codecov bot commented Jan 23, 2021

Codecov Report

Merging #554 (56d6ec7) into master (66efef7) will increase coverage by 0.14%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##             master    #554      +/-   ##
===========================================
+ Coverage      0.87%   1.02%   +0.14%     
- Complexity      432     434       +2     
===========================================
  Files            13      13              
  Lines          1364    1366       +2     
===========================================
+ Hits             12      14       +2     
  Misses         1352    1352              
Flag Coverage Δ Complexity Δ
integration 0.00% <0.00%> (ø) 0.00 <0.00> (ø)
unittests 1.02% <100.00%> (+0.14%) 0.00 <0.00> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ Complexity Δ
lib/Service/JsonService.php 100.00% <100.00%> (ø) 9.00 <0.00> (+2.00)

@christianlupus christianlupus merged commit b018667 into master Jan 23, 2021
@delete-merged-branch delete-merged-branch bot deleted the fix/jsonService-context branch January 23, 2021 17:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Backend Issue or PR related to the backend code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants