Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Demo: commit untested code #57

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Demo correct commit SHA incl. github action merge commit

51b6a48
Select commit
Loading
Failed to load commit list.
Open

Demo: commit untested code #57

Demo correct commit SHA incl. github action merge commit
51b6a48
Select commit
Loading
Failed to load commit list.
UndercoverCI / coverage failed Jun 27, 2024 in 30s

Complete

🚨 UndercoverCI has detected 1 warning in this changeset.

Details

Revision 51b6a48 has modified the following 3 code locations. Results marked with ⚠️ have untested lines added or changed in this commit, look into them!

file name coverage branches
app/controllers/application_controller.rb ⚠️ instance method check_cookie 0.0 0/2
app/controllers/application_controller.rb class ApplicationController 0.625
app/controllers/home_controller.rb instance method index 1.0

Annotations

Check warning on line 8 in app/controllers/application_controller.rb

See this annotation in the file changed.

@undercoverci undercoverci / coverage

Untested instance method 

Instance method `check_cookie` is missing coverage for lines 3..4,7 (node coverage: 0.0).
Missing branch coverage found in line 3.
Raw output 
2:   def check_cookie hits: n/a
3:     if params[:x] != ENV.fetch("CODE", "beta") hits: 0 branches: 0/2
4:       redirect_to(bounce_home_index_url) and return hits: 0
5:     end hits: n/a
6: 
7:     cookies[:beta_sign_in] = { value: 1, expires: 1.year } hits: 0
8:   end hits: n/a
View more details on UndercoverCI