hackpad-cli 0.0.7 → 0.1.0

data/spec/lib/hackpad/cli/config_spec.rb

@@ -1,7 +1,7 @@
 # encoding: utf-8
 
 require 'spec_helper'
-require "hackpad/cli/config"
+require 'hackpad/cli/config'
 
 describe Hackpad::Cli::Config do
 
@@ -10,21 +10,22 @@ describe Hackpad::Cli::Config do
   let(:options) { { configdir: configdir, workspace: 'default' } }
 
   before :each do
-    FileUtils.mkdir_p configdir unless Dir.exists?(configdir)
+    FileUtils.mkdir_p configdir unless Dir.exist?(configdir)
   end
 
   after :each do
-    FileUtils.rm configfile if File.exists? configfile
+    FileUtils.rm configfile if File.exist?(configfile)
   end
 
-  describe ".load" do
-    let(:config) { {'xx' => 'oo'} }
+  describe '.load' do
+    let(:config) { { 'xx' => 'oo' } }
 
-    context "when there is no config file," do
-      it "calls for setup" do
-        subject.stub(:setup).with(configfile)
-        File.open(configfile, "w") do |f|
-          f.write YAML::dump(config)
+    context 'when there is no config file,' do
+      it 'calls for setup' do
+        Dir.stub(:exists?).and_return false
+        subject.stub(:setup).with(configfile, STDIN, STDOUT)
+        File.open(configfile, 'w') do |f|
+          f.write YAML.dump(config)
         end
         expect(subject.load options).to eq config
       end
@@ -32,12 +33,12 @@ describe Hackpad::Cli::Config do
 
   end
 
-  describe ".setup" do
-    context "when normal input is provided," do
+  describe '.setup' do
+    context 'when normal input is provided,' do
       let(:input) { StringIO.new }
       let(:output) { StringIO.new }
-      it "handles setup interactively" do
-        input.stub(:gets).and_return("client_id","secret","site")
+      it 'handles setup interactively' do
+        input.stub(:gets).and_return('client_id', 'secret', 'site')
         subject.send :setup, configfile, input, output
         expect(File.read configfile).to eq "---\nclient_id: client_id\nsecret: secret\nsite: site\n"
       end

data/spec/lib/hackpad/cli/pad_spec.rb

@@ -1,41 +1,97 @@
 # encoding: utf-8
 
 require 'spec_helper'
-require "hackpad/cli"
-require "hackpad/cli/pad"
-require "hackpad/cli/api"
-require "hackpad/cli/store"
+require 'hackpad/cli'
+require 'hackpad/cli/pad'
+require 'hackpad/cli/api'
+require 'hackpad/cli/store'
 
 describe Hackpad::Cli::Pad do
 
-  before :each do
-    Hackpad::Cli::Api.stub(:read).with('123', 'txt').and_return("content\nand body")
-    Hackpad::Cli::Api.stub(:read_options).with('123').and_return({"success" => "true", "options" => {}})
-    options = {
-      configdir: File.expand_path('../../../../files', __FILE__),
-      workspace: 'default'
-    }
-    Hackpad::Cli::Store.prepare options
-    @pad = Hackpad::Cli::Pad.new "123"
-    @pad.load 'txt'
-  end
+  let(:pad) { Hackpad::Cli::Pad.new '123' }
 
-  after :each do
-    FileUtils.rm_rf File.expand_path('../../../../files/default', __FILE__)
+  describe '.new' do
+    it { expect(pad.id).to eq '123' }
   end
 
-  it "creates a new pad object" do
-    expect(@pad.id).to eq "123"
+  describe '.cached?' do
+    before { Hackpad::Cli::Store.stub(:exists?).and_return true }
+    it { expect(pad.cached?).to be_true }
   end
 
-  it "Can extract the title" do
-    expect(@pad.title).to eq "content"
+  context 'when the pad has no data,' do
+    describe '.title' do
+      it { expect(pad.title).to eq nil }
+    end
+    describe '.chars' do
+      it { expect(pad.chars).to eq nil }
+    end
+    describe '.lines' do
+      it { expect(pad.lines).to eq nil }
+    end
   end
-  it "Can count chars from content" do
-    expect(@pad.chars).to be 16
+
+  context 'when the pad has only a content,' do
+    before { pad.instance_variable_set(:@content, "This\nis\nInformation!") }
+    describe '.title' do
+      it { expect(pad.title).to eq 'This' }
+    end
+    describe '.chars' do
+      it { expect(pad.chars).to eq 20 }
+    end
+    describe '.lines' do
+      it { expect(pad.lines).to eq 3 }
+    end
   end
-  it "Can count lines from content" do
-    expect(@pad.lines).to be 2
+
+  context 'when a pad is cached,' do
+    before { Hackpad::Cli::Store.stub(:exists?).and_return true }
+    context "when we don't want a refresh," do
+      describe '.load' do
+        context 'when unknown format is asked,' do
+          it { expect { pad.load 'xxx' }.to raise_error(Hackpad::Cli::UnknownFormat) }
+        end
+        context 'when pad has no id,' do
+          before { pad.send(:remove_instance_variable, :@id) }
+          it { expect { pad.load 'txt' }.to raise_error(Hackpad::Cli::UndefinedPad) }
+        end
+        context 'when all is ok,' do
+          before { pad.stub(:load_from_cache) }
+          it { expect { pad.load 'txt' }.not_to raise_error }
+        end
+      end
+      describe '.load_from_cache' do
+        let(:meta) { { 'options' => { 'guestPolicy' => 'open', 'isModerated' => false }, 'cached_at' => 'some time' } }
+        before { Hackpad::Cli::Store.stub(:read).with('123', 'txt').and_return("This\nis\nInformation!") }
+        before { Hackpad::Cli::Store.stub(:read_options).with('123').and_return(meta) }
+        before { pad.load_from_cache 'txt' }
+        it { expect(pad.content).to eq "This\nis\nInformation!" }
+        it { expect(pad.guest_policy).to eq 'open' }
+        it { expect(pad.moderated).to be_false }
+        it { expect(pad.cached_at).to eq 'some time' }
+      end
+    end
+    context 'when we want a refresh,' do
+      describe '.load' do
+        before { pad.stub(:load_from_api) }
+        it { expect { pad.load 'txt', true }.not_to raise_error }
+      end
+      describe '.load_from_api' do
+        let(:meta) { { 'options' => { 'guestPolicy' => 'open', 'isModerated' => false }, 'cached_at' => 'some time' } }
+        before { Hackpad::Cli::Api.stub(:read).with('123', 'txt').and_return("This\nis\nInformation!") }
+        before { Hackpad::Cli::Api.stub(:read_options).with('123').and_return(meta) }
+        context 'when we want to save to cache,' do
+          before { Hackpad::Cli::Store.stub(:save) }
+          before { Hackpad::Cli::Store.stub(:save_options) }
+          before { pad.load_from_api 'txt' }
+          it { expect(pad.content).to eq "This\nis\nInformation!" }
+        end
+        context "when we don't want to save to cache," do
+          before { pad.load_from_api 'txt', false }
+          it { expect(pad.content).to eq "This\nis\nInformation!" }
+        end
+      end
+    end
   end
 
 end

data/spec/lib/hackpad/cli/padlist_spec.rb

@@ -0,0 +1,70 @@
+# encoding: utf-8
+
+require 'spec_helper'
+require 'hackpad/cli/padlist'
+
+describe Hackpad::Cli::Padlist do
+
+  describe '.get_list' do
+    let(:output) { StringIO.new }
+
+    context 'when no refresh is asked and cache exists,' do
+      before { Hackpad::Cli::Store.stub(:exists?).and_return(true) }
+      before { Hackpad::Cli::Store.stub(:read_list).and_return(Array.new) }
+      it { expect(subject.get_list).to be_an(Array) }
+    end
+
+    context "when no refresh is asked and cache don't exist," do
+      let(:pad1) { OpenStruct.new(id: '123', title: 'title1') }
+      let(:pad2) { OpenStruct.new(id: '456', title: 'title2') }
+      before { Hackpad::Cli::Api.stub(:list).and_return(%w(123 456)) }
+      before { Hackpad::Cli::Store.stub(:exists?).and_return(false, false) }
+      before { Hackpad::Cli::Padlist.stub(:get_pad).and_return(pad1, pad2) }
+      before { Hackpad::Cli::Store.stub(:save_list) }
+      it { expect(subject.get_list(false, output)).to be_an(Array) }
+      it { expect(subject.get_list(false, output).count).to be 2 }
+      it { expect(subject.get_list(false, output)[1].title).to eq 'title2' }
+    end
+
+  end
+
+  describe '.get_pad' do
+    context 'when no refresh is asked,' do
+      let(:pad1) { OpenStruct.new(id: '123', title: 'title1') }
+      let(:pad) { double Hackpad::Cli::Pad }
+      before { Hackpad::Cli::Pad.stub(:new).and_return pad }
+      before { pad.stub(:load) }
+      before { pad.stub(:title).and_return('title1') }
+      it { expect(subject.get_pad('123')).to eq pad1 }
+    end
+  end
+
+  describe '.check_list' do
+    before { Hackpad::Cli::Api.stub(:list).and_return(%w(123 456)) }
+    let(:pad) { double Hackpad::Cli::Pad }
+    before { Hackpad::Cli::Pad.stub(:new).and_return(pad, pad) }
+
+    context 'when there is no new pad,' do
+      before { pad.stub(:cached?).and_return(true, true) }
+      it { expect(subject.check_list).to eq [] }
+    end
+
+    context 'when there is no one new pad,' do
+      let(:pad2) { OpenStruct.new(id: '456', title: 'title2') }
+      before { pad.stub(:cached?).and_return(true, false) }
+      before { pad.stub(:load) }
+      before { pad.stub(:title).and_return('title2') }
+      it { expect(subject.check_list).to eq [pad2] }
+    end
+
+    context 'when there is no 2 new pads,' do
+      let(:pad1) { OpenStruct.new(id: '123', title: 'title1') }
+      let(:pad2) { OpenStruct.new(id: '456', title: 'title2') }
+      before { pad.stub(:cached?).and_return(false, false) }
+      before { pad.stub(:load).and_return(nil, nil) }
+      before { pad.stub(:title).and_return('title1', 'title2') }
+      it { expect(subject.check_list).to eq [pad1, pad2] }
+    end
+  end
+
+end

data/spec/lib/hackpad/cli/runner_spec.rb

@@ -1,8 +1,8 @@
 # encoding: utf-8
 
 require 'spec_helper'
-require "hackpad/cli/runner"
-require "hackpad/cli/client"
+require 'hackpad/cli/runner'
+require 'hackpad/cli/client'
 
 describe Hackpad::Cli::Runner do
 
@@ -12,39 +12,67 @@ describe Hackpad::Cli::Runner do
     Hackpad::Cli::Client.stub(:new, {}).and_return(Object)
   end
 
-  it "calls the search method in client class" do
+  it 'calls the stats method in client class' do
+    Object.stub(:stats)
+    cli.shell.mute do
+      cli.stats
+    end
+  end
+
+  it 'calls the search method in client class' do
     Object.stub(:search)
     cli.shell.mute do
-      cli.search "xxx"
+      cli.search 'xxx'
     end
   end
 
-  it "calls the list method in client class" do
+  it 'calls the list method in client class' do
     Object.stub(:list)
     cli.shell.mute do
       cli.list
     end
   end
 
-  it "calls the list method in client class" do
+  it 'calls the list method in client class' do
     Object.stub(:list)
     cli.shell.mute do
       cli.list
     end
   end
 
-  it "calls the info method in client class" do
+  it 'calls the check method in client class' do
+    Object.stub(:check)
+    cli.shell.mute do
+      cli.check
+    end
+  end
+
+  it 'calls the version method in client class' do
+    STDOUT.stub(:puts).with(Hackpad::Cli::VERSION)
+    cli.shell.mute do
+      cli.version
+    end
+  end
+
+  it 'calls the info method in client class' do
     Object.stub(:info)
     cli.shell.mute do
       cli.info 'pad'
     end
   end
 
-  it "calls the show method in client class" do
+  it 'calls the show method in client class' do
     Object.stub(:show)
     cli.shell.mute do
       cli.show 'pad', 'md'
     end
   end
 
+  it 'calls the colors method in client class' do
+    String.stub(:color_matrix).with(' xoxo ')
+    cli.shell.mute do
+      cli.colors
+    end
+  end
+
 end

data/spec/lib/hackpad/cli/store_spec.rb

@@ -1,7 +1,8 @@
 # encoding: utf-8
 
 require 'spec_helper'
-require "hackpad/cli/store"
+require 'hackpad/cli/store'
+require 'hackpad/cli/pad'
 
 describe Hackpad::Cli::Store do
 
@@ -12,36 +13,36 @@ describe Hackpad::Cli::Store do
     subject.prepare options
   end
 
-  describe ".read_list" do
+  describe '.read_list' do
     before { File.stub(:read).and_return("gy23ui first one\ngy3u4 second one\n23489g [some time] third") }
     let(:list) { subject.read_list }
     it { expect(list).to be_an Array }
     it { expect(list[0]).to be_an OpenStruct }
-    it { expect(list[0].id).to eq "gy23ui" }
-    it { expect(list[0].title).to eq "first one" }
-    it { expect(list[2].id).to eq "23489g" }
-    it { expect(list[2].title).to eq "third" }
-    it { expect(list[2].cached_at).to eq "some time" }
+    it { expect(list[0].id).to eq 'gy23ui' }
+    it { expect(list[0].title).to eq 'first one' }
+    it { expect(list[2].id).to eq '23489g' }
+    it { expect(list[2].title).to eq 'third' }
+    it { expect(list[2].cached_at).to eq 'some time' }
   end
 
-  describe ".exists?" do
+  describe '.exists?' do
 
-    context "when refresh option is set," do
+    context 'when refresh option is set,' do
       let(:options) { { configdir: configdir, workspace: 'default', refresh: true } }
-      before {
+      before do
         subject.prepare options
         FileUtils.touch File.join(configdir, 'default', 'pads', 'txt', 'xxx')
-      }
+      end
       after { FileUtils.rm File.join(configdir, 'default', 'pads', 'txt', 'xxx') }
       it { expect(subject.exists? 'txt', 'xxx').to be false }
     end
 
-    context "when refresh option is not set," do
+    context 'when refresh option is not set,' do
       context "when config file don't exist," do
         it { expect(subject.exists? 'txt', 'xxx').to be false }
       end
 
-      context "when configfile exists," do
+      context 'when configfile exists,' do
         before { FileUtils.touch File.join(configdir, 'default', 'pads', 'txt', 'xxx') }
         after { FileUtils.rm File.join(configdir, 'default', 'pads', 'txt', 'xxx') }
         it { expect(subject.exists? 'txt', 'xxx').to be true }
@@ -50,4 +51,84 @@ describe Hackpad::Cli::Store do
 
   end
 
+  describe '.save' do
+    let(:padfile) { File.join(configdir, 'default', 'pads', 'txt', 'xxx') }
+    let(:content) { "This is content\n" }
+    before { subject.prepare options }
+    let(:pad) { double Hackpad::Cli::Pad }
+    before { pad.stub(:id).and_return 'xxx' }
+    before { pad.stub(:content).and_return content }
+    after { FileUtils.rm padfile }
+    it do
+      subject.save pad, 'txt'
+      expect(File.read(padfile)).to eq content
+    end
+  end
+
+  describe '.save_options' do
+    let(:padfile) { File.join(configdir, 'default', 'pads', 'meta', 'xxx') }
+    let(:content) { { thing: '123', other: 'option' } }
+    before { subject.prepare options }
+    after { FileUtils.rm padfile }
+    it do
+      subject.save_options 'xxx', content
+      expect(File.read(padfile)).to eq "{\n  \"thing\": \"123\",\n  \"other\": \"option\"\n}\n"
+    end
+  end
+
+  describe '.save_list' do
+    let(:padfile) { File.join(configdir, 'default', 'pads', 'padlist') }
+    let(:pads) { [OpenStruct.new(id: '123', cached_at: 'some time', title: 'title1')] }
+    before { subject.prepare options }
+    after { FileUtils.rm padfile }
+    it do
+      subject.save_list pads
+      expect(File.read(padfile)).to eq "123 [some time] title1\n"
+    end
+  end
+
+  describe '.read' do
+    let(:padfile) { File.join(configdir, 'default', 'pads', 'txt', 'xxx') }
+    let(:content) { "This is content\n" }
+    before { subject.prepare options }
+    let(:pad) { double Hackpad::Cli::Pad }
+    before { pad.stub(:id).and_return 'xxx' }
+    before { pad.stub(:content).and_return content }
+    before { subject.save pad, 'txt' }
+    after { FileUtils.rm padfile }
+
+    it { expect(subject.read 'xxx', 'txt').to eq content }
+  end
+
+  describe '.read_option' do
+    let(:padfile) { File.join(configdir, 'default', 'pads', 'meta', 'xxx') }
+    let(:content) { { 'thing' => '123', 'other' => 'option' } }
+    before { subject.prepare options }
+    before { subject.save_options 'xxx', content }
+    after { FileUtils.rm padfile }
+    it { expect(subject.read_options 'xxx').to eq content }
+  end
+
+  describe '.count_pads' do
+    let(:padfile) { File.join(configdir, 'default', 'pads', 'meta', 'xxx') }
+    let(:content) { { 'thing' => '123', 'other' => 'option' } }
+    before { subject.prepare options }
+    before { subject.save_options 'xxx', content }
+    after { FileUtils.rm padfile }
+    it { expect(subject.count_pads).to be 1 }
+  end
+
+  describe '.last_refresh' do
+    let(:timestamp) { Time.new(2012, 10, 31) }
+    let(:padlist) { File.join(configdir, 'default', 'pads', 'padlist') }
+    let(:pads) { [OpenStruct.new(id: '123', cached_at: 'some time', title: 'title1')] }
+    before { subject.prepare options }
+    before do
+      subject.save_list pads
+      FileUtils.touch padlist, mtime: timestamp
+    end
+    after { FileUtils.rm padlist }
+    it { expect(subject.last_refresh).to eq timestamp }
+  end
+
 end