Contents

# frozen_string_literal: true

require "rubocop"

# :nocov:
module RuboCop
  module Cop
    module Ariadne
      # This cop ensures that tags are not set with ||=
      #
      # bad
      # @attributes[:tag] ||= :h1
      #
      # good
      # @attributes[:tag] = fetch_or_raise(TAG_OPTIONS, tag)
      #
      # good
      # @attributes[:tag] = :h2
      # TODO: TEST THIS
      class NoTagMemoize < Base
        INVALID_MESSAGE = <<~STR
          Avoid `[:tag] ||=`. Instead, try one of the following:
            - Don't allow consumers to update the tag by having a fixed tag (e.g. `attributes[:tag] = :div`)
            - Use the `fetch_or_raise` helper to only allow a tag from a restricted list.
        STR

        def_node_search :tag_memoized?, <<~PATTERN
          (or-asgn
            (send
              _
              _
              (sym :tag)
            )
            _
          )
        PATTERN

        def on_or_asgn(node)
          add_offense(node, message: INVALID_MESSAGE) if tag_memoized?(node)
        end
      end
    end
  end
end

Version data entries

2 entries across 2 versions & 1 rubygems

Version	Path
ariadne_view_components-0.0.93.2	lib/rubocop/cop/ariadne/no_tag_memoize.rb
ariadne_view_components-0.0.93.1	lib/rubocop/cop/ariadne/no_tag_memoize.rb