rds-rotate-db-snapshots 0.5.2 → 1.0.0

data/spec/lib/rds_rotate_db_snapshots/actions_spec.rb

@@ -1,116 +0,0 @@
-require 'helper'
-class Test
-  include RdsRotateDbSnapshots::Actions
-
-  attr_reader :options
-
-  def initialize(script_name: nil, options: {})
-    @script_name = script_name
-    @options = options
-    @backoff_counter = 0
-  end
-
-  def rds_client
-    @rds_client ||= RdsRotateDbSnapshots::RdsClient.new(@options)
-  end
-  alias client rds_client
-
-  def reset_backoff
-    @backoff_counter = 0
-  end
-
-  def time_periods
-    @options[:time_periods]
-  end
-
-  private
-
-  def backoff
-    @backoff_counter += 1
-  end
-end
-
-describe RdsRotateDbSnapshots::Actions do
-  let(:test_class) { Test.new(script_name: 'test', options: options) }
-  let(:client) { double("client") }
-  let(:time_periods) do
-    {
-      hourly: { seconds: 60 * 60, format: '%Y-%m-%d-%H', keep: 0, keeping: {} },
-      daily: { seconds: 24 * 60 * 60, format: '%Y-%m-%d', keep: 1, keeping: {} },
-      weekly: { seconds: 7 * 24 * 60 * 60, format: '%Y-%W', keep: 0, keeping: {} },
-      monthly: { seconds: 30 * 24 * 60 * 60, format: '%Y-%m', keep: 0, keeping: {} }
-    }
-  end
-  let(:options) do
-    {
-      aws_access_key: 'ACCESS_KEY',
-      aws_secret_access_key: 'SECRET_KEY',
-      aws_region: 'REGION',
-      pattern: 'test',
-      dry_run: false,
-      backoff_limit: 15,
-      time_periods: time_periods
-    }
-  end
-  let(:rds_snapshots) do
-    [
-      { snapshot_create_time: Time.now - (24 * 3600), db_instance_identifier: 'test_db',
-        db_snapshot_identifier: 'test_snapshot' },
-      { snapshot_create_time: Time.now, db_instance_identifier: 'test_db', db_snapshot_identifier: 'test_snapshot2' }
-    ]
-  end
-
-  before do
-    allow(client).to receive(:describe_db_snapshots).and_return(rds_snapshots)
-    allow(test_class).to receive(:client).and_return(client)
-  end
-
-  describe "#rotate_em" do
-    it "deletes the snapshots that are not part of the specified time periods" do
-      expect(client).to receive(:delete_db_snapshot)
-      test_class.rotate_em(rds_snapshots)
-    end
-
-    context 'when the snapshot is not matched with pattern' do
-      let(:rds_snapshots) do
-        [
-          { snapshot_create_time: Time.now - (49 * 3600), db_instance_identifier: 'test_db',
-            db_snapshot_identifier: 'other_snapshot' },
-          { snapshot_create_time: Time.now - (48 * 3600), db_instance_identifier: 'test_db',
-            db_snapshot_identifier: 'test_snapshot' },
-          { snapshot_create_time: Time.now, db_instance_identifier: 'test_db',
-            db_snapshot_identifier: 'test_snapshot2' }
-        ]
-      end
-
-      it "deletes the snapshots that are matched with pattern" do
-        expect(client).to receive(:delete_db_snapshot).with(db_snapshot_identifier: 'test_snapshot')
-        test_class.rotate_em(rds_snapshots)
-      end
-    end
-  end
-
-  describe "#create_snapshot" do
-    it "creates a snapshot with the specified name" do
-      expect(client).to receive(:create_db_snapshot)
-      test_class.create_snapshot('test', ['test_db'])
-    end
-
-    context "when snaphost name is invalid" do
-      it "raises an error SystemExit" do
-        expect { test_class.create_snapshot('$#', ['test_db']) }.to raise_error(SystemExit)
-      end
-    end
-  end
-
-  describe "#get_db_snapshots" do
-    let(:snapshots) { double('snapshots', db_snapshots: rds_snapshots) }
-
-    it "returns the list of snapshots from the client" do
-      allow(snapshots).to receive(:[]).with(:marker).and_return(nil)
-      expect(client).to receive(:describe_db_snapshots).and_return(snapshots)
-      snapshots = test_class.get_db_snapshots(options)
-      expect(snapshots).to eq(rds_snapshots)
-    end
-  end
-end

data/spec/lib/rds_rotate_db_snapshots/options_parser_spec.rb

@@ -1,30 +0,0 @@
-require 'helper'
-
-describe RdsRotateDbSnapshots::OptionsParser do
-  subject { described_class.new(script_name: script_name, cli: true).parse! }
-
-  let(:script_name) { "rds_rotate_snapshots.rb" }
-
-  describe "#parse!" do
-    before { ARGV.clear }
-
-    it "parses options correctly" do
-      ARGV.push("--aws-access-key", "ACCESS_KEY", "--aws-secret-access-key", "SECRET_KEY", "--aws-region", "REGION",
-                "--pattern", "PATTERN", "--backoff-limit", "20", "--create-snapshot", "snapshot")
-      options = subject
-
-      expect(options[:aws_access_key]).to eq("ACCESS_KEY")
-      expect(options[:aws_secret_access_key]).to eq("SECRET_KEY")
-      expect(options[:aws_region]).to eq("REGION")
-      expect(options[:pattern]).to eq("PATTERN")
-      expect(options[:backoff_limit]).to eq("20")
-      expect(options[:create_snapshot]).to eq("snapshot")
-    end
-
-    it "raises NotImplementedError when by-tags option is passed and it is not implemented" do
-      ARGV.push("--by-tags", "tag=value,tag2=value")
-
-      expect { subject }.to raise_error(RdsRotateDbSnapshots::OptionsParser::NotImplementedError)
-    end
-  end
-end

data/spec/lib/rds_rotate_db_snapshots/rds_client_spec.rb

@@ -1,34 +0,0 @@
-require 'helper'
-
-describe RdsRotateDbSnapshots::RdsClient do
-  let(:options) do
-    {
-    aws_access_key: "ACCESS_KEY",
-    aws_secret_access_key: "SECRET_KEY",
-    aws_session_token: "SESSION_TOKEN",
-    aws_region: "REGION"
-  }
-  end
-  let(:rds_client) { described_class.new(options) }
-
-  it 'configures the client with the correct credentials and region' do
-    expect(rds_client.instance_variable_get(:@client).config.credentials)
-      .to have_attributes(access_key_id: "ACCESS_KEY", secret_access_key: "SECRET_KEY", session_token: "SESSION_TOKEN")
-    expect(rds_client.instance_variable_get(:@client).config.region).to eq("REGION")
-  end
-
-  it 'delegates describe_db_snapshots method to the @client object' do
-    expect(rds_client.instance_variable_get(:@client)).to receive(:describe_db_snapshots)
-    rds_client.describe_db_snapshots
-  end
-
-  it 'delegates create_db_snapshot method to the @client object' do
-    expect(rds_client.instance_variable_get(:@client)).to receive(:create_db_snapshot)
-    rds_client.create_db_snapshot("test-snapshot")
-  end
-
-  it 'delegates delete_db_snapshot method to the @client object' do
-    expect(rds_client.instance_variable_get(:@client)).to receive(:delete_db_snapshot)
-    rds_client.delete_db_snapshot("test-snapshot")
-  end
-end

data/spec/lib/rds_rotate_db_snapshots_spec.rb

@@ -1,63 +0,0 @@
-require "helper"
-
-describe RdsRotateDbSnapshots do
-  subject { described_class.new(script_name: script_name, cli: cli) }
-
-  let(:script_name) { "test" }
-  let(:cli) { true }
-
-  before do
-    allow(Aws::RDS::Client).to receive(:new)
-  end
-
-  describe "on include" do
-    it "adds action methods to the base class" do
-      expect(described_class.instance_methods).to include(:rotate_em)
-      expect(described_class.instance_methods).to include(:create_snapshot)
-      expect(described_class.instance_methods).to include(:get_db_snapshots)
-      expect(described_class.instance_methods).to include(:rotate_by_tags)
-      expect(described_class.instance_methods).to include(:client)
-      expect(described_class.instance_methods).to include(:time_periods)
-    end
-  end
-
-  describe "#client" do
-    it "returns an RdsClient" do
-      expect(subject.client).to be_a(RdsRotateDbSnapshots::RdsClient)
-    end
-  end
-
-  describe "#rds_client" do
-    it "returns an RdsClient" do
-      expect(subject.rds_client).to be_a(RdsRotateDbSnapshots::RdsClient)
-    end
-  end
-
-  describe "#reset_backoff" do
-    it "resets backoff counter" do
-      subject.instance_variable_set(:@backoff_counter, 1)
-      subject.reset_backoff
-      expect(subject.instance_variable_get(:@backoff_counter)).to eq(0)
-    end
-  end
-
-  describe "#time_periods" do
-    it "returns time periods" do
-      expect(subject.time_periods).to eq(
-        daily: { format: "%Y-%m-%d", keep: 0, keeping: {}, seconds: 86_400 },
-        hourly: { format: "%Y-%m-%d-%H", keep: 0, keeping: {}, seconds: 3600 },
-        monthly: { format: "%Y-%m", keep: 0, keeping: {}, seconds: 2_592_000 },
-        weekly: { format: "%Y-%W", keep: 0, keeping: {}, seconds: 604_800 },
-        yearly: { format: "%Y", keep: 0, keeping: {}, seconds: 31_104_000 }
-      )
-    end
-  end
-
-  describe "#backoff" do
-    it "backs off" do
-      subject.instance_variable_set(:@backoff_counter, 1)
-      expect(subject).to receive(:sleep)
-      subject.send(:backoff)
-    end
-  end
-end