hackpad-cli 0.0.7 → 0.1.0

data/lib/hackpad/cli/pad.rb

@@ -1,5 +1,5 @@
-require_relative "store"
-require_relative "api"
+require_relative 'store'
+require_relative 'api'
 
 module Hackpad
   module Cli
@@ -28,40 +28,39 @@ module Hackpad
         @content.lines.count if @content
       end
 
-      def load(ext, refresh=false)
-        raise UnknownFormat unless FORMATS.include? ext
-        raise UndefinedPad unless @id
-        if refresh or !Store.exists? ext, @id
-          load_from_api ext
+      def load(ext, refresh = false, save = true)
+        fail UnknownFormat unless FORMATS.include? ext
+        fail UndefinedPad unless @id
+        if refresh || !Store.exists?(ext, @id)
+          load_from_api ext, save
         else
           load_from_cache ext
         end
       end
 
-      def load_from_api(ext)
+      def load_from_api(ext, dosave = true)
         @content = Api.read @id, ext
-        Store.save self, ext
+        dosave && Store.save(self, ext)
         options = Api.read_options @id
         @guest_policy = options['options']['guestPolicy']
-        @moderated = !!options['options']['isModerated']
+        @moderated = options['options']['isModerated']
         options['cached_at'] = Time.now
         @cached_at = options['cached_at']
-        Store.save_meta @id, options
+        dosave && Store.save_options(@id, options)
       end
 
       def load_from_cache(ext)
         @content = Store.read @id, ext
         options = Store.read_options @id
         @guest_policy = options['options']['guestPolicy']
-        @moderated = !!options['options']['isModerated']
+        @moderated = options['options']['isModerated']
         @cached_at = options['cached_at']
       end
 
-      def is_cached?
+      def cached?
         Store.exists? 'meta', @id
       end
 
     end
   end
 end
-

data/lib/hackpad/cli/padlist.rb

@@ -1,25 +1,23 @@
 require 'ostruct'
-require_relative "store"
-require_relative "api"
-require_relative "pad"
+require_relative 'store'
+require_relative 'api'
+require_relative 'pad'
 
 module Hackpad
   module Cli
     module Padlist
-      extend self
+      module_function
 
-      def get_list(refresh=false)
+      def get_list(refresh = false, output = STDOUT)
         all = []
-        if refresh or !Store.exists? "padlist"
-          print "Refreshing "
+        if refresh || !Store.exists?('padlist')
+          output.print 'Refreshing '
           list = Api.list
           list.each do |a|
-            print "."
-            pad = Pad.new a
-            pad.load 'txt', refresh
-            all << OpenStruct.new( id: a, title: pad.title )
+            output.print '.'
+            all << get_pad(a, refresh)
           end
-          puts " all done."
+          output.puts ' all done.'
           Store.save_list all
         else
           all = Store.read_list
@@ -27,13 +25,20 @@ module Hackpad
         all
       end
 
+      def get_pad(id, refresh = false)
+        pad = Pad.new id
+        pad.load 'txt', refresh
+        OpenStruct.new(id: id, title: pad.title)
+      end
+
       def check_list
         all = []
         list = Api.list
         list.each do |a|
           pad = Pad.new a
-          if !pad.is_cached?
-            all << OpenStruct.new( id: a, title: pad.title )
+          unless pad.cached?
+            pad.load 'txt', false, false
+            all << OpenStruct.new(id: a, title: pad.title)
           end
         end
         all
@@ -42,4 +47,3 @@ module Hackpad
     end
   end
 end
-

data/lib/hackpad/cli/plain_colors.rb

@@ -1,7 +1,7 @@
 require 'colorize'
 
 class String
-  def colorize(params)
+  def colorize(_)
     self
   end
 end

data/lib/hackpad/cli/runner.rb

@@ -1,74 +1,89 @@
-require "thor"
-require "yaml"
-require_relative "client"
-require_relative "version"
+require 'thor'
+require 'yaml'
+require_relative 'client'
+require_relative 'version'
 
 module Hackpad
   module Cli
     class Runner < Thor
 
+      refresh_option = [
+        :refresh, {
+          aliases: '-r',
+          type:    'boolean',
+          default: false,
+          desc:    'Add this if you want refresh the cache.'
+        }
+      ]
+
+      url_option = [
+        :urls, {
+          aliases: '-u',
+          type:    'boolean',
+          default: false,
+          desc:    'Displays urls rather than pad ids.'
+        }
+      ]
+
       class_option :configdir,
-        aliases: "-c",
-        default: File.join(ENV["HOME"], ".hackpad-cli/"),
-        desc: "Path to the hackpad-cli directory to use."
+        aliases: '-c',
+        default: File.join(ENV['HOME'], '.hackpad-cli/'),
+        desc:    'Path to the hackpad-cli directory to use.'
 
       class_option :workspace,
-        aliases: "-w",
-        default: "default",
-        desc: "Name of the workspace to use."
-
-      class_option :refresh,
-        aliases: "-r",
-        type: 'boolean',
-        default: false,
-        desc: "Add this if you want refresh the cache."
-
-      class_option :urls,
-        aliases: "-u",
-        type: 'boolean',
-        default: false,
-        desc: "Displays urls rather than pad ids."
+        aliases: '-w',
+        default: 'default',
+        desc:    'Name of the workspace to use.'
 
       class_option :plain,
-        aliases: "-p",
-        type: 'boolean',
+        aliases: '-p',
+        type:    'boolean',
         default: false,
-        desc: "Add this if you don't want colors."
+        desc:    "Add this if you don't want colors."
 
       default_task :help
 
-      desc "search [term]", "Lists available pads matching [term]."
+      desc 'stats', 'Lists configuration values.'
+      def stats
+        Hackpad::Cli::Client.new(options).stats
+      end
+
+      desc 'search [term]', 'Lists available pads matching [term] (options: -u to show urls)'
+      method_option(*url_option)
       def search(term)
         Hackpad::Cli::Client.new(options).search term
       end
 
-      desc "list", "Lists available pads."
+      desc 'list', 'Lists available pads (options: -u to show urls, -r to refresh).'
+      method_option(*refresh_option)
+      method_option(*url_option)
       def list
         Hackpad::Cli::Client.new(options).list
       end
 
-      desc "check", "Checks differences between local cache and remote list."
+      desc 'check', 'Checks differences between local cache and remote list (options: -u to show urls).'
+      method_option(*url_option)
       def check
         Hackpad::Cli::Client.new(options).check
       end
 
-      desc "info [pad_id]", "gets info for the pad <pad_id>."
+      desc 'info [pad_id]', 'gets info for the pad <pad_id>.'
       def info(pad)
         Hackpad::Cli::Client.new(options).info pad
       end
 
-      desc "show [pad_id] [format]", "shows pad <pad_id> in format [html,txt,md] (default txt)."
-      def show(pad,format='txt')
+      desc 'show [pad_id] [format]', 'shows pad <pad_id> in format [html,txt,md], default txt (options: -r to refresh).'
+      method_option(*refresh_option)
+      def show(pad, format = 'txt')
         Hackpad::Cli::Client.new(options).show pad, format
       end
 
-      desc "version", "Displays the hackpad-cli version."
+      desc 'version', 'Displays the hackpad-cli version.'
       def version
         puts Hackpad::Cli::VERSION
       end
 
-
-      desc "colors", "displays colorize color matrix.", hide: true
+      desc 'colors', 'displays colorize color matrix.', hide: true
       def colors
         require 'colorize'
         String.color_matrix ' xoxo '

data/lib/hackpad/cli/store.rb

@@ -5,13 +5,13 @@ require_relative '../cli'
 module Hackpad
   module Cli
     module Store
-      extend self
+      module_function
 
       def prepare(config)
         @refresh = config[:refresh]
         dir = File.join(config[:configdir], config[:workspace])
         @pads_dir = File.join(dir, 'pads')
-        @list_cache = File.join(dir, 'pads.list')
+        @list_cache = File.join(@pads_dir, 'padlist')
         prepare_dirs @pads_dir
       end
 
@@ -20,7 +20,7 @@ module Hackpad
       end
 
       def exists?(*path)
-        !@refresh && File.exists?(File.join(@pads_dir, *path))
+        !@refresh && File.exist?(File.join(@pads_dir, *path))
       end
 
       def save(pad, ext)
@@ -29,16 +29,16 @@ module Hackpad
         end
       end
 
-      def save_meta(id, options)
+      def save_options(id, options)
         File.open(File.join(@pads_dir, 'meta', id), 'w') do |f|
           f.puts JSON.pretty_generate(options)
         end
       end
 
       def save_list(pads)
-        File.open(File.join(@pads_dir, 'padlist'), 'w') do |f|
+        File.open(@list_cache, 'w') do |f|
           pads.each do |p|
-            f.puts "#{p.id} #{p.title}"
+            f.puts "#{p.id} [#{p.cached_at}] #{p.title}"
           end
         end
       end
@@ -54,11 +54,19 @@ module Hackpad
       end
 
       def read_list
-        File.read(File.join(@pads_dir, 'padlist')).lines.reduce([]) { |a,line|
+        File.read(@list_cache).lines.reduce([]) do |a, line|
           /(?<id>[a-zA-Z0-9]*) (\[(?<cached_at>[-a-zA-Z0-9: ]*)\] )?(?<title>.*)/ =~ line
-          a << OpenStruct.new( id: id, title: title, cached_at: cached_at )
+          a << OpenStruct.new(id: id, title: title, cached_at: cached_at)
           a
-        }
+        end
+      end
+
+      def count_pads
+        Dir.glob(File.join(@pads_dir, 'meta', '*')).count
+      end
+
+      def last_refresh
+        File.mtime(@list_cache) if File.exist?(@list_cache)
       end
 
     end

data/lib/hackpad/cli/version.rb

@@ -1,5 +1,5 @@
 module Hackpad
   module Cli
-    VERSION = "0.0.7"
+    VERSION = '0.1.0'
   end
 end

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

@@ -0,0 +1,89 @@
+# encoding: utf-8
+
+require 'spec_helper'
+require 'webmock/rspec'
+require 'hackpad/cli/api'
+
+WebMock.disable_net_connect!(allow: 'codeclimate.com')
+
+describe Hackpad::Cli::Api do
+
+  describe '.search' do
+    let(:config) { { 'site' => 'http://x.hackpad.com', 'client_id' => '123', 'secret' => 'aaa' } }
+    before { Hackpad::Cli::Api.prepare config }
+    context 'when just a simple term is provided,' do
+      it 'returns expected json' do
+        stub_request(:get, 'http://x.hackpad.com/api/1.0/search?limit=100&q=term&start=0')
+          .to_return(body: '[{"title":"API 1.0 Docs","id":"C0E68BD495E9","snippet":"example"}]', status: 200)
+        expect(subject.search('term')).to eq([{
+          'title' => 'API 1.0 Docs',
+          'id' => 'C0E68BD495E9',
+          'snippet' => 'example'
+        }])
+      end
+    end
+  end
+
+  describe '.list' do
+    let(:config) { { 'site' => 'http://x.hackpad.com', 'client_id' => '123', 'secret' => 'aaa' } }
+    before { Hackpad::Cli::Api.prepare config }
+    it 'returns expected json' do
+      stub_request(:get, 'http://x.hackpad.com/api/1.0/pads/all')
+        .to_return(body: '["aaa","bbb"]', status: 200)
+      expect(subject.list).to eq(%w(aaa bbb))
+    end
+  end
+
+  describe '.read_options' do
+    let(:config) { { 'site' => 'http://x.hackpad.com', 'client_id' => '123', 'secret' => 'aaa' } }
+    before { Hackpad::Cli::Api.prepare config }
+    it 'returns expected json' do
+      stub_request(:get, 'http://x.hackpad.com/api/1.0/pad/aaa/options')
+        .to_return(body: '{ "options": "xxx"}', status: 200)
+      expect(subject.read_options('aaa')).to eq('options' => 'xxx')
+    end
+  end
+
+  describe '.read' do
+    let(:config) { { 'site' => 'http://x.hackpad.com', 'client_id' => '123', 'secret' => 'aaa' } }
+    before { Hackpad::Cli::Api.prepare config }
+    it 'returns expected json' do
+      stub_request(:get, 'http://x.hackpad.com/api/1.0/pad/aaa/content.html')
+        .to_return(body: '<b>blah</b>', status: 200)
+      expect(subject.read('aaa', 'html')).to eq('<b>blah</b>')
+    end
+  end
+
+  describe '.get' do
+    let(:config) { { 'site' => 'http://x.hackpad.com', 'client_id' => '123', 'secret' => 'aaa' } }
+    before { Hackpad::Cli::Api.prepare config }
+    context 'when proper crendential are provided,' do
+      it 'all goes well' do
+        stub_request(:get, 'http://x.hackpad.com/api/1.0/xxx')
+          .with(headers: {
+            'Accept' => '*/*',
+            'Authorization' => /OAuth oauth_consumer_key="123"/,
+            'User-Agent' => /hackpad-cli v#{Hackpad::Cli::VERSION}/
+          })
+          .to_return(status: 200, body: '{"some": "result"}')
+        expect(Hackpad::Cli::Api.get('/api/1.0/xxx')).to eq('some' => 'result')
+      end
+    end
+    context 'when api endpoint is not found' do
+      it 'throws an exception' do
+        stub_request(:get, 'http://x.hackpad.com/api/1.0/xxx')
+          .with(headers: {
+            'Accept' => '*/*',
+            'Authorization' => /OAuth oauth_consumer_key="123"/,
+            'User-Agent' => /hackpad-cli v#{Hackpad::Cli::VERSION}/
+          })
+          .to_return(status: 404, body: '{"some": "result"}')
+        expect { Hackpad::Cli::Api.get('/api/1.0/xxx') }
+          .to raise_error(Hackpad::Cli::ApiException, 'HTTP error, code 404')
+      end
+
+    end
+
+  end
+
+end

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

@@ -1,122 +1,219 @@
 # encoding: utf-8
 
 require 'spec_helper'
-require "hackpad/cli/client"
+require 'hackpad/cli/client'
 
 describe Hackpad::Cli::Client do
   let(:configdir) { File.expand_path('../../../../files', __FILE__) }
   let(:options) { { configdir: configdir, workspace: 'default' } }
+  let(:format) { "%-20s %s\n" }
 
-  describe ".new" do
+  describe '.new' do
     before { Hackpad::Cli::Api.stub(:prepare) }
     before { Hackpad::Cli::Store.stub(:prepare) }
     before { Hackpad::Cli::Config.stub(:load) }
 
-    context "when default options are passed," do
+    context 'when default options are passed,' do
       let(:client) { Hackpad::Cli::Client.new options }
       it { expect(client).to be_a Hackpad::Cli::Client }
-      context "when colorization is expected," do
-        it { expect("x".blue).to eq "\e[0;34;49mx\e[0m" }
+      context 'when colorization is expected,' do
+        it { expect('x'.blue).to eq "\e[0;34;49mx\e[0m" }
       end
     end
 
-    context "when plain text is required," do
-      context "when colorization is not expected," do
-        before { Hackpad::Cli::Client.new options.merge({plain: true}) }
-        after { load "colorize.rb" }
-        it { expect("x".blue).to eq "x" }
+    context 'when plain text is required,' do
+      context 'when colorization is not expected,' do
+        before { Hackpad::Cli::Client.new options.merge(plain: true) }
+        after { load 'colorize.rb' }
+        it { expect('x'.blue).to eq 'x' }
       end
     end
   end
 
-  describe ".search" do
+  describe '.stats' do
+    let(:timestamp) { Time.new(2013, 10, 2) }
+    before { Hackpad::Cli::Store.stub(:prepare) }
+    before { Hackpad::Cli::Config.stub(:load).and_return('site' => 'http://test.dev') }
+    before { Hackpad::Cli::Store.stub(:count_pads).and_return(12) }
+    before { Hackpad::Cli::Store.stub(:last_refresh).and_return(timestamp) }
+    let(:client) { Hackpad::Cli::Client.new options }
+    it do
+      expect(STDOUT).to receive(:printf).with(format, 'Site', "\e[0;34;49mhttp://test.dev\e[0m")
+      expect(STDOUT).to receive(:printf).with(format, 'Cached Pads', 12)
+      expect(STDOUT).to receive(:printf).with(format, 'Last Refresh', timestamp)
+      client.stats
+    end
+  end
+
+  describe '.search' do
     before { Hackpad::Cli::Api.stub(:prepare) }
     before { Hackpad::Cli::Store.stub(:prepare) }
-    before { Hackpad::Cli::Config.stub(:load).and_return({'site' => 'http://test.dev'}) }
-    before {
-      Hackpad::Cli::Api.stub(:search).with("xxx",0).and_return(
-        [ {
-          "title" => "xtitle",
-          "id" => "xxxxxx",
-          "snippet" => "context <b class=\"hit\">x</b> context"
-        } ]
-      )
-    }
-    context "when default options are used," do
+    before { Hackpad::Cli::Config.stub(:load).and_return('site' => 'http://test.dev') }
+    before do
+      Hackpad::Cli::Api.stub(:search)
+        .with('xxx', 0)
+        .and_return(
+          [{
+            'title' => 'xtitle',
+            'id' => 'xxxxxx',
+            'snippet' => 'context <b class="hit">x</b> context'
+          }]
+        )
+    end
+    context 'when default options are used,' do
       let(:client) { Hackpad::Cli::Client.new options }
-      it {
+      it do
         expect(STDOUT).to receive(:puts).with("\e[1;39;49mxxxxxx\e[0m - \e[0;33;49mxtitle\e[0m")
         expect(STDOUT).to receive(:puts).with("   context \e[1;36;49mx\e[0m context")
-        client.search "xxx"
-      }
+        client.search 'xxx'
+      end
     end
-    context "when options sets urls to true," do
-      let(:client) { Hackpad::Cli::Client.new options.merge({urls: true}) }
-      it {
+    context 'when options sets urls to true,' do
+      let(:client) { Hackpad::Cli::Client.new options.merge(urls: true) }
+      it do
         expect(STDOUT).to receive(:puts).with("http://test.dev/\e[1;39;49mxxxxxx\e[0m - \e[0;33;49mxtitle\e[0m")
         expect(STDOUT).to receive(:puts).with("   context \e[1;36;49mx\e[0m context")
-        client.search "xxx"
-      }
+        client.search 'xxx'
+      end
     end
   end
 
-  describe ".list" do
+  describe '.list' do
     before { Hackpad::Cli::Api.stub(:prepare) }
     before { Hackpad::Cli::Store.stub(:prepare) }
-    before { Hackpad::Cli::Config.stub(:load).and_return({'site' => 'http://test.dev'}) }
-    before { Hackpad::Cli::Padlist.stub(:get_list).and_return( [ OpenStruct.new( id: 'xxxxxx', title: 'xtitle' ) ] ) }
-    context "when default options are used," do
+    before { Hackpad::Cli::Config.stub(:load).and_return('site' => 'http://test.dev') }
+    before do
+      Hackpad::Cli::Padlist.stub(:get_list)
+        .and_return([OpenStruct.new(id: 'xxxxxx', title: 'xtitle')])
+    end
+    context 'when default options are used,' do
       let(:client) { Hackpad::Cli::Client.new options }
-      it {
-        expect(STDOUT).to receive(:puts).with(["xxxxxx - xtitle"])
+      it do
+        expect(STDOUT).to receive(:puts).with(['xxxxxx - xtitle'])
         client.list
-      }
+      end
     end
-    context "when options sets urls to true," do
-      let(:client) { Hackpad::Cli::Client.new options.merge({urls: true}) }
-      it {
-        expect(STDOUT).to receive(:puts).with(["http://test.dev/xxxxxx - xtitle"])
+    context 'when options sets urls to true,' do
+      let(:client) { Hackpad::Cli::Client.new options.merge(urls: true) }
+      it do
+        expect(STDOUT).to receive(:puts).with(['http://test.dev/xxxxxx - xtitle'])
         client.list
-      }
+      end
     end
   end
 
-  describe ".check" do
+  describe '.check' do
     before { Hackpad::Cli::Api.stub(:prepare) }
     before { Hackpad::Cli::Store.stub(:prepare) }
-    before { Hackpad::Cli::Config.stub(:load).and_return({'site' => 'http://test.dev'}) }
-    before { Hackpad::Cli::Padlist.stub(:check_list).and_return( [ OpenStruct.new( id: 'xxxxxx', title: 'xtitle' ) ] ) }
-    context "when default options are used," do
-      let(:client) { Hackpad::Cli::Client.new options }
-      it {
-        expect(STDOUT).to receive(:puts).with("New pads:")
-        expect(STDOUT).to receive(:puts).with(["xxxxxx - xtitle"])
-        client.check
-      }
+    before { Hackpad::Cli::Config.stub(:load).and_return('site' => 'http://test.dev') }
+
+    context 'when there is a new pad,' do
+      before do
+        Hackpad::Cli::Padlist.stub(:check_list)
+          .and_return([OpenStruct.new(id: 'xxxxxx', title: 'xtitle')])
+      end
+      context 'when default options are used,' do
+        let(:client) { Hackpad::Cli::Client.new options }
+        it do
+          expect(STDOUT).to receive(:puts).with('New pads:')
+          expect(STDOUT).to receive(:puts).with(['xxxxxx - xtitle'])
+          client.check
+        end
+      end
+      context 'when options sets urls to true,' do
+        let(:client) { Hackpad::Cli::Client.new options.merge(urls: true) }
+        it do
+          expect(STDOUT).to receive(:puts).with('New pads:')
+          expect(STDOUT).to receive(:puts).with(['http://test.dev/xxxxxx - xtitle'])
+          client.check
+        end
+      end
     end
-    context "when options sets urls to true," do
-      let(:client) { Hackpad::Cli::Client.new options.merge({urls: true}) }
-      it {
-        expect(STDOUT).to receive(:puts).with("New pads:")
-        expect(STDOUT).to receive(:puts).with(["http://test.dev/xxxxxx - xtitle"])
+    context 'when there is no new pad,' do
+      before { Hackpad::Cli::Padlist.stub(:check_list).and_return([]) }
+      let(:client) { Hackpad::Cli::Client.new options }
+      it do
+        expect(STDOUT).to receive(:puts).with('New pads:')
+        expect(STDOUT).to receive(:puts).with('There is no new pad.')
         client.check
-      }
+      end
     end
   end
 
-  describe ".info" do
+  describe '.info' do
     before { Hackpad::Cli::Api.stub(:prepare) }
     before { Hackpad::Cli::Store.stub(:prepare) }
-    before { Hackpad::Cli::Config.stub(:load).and_return({'site' => 'http://test.dev'}) }
-    context "when unknown id is given" do
-      pending "todo"
+    before { Hackpad::Cli::Config.stub(:load).and_return('site' => 'http://test.dev') }
+    let(:client) { Hackpad::Cli::Client.new options }
+    let(:pad) { double Hackpad::Cli::Pad }
+    before { Hackpad::Cli::Pad.stub(:new).with('123').and_return pad }
+
+    context 'when unknown id is given,' do
+      before { pad.stub(:load).and_raise Hackpad::Cli::UndefinedPad }
+      it { expect { client.info('123') }.to raise_error(Hackpad::Cli::UndefinedPad) }
     end
-    context "when id is an existing pad" do
 
+    context 'when id is an existing pad,' do
+      before { pad.stub(:load) }
+      before { pad.stub(:title).and_return('title1') }
+      before { pad.stub(:chars).and_return(20) }
+      before { pad.stub(:lines).and_return(2) }
+      before { pad.stub(:guest_policy).and_return('open') }
+      before { pad.stub(:moderated).and_return('false') }
+      before { pad.stub(:cached_at).and_return }
+      it do
+        expect(STDOUT).to receive(:printf).with(format, 'Id', "\e[1;39;49m123\e[0m")
+        expect(STDOUT).to receive(:printf).with(format, 'Title', "\e[0;33;49mtitle1\e[0m")
+        expect(STDOUT).to receive(:printf).with(format, 'URI', 'http://test.dev/123')
+        expect(STDOUT).to receive(:printf).with(format, 'Chars', '20')
+        expect(STDOUT).to receive(:printf).with(format, 'Lines', '2')
+        expect(STDOUT).to receive(:printf).with(format, 'Guest Policy', 'open')
+        expect(STDOUT).to receive(:printf).with(format, 'Moderated', 'false')
+        expect(STDOUT).to receive(:printf).with(format, 'Cached', 'unknown')
+        client.info '123'
+      end
     end
+
   end
 
-  pending "Hackpad::Cli::Client.info"
-  pending "Hackpad::Cli::Client.show"
+  describe '.show' do
+    before { Hackpad::Cli::Api.stub(:prepare) }
+    before { Hackpad::Cli::Store.stub(:prepare) }
+    before { Hackpad::Cli::Config.stub(:load).and_return('site' => 'http://test.dev') }
+    let(:client) { Hackpad::Cli::Client.new options }
+    let(:pad) { double Hackpad::Cli::Pad }
+    before { Hackpad::Cli::Pad.stub(:new).with('123').and_return pad }
+    before { pad.stub(:load) }
+
+    context 'when a txt version is asked,' do
+      before { pad.stub(:content).and_return('this is content') }
+      it do
+        expect(STDOUT).to receive(:puts).with('this is content')
+        client.show '123', 'txt'
+      end
+    end
+
+    context 'when a html version is asked,' do
+      before { pad.stub(:content).and_return('<ul><li>this is content</li></ul>') }
+      it do
+        expect(STDOUT).to receive(:puts).with('<ul><li>this is content</li></ul>')
+        client.show '123', 'html'
+      end
+    end
+
+    context 'when a markdown version is asked,' do
+      before { pad.stub(:content).and_return('<ul><li>this is content</li></ul>') }
+      before do
+        ReverseMarkdown.stub(:convert)
+          .with('<ul><li>this is content</li></ul>', github_flavored: true)
+          .and_return('- this is content')
+      end
+      it do
+        expect(STDOUT).to receive(:puts).with('- this is content')
+        client.show '123', 'md'
+      end
+    end
+
+  end
 
 end