groovenauts-thor 0.19.1 → 0.19.1.1

data/spec/actions/create_link_spec.rb

@@ -0,0 +1,96 @@
+require "helper"
+require "thor/actions"
+require "tempfile"
+
+describe Thor::Actions::CreateLink do
+  before do
+    @silence = false
+    @hardlink_to = File.join(Dir.tmpdir, "linkdest.rb")
+    ::FileUtils.rm_rf(destination_root)
+    ::FileUtils.rm_rf(@hardlink_to)
+  end
+
+  def create_link(destination = nil, config = {}, options = {})
+    @base = MyCounter.new([1, 2], options, :destination_root => destination_root)
+    allow(@base).to receive(:file_name).and_return("rdoc")
+
+    @tempfile = Tempfile.new("config.rb")
+
+    @action = Thor::Actions::CreateLink.new(@base, destination, @tempfile.path,
+                                            {:verbose => !@silence}.merge(config))
+  end
+
+  def invoke!
+    capture(:stdout) { @action.invoke! }
+  end
+
+  def revoke!
+    capture(:stdout) { @action.revoke! }
+  end
+
+  def silence!
+    @silence = true
+  end
+
+  describe "#invoke!" do
+    it "creates a symbolic link for :symbolic => true" do
+      create_link("doc/config.rb", :symbolic => true)
+      invoke!
+      destination_path = File.join(destination_root, "doc/config.rb")
+      expect(File.exist?(destination_path)).to be true
+      expect(File.symlink?(destination_path)).to be true
+    end
+
+    it "creates a hard link for :symbolic => false" do
+      create_link(@hardlink_to, :symbolic => false)
+      invoke!
+      destination_path = @hardlink_to
+      expect(File.exist?(destination_path)).to be true
+      expect(File.symlink?(destination_path)).to be false
+    end
+
+    it "creates a symbolic link by default" do
+      create_link("doc/config.rb")
+      invoke!
+      destination_path = File.join(destination_root, "doc/config.rb")
+      expect(File.exist?(destination_path)).to be true
+      expect(File.symlink?(destination_path)).to be true
+    end
+
+    it "does not create a link if pretending" do
+      create_link("doc/config.rb", {}, :pretend => true)
+      invoke!
+      expect(File.exist?(File.join(destination_root, "doc/config.rb"))).to be false
+    end
+
+    it "shows created status to the user" do
+      create_link("doc/config.rb")
+      expect(invoke!).to eq("      create  doc/config.rb\n")
+    end
+
+    it "does not show any information if log status is false" do
+      silence!
+      create_link("doc/config.rb")
+      expect(invoke!).to be_empty
+    end
+  end
+
+  describe "#identical?" do
+    it "returns true if the destination link exists and is identical" do
+      create_link("doc/config.rb")
+      expect(@action.identical?).to be false
+      invoke!
+      expect(@action.identical?).to be true
+    end
+  end
+
+  describe "#revoke!" do
+    it "removes the symbolic link of non-existent destination" do
+      create_link("doc/config.rb")
+      invoke!
+      File.delete(@tempfile.path)
+      revoke!
+      expect(File.symlink?(@action.destination)).to be false
+    end
+  end
+end

data/spec/actions/directory_spec.rb

@@ -0,0 +1,169 @@
+require "helper"
+require "thor/actions"
+
+describe Thor::Actions::Directory do
+  before do
+    ::FileUtils.rm_rf(destination_root)
+    allow(invoker).to receive(:file_name).and_return("rdoc")
+  end
+
+  def invoker
+    @invoker ||= WhinyGenerator.new([1, 2], {}, :destination_root => destination_root)
+  end
+
+  def revoker
+    @revoker ||= WhinyGenerator.new([1, 2], {}, :destination_root => destination_root, :behavior => :revoke)
+  end
+
+  def invoke!(*args, &block)
+    capture(:stdout) { invoker.directory(*args, &block) }
+  end
+
+  def revoke!(*args, &block)
+    capture(:stdout) { revoker.directory(*args, &block) }
+  end
+
+  def exists_and_identical?(source_path, destination_path)
+    %w[config.rb README].each do |file|
+      source      = File.join(source_root, source_path, file)
+      destination = File.join(destination_root, destination_path, file)
+
+      expect(File.exist?(destination)).to be true
+      expect(FileUtils.identical?(source, destination)).to be true
+    end
+  end
+
+  describe "#invoke!" do
+    it "raises an error if the source does not exist" do
+      expect do
+        invoke! "unknown"
+      end.to raise_error(Thor::Error, /Could not find "unknown" in any of your source paths/)
+    end
+
+    it "does not create a directory in pretend mode" do
+      invoke! "doc", "ghost", :pretend => true
+      expect(File.exist?("ghost")).to be false
+    end
+
+    it "copies the whole directory recursively to the default destination" do
+      invoke! "doc"
+      exists_and_identical?("doc", "doc")
+    end
+
+    it "copies the whole directory recursively to the specified destination" do
+      invoke! "doc", "docs"
+      exists_and_identical?("doc", "docs")
+    end
+
+    it "copies only the first level files if recursive" do
+      invoke! ".", "commands", :recursive => false
+
+      file = File.join(destination_root, "commands", "group.thor")
+      expect(File.exist?(file)).to be true
+
+      file = File.join(destination_root, "commands", "doc")
+      expect(File.exist?(file)).to be false
+
+      file = File.join(destination_root, "commands", "doc", "README")
+      expect(File.exist?(file)).to be false
+    end
+
+    it "ignores files within excluding/ directories when exclude_pattern is provided" do
+      invoke! "doc", "docs", :exclude_pattern => /excluding\//
+      file = File.join(destination_root, "docs", "excluding", "rdoc.rb")
+      expect(File.exist?(file)).to be false
+    end
+
+    it "copies and evaluates files within excluding/ directory when no exclude_pattern is present" do
+      invoke! "doc", "docs"
+      file = File.join(destination_root, "docs", "excluding", "rdoc.rb")
+      expect(File.exist?(file)).to be true
+      expect(File.read(file)).to eq("BAR = BAR\n")
+    end
+
+    it "copies files from the source relative to the current path" do
+      invoker.inside "doc" do
+        invoke! "."
+      end
+      exists_and_identical?("doc", "doc")
+    end
+
+    it "copies and evaluates templates" do
+      invoke! "doc", "docs"
+      file = File.join(destination_root, "docs", "rdoc.rb")
+      expect(File.exist?(file)).to be true
+      expect(File.read(file)).to eq("FOO = FOO\n")
+    end
+
+    it "copies directories and preserves file mode" do
+      invoke! "preserve", "preserved", :mode => :preserve
+      original = File.join(source_root, "preserve", "script.sh")
+      copy = File.join(destination_root, "preserved", "script.sh")
+      expect(File.stat(original).mode).to eq(File.stat(copy).mode)
+    end
+
+    it "copies directories" do
+      invoke! "doc", "docs"
+      file = File.join(destination_root, "docs", "components")
+      expect(File.exist?(file)).to be true
+      expect(File.directory?(file)).to be true
+    end
+
+    it "does not copy .empty_directory files" do
+      invoke! "doc", "docs"
+      file = File.join(destination_root, "docs", "components", ".empty_directory")
+      expect(File.exist?(file)).to be false
+    end
+
+    it "copies directories even if they are empty" do
+      invoke! "doc/components", "docs/components"
+      file = File.join(destination_root, "docs", "components")
+      expect(File.exist?(file)).to be true
+    end
+
+    it "does not copy empty directories twice" do
+      content = invoke!("doc/components", "docs/components")
+      expect(content).not_to match(/exist/)
+    end
+
+    it "logs status" do
+      content = invoke!("doc")
+      expect(content).to match(/create  doc\/README/)
+      expect(content).to match(/create  doc\/config\.rb/)
+      expect(content).to match(/create  doc\/rdoc\.rb/)
+      expect(content).to match(/create  doc\/components/)
+    end
+
+    it "yields a block" do
+      checked = false
+      invoke!("doc") do |content|
+        checked ||= !!(content =~ /FOO/)
+      end
+      expect(checked).to be true
+    end
+
+    it "works with glob characters in the path" do
+      content = invoke!("app{1}")
+      expect(content).to match(/create  app\{1\}\/README/)
+    end
+  end
+
+  describe "#revoke!" do
+    it "removes the destination file" do
+      invoke! "doc"
+      revoke! "doc"
+
+      expect(File.exist?(File.join(destination_root, "doc", "README"))).to be false
+      expect(File.exist?(File.join(destination_root, "doc", "config.rb"))).to be false
+      expect(File.exist?(File.join(destination_root, "doc", "components"))).to be false
+    end
+
+    it "works with glob characters in the path" do
+      invoke! "app{1}"
+      expect(File.exist?(File.join(destination_root, "app{1}", "README"))).to be true
+
+      revoke! "app{1}"
+      expect(File.exist?(File.join(destination_root, "app{1}", "README"))).to be false
+    end
+  end
+end

data/spec/actions/empty_directory_spec.rb

@@ -0,0 +1,129 @@
+require "helper"
+require "thor/actions"
+
+describe Thor::Actions::EmptyDirectory do
+  before do
+    ::FileUtils.rm_rf(destination_root)
+  end
+
+  def empty_directory(destination, options = {})
+    @action = Thor::Actions::EmptyDirectory.new(base, destination)
+  end
+
+  def invoke!
+    capture(:stdout) { @action.invoke! }
+  end
+
+  def revoke!
+    capture(:stdout) { @action.revoke! }
+  end
+
+  def base
+    @base ||= MyCounter.new([1, 2], {}, :destination_root => destination_root)
+  end
+
+  describe "#destination" do
+    it "returns the full destination with the destination_root" do
+      expect(empty_directory("doc").destination).to eq(File.join(destination_root, "doc"))
+    end
+
+    it "takes relative root into account" do
+      base.inside("doc") do
+        expect(empty_directory("contents").destination).to eq(File.join(destination_root, "doc", "contents"))
+      end
+    end
+  end
+
+  describe "#relative_destination" do
+    it "returns the relative destination to the original destination root" do
+      base.inside("doc") do
+        expect(empty_directory("contents").relative_destination).to eq("doc/contents")
+      end
+    end
+  end
+
+  describe "#given_destination" do
+    it "returns the destination supplied by the user" do
+      base.inside("doc") do
+        expect(empty_directory("contents").given_destination).to eq("contents")
+      end
+    end
+  end
+
+  describe "#invoke!" do
+    it "copies the file to the specified destination" do
+      empty_directory("doc")
+      invoke!
+      expect(File.exist?(File.join(destination_root, "doc"))).to be true
+    end
+
+    it "shows created status to the user" do
+      empty_directory("doc")
+      expect(invoke!).to eq("      create  doc\n")
+    end
+
+    it "does not create a directory if pretending" do
+      base.inside("foo", :pretend => true) do
+        empty_directory("ghost")
+      end
+      expect(File.exist?(File.join(base.destination_root, "ghost"))).to be false
+    end
+
+    describe "when directory exists" do
+      it "shows exist status" do
+        empty_directory("doc")
+        invoke!
+        expect(invoke!).to eq("       exist  doc\n")
+      end
+    end
+  end
+
+  describe "#revoke!" do
+    it "removes the destination file" do
+      empty_directory("doc")
+      invoke!
+      revoke!
+      expect(File.exist?(@action.destination)).to be false
+    end
+  end
+
+  describe "#exists?" do
+    it "returns true if the destination file exists" do
+      empty_directory("doc")
+      expect(@action.exists?).to be false
+      invoke!
+      expect(@action.exists?).to be true
+    end
+  end
+
+  context "protected methods" do
+    describe "#convert_encoded_instructions" do
+      before do
+        empty_directory("test_dir")
+        allow(@action.base).to receive(:file_name).and_return("expected")
+      end
+
+      it "accepts and executes a 'legal' %\w+% encoded instruction" do
+        expect(@action.send(:convert_encoded_instructions, "%file_name%.txt")).to eq("expected.txt")
+      end
+
+      it "accepts and executes a private %\w+% encoded instruction" do
+        @action.base.extend Module.new {
+          def private_file_name
+            "expected"
+          end
+          private :private_file_name
+        }
+        expect(@action.send(:convert_encoded_instructions, "%private_file_name%.txt")).to eq("expected.txt")
+      end
+
+      it "ignores an 'illegal' %\w+% encoded instruction" do
+        expect(@action.send(:convert_encoded_instructions, "%some_name%.txt")).to eq("%some_name%.txt")
+      end
+
+      it "ignores incorrectly encoded instruction" do
+        expect(@action.send(:convert_encoded_instructions, "%some.name%.txt")).to eq("%some.name%.txt")
+      end
+    end
+  end
+end

data/spec/actions/file_manipulation_spec.rb

@@ -0,0 +1,392 @@
+require "helper"
+
+class Application; end
+
+describe Thor::Actions do
+  def runner(options = {})
+    @runner ||= MyCounter.new([1], options, :destination_root => destination_root)
+  end
+
+  def action(*args, &block)
+    capture(:stdout) { runner.send(*args, &block) }
+  end
+
+  def exists_and_identical?(source, destination)
+    destination = File.join(destination_root, destination)
+    expect(File.exist?(destination)).to be true
+
+    source = File.join(source_root, source)
+    expect(FileUtils).to be_identical(source, destination)
+  end
+
+  def file
+    File.join(destination_root, "foo")
+  end
+
+  before do
+    ::FileUtils.rm_rf(destination_root)
+  end
+
+  describe "#chmod" do
+    it "executes the command given" do
+      expect(FileUtils).to receive(:chmod_R).with(0755, file) # rubocop:disable SymbolName
+      action :chmod, "foo", 0755
+    end
+
+    it "does not execute the command if pretending" do
+      expect(FileUtils).not_to receive(:chmod_R) # rubocop:disable SymbolName
+      runner(:pretend => true)
+      action :chmod, "foo", 0755
+    end
+
+    it "logs status" do
+      expect(FileUtils).to receive(:chmod_R).with(0755, file) # rubocop:disable SymbolName
+      expect(action(:chmod, "foo", 0755)).to eq("       chmod  foo\n")
+    end
+
+    it "does not log status if required" do
+      expect(FileUtils).to receive(:chmod_R).with(0755, file) # rubocop:disable SymbolName
+      expect(action(:chmod, "foo", 0755, :verbose => false)).to be_empty
+    end
+  end
+
+  describe "#copy_file" do
+    it "copies file from source to default destination" do
+      action :copy_file, "command.thor"
+      exists_and_identical?("command.thor", "command.thor")
+    end
+
+    it "copies file from source to the specified destination" do
+      action :copy_file, "command.thor", "foo.thor"
+      exists_and_identical?("command.thor", "foo.thor")
+    end
+
+    it "copies file from the source relative to the current path" do
+      runner.inside("doc") do
+        action :copy_file, "README"
+      end
+      exists_and_identical?("doc/README", "doc/README")
+    end
+
+    it "copies file from source to default destination and preserves file mode" do
+      action :copy_file, "preserve/script.sh", :mode => :preserve
+      original = File.join(source_root, "preserve/script.sh")
+      copy = File.join(destination_root, "preserve/script.sh")
+      expect(File.stat(original).mode).to eq(File.stat(copy).mode)
+    end
+
+    it "logs status" do
+      expect(action(:copy_file, "command.thor")).to eq("      create  command.thor\n")
+    end
+
+    it "accepts a block to change output" do
+      action :copy_file, "command.thor" do |content|
+        "OMG" + content
+      end
+      expect(File.read(File.join(destination_root, "command.thor"))).to match(/^OMG/)
+    end
+  end
+
+  describe "#link_file" do
+    it "links file from source to default destination" do
+      action :link_file, "command.thor"
+      exists_and_identical?("command.thor", "command.thor")
+    end
+
+    it "links file from source to the specified destination" do
+      action :link_file, "command.thor", "foo.thor"
+      exists_and_identical?("command.thor", "foo.thor")
+    end
+
+    it "links file from the source relative to the current path" do
+      runner.inside("doc") do
+        action :link_file, "README"
+      end
+      exists_and_identical?("doc/README", "doc/README")
+    end
+
+    it "logs status" do
+      expect(action(:link_file, "command.thor")).to eq("      create  command.thor\n")
+    end
+  end
+
+  describe "#get" do
+    it "copies file from source to the specified destination" do
+      action :get, "doc/README", "docs/README"
+      exists_and_identical?("doc/README", "docs/README")
+    end
+
+    it "uses just the source basename as destination if none is specified" do
+      action :get, "doc/README"
+      exists_and_identical?("doc/README", "README")
+    end
+
+    it "allows the destination to be set as a block result" do
+      action(:get, "doc/README") { |c| "docs/README" }
+      exists_and_identical?("doc/README", "docs/README")
+    end
+
+    it "yields file content to a block" do
+      action :get, "doc/README" do |content|
+        expect(content).to eq("__start__\nREADME\n__end__\n")
+      end
+    end
+
+    it "logs status" do
+      expect(action(:get, "doc/README", "docs/README")).to eq("      create  docs/README\n")
+    end
+
+    it "accepts http remote sources" do
+      body = "__start__\nHTTPFILE\n__end__\n"
+      FakeWeb.register_uri(:get, "http://example.com/file.txt", :body => body)
+      action :get, "http://example.com/file.txt" do |content|
+        expect(content).to eq(body)
+      end
+      FakeWeb.clean_registry
+    end
+
+    it "accepts https remote sources" do
+      body = "__start__\nHTTPSFILE\n__end__\n"
+      FakeWeb.register_uri(:get, "https://example.com/file.txt", :body => body)
+      action :get, "https://example.com/file.txt" do |content|
+        expect(content).to eq(body)
+      end
+      FakeWeb.clean_registry
+    end
+  end
+
+  describe "#template" do
+    it "allows using block helpers in the template" do
+      action :template, "doc/block_helper.rb"
+
+      file = File.join(destination_root, "doc/block_helper.rb")
+      expect(File.read(file)).to eq("Hello world!")
+    end
+
+    it "evaluates the template given as source" do
+      runner.instance_variable_set("@klass", "Config")
+      action :template, "doc/config.rb"
+
+      file = File.join(destination_root, "doc/config.rb")
+      expect(File.read(file)).to eq("class Config; end\n")
+    end
+
+    it "copies the template to the specified destination" do
+      runner.instance_variable_set("@klass", "Config")
+      action :template, "doc/config.rb", "doc/configuration.rb"
+      file = File.join(destination_root, "doc/configuration.rb")
+      expect(File.exist?(file)).to be true
+    end
+
+    it "converts encoded instructions" do
+      expect(runner).to receive(:file_name).and_return("rdoc")
+      action :template, "doc/%file_name%.rb.tt"
+      file = File.join(destination_root, "doc/rdoc.rb")
+      expect(File.exist?(file)).to be true
+    end
+
+    it "accepts filename without .tt for template method" do
+      expect(runner).to receive(:file_name).and_return("rdoc")
+      action :template, "doc/%file_name%.rb"
+      file = File.join(destination_root, "doc/rdoc.rb")
+      expect(File.exist?(file)).to be true
+    end
+
+    it "logs status" do
+      runner.instance_variable_set("@klass", "Config")
+      expect(capture(:stdout) { runner.template("doc/config.rb") }).to eq("      create  doc/config.rb\n")
+    end
+
+    it "accepts a block to change output" do
+      runner.instance_variable_set("@klass", "Config")
+      action :template, "doc/config.rb" do |content|
+        "OMG" + content
+      end
+      expect(File.read(File.join(destination_root, "doc/config.rb"))).to match(/^OMG/)
+    end
+
+    it "guesses the destination name when given only a source" do
+      action :template, "doc/config.yaml.tt"
+
+      file = File.join(destination_root, "doc/config.yaml")
+      expect(File.exist?(file)).to be true
+    end
+  end
+
+  describe "when changing existent files" do
+    before do
+      ::FileUtils.cp_r(source_root, destination_root)
+    end
+
+    def file
+      File.join(destination_root, "doc", "README")
+    end
+
+    describe "#remove_file" do
+      it "removes the file given" do
+        action :remove_file, "doc/README"
+        expect(File.exist?(file)).to be false
+      end
+
+      it "removes directories too" do
+        action :remove_dir, "doc"
+        expect(File.exist?(File.join(destination_root, "doc"))).to be false
+      end
+
+      it "does not remove if pretending" do
+        runner(:pretend => true)
+        action :remove_file, "doc/README"
+        expect(File.exist?(file)).to be true
+      end
+
+      it "logs status" do
+        expect(action(:remove_file, "doc/README")).to eq("      remove  doc/README\n")
+      end
+
+      it "does not log status if required" do
+        expect(action(:remove_file, "doc/README", :verbose => false)).to be_empty
+      end
+    end
+
+    describe "#gsub_file" do
+      it "replaces the content in the file" do
+        action :gsub_file, "doc/README", "__start__", "START"
+        expect(File.binread(file)).to eq("START\nREADME\n__end__\n")
+      end
+
+      it "does not replace if pretending" do
+        runner(:pretend => true)
+        action :gsub_file, "doc/README", "__start__", "START"
+        expect(File.binread(file)).to eq("__start__\nREADME\n__end__\n")
+      end
+
+      it "accepts a block" do
+        action(:gsub_file, "doc/README", "__start__") { |match| match.gsub("__", "").upcase  }
+        expect(File.binread(file)).to eq("START\nREADME\n__end__\n")
+      end
+
+      it "logs status" do
+        expect(action(:gsub_file, "doc/README", "__start__", "START")).to eq("        gsub  doc/README\n")
+      end
+
+      it "does not log status if required" do
+        expect(action(:gsub_file, file, "__", :verbose => false) { |match| match * 2 }).to be_empty
+      end
+    end
+
+    describe "#append_to_file" do
+      it "appends content to the file" do
+        action :append_to_file, "doc/README", "END\n"
+        expect(File.binread(file)).to eq("__start__\nREADME\n__end__\nEND\n")
+      end
+
+      it "accepts a block" do
+        action(:append_to_file, "doc/README") { "END\n" }
+        expect(File.binread(file)).to eq("__start__\nREADME\n__end__\nEND\n")
+      end
+
+      it "logs status" do
+        expect(action(:append_to_file, "doc/README", "END")).to eq("      append  doc/README\n")
+      end
+    end
+
+    describe "#prepend_to_file" do
+      it "prepends content to the file" do
+        action :prepend_to_file, "doc/README", "START\n"
+        expect(File.binread(file)).to eq("START\n__start__\nREADME\n__end__\n")
+      end
+
+      it "accepts a block" do
+        action(:prepend_to_file, "doc/README") { "START\n" }
+        expect(File.binread(file)).to eq("START\n__start__\nREADME\n__end__\n")
+      end
+
+      it "logs status" do
+        expect(action(:prepend_to_file, "doc/README", "START")).to eq("     prepend  doc/README\n")
+      end
+    end
+
+    describe "#inject_into_class" do
+      def file
+        File.join(destination_root, "application.rb")
+      end
+
+      it "appends content to a class" do
+        action :inject_into_class, "application.rb", Application, "  filter_parameters :password\n"
+        expect(File.binread(file)).to eq("class Application < Base\n  filter_parameters :password\nend\n")
+      end
+
+      it "accepts a block" do
+        action(:inject_into_class, "application.rb", Application) { "  filter_parameters :password\n" }
+        expect(File.binread(file)).to eq("class Application < Base\n  filter_parameters :password\nend\n")
+      end
+
+      it "logs status" do
+        expect(action(:inject_into_class, "application.rb", Application, "  filter_parameters :password\n")).to eq("      insert  application.rb\n")
+      end
+
+      it "does not append if class name does not match" do
+        action :inject_into_class, "application.rb", "App", "  filter_parameters :password\n"
+        expect(File.binread(file)).to eq("class Application < Base\nend\n")
+      end
+    end
+  end
+
+  describe "when adjusting comments" do
+    before do
+      ::FileUtils.cp_r(source_root, destination_root)
+    end
+
+    def file
+      File.join(destination_root, "doc", "COMMENTER")
+    end
+
+    unmodified_comments_file = /__start__\n # greenblue\n#\n# yellowblue\n#yellowred\n #greenred\norange\n    purple\n  ind#igo\n  # ind#igo\n__end__/
+
+    describe "#uncomment_lines" do
+      it "uncomments all matching lines in the file" do
+        action :uncomment_lines, "doc/COMMENTER", "green"
+        expect(File.binread(file)).to match(/__start__\n greenblue\n#\n# yellowblue\n#yellowred\n greenred\norange\n    purple\n  ind#igo\n  # ind#igo\n__end__/)
+
+        action :uncomment_lines, "doc/COMMENTER", "red"
+        expect(File.binread(file)).to match(/__start__\n greenblue\n#\n# yellowblue\nyellowred\n greenred\norange\n    purple\n  ind#igo\n  # ind#igo\n__end__/)
+      end
+
+      it "correctly uncomments lines with hashes in them" do
+        action :uncomment_lines, "doc/COMMENTER", "ind#igo"
+        expect(File.binread(file)).to match(/__start__\n # greenblue\n#\n# yellowblue\n#yellowred\n #greenred\norange\n    purple\n  ind#igo\n  ind#igo\n__end__/)
+      end
+
+      it "does not modify already uncommented lines in the file" do
+        action :uncomment_lines, "doc/COMMENTER", "orange"
+        action :uncomment_lines, "doc/COMMENTER", "purple"
+        expect(File.binread(file)).to match(unmodified_comments_file)
+      end
+
+      it "does not uncomment the wrong line when uncommenting lines preceded by blank commented line" do
+        action :uncomment_lines, "doc/COMMENTER", "yellow"
+        expect(File.binread(file)).to match(/__start__\n # greenblue\n#\nyellowblue\nyellowred\n #greenred\norange\n    purple\n  ind#igo\n  # ind#igo\n__end__/)
+      end
+    end
+
+    describe "#comment_lines" do
+      it "comments lines which are not commented" do
+        action :comment_lines, "doc/COMMENTER", "orange"
+        expect(File.binread(file)).to match(/__start__\n # greenblue\n#\n# yellowblue\n#yellowred\n #greenred\n# orange\n    purple\n  ind#igo\n  # ind#igo\n__end__/)
+
+        action :comment_lines, "doc/COMMENTER", "purple"
+        expect(File.binread(file)).to match(/__start__\n # greenblue\n#\n# yellowblue\n#yellowred\n #greenred\n# orange\n    # purple\n  ind#igo\n  # ind#igo\n__end__/)
+      end
+
+      it "correctly comments lines with hashes in them" do
+        action :comment_lines, "doc/COMMENTER", "ind#igo"
+        expect(File.binread(file)).to match(/__start__\n # greenblue\n#\n# yellowblue\n#yellowred\n #greenred\norange\n    purple\n  # ind#igo\n  # ind#igo\n__end__/)
+      end
+
+      it "does not modify already commented lines" do
+        action :comment_lines, "doc/COMMENTER", "green"
+        expect(File.binread(file)).to match(unmodified_comments_file)
+      end
+    end
+  end
+end