Sign upLogin

18F/e-manifest

View on GitHub
Star
app/models/cdx/user.rb

Summary

Maintainability
A
0 mins
Test Coverage

Redundant curly braces around a hash parameter.
Open

      :authenticate, {
        message: {
          userId: opts[:user_id],
          password: opts[:password]
        }
Severity: Minor
Found in app/models/cdx/user.rb by rubocop

This cop checks for braces around the last parameter in a method call if the last parameter is a hash. It supports braces, no_braces and context_dependent styles.

Example: EnforcedStyle: braces

# The `braces` style enforces braces around all method
# parameters that are hashes.

# bad
some_method(x, y, a: 1, b: 2)

# good
some_method(x, y, {a: 1, b: 2})

Example: EnforcedStyle: no_braces (default)

# The `no_braces` style checks that the last parameter doesn't
# have braces around it.

# bad
some_method(x, y, {a: 1, b: 2})

# good
some_method(x, y, a: 1, b: 2)

Example: EnforcedStyle: context_dependent

# The `context_dependent` style checks that the last parameter
# doesn't have braces around it, but requires braces if the
# second to last parameter is also a hash literal.

# bad
some_method(x, y, {a: 1, b: 2})
some_method(x, y, {a: 1, b: 2}, a: 1, b: 2)

# good
some_method(x, y, a: 1, b: 2)
some_method(x, y, {a: 1, b: 2}, {a: 1, b: 2})

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

class CDX::User < CDX::LoggedRequest
Severity: Minor
Found in app/models/cdx/user.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.

Avoid comma after the last item of a hash.
Open

      MiddleInitial: user_data[:middle_initial],
Severity: Minor
Found in app/models/cdx/user.rb by rubocop

This cop checks for trailing comma in array and hash literals.

Example: EnforcedStyleForMultiline: consistent_comma

# bad
a = [1, 2,]

# good
a = [
  1, 2,
  3,
]

# good
a = [
  1,
  2,
]

Example: EnforcedStyleForMultiline: comma

# bad
a = [1, 2,]

# good
a = [
  1,
  2,
]

Example: EnforcedStyleForMultiline: no_comma (default)

# bad
a = [1, 2,]

# good
a = [
  1,
  2
]

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

      output_stream.puts ANSI.blue{ response.hash.pretty_inspect }
Severity: Minor
Found in app/models/cdx/user.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 }

Line is too long. [81/80]
Open

    @user_data ||= response.hash[:envelope][:body][:authenticate_response][:user]
Severity: Minor
Found in app/models/cdx/user.rb by rubocop

Space missing to the left of {.
Open

      output_stream.puts ANSI.blue{ response.hash.pretty_inspect }
Severity: Minor
Found in app/models/cdx/user.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
}

Missing top-level class documentation comment.
Open

class CDX::User < CDX::LoggedRequest
Severity: Minor
Found in app/models/cdx/user.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

There are no issues that match your filters.

Category
Status