Rename from ChatOps to Chatops

Author: Ben Lavender

README.md

@@ -5,8 +5,8 @@ Rails helpers for easy, JSON-RPC based chatops.
 A minimal controller example:
 
 ```ruby
-class ChatOpsController < ApplicationController
-  include ::ChatOps::Controller
+class ChatopsController < ApplicationController
+  include ::Chatops::Controller
 
   chatops_namespace :echo
 
@@ -79,7 +79,7 @@ You can return `jsonrpc_success` with a string to return text to chat. If you
 have an input validation or other handle-able error, you can use
 `jsonrpc_failure` to send a helpful error message.
 
-ChatOps are regular old rails controller actions, and you can use niceties like
+Chatops are regular old rails controller actions, and you can use niceties like
 `before_action` and friends. `before_action :echo, :load_user` for the above
 case would call `load_user` before running `echo`.
 

lib/chatops.rb

@@ -1,4 +1,4 @@
-module ChatOps
+module Chatops
   def self.public_key
     ENV[public_key_env_var_name]
   end

lib/chatops/controller.rb

@@ -1,6 +1,6 @@
 require "chatops"
 
-module ChatOps
+module Chatops
   module Controller
     class ConfigurationError < StandardError ; end
     extend ActiveSupport::Concern
@@ -103,22 +103,22 @@ def ensure_chatops_authenticated
       signature_string = [@chatops_url, @chatops_nonce, @chatops_timestamp, body].join("\n")
       # We return this just to aid client debugging.
       response.headers["Chatops-Signature-String"] = signature_string
-      raise ConfigurationError.new("You need to add a client's public key in .pem format via #{ChatOps.public_key_env_var_name}") unless ChatOps.public_key.present?
-      if signature_valid?(ChatOps.public_key, @chatops_signature, signature_string) ||
-          signature_valid?(ChatOps.alt_public_key, @chatops_signature, signature_string)
+      raise ConfigurationError.new("You need to add a client's public key in .pem format via #{Chatops.public_key_env_var_name}") unless Chatops.public_key.present?
+      if signature_valid?(Chatops.public_key, @chatops_signature, signature_string) ||
+          signature_valid?(Chatops.alt_public_key, @chatops_signature, signature_string)
           return true
       end
       return render :status => :forbidden, :plain => "Not authorized"
     end
 
     def ensure_valid_chatops_url
-      unless ChatOps.auth_base_url.present?
-        raise ConfigurationError.new("You need to set the server's base URL to authenticate chatops RPC via #{ChatOps.auth_base_url_env_var_name}")
+      unless Chatops.auth_base_url.present?
+        raise ConfigurationError.new("You need to set the server's base URL to authenticate chatops RPC via #{Chatops.auth_base_url_env_var_name}")
       end
-      if ChatOps.auth_base_url[-1] == "/"
-        raise ConfigurationError.new("Don't include a trailing slash in #{ChatOps.auth_base_url_env_var_name}; the rails path will be appended and it must match exactly.")
+      if Chatops.auth_base_url[-1] == "/"
+        raise ConfigurationError.new("Don't include a trailing slash in #{Chatops.auth_base_url_env_var_name}; the rails path will be appended and it must match exactly.")
       end
-      @chatops_url = ChatOps.auth_base_url + request.path
+      @chatops_url = Chatops.auth_base_url + request.path
     end
 
     def ensure_valid_chatops_nonce

lib/chatops/controller/rspec.rb

@@ -1,5 +1,5 @@
 require "chatops/controller/test_case"
 
 RSpec.configure do |config|
-  config.include ChatOps::Controller::TestCaseHelpers
+  config.include Chatops::Controller::TestCaseHelpers
 end

lib/chatops/controller/test_case.rb

@@ -1,5 +1,5 @@
 require "chatops/controller/test_case_helpers"
 
-class ChatOps::Controller::TestCase < ActionController::TestCase
-  include ChatOps::Controller::TestCaseHelpers
+class Chatops::Controller::TestCase < ActionController::TestCase
+  include Chatops::Controller::TestCaseHelpers
 end

lib/chatops/controller/test_case_helpers.rb

@@ -1,4 +1,4 @@
-module ChatOps::Controller::TestCaseHelpers
+module Chatops::Controller::TestCaseHelpers
 
   class NoMatchingCommandRegex < StandardError ; end
 

lib/chatops_controller.rb

@@ -1,6 +1,2 @@
 require "chatops_controller/version"
 require "chatops/controller"
-
-::ActiveSupport::Inflector.inflections(:en) do |inflect|
-  inflect.acronym("ChatOps")
-end

spec/lib/chatops/controller_spec.rb

@@ -4,9 +4,9 @@
 
 describe ActionController::Base, type: :controller do
   controller do
-    include ChatOps::Controller
+    include Chatops::Controller
     chatops_namespace :test
-    chatops_help "ChatOps of and relating to testing"
+    chatops_help "Chatops of and relating to testing"
     chatops_error_response "Try checking haystack?"
 
     before_action :ensure_app_given, :only => [:wcid]
@@ -137,7 +137,7 @@ def rails_flexible_post(path, outer_params, jsonrpc_params = nil)
     ENV.delete "CHATOPS_AUTH_BASE_URL"
     expect {
       get :list
-    }.to raise_error(ChatOps::Controller::ConfigurationError)
+    }.to raise_error(Chatops::Controller::ConfigurationError)
   end
 
   it "raises an error trying to auth without a public key" do
@@ -152,7 +152,7 @@ def rails_flexible_post(path, outer_params, jsonrpc_params = nil)
     ENV.delete "CHATOPS_AUTH_PUBLIC_KEY"
     expect {
       get :list
-    }.to raise_error(ChatOps::Controller::ConfigurationError)
+    }.to raise_error(Chatops::Controller::ConfigurationError)
   end
 
   it "doesn't authenticate with the wrong public key'" do
@@ -214,7 +214,7 @@ def rails_flexible_post(path, outer_params, jsonrpc_params = nil)
       expect(response.status).to eq 200
       expect(json_response).to eq({
         "namespace" => "test",
-        "help" => "ChatOps of and relating to testing",
+        "help" => "Chatops of and relating to testing",
         "error_response" => "Try checking haystack?",
         "methods" => {
           "wcid" => {
@@ -363,7 +363,7 @@ def rails_flexible_post(path, outer_params, jsonrpc_params = nil)
       it "anchors regexes" do
         expect {
           chat "too bad that this message doesn't start with where can i deploy foobar", "bhuga"
-        }.to raise_error(ChatOps::Controller::TestCaseHelpers::NoMatchingCommandRegex)
+        }.to raise_error(Chatops::Controller::TestCaseHelpers::NoMatchingCommandRegex)
       end
     end
   end