laser-cutter 0.3.2 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 281b7861f4452cfb6323f9d2a4a05aff6db5a742
-  data.tar.gz: dd4e65121d84f8a285dd3c603d1674a5505af9f4
+  metadata.gz: df5fc0b8025663e5d008e970a56fa3fe49f5a275
+  data.tar.gz: c9a4201c05edf34506255706906d979249047b92
 SHA512:
-  metadata.gz: d42068728244eb2d1125a9f3e759d5c00e1623a15f29e0eede7deb4eb3a5711f717434f81d2c99fd400d346e3e9b6832e47cb932830171b55cc9fa551f0d979a
-  data.tar.gz: 65ab7c551ef9f144dc8f91e9898d526e60ac42246c779cb9d480aea6d2f3e36e44514ccc3ff7ff802bdf30097d34d3127adc4a928ec9fa9bb78317302d0bb104
+  metadata.gz: 88b3fbacb5fbc6f0049f7799a7721f9b22ccff4ddfe313496a051b931ce5b9184efd273b68981234038f3d5967e1b073d0d8bbe6aa88a35a925bd4f2ab7b343f
+  data.tar.gz: 4c816f055a6f32109305ece7f3b7e2cb98d69d71b70d2e8cc6f7c566edd475c3cb6cb8a53de4a1598187f0b88ba170026acecd1a09329b0d8f0f4567e2581c08

data/.gitignore

@@ -24,3 +24,4 @@ tmp
 mkmf.log
 .idea/
 **output.pdf
+.ruby_version

data/.travis.yml

@@ -1,7 +1,7 @@
 language: ruby
 rvm:
   - 1.9.3
-script: "bundle exec rspec"
+script: "CODECLIMATE_REPO_TOKEN=6520104decf84e2f6c2c169d30d67976bfcdaa48cd89b4ec34f6e637f2a0ae02 bundle exec rspec"
 notifications:
   email:
     recipients:

data/Gemfile

@@ -2,3 +2,5 @@ source 'https://rubygems.org'
 
 # Specify your gem's dependencies in laser-cutter-cutter.gemspec
 gemspec
+
+gem "codeclimate-test-reporter", group: :test, require: nil

data/README.md

@@ -1,68 +1,28 @@
 [![Gem Version](https://badge.fury.io/rb/laser-cutter.svg)](http://badge.fury.io/rb/laser-cutter)
 [![Build status](https://secure.travis-ci.org/kigster/laser-cutter.png)](http://travis-ci.org/kigster/laser-cutter)
 [![Code Climate](https://codeclimate.com/github/kigster/laser-cutter.png)](https://codeclimate.com/github/kigster/laser-cutter)
+[![Test Coverage](https://codeclimate.com/github/kigster/laser-cutter/badges/coverage.svg)](https://codeclimate.com/github/kigster/laser-cutter)
 
 ## LaserCutter
 
 Similar to [BoxMaker](https://github.com/rahulbot/boxmaker/) (which is written in Java a long time ago), 
-this ruby gem generates PDFs that can be used as a basis for cutting boxes on a typical laser cutter. 
+this ruby gem generates PDFs that can be used as a basis for creating a "snap-in" boxes with notched
+sides on a typical laser cutter, by providing dimensions, material thickness and output file name. 
 
-Unlike ```BoxMaker```, this gem has a lot of automated tests around creating the geometry of the notches
-and calculating locations. This welcomes additional feature contributions from other developers,
-as existing test suite offers confidence around not introducing bugs or regressions.
+For more detailed comparison with BoxMaker and motivation behind this project, please see the section 
+at the bottom of this README.
 
-BoxMaker's algorithm _ensures that the same notch length is across all sides, but sacrifices
-symmetry as a result_.  So you may have a front panel's left and right edges be simply non symmetric. 
-And that might be entirely OK with you :)
- 
-```laser-cutter```'s algorithm will create a _symmetric design for most panels_, but it might sacrifice
-identical notch length. Depending on the box dimensions you may end up with a slightly different notch 
-length on each side of the box.
-
-Having said that, one of the design goals of this ruby version is to provide a highly extensible platform,
-where alternative algorithms can be implemented and supported by command line options.
-
-```laser-cutter``` has quite a few options that allow you to set stroke width, page size,
-layout, margins, padding (spacing between boxes), open the PDF file using system viewer right
-after generation, and many more are coming soon.
-
-The choice ultimately comes down to the preference and feature set, so here I show you two boxes made with
-each program, so you can pick what you prefer. 
-
-#### Disclaimer
-
-Important quick note that the author believes that BoxMaker is a greatly useful piece of software 
-generously open sourced by the author, and so in no way this project disputes it's viability.  
-In fact BoxMaker was an inspiration for this project. This project simply attempts to advance 
-further and provide additional solutions to this complex problem of creating 3D designs on a 2D
-surface using laser cutter.
-
-### Example Outputs
-
-Below are two examples of boxes with identical dimensions produced with ```laser-cutter``` and ```boxmaker```:
-
-#### BoxMaker 
-
-```bash
-> java -cp BOX.jar com.rahulbotics.boxmaker.BoxMaker -i -W 2.5 -H 2 -D 1 -T 0.25 -n 0.5 -f file.pdf
-```
-
-![BoxMaker Example](doc/boxmaker.jpg).
+One of the design goals of this project is to provide a highly extensible platform for creating 
+laser-cut designs, where alternative strategies can be added over time, and supported by various 
+command line options, and perhaps a light weight web application.  If you are interested in 
+contributing to the project, please see [contributing](CONTRIBUTING.md) for more details. 
 
-#### LaserCutter 
+```laser-cutter``` supports many flexible command line options that allow setting dimensions, 
+stroke width, page size, layout, margins, padding (spacing between the boxes), and many more.  
 
-```bash
-> laser-cutter -u in -s 2.5x1x2/0.25/0.5 -o file.pdf
-```
-
-![LaserCutter Example](doc/laser-cutter.jpg).
+## Dependencies
 
-## Future Features
-
-* Creating T-style joins, using various sized nuts and bolts (such as common #4-40 and M2 sizes)
-* Creating lids and front panels that are larger than the box 
-* Rendering to CSS to show live preview on a web app
-* Your brilliant idea can be here!  Please see [contributing](CONTRIBUTING.md) for more info.
+The gem depends primarily on [Prawn](http://prawnpdf.org) – a fantastic PDF generation library. 
 
 ## Installation
 
@@ -81,44 +41,110 @@ Or install it yourself as:
 ## Usage
 
 ```bash
-Usage: laser-cutter [options]
-       laser-cutter --list-all-page-sizes --units in
-
-Examples:
-  1. Create a box defined in inches, and open PDF in preview right after:
-
-       laser-cutter --units in -s 3x2x2/0.125/0.5  -O -o box.pdf
-
-  2. Create a box defined in millimeters, print verbose info and set
-     page size to A3, and layout to landscape, and stroke width to 2mm:
-
-       laser-cutter -w70 -h20 -d50 -t4.3 -n5 -PA3 -L landscape -S 0.5 -v -O -o box.pdf
-
-
-Specific options:
-    -s, --size WxHxD/T/N             Combined internal dimensions: W = width, H = height,
-                                     D = depth, T = thickness, N = notch length
+Usage: laser-cutter [options] -o filename.pdf
+   eg: laser-cutter -i -s 1x1.5x2/0.125/0.125 -O -o box.pdf
 
+Specific Options:
     -w, --width WIDTH                Internal width of the box
     -h, --height HEIGHT              Internal height of the box
     -d, --depth DEPTH                Internal depth of the box
     -t, --thickness THICKNESS        Thickness of the box material
     -n, --notch NOTCH                Preferred notch length (used only as a guide)
-    -o, --file FILE                  Output filename of the PDF
-    -u, --units UNITS                Either 'mm' (default) or 'in'
+
     -m, --margin MARGIN              Margins from the edge of the document
     -p, --padding PADDING            Space between the boxes on the page
+    -S, --stroke WIDTH               Numeric stroke width of the line
     -P, --page_size LETTER           Page size, see --list-all-page-sizes for more info.
     -L, --page_layout portrait       Page layout, other option is 'landscape'
-    -S, --stroke WIDTH               Numeric stroke width of the line
+
     -O, --open                       Open generated file with system viewer before exiting
+    -u, --units UNITS                Either 'mm' (default) or 'in'
+
     -A, --list-all-page-sizes        Print all available page sizes with dimensions and exit
     -v, --[no-]verbose               Run verbosely
 
-Common options:
+        --examples                   Show detailed usage examples
         --help                       Show this message
         --version                    Show version
-```       
+
+Common Options:
+    -o, --file FILE                  Required output filename of the PDF
+    -s, --size WxHxD/T/N             Combined internal dimensions: W = width, H = height,
+                                     D = depth, T = thickness, N = notch length
+
+    -i, --inches                     Switch measurements to inches instead of millimeters
+
+Examples:
+  1. Create a box defined in inches, and open PDF in preview right after:
+
+       laser-cutter -i -s 3x2x2/0.125/0.5 -O -o box.pdf
+
+  2. Create a box defined in millimeters, print verbose info and set
+     page size to A3, and layout to landscape, and stroke width to 1/2mm:
+
+       laser-cutter -w70 -h20 -d50 -t4.3 -n5 -PA3 -L landscape -S0.5 -v -O -o box.pdf
+
+  3. List all possible page sizes in metric or imperial systems:
+
+       laser-cutter --list-all-page-sizes
+       laser-cutter --list-all-page-sizes --inches
+```                 
+
+## Future Features
+
+* Extensibility with various layout strategies, notch drawing strategies, basically plug and play
+  model for adding new algorithms for path creation and box joining
+* Support more shapes than just box
+* Create T-style joins, using various standard sizes of nuts and bolts (such as common #4-40 and M2 sizes)
+* Supporting lids and front panels, that are larger than the box itself and have holes for notches. 
+* A web-app that uses the gem and renders box live in CSS using CSS 2D to show preview.
+* Your brilliant idea can be here too!  Please see [contributing](CONTRIBUTING.md) for more info.
+
+## Comparison with BoxMaker
+
+It's important to note that the author believes that BoxMaker is a greatly useful piece of software 
+generously open sourced by the author, and so in no way this project disputes BoxMaker's viability. 
+  
+In fact BoxMaker was an inspiration for this project. Laser-Cutter library attempts to further advance 
+concept of programmatically creating laser-cut boxes, provide additional tuning, options, strategies
+and most importantly – extensibility.  
+
+Unlike ```BoxMaker```, this gem has a suit of automated tests (rspecs) around creating the geometry. 
+In addition, we welcome new feature contributions, or bug fixes from other developers, and in that 
+regard rspecs offer confidence that functionality still works.
+
+BoxMaker's algorithm _tries to ensures that the same notch length is across all sides, but sacrifices
+symmetry as a result_.  So you may have a front panel's left and right edges be simply non symmetric. 
+And that might be entirely OK with you :)
+ 
+```laser-cutter```'s algorithm will create a _symmetric design for most panels_, but it might sacrifice
+identical notch length. Depending on the box dimensions you may end up with a slightly different notch 
+length on each side of the box.
+
+The choice ultimately comes down to the preference and feature set, so here I show you two boxes made with
+each program, so you can pick what you prefer. 
+
+### Example Outputs
+
+Below are two examples of boxes with identical dimensions produced with ```laser-cutter``` and ```boxmaker```:
+
+This is how you would make a box with Adam Phelp's fork of BoxMaker (which adds flags and a lot of 
+niceties): 
+
+```bash
+git clone https://github.com/aphelps/boxmaker && cd boxmaker && ant
+java -cp BOX.jar com.rahulbotics.boxmaker.BoxMaker \
+      -i -W 1 -H 2 -D 1.5 -T 0.125 -n 0.125 -f box.pdf
+```
+
+And laser-cutter:
+
+```bash
+gem install laser-cutter
+laser-cutter -i -s 1x1.5x2/0.125/0.125 -O -o box.pdf
+```
+
+![LaserCutter Comparison](doc/comparison.jpg).
 
 ## Contributing
 

data/bin/laser-cutter

@@ -8,122 +8,107 @@ require 'hashie/mash'
 module Laser
   module Cutter
     class OptParse
+      def self.puts_error(e)
+        STDERR.puts "Whoops, #{e}".red
+        STDERR.puts "Try --help or --examples for more info...".yellow
+      end
+
       def self.parse(args)
         banner_text = <<-EOF
-Usage: laser-cutter [options]
-       laser-cutter --list-all-page-sizes --units in
+#{('Laser-Cutter v'+ Laser::Cutter::VERSION).bold}
+
+Usage: laser-cutter [options] -o filename.pdf
+   eg: laser-cutter -i -s 1x1.5x2/0.125/0.125 -O -o box.pdf
+EOF
+
+        examples = <<-EOF
 
 Examples:
   1. Create a box defined in inches, and open PDF in preview right after:
 
-       laser-cutter --units in -s 3x2x2/0.125/0.5  -O -o box.pdf
+       laser-cutter -i -s 3x2x2/0.125/0.5 -O -o box.pdf
 
   2. Create a box defined in millimeters, print verbose info and set
-     page size to A3, and layout to landscape, and stroke width to 2mm:
+     page size to A3, and layout to landscape, and stroke width to 1/2mm:
 
-       laser-cutter -w70 -h20 -d50 -t4.3 -n5 -PA3 -L landscape -S 0.5 -v -O -o box.pdf
-EOF
+       laser-cutter -w70 -h20 -d50 -t4.3 -n5 -PA3 -L landscape -S0.5 -v -O -o box.pdf
+
+  3. List all possible page sizes in metric or imperial systems:
 
+       laser-cutter --list-all-page-sizes
+       laser-cutter --list-all-page-sizes --inches
+EOF
         options = Hashie::Mash.new
         options.verbose = false
         options.units = 'mm'
 
         opt_parser = OptionParser.new do |opts|
-          opts.banner = banner_text.cyan
-          opts.separator ""
-          opts.separator "Specific options:"
-
-          opts.on("-s", "--size WxHxD/T/N",
-              "Combined internal dimensions: W = width, H = height,\n#{" " * 37}D = depth, T = thickness, N = notch length\n\n") do |size|
-            options.size = size
-          end
-
+          opts.banner = banner_text.blue
+          opts.separator "Specific Options:"
           opts.on("-w", "--width WIDTH", "Internal width of the box") { |value|  options.width = value }
           opts.on("-h", "--height HEIGHT", "Internal height of the box") { |value|  options.height = value }
           opts.on("-d", "--depth DEPTH", "Internal depth of the box") { |value|  options.depth= value }
-
           opts.on("-t", "--thickness THICKNESS", "Thickness of the box material") { |value|  options.thickness = value }
           opts.on("-n", "--notch NOTCH", "Preferred notch length (used only as a guide)") { |value|  options.notch = value }
-          opts.on("-o", "--file FILE", "Output filename of the PDF") { |value|  options.file = value }
-
-          opts.on("-u", "--units UNITS", "Either 'mm' (default) or 'in'") { |value|  options.units = value }
+          opts.separator ""
           opts.on("-m", "--margin MARGIN", "Margins from the edge of the document") { |value|  options.margin = value }
           opts.on("-p", "--padding PADDING", "Space between the boxes on the page") { |value|  options.padding = value }
-
+          opts.on("-S", "--stroke WIDTH", "Numeric stroke width of the line") { |value|  options.stroke = value }
           opts.on("-P", "--page_size LETTER", "Page size, see --list-all-page-sizes for more info.") { |value|  options.page_size = value }
           opts.on("-L", "--page_layout portrait", "Page layout, other option is 'landscape' ") { |value|  options.page_layout = value }
-          opts.on("-S", "--stroke WIDTH", "Numeric stroke width of the line") { |value|  options.stroke = value }
-
+          opts.separator ""
           opts.on("-O", "--open", "Open generated file with system viewer before exiting") { |v| options.open = v }
-
+          opts.on("-u", "--units UNITS", "Either 'mm' (default) or 'in'") { |value|  options.units = value }
+          opts.separator ""
           opts.on("-A", "--list-all-page-sizes", "Print all available page sizes with dimensions and exit") { |v| options.list_all_page_sizes = true }
-
           opts.on("-v", "--[no-]verbose", "Run verbosely") { |v| options.verbose = v }
-
           opts.separator ""
-          opts.separator "Common options:"
-
-          opts.on_tail("--help", "Show this message") do
-            puts opts
-            exit
-          end
-          opts.on_tail("--version", "Show version") do
-            puts Laser::Cutter::VERSION
-            exit
+          opts.on("--examples", "Show detailed usage examples") {  puts opts; puts examples.yellow; exit  }
+          opts.on("--help", "Show this message") { puts opts; exit }
+          opts.on("--version", "Show version") {  puts Laser::Cutter::VERSION; exit }
+          opts.separator ""
+          opts.separator "Common Options:"
+          opts.on_tail("-o", "--file FILE", "Required output filename of the PDF") { |value|  options.file = value }
+          opts.on_tail("-s", "--size WxHxD/T/N",
+                       "Combined internal dimensions: W = width, H = height,\n#{" " * 37}D = depth, T = thickness, N = notch length\n\n") do |size|
+            options.size = size
           end
+          opts.on_tail("-i", "--inches", "Switch measurements to inches instead of millimeters") { |value|  options.units = 'in' }
         end
+
         opt_parser.parse!(args)
-        options
-      rescue OptionParser::InvalidOption, OptionParser::MissingArgument => e
-        puts opt_parser.help
-        puts
-        puts "#{e.message}".red
+
+        config = Laser::Cutter::Configuration.new(options.to_hash)
+        if config.list_all_page_sizes
+          puts config.all_page_sizes
+          exit 0
+        end
+
+        if options.verbose
+          puts "Starting with the following configuration:"
+          puts JSON.pretty_generate(config.to_hash).green
+        end
+
+        config.validate!
+        config
+      rescue OptionParser::InvalidOption, OptionParser::MissingArgument, Laser::Cutter::MissingOption => e
+        puts opt_parser.banner.blue
+        puts_error(e)
         exit 1
       end # end parse()
     end
   end
 end # class OptParse
 
-options = Laser::Cutter::OptParse.parse(ARGV)
-
-if options.list_all_page_sizes
-  puts
-  unit = 1.0 / 72.0 # PDF units per inch
-  multiplier = options.units == 'in' ? 1.0 : 25.4
-  hash = PDF::Core::PageGeometry::SIZES
-  hash.keys.sort.each do |k|
-    printf("\t%10s:\t%6.1f x %6.1f\n",
-           k,
-           multiplier * hash[k][0].to_f * unit,
-           multiplier * hash[k][1].to_f * unit )
-  end
-  exit 0
-end
-
-config = Laser::Cutter::Configuration.new(options.to_hash)
-
-if options.verbose
-  puts "Starting with the following configuration:"
-  puts JSON.pretty_generate(config.to_hash).green
-end
-
-begin
-  config.validate!
-rescue Exception => e
-  STDERR.puts "\nSorry, #{e}".red
-  STDERR.puts "Try --help for more info..."
-  exit 1
-end
+config = Laser::Cutter::OptParse.parse(ARGV)
 
 begin
-  box = Laser::Cutter::Box.new(config)
-  Laser::Cutter::Renderer::BoxRenderer.new(box, config).render
+  Laser::Cutter::Renderer::BoxRenderer.new(config).render
   if config.open
     `open #{config.file}`
   end
 rescue Exception => e
-  puts "#{e}".red
-  STDERR.puts "Try --help for more info...".yellow
+  puts_error(e)
   if options.verbose
     STDERR.puts "Exception: #{e.inspect}".red
     STDERR.puts e.backtrace.join("\n").red

data/lib/laser-cutter/box.rb

@@ -1,5 +1,7 @@
 module Laser
   module Cutter
+    # Note: this class badly needs refactoring and tests.  Both are coming.
+
     class Box
       # Everything is in millimeters
 
@@ -7,7 +9,7 @@ module Laser
       attr_accessor :padding, :units
 
       attr_accessor :front, :back, :top, :bottom, :left, :right
-      attr_accessor :faces, :bounds, :conf
+      attr_accessor :faces, :bounds, :conf, :corner_face
 
       def initialize(config = {})
         self.dim = Geometry::Dimensions.new(config['width'], config['height'], config['depth'])
@@ -18,32 +20,35 @@ module Laser
         self.units = config['units']
 
         create_faces! # generates dimensions for each side
-        self.faces = [top, front, bottom, back, left, right]
+        self.faces =     [top, front, bottom, back, left, right]
 
         position_faces!
+
         self.conf   = {
-            valign: [ :out, :out,  :out,  :out, :in, :in],
-            halign: [ :in,  :out,  :in,   :out, :in, :in],
-            corners:[ :no,  :yes,  :no,   :yes, :no, :no]
+            valign: [    :out, :out,  :out,   :out, :in, :in],
+            halign: [    :in,  :out,  :in,    :out, :in, :in],
+            corners: {
+                front: [ :no,  :yes,  :no,    :yes, :no, :no], # our default choice, but may not work
+                top:   [ :yes, :no,   :yes,   :no,  :no, :no]  # 2nd choice, has to work if 1st doesn't
+            },
         }
-
         self
       end
 
+
       # Returns an flattened array of lines representing notched
       # rectangle.
       def notches
-        generate_bounding_boxes!
+        corner_face = pick_corners_face
 
         notches = []
-
         faces.each_with_index do |face, face_index|
-          bound = bounds[face_index]
+          bound = face_bounding_rect(face)
           bound.sides.each_with_index do |bounding_side, side_index |
             key = side_index.odd? ? :valign : :halign
             path = Geometry::PathGenerator.new(:notch_width => notch_width,
                                                :center_out => (self.conf[key][face_index] == :out) ,
-                                               :fill_corners => (self.conf[:corners][face_index] == :yes && side_index.odd?),
+                                               :fill_corners => (self.conf[:corners][corner_face][face_index] == :yes && side_index.odd?),
                                                :thickness => thickness
             ).path(Geometry::Edge.new(bounding_side, face.sides[side_index], self.notch_width))
             notches << path.create_lines
@@ -67,15 +72,11 @@ module Laser
 
       private
 
-      def generate_bounding_boxes!
-        self.bounds = []
-        self.bounds = faces.map do |face|
-          b = face.clone
-          b.move_to(b.position.move_by(-thickness, -thickness))
-          b.p2 = b.p2.move_by(2 * thickness, 2 * thickness)
-          b.relocate!
-          b
-        end
+      def face_bounding_rect(face)
+        b = face.clone
+        b.move_to(b.position.move_by(-thickness, -thickness))
+        b.p2 = b.p2.move_by(2 * thickness, 2 * thickness)
+        b.relocate!
       end
 
       #___________________________________________________________________
@@ -131,6 +132,15 @@ module Laser
         self.right = Geometry::Rect.create(zero, dim.d, dim.h, "right")
       end
 
+      def pick_corners_face
+        b = face_bounding_rect(front)
+        edges = []
+        front.sides[0..1].each_with_index do |face, index|
+          edges << Geometry::Edge.new(b.sides[index], face, notch_width)
+        end
+        edges.map(&:notch_count).all?{|c| c % 4 == 3} ? :top : :front
+      end
+
     end
   end
 end

data/lib/laser-cutter/configuration.rb

@@ -1,9 +1,11 @@
 require 'hashie/mash'
 require 'prawn/measurement_extensions'
-
+require 'pdf/core/page_geometry'
 
 module Laser
   module Cutter
+    class MissingOption < RuntimeError
+    end
     class Configuration < Hashie::Mash
       DEFAULTS = {
           units: 'mm',
@@ -49,7 +51,23 @@ module Laser
         REQUIRED.each do |k|
           missing << k if self[k].nil?
         end
-        raise "#{missing.join(', ')} #{missing.size > 1 ? 'are' : 'is'} required, but missing." unless missing.empty?
+        unless missing.empty?
+          raise MissingOption.new("#{missing.join(', ')} #{missing.size > 1 ? 'are' : 'is'} required, but missing.")
+        end
+      end
+
+      def all_page_sizes
+        unit = 1.0 / 72.0 # PDF units per inch
+        multiplier = (self.units == 'in') ? 1.0 : 25.4
+        h = PDF::Core::PageGeometry::SIZES
+        output = "\n"
+        h.keys.sort.each do |k|
+          output << sprintf("\t%10s:\t%6.1f x %6.1f\n",
+                 k,
+                 multiplier * h[k][0].to_f * unit,
+                 multiplier * h[k][1].to_f * unit )
+        end
+        output
       end
     end
   end

data/lib/laser-cutter/renderer/box_renderer.rb

@@ -5,12 +5,15 @@ module Laser
     module Renderer
       class BoxRenderer < AbstractRenderer
         alias_method :box, :subject
+        def initialize(options = {})
+          self.options = options
+          self.subject = Laser::Cutter::Box.new(options)
+        end
 
         def render pdf = nil
           pdf = Prawn::Document.new(:margin => options.margin.send(options.units),
                                     :page_size => options.page_size,
                                     :page_layout => options.page_layout.to_sym)
-
           header = <<-EOF
 
           Produced with Laser Cutter Ruby Gem (v#{Laser::Cutter::VERSION})

data/lib/laser-cutter/version.rb

@@ -1,5 +1,5 @@
 module Laser
   module Cutter
-    VERSION = "0.3.2"
+    VERSION = "0.4.0"
   end
 end

data/spec/configuration_spec.rb

@@ -3,10 +3,10 @@ require_relative 'spec_helper'
 module Laser
   module Cutter
     describe Configuration do
-      let(:config) { Laser::Cutter::Configuration.new(opts)}
+      let(:config) { Laser::Cutter::Configuration.new(opts) }
 
       context 'option parsing' do
-        let(:opts) { { "size" => "2x3x2/0.125/0.5", "inches" => true} }
+        let(:opts) { {"size" => "2x3x2/0.125/0.5", "inches" => true} }
         it 'should be able to parse size options' do
           expect(config.width).to eql(2.0)
           expect(config.height).to eql(3.0)
@@ -15,14 +15,30 @@ module Laser
           expect(config.notch).to eql(0.5)
         end
       end
-      context 'validate' do
-        let(:opts)  {{ "height" => "23" }}
+      context '#validate' do
+        let(:opts) { {"height" => "23"} }
         it 'should be able to validate missing options' do
           expect(config.height).to eql(23.0)
-          expect { config.validate! } .to raise_error(RuntimeError)
+          expect { config.validate! }.to raise_error(Laser::Cutter::MissingOption)
         end
       end
+      context '#list_page_sizes' do
+        context 'formatting of output' do
+          context 'when using inches' do
+            let(:opts) { {"units" => "in"} }
+            it 'should return the list in inches' do
+              expect(config.all_page_sizes).to match %r(.*B10\:\s+1\.2\s+x\s+1\.7)
+            end
+          end
+          context 'when using mm' do
+            let(:opts) { {"units" => "mm"} }
+            it 'should return the list in mm' do
+              expect(config.all_page_sizes).to match %r(.*B10\:\s+31\.0\s+x\s+44\.0)
+            end
+          end
+        end
 
+      end
     end
   end
 end

data/spec/renderer_spec.rb

@@ -6,7 +6,7 @@ module Laser
       describe 'BoxRenderer' do
         context '#render' do
           let(:box) { Laser::Cutter::Box.new(config) }
-          let(:renderer) { BoxRenderer.new(box, config) }
+          let(:renderer) { BoxRenderer.new(config) }
           let(:file) { File.expand_path("../../laser-cutter-pdf-test.#{$$}.pdf", __FILE__) }
 
           def render_file filename

data/spec/spec_helper.rb

@@ -5,12 +5,18 @@
 #
 # See http://rubydoc.info/gems/rspec-core/RSpec/Core/Configuration
 
+if ENV['CODECLIMATE_REPO_TOKEN']
+  require 'codeclimate-test-reporter'
+  CodeClimate::TestReporter.start
+end
+
 ENV['BUNDLE_GEMFILE'] ||= File.expand_path('../Gemfile', __FILE__)
 require 'rubygems'
 require 'bundler/setup' if File.exists?(ENV['BUNDLE_GEMFILE'])
 require 'laser-cutter'
+require "codeclimate-test-reporter"
+
 
-#Dir['spec/support/**/*.rb'].each { |filename| require_relative "../#{filename}" }
 
 RSpec.configure do |config|
   config.treat_symbols_as_metadata_keys_with_true_values = true

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: laser-cutter
 version: !ruby/object:Gem::Version
-  version: 0.3.2
+  version: 0.4.0
 platform: ruby
 authors:
 - Konstantin Gredeskoul
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-09-22 00:00:00.000000000 Z
+date: 2014-09-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: prawn
@@ -115,8 +115,7 @@ files:
 - README.md
 - Rakefile
 - bin/laser-cutter
-- doc/boxmaker.jpg
-- doc/laser-cutter.jpg
+- doc/comparison.jpg
 - laser-cutter.gemspec
 - lib/laser-cutter.rb
 - lib/laser-cutter/box.rb
@@ -180,4 +179,3 @@ test_files:
 - spec/rect_spec.rb
 - spec/renderer_spec.rb
 - spec/spec_helper.rb
-has_rdoc: