Skip to content

Commit

Permalink
[ci] Remove test for accidentally added debugger statement
Browse files Browse the repository at this point in the history
Nowadays 'debugger' in the code would throw a NameError error because it's
not available anymore. We could replace this by checking for pry, but such
things should be found by running the tests and / or by our code reviews.
  • Loading branch information
bgeuken committed Aug 17, 2015
1 parent e5ffb8a commit 21b9dcb
Showing 1 changed file with 0 additions and 10 deletions.
10 changes: 0 additions & 10 deletions src/api/test/unit/code_quality_test.rb
Original file line number Diff line number Diff line change
Expand Up @@ -54,16 +54,6 @@ def setup
end
end

# Checks that no 'debugger' statement is present in ruby code
test 'no ruby debugger statement' do
@ruby_files.each do |ruby_file|
File.open(ruby_file).each_with_index do |line, number|
assert(false, "#{ruby_file}:#{number + 1} 'debugger' statement found!") if line.match(/^\s*debugger/)
assert(false, "#{ruby_file}:#{number + 1} 'save_and_open_page' statement found!") if line.match(/^\s*save_and_open_page/)
end
end
end

# our current exceptions
BlackList = {
'ApplicationController#extract_ldap_user' => 119.9,
Expand Down

0 comments on commit 21b9dcb

Please sign in to comment.