Issue with controller_path for cache keys

[mmhan]

controller_path seems to be used for both cache keys and view paths

The method is as seen below.

https://github.com/trailblazer/trailblazer-cells/blob/master/lib/trailblazer/cell.rb#L34:L37

Currently it is producing duplicate cache keys for different cells as below

# Comment::Cell::Show #=> comment/view/
# Comment::Cell::Summary #=> comment/views/

My expectation is that cache keys should be

# Comment::Cell::Show #=> comment/view/show/
# Comment::Cell::Summary #=> comment/views/summary/

[mmhan]

Although it is arguable that I should be using states for Comment::Cell::Show and Comment::Cell::Summary, for my particular use case it is necessary. My current work around is override state_cache_key as below.

class Application::Cell < Trailblazer::Cell
  class << self
    # Computes the complete, namespaced cache key for +state+.
    def state_cache_key(state, key_parts={})
      last_name = util.underscore(name.sub(/(.+::Cell::)/, ''))
      expand_cache_key([controller_path, last_name, state, key_parts])
    end
  end