klipbook 1.0.2 → 2.0.0

data/spec/lib/klipbook/{book_spec.rb → sources/book_spec.rb}

@@ -7,7 +7,7 @@ describe Klipbook::Book do
 
     context 'with no author' do
       let(:book) do
-        Klipbook::Book.new do |b|
+        Klipbook::Book.new.tap do |b|
           b.title = 'Book title'
         end
       end
@@ -19,7 +19,7 @@ describe Klipbook::Book do
 
     context 'with an author' do
       let(:book) do
-        Klipbook::Book.new do |b|
+        Klipbook::Book.new.tap do |b|
           b.title = 'Book title'
           b.author = 'Rob Ripjaw'
         end

data/spec/lib/klipbook/sources/kindle_device/file_spec.rb

@@ -2,18 +2,38 @@ require 'spec_helper'
 
 describe Klipbook::Sources::KindleDevice::File do
 
-  let(:file) { Klipbook::Sources::KindleDevice::File.new('file text', max_books, file_parser) }
+  let(:file) { Klipbook::Sources::KindleDevice::File.new('file path', max_books, file_parser) }
 
-  let (:max_books) { 30 }
+  let(:max_books) { 30 }
 
-  let (:entries) { [] }
+  let(:entries) { [] }
 
-  let (:file_parser) do
+  let(:file_parser) do
     Object.new.tap do |fake_parser|
       stub(fake_parser).extract_entries { entries }
     end
   end
 
+  let(:input_file) do
+    Object.new.tap do |fake_file|
+      stub(fake_file).read { 'file text' }
+    end
+  end
+
+  before :each do
+    stub(File).open.returns do |*args|
+      passed_block = args.last
+      passed_block.call(input_file)
+    end
+  end
+
+  describe 'initially' do
+    it 'reads the contents of the input file' do
+      file
+      File.should have_received.open('file path', 'r')
+    end
+  end
+
   describe '#books' do
     subject { file.books }
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: klipbook
 version: !ruby/object:Gem::Version
-  version: 1.0.2
+  version: 2.0.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-12-12 00:00:00.000000000 Z
+date: 2013-02-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: gli
@@ -140,7 +140,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: pry
+  name: pry-debugger
   requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
@@ -276,34 +276,39 @@ files:
 - features/collate.feature
 - features/fixtures/clippings-for-three-books.txt
 - features/list.feature
+- features/pretty_print.feature
 - features/step_definitions/collate_steps.rb
 - features/step_definitions/list_steps.rb
+- features/step_definitions/pretty_print_steps.rb
 - features/support/env.rb
 - klipbook.gemspec
 - lib/klipbook.rb
-- lib/klipbook/blank.rb
-- lib/klipbook/book.rb
-- lib/klipbook/clipping.rb
-- lib/klipbook/collator.rb
+- lib/klipbook/collate/book_file.rb
+- lib/klipbook/commands/collate.rb
+- lib/klipbook/commands/list_books.rb
+- lib/klipbook/commands/pretty_print.rb
 - lib/klipbook/config.rb
-- lib/klipbook/fetcher.rb
-- lib/klipbook/invalid_source_error.rb
-- lib/klipbook/output/book_helpers.rb
-- lib/klipbook/output/html_book_summary.erb
-- lib/klipbook/output/html_summary_writer.rb
-- lib/klipbook/printer.rb
+- lib/klipbook/pretty_print/html_book_summary.erb
+- lib/klipbook/pretty_print/html_printer.rb
 - lib/klipbook/sources/amazon_site/book_scraper.rb
 - lib/klipbook/sources/amazon_site/site_scraper.rb
+- lib/klipbook/sources/book.rb
+- lib/klipbook/sources/book_source.rb
+- lib/klipbook/sources/clipping.rb
+- lib/klipbook/sources/invalid_source_error.rb
 - lib/klipbook/sources/kindle_device/entry.rb
 - lib/klipbook/sources/kindle_device/entry_parser.rb
 - lib/klipbook/sources/kindle_device/file.rb
 - lib/klipbook/sources/kindle_device/file_parser.rb
+- lib/klipbook/util/blank.rb
+- lib/klipbook/util/struct_to_json.rb
 - lib/klipbook/version.rb
-- spec/lib/klipbook/book_spec.rb
-- spec/lib/klipbook/collator_spec.rb
-- spec/lib/klipbook/fetcher_spec.rb
-- spec/lib/klipbook/output/html_summary_writer_spec.rb
-- spec/lib/klipbook/printer_spec.rb
+- spec/lib/klipbook/collate/book_file_spec.rb
+- spec/lib/klipbook/commands/list_books_spec.rb
+- spec/lib/klipbook/commands/pretty_print_spec.rb
+- spec/lib/klipbook/pretty_print/html_printer_spec.rb
+- spec/lib/klipbook/sources/book_source_spec.rb
+- spec/lib/klipbook/sources/book_spec.rb
 - spec/lib/klipbook/sources/kindle_device/entry_parser_spec.rb
 - spec/lib/klipbook/sources/kindle_device/file_parser_spec.rb
 - spec/lib/klipbook/sources/kindle_device/file_spec.rb
@@ -325,7 +330,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -4595890160696464397
+      hash: -33745898634206631
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:

data/lib/klipbook/book.rb

@@ -1,18 +0,0 @@
-module Klipbook
-  class Book
-    attr_accessor :asin, :author, :title, :last_update, :clippings
-
-    def initialize
-      yield self if block_given?
-    end
-
-    def title_and_author
-      author_txt = author ? " by #{author}" : ''
-      "#{title}#{author_txt}"
-    end
-
-    def get_binding
-      binding
-    end
-  end
-end

data/lib/klipbook/clipping.rb

@@ -1,9 +0,0 @@
-module Klipbook
-  class Clipping
-    attr_accessor :annotation_id, :text, :location, :type, :page
-
-    def initialize
-      yield self if block_given?
-    end
-  end
-end

data/lib/klipbook/collator.rb

@@ -1,17 +0,0 @@
-module Klipbook
-  class Collator
-    def initialize(books, summary_writer=Klipbook::Output::HtmlSummaryWriter.new)
-      @books = books
-      @summary_writer = summary_writer
-    end
-
-    def collate_books(output_dir, force, message_stream=$stdout)
-      message_stream.puts "Using output directory: #{output_dir}"
-
-      @books.each do |book|
-        @summary_writer.write(book, output_dir, force)
-      end
-    end
-  end
-end
-

data/lib/klipbook/output/book_helpers.rb

@@ -1,12 +0,0 @@
-module Klipbook::Output
-  module BookHelpers
-    def location_html(location)
-      if self.asin
-        "<a href=\"kindle://book?action=open&asin=#{asin}&location=#{location}\">loc #{location}</a>"
-      else
-        "loc #{location}"
-      end
-    end
-  end
-end
-

data/lib/klipbook/printer.rb

@@ -1,18 +0,0 @@
-module Klipbook
-  class Printer
-    def initialize(books)
-      @books = books
-    end
-
-    def print(output=$stdout)
-      if @books.empty?
-        output.puts 'No books available'
-      else
-        output.puts 'Book list:'
-        @books.each_with_index do |book, index|
-          output.puts "[#{index + 1}] #{book.title_and_author}"
-        end
-      end
-    end
-  end
-end

data/spec/lib/klipbook/collator_spec.rb

@@ -1,40 +0,0 @@
-require 'spec_helper'
-
-describe Klipbook::Collator do
-
-  let (:it) { Klipbook::Collator.new(books, summary_writer) }
-
-  let (:summary_writer) do
-    Object.new.tap do |fake_writer|
-      stub(fake_writer).write
-    end
-  end
-
-  let (:message_stream) do
-    Object.new.tap do |fake_stream|
-      stub(fake_stream).puts
-    end
-  end
-
-  let (:book_one) { Klipbook::Book.new }
-  let (:book_two) { Klipbook::Book.new }
-  let (:books) { [ book_one, book_two ] }
-
-  let(:output_dir) { 'fake output dir' }
-
-  describe '#collate_books' do
-
-    subject { it.collate_books(output_dir, true, message_stream) }
-
-    it 'prints a message displaying the output directory' do
-      subject
-      message_stream.should have_received.puts('Using output directory: fake output dir')
-    end
-
-    it 'passes each book to the summary writer' do
-      subject
-      summary_writer.should have_received.write(book_one, output_dir, true)
-      summary_writer.should have_received.write(book_two, output_dir, true)
-    end
-  end
-end