18F/e-manifest

View on GitHub
app/models/cdx/handle_error.rb

Summary

Maintainability
A
0 mins
Test Coverage

Surrounding space missing in default value assignment.
Open

  def initialize(error, output_stream=$stdout)
Severity: Minor
Found in app/models/cdx/handle_error.rb by rubocop

Checks that the equals signs in parameter default assignments have or don't have surrounding space depending on configuration.

Example:

# bad
def some_method(arg1=:default, arg2=nil, arg3=[])
  # do something...
end

# good
def some_method(arg1 = :default, arg2 = nil, arg3 = [])
  # do something...
end

Space missing to the left of {.
Open

      output_stream.puts ANSI.red{ error_hash.pretty_inspect }
Severity: Minor
Found in app/models/cdx/handle_error.rb by rubocop

Checks that block braces have or don't have a space before the opening brace depending on configuration.

Example:

# bad
foo.map{ |a|
  a.bar.to_s
}

# good
foo.map { |a|
  a.bar.to_s
}

Parenthesize the param ANSI.red{ error_hash.pretty_inspect } to make sure that the block will be associated with the ANSI.red method call.
Open

      output_stream.puts ANSI.red{ error_hash.pretty_inspect }
Severity: Minor
Found in app/models/cdx/handle_error.rb by rubocop

This cop checks for ambiguous block association with method when param passed without parentheses.

Example:

# bad
some_method a { |val| puts val }

Example:

# good
# With parentheses, there's no ambiguity.
some_method(a) { |val| puts val }

# good
# Operator methods require no disambiguation
foo == bar { |b| b.baz }

# good
# Lambda arguments require no disambiguation
foo = ->(bar) { bar.baz }

Use nested module/class definitions instead of compact style.
Open

class CDX::HandleError
Severity: Minor
Found in app/models/cdx/handle_error.rb by rubocop

This cop checks the style of children definitions at classes and modules. Basically there are two different styles:

Example: EnforcedStyle: nested (default)

# good
# have each child on its own line
class Foo
  class Bar
  end
end

Example: EnforcedStyle: compact

# good
# combine definitions as much as possible
class Foo::Bar
end

The compact style is only forced for classes/modules with one child.

Line is too long. [82/80]
Open

      parent = fault_detail[:register_auth_fault] || fault_detail[:register_fault]
Severity: Minor
Found in app/models/cdx/handle_error.rb by rubocop

Missing top-level class documentation comment.
Open

class CDX::HandleError
Severity: Minor
Found in app/models/cdx/handle_error.rb by rubocop

This cop checks for missing top-level documentation of classes and modules. Classes with no body are exempt from the check and so are namespace modules - modules that have nothing in their bodies except classes, other modules, or constant definitions.

The documentation requirement is annulled if the class or module has a "#:nodoc:" comment next to it. Likewise, "#:nodoc: all" does the same for all its children.

Example:

# bad
class Person
  # ...
end

# good
# Description/Explanation of Person class
class Person
  # ...
end

Use == if you meant to do a comparison or wrap the expression in parentheses to indicate you meant to assign in a condition.
Open

    if fault_detail = error_hash[:fault][:detail]
Severity: Minor
Found in app/models/cdx/handle_error.rb by rubocop

This cop checks for assignments in the conditions of if/while/until.

Example:

# bad

if some_var = true
  do_something
end

Example:

# good

if some_var == true
  do_something
end

There are no issues that match your filters.

Category
Status