bard 2.2.0 → 3.1.0

data/spec/bard/git_spec.rb

@@ -1,101 +0,0 @@
-require "spec_helper"
-require "bard/plugins/git"
-
-describe Bard::Git do
-  describe ".current_branch" do
-    it "should return the current branch" do
-      allow(Bard::Git).to receive(:`).with("git symbolic-ref HEAD 2>&1").and_return("refs/heads/master\n")
-      expect(Bard::Git.current_branch).to eq("master")
-    end
-
-    it "should return false if not on a branch" do
-      allow(Bard::Git).to receive(:`).with("git symbolic-ref HEAD 2>&1").and_return("fatal: ref HEAD is not a symbolic ref\n")
-      expect(Bard::Git.current_branch).to be_falsey
-    end
-  end
-
-  describe ".fast_forward_merge?" do
-    it "should return true if the root is an ancestor of the branch" do
-      allow(Bard::Git).to receive(:sha_of).with("root").and_return("root_sha")
-      allow(Bard::Git).to receive(:sha_of).with("branch").and_return("branch_sha")
-      allow(Bard::Git).to receive(:`).with("git merge-base root_sha branch_sha").and_return("root_sha\n")
-      expect(Bard::Git.fast_forward_merge?("root", "branch")).to be_truthy
-    end
-
-    it "should return false if the root is not an ancestor of the branch" do
-      allow(Bard::Git).to receive(:sha_of).with("root").and_return("root_sha")
-      allow(Bard::Git).to receive(:sha_of).with("branch").and_return("branch_sha")
-      allow(Bard::Git).to receive(:`).with("git merge-base root_sha branch_sha").and_return("other_sha\n")
-      expect(Bard::Git.fast_forward_merge?("root", "branch")).to be_falsey
-    end
-  end
-
-  describe ".up_to_date_with_remote?" do
-    it "should return true if the local branch is up to date with the remote" do
-      allow(Bard::Git).to receive(:sha_of).with("branch").and_return("sha")
-      allow(Bard::Git).to receive(:sha_of).with("origin/branch").and_return("sha")
-      expect(Bard::Git.up_to_date_with_remote?("branch")).to be_truthy
-    end
-
-    it "should return false if the local branch is not up to date with the remote" do
-      allow(Bard::Git).to receive(:sha_of).with("branch").and_return("sha1")
-      allow(Bard::Git).to receive(:sha_of).with("origin/branch").and_return("sha2")
-      expect(Bard::Git.up_to_date_with_remote?("branch")).to be_falsey
-    end
-  end
-
-  describe ".sha_of" do
-    it "should return the sha of a ref" do
-      allow(Bard::Git).to receive(:`).with("git rev-parse ref 2>/dev/null") {
-        `true` # sets $? to success
-        "sha\n"
-      }
-      expect(Bard::Git.sha_of("ref")).to eq("sha")
-    end
-
-    it "should return nil if the ref does not exist" do
-      allow(Bard::Git).to receive(:`).with("git rev-parse ref 2>/dev/null") {
-        `false` # sets $? to failure
-        "ref: fatal: ambiguous argument 'ref': unknown revision or path not in the working tree.\n"
-      }
-      expect(Bard::Git.sha_of("ref")).to be_nil
-    end
-  end
-
-  describe ".github_pages_url" do
-    def stub_origin(remote)
-      allow(Bard::Git).to receive(:`).with("git remote get-url origin").and_return("#{remote}\n")
-    end
-
-    it "derives a project pages url from an ssh remote" do
-      stub_origin("git@github.com:botandrose/bard")
-      expect(Bard::Git.github_pages_url).to eq("https://botandrose.github.io/bard/")
-    end
-
-    it "derives a project pages url from an https remote with a .git suffix" do
-      stub_origin("https://github.com/botandrose/bard.git")
-      expect(Bard::Git.github_pages_url).to eq("https://botandrose.github.io/bard/")
-    end
-
-    it "lowercases the owner in the pages host" do
-      stub_origin("git@github.com:BotAndRose/Bard.git")
-      expect(Bard::Git.github_pages_url).to eq("https://botandrose.github.io/Bard/")
-    end
-
-    it "uses the root url for a user/org pages repo" do
-      stub_origin("git@github.com:botandrose/botandrose.github.io")
-      expect(Bard::Git.github_pages_url).to eq("https://botandrose.github.io/")
-    end
-
-    it "raises for a non-github remote" do
-      stub_origin("git@gitlab.com:botandrose/bard.git")
-      expect { Bard::Git.github_pages_url }.to raise_error(/not a github\.com repository/)
-    end
-
-    it "raises when there is no origin remote" do
-      allow(Bard::Git).to receive(:`).with("git remote get-url origin").and_return("\n")
-      expect { Bard::Git.github_pages_url }.to raise_error(/not a github\.com repository/)
-    end
-  end
-end
-

data/spec/bard/github_spec.rb

@@ -1,45 +0,0 @@
-require "spec_helper"
-require "bard/plugins/github"
-
-describe Bard::Github do
-  let(:github) { Bard::Github.new("test-project") }
-
-  before do
-    allow(Bard::Secrets).to receive(:fetch).with("github-apikey").and_return("12345")
-  end
-
-  describe "#get" do
-    it "should make a GET request to the GitHub API" do
-      stub_request(:get, "https://api.github.com/repos/botandrosedesign/test-project/path").to_return(body: "{}")
-      github.get("path")
-    end
-  end
-
-  describe "#post" do
-    it "should make a POST request to the GitHub API" do
-      stub_request(:post, "https://api.github.com/repos/botandrosedesign/test-project/path").with(body: "{\"foo\":\"bar\"}").to_return(body: "{}")
-      github.post("path", { foo: "bar" })
-    end
-  end
-
-  describe "#put" do
-    it "should make a PUT request to the GitHub API" do
-      stub_request(:put, "https://api.github.com/repos/botandrosedesign/test-project/path").with(body: "{\"foo\":\"bar\"}").to_return(body: "{}")
-      github.put("path", { foo: "bar" })
-    end
-  end
-
-  describe "#patch" do
-    it "should make a PATCH request to the GitHub API" do
-      stub_request(:patch, "https://api.github.com/repos/botandrosedesign/test-project/path").with(body: "{\"foo\":\"bar\"}").to_return(body: "{}")
-      github.patch("path", { foo: "bar" })
-    end
-  end
-
-  describe "#delete" do
-    it "should make a DELETE request to the GitHub API" do
-      stub_request(:delete, "https://api.github.com/repos/botandrosedesign/test-project").with(body: "{\"foo\":\"bar\"}").to_return(body: "{}")
-      github.delete(nil, { foo: "bar" })
-    end
-  end
-end

data/spec/bard/ping_spec.rb

@@ -1,60 +0,0 @@
-require "spec_helper"
-require "bard/plugins/ping/check"
-
-describe Bard::Ping do
-  let(:target) { double("target", ping: ["http://example.com"]) }
-  let(:ping) { described_class.new(target) }
-
-  def success_response
-    Net::HTTPSuccess.new(1.0, "200", "OK")
-  end
-
-  def not_found_response
-    Net::HTTPNotFound.new(1.0, "404", "Not Found")
-  end
-
-  context "when the target is reachable" do
-    it "returns an empty array" do
-      allow(ping).to receive(:http_get).and_return(success_response)
-      expect(ping.call).to be_empty
-    end
-  end
-
-  context "when the target is not reachable" do
-    it "returns the url" do
-      allow(ping).to receive(:http_get).and_return(not_found_response)
-      expect(ping.call).to eq(["http://example.com"])
-    end
-  end
-
-  context "when there is a redirect" do
-    it "follows the redirect and returns an empty array" do
-      redirect_response = Net::HTTPRedirection.new(1.0, "301", "Moved Permanently")
-      redirect_response["location"] = "/new"
-      allow(ping).to receive(:http_get).with(URI("http://example.com")).and_return(redirect_response)
-      allow(ping).to receive(:http_get).with(URI("http://example.com/new")).and_return(success_response)
-      expect(ping.call).to be_empty
-    end
-  end
-
-  context "when a transient error occurs" do
-    it "retries once before marking down" do
-      calls = 0
-      allow(ping).to receive(:http_get) do
-        calls += 1
-        raise Errno::ECONNRESET if calls == 1
-
-        success_response
-      end
-
-      expect(ping.call).to be_empty
-    end
-  end
-
-  context "when errors persist across retries" do
-    it "returns the url" do
-      allow(ping).to receive(:http_get).and_raise(Errno::ECONNREFUSED)
-      expect(ping.call).to eq(["http://example.com"])
-    end
-  end
-end

data/spec/bard/ssh_copy_spec.rb

@@ -1,44 +0,0 @@
-require "spec_helper"
-require "bard/plugins/ssh/copy"
-
-describe Bard::SSH::Copy do
-  let(:ssh_server) { double("ssh_server", gateway: nil, ssh_key: nil, port: "22", ssh_uri: double(port: 22, user: "user", host: "example.com")) }
-  let(:production) { double("production", key: :production, server: ssh_server, scp_uri: "user@example.com:/path/to/file", rsync_uri: "user@example.com:/path/to/", path: "/path/to", has_capability?: true) }
-  let(:local) { double("local", key: :local, has_capability?: false) }
-
-  describe ".can_handle?" do
-    it "handles pairs where at least one target has ssh capability" do
-      expect(Bard::SSH::Copy.can_handle?(local, production)).to be true
-      expect(Bard::SSH::Copy.can_handle?(production, local)).to be true
-      expect(Bard::SSH::Copy.can_handle?(production, production)).to be true
-    end
-
-    it "does not handle pairs where neither target has ssh capability" do
-      expect(Bard::SSH::Copy.can_handle?(local, local)).to be false
-    end
-  end
-
-  context ".file via Bard::Copy dispatch" do
-    it "should copy a file from a remote server to the local machine" do
-      expect(Bard::Command).to receive(:run!).with("scp -o StrictHostKeyChecking=no -o UserKnownHostsFile=/dev/null -o LogLevel=ERROR user@example.com:/path/to/file path/to/file", verbose: false)
-      Bard::Copy.file "path/to/file", from: production, to: local
-    end
-
-    it "should copy a file from the local machine to a remote server" do
-      expect(Bard::Command).to receive(:run!).with("scp -o StrictHostKeyChecking=no -o UserKnownHostsFile=/dev/null -o LogLevel=ERROR path/to/file user@example.com:/path/to/file", verbose: false)
-      Bard::Copy.file "path/to/file", from: local, to: production
-    end
-  end
-
-  context ".dir via Bard::Copy dispatch" do
-    it "should copy a directory from a remote server to the local machine" do
-      expect(Bard::Command).to receive(:run!).with("rsync -e'ssh  -p22' --delete --info=progress2 -az user@example.com:/path/to/ ./path/", verbose: false)
-      Bard::Copy.dir "path/to", from: production, to: local
-    end
-
-    it "should copy a directory from the local machine to a remote server" do
-      expect(Bard::Command).to receive(:run!).with("rsync -e'ssh  -p22' --delete --info=progress2 -az ./path/to user@example.com:/path/to/", verbose: false)
-      Bard::Copy.dir "path/to", from: local, to: production
-    end
-  end
-end