dropio 0.9.1 → 1.0

data/spec/dropio/drop_spec.rb

@@ -1,24 +1,150 @@
 require File.dirname(__FILE__) + '/../spec_helper.rb'
 
 describe Drop do
+  
+  before(:each) do
+    @client = Dropio::Client.new
+    @api = stub(Dropio::Api)
+    @client.service = @api
+    
+    Dropio::Resource.stub!(:client).and_return(@client)
+    Dropio::Resource.client.should == @client
+    Dropio::Resource.client.service.should == @api
+    
+    @mydrop = Dropio::Drop.new(:name => "test_drop",:admin_token => "admin_token")
+  end
+  
   it "should have the attributes of a Drop" do
-    Drop.new.should respond_to(:name, :email, :voicemail, :conference, :fax,
-                               :rss, :guest_token, :admin_token,
-                               :expiration_length, :guests_can_comment, :guests_can_comment, 
-                               :guests_can_delete, :max_bytes,
-                               :current_bytes, :hidden_upload_url,
-                               :upload_url, :password, :admin_password, :premium_code)
+    Drop.new.should respond_to(:name, :email, :voicemail, :conference, :fax, :rss, :guest_token, :description,
+                  :admin_token, :expires_at, :expiration_length, :guests_can_comment, :guests_can_add, :guests_can_delete,
+                  :max_bytes, :current_bytes, :hidden_upload_url, :asset_count, :chat_password, :default_view, 
+                  :password, :admin_password, :premium_code, :admin_email, :email_key)
   end
   
   it "should find drops by name" do
-    mydrop = stub(Drop)
-    Client.instance.should_receive(:find_drop).with("mydrop", nil).and_return(mydrop)
-    Drop.find("mydrop").should == mydrop
+    @client.should_receive(:handle).with(:drop,{}).and_return(@mydrop)
+    @api.should_receive(:drop).with("mydrop", nil).and_return({})
+    Drop.find("mydrop").should == @mydrop
   end
 
   it "should find drops by name and token" do
-    mydrop = stub(Drop)
-    Client.instance.should_receive(:find_drop).with("mydrop", "d85a6").and_return(mydrop)
-    Drop.find("mydrop", "d85a6").should == mydrop
+    @client.should_receive(:handle).with(:drop,{}).and_return(@mydrop)
+    @api.should_receive(:drop).with("mydrop", "d85a6").and_return({})
+    Drop.find("mydrop", "d85a6").should == @mydrop
+  end
+  
+  it "should have a default token and it should default to the admin" do
+    @mydrop.admin_token = "tester"
+    @mydrop.default_token.should == "tester"
+  end
+  
+  it "should find a set of related assets" do
+    @asset = stub(Asset)
+    @asset.should_receive(:drop=).once
+    @client.should_receive(:handle).with(:assets,{}).and_return([@asset])
+    @api.stub!(:assets).with(@mydrop.name,1,:oldest,@mydrop.default_token).and_return({})
+    @mydrop.assets.should == [@asset]
+  end
+  
+  it "should be able to create a new Drop" do
+    @client.should_receive(:handle).with(:drop,{}).and_return(@mydrop)
+    @api.should_receive(:create_drop).with({:name => "tester"}).and_return({})
+    Drop.create({:name => "tester"}).should == @mydrop
+  end
+  
+  it "should fetch the upload embed code" do
+    @client.should_receive(:handle).with(:response,{}).and_return({"result" => "Success"})
+    @api.should_receive(:drop_upload_code).with(@mydrop.name,@mydrop.default_token).and_return({})
+    @mydrop.upload_code
+  end
+  
+  it "should be able to empty itself" do
+    @client.should_receive(:handle).with(:response,{}).and_return({})
+    @api.should_receive(:empty_drop).with(@mydrop.name,@mydrop.admin_token).and_return({})
+    @mydrop.empty
+  end
+  
+  it "should be able to promote a nick" do
+    @client.should_receive(:handle).with(:response,{}).and_return({"result" => "Success"})
+    @api.should_receive(:promote_nick).with(@mydrop.name,"jake",@mydrop.admin_token).and_return({})
+    @mydrop.promote("jake")
+  end
+  
+  it "should save itself" do
+    @client.should_receive(:handle).with(:drop,{}).and_return(@mydrop)
+    expected_hash = {:password=>"test_password", :expiration_length=>nil, :admin_password=>nil, :guests_can_comment=>nil, 
+                     :premium_code=>nil, :guests_can_add=>nil, :chat_password=>nil, :guests_can_delete=>nil, 
+                     :admin_email=>nil, :default_view=>nil, :email_key=>nil, :description=>nil}
+    @api.should_receive(:update_drop).with(@mydrop.name,@mydrop.admin_token,expected_hash).and_return({})
+    @mydrop.password = "test_password"
+    @mydrop.save
+  end
+  
+  it "should destroy itself" do
+    @client.should_receive(:handle).with(:response,{}).and_return({"result" => "Success"})
+    @api.should_receive(:delete_drop).with(@mydrop.name,@mydrop.admin_token).and_return({})
+    @mydrop.destroy!
+  end
+  
+  it "should add files from a url" do
+    @asset = stub(Asset)
+    @asset.should_receive(:drop=).once
+    @client.should_receive(:handle).with(:asset,{}).and_return(@asset)
+    @api.should_receive(:add_file_from_url).with(@mydrop.name,"http://myurl.com/myfile.txt",@mydrop.default_token).and_return({})
+    @mydrop.add_file_from_url("http://myurl.com/myfile.txt").should == @asset
+  end
+  
+  it "should add files from a path" do
+    @asset = stub(Asset)
+    @asset.should_receive(:drop=).once
+    @client.should_receive(:handle).with(:asset,{}).and_return(@asset)
+    @api.should_receive(:add_file).with(@mydrop.name,"/mypath/myfile.txt",@mydrop.default_token, nil).and_return({})
+    @mydrop.add_file("/mypath/myfile.txt").should == @asset
+  end
+  
+  it "should create notes from title and contents" do
+    @asset = stub(Asset)
+    @asset.should_receive(:drop=).once
+    @client.should_receive(:handle).with(:asset,{}).and_return(@asset)
+    @api.should_receive(:create_note).with(@mydrop.name,"contents", "title",@mydrop.default_token).and_return({})
+    @mydrop.create_note("contents","title").should == @asset
+  end
+  
+  it "should create links from a url, title, and description" do
+    @asset = stub(Asset)
+    @asset.should_receive(:drop=).once
+    @client.should_receive(:handle).with(:asset,{}).and_return(@asset)
+    @api.should_receive(:create_link).with(@mydrop.name,"http://drop.io","drop.io","The best!",@mydrop.default_token).and_return({})
+    @mydrop.create_link("http://drop.io","drop.io","The best!").should == @asset
   end
+  
+  it "should be able to create a twitter subscription" do
+    @sub = stub(Subscription)
+    @sub.should_receive(:drop=).once
+    @client.should_receive(:handle).with(:subscription,{}).and_return(@sub)
+    @api.should_receive(:create_twitter_subscription).with(@mydrop.name,"mytwitter","pass",nil,{},@mydrop.default_token).and_return({})
+    @mydrop.create_twitter_subscription("mytwitter","pass")
+  end
+
+  it "should be able to create email subscriptions" do
+    @sub = stub(Subscription)
+    @sub.should_receive(:drop=).once
+    @client.should_receive(:handle).with(:subscription,{}).and_return(@sub)
+    @api.should_receive(:create_email_subscription).with(@mydrop.name,"jake@dropio.com","My welcome message",nil,nil,nil,{},@mydrop.default_token).and_return({})
+    @mydrop.create_email_subscription("jake@dropio.com","My welcome message")
+  end
+  
+  it "should be able to get a list of subscriptions back" do
+    @sub = stub(Subscription)
+    @sub.should_receive(:drop=).once
+    @client.should_receive(:handle).with(:subscriptions,{}).and_return([@sub])
+    @api.stub!(:subscriptions).with(@mydrop.name, @mydrop.admin_token).and_return({})
+    @mydrop.subscriptions.should == [@sub]
+  end
+  
+  it "should generate a signed url" do
+    @api.should_receive(:generate_drop_url).with(@mydrop.name,@mydrop.default_token)    
+    @mydrop.generate_url
+  end
+  
 end

data/spec/dropio_spec.rb

@@ -2,7 +2,7 @@ require File.dirname(__FILE__) + '/spec_helper.rb'
 
 describe Dropio do
   it "should store the API key" do
-    Dropio.api_key = "83a05513ddddb73e75c9d8146c115f7fd8e90de6"
-    Dropio.api_key.should == "83a05513ddddb73e75c9d8146c115f7fd8e90de6"
+    Dropio::Config.api_key = "83a05513ddddb73e75c9d8146c115f7fd8e90de6"
+    Dropio::Config.api_key.should == "83a05513ddddb73e75c9d8146c115f7fd8e90de6"
   end
 end

data/spec/spec_helper.rb

@@ -1,5 +1,8 @@
 require 'rubygems'
 require 'spec'
+require 'fakeweb'
+
+FakeWeb.allow_net_connect = false
 
 lib_dir = File.dirname(__FILE__) + '/../lib'
 $:.unshift lib_dir unless $:.include?(lib_dir)

metadata

@@ -1,15 +1,15 @@
 --- !ruby/object:Gem::Specification 
 name: dropio
 version: !ruby/object:Gem::Version 
-  version: 0.9.1
+  version: "1.0"
 platform: ruby
 authors: 
-- Jake Good, Peter Jaros
+- Jake Good
 autorequire: 
 bindir: bin
 cert_chain: []
 
-date: 2008-11-11 00:00:00 -05:00
+date: 2009-09-09 00:00:00 -05:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -23,7 +23,17 @@ dependencies:
         version: "0"
     version: 
 - !ruby/object:Gem::Dependency 
-  name: json
+  name: httparty
+  type: :runtime
+  version_requirement: 
+  version_requirements: !ruby/object:Gem::Requirement 
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        version: "0"
+    version: 
+- !ruby/object:Gem::Dependency 
+  name: multipart-post
   type: :runtime
   version_requirement: 
   version_requirements: !ruby/object:Gem::Requirement 
@@ -42,10 +52,9 @@ dependencies:
       - !ruby/object:Gem::Version 
         version: "0"
     version: 
-description: A Ruby client library for the Drop.io API (http://api.drop.io)
+description: A Ruby client library for the Drop.io API (http://api.drop.io). Please email jake@dropio.com with any questions.
 email: 
 - jake@dropio.com
-- peeja@dropio.com
 executables: []
 
 extensions: []
@@ -56,30 +65,33 @@ extra_rdoc_files:
 - Todo.rdoc
 files: 
 - History.rdoc
+- lib/dropio/api.rb
 - lib/dropio/asset.rb
-- lib/dropio/client/mapper.rb
-- lib/dropio/client/multipart_post.rb
 - lib/dropio/client.rb
 - lib/dropio/comment.rb
 - lib/dropio/drop.rb
 - lib/dropio/resource.rb
+- lib/dropio/subscription.rb
 - lib/dropio.rb
 - LICENSE.txt
 - Manifest
 - Rakefile
 - Readme.rdoc
+- spec/dropio/api_spec.rb
 - spec/dropio/asset_spec.rb
-- spec/dropio/client/mapper_spec.rb
 - spec/dropio/client_spec.rb
 - spec/dropio/comment_spec.rb
 - spec/dropio/drop_spec.rb
+- spec/dropio/subscription_spec.rb
 - spec/dropio_spec.rb
 - spec/spec.opts
 - spec/spec_helper.rb
 - Todo.rdoc
 - dropio.gemspec
 has_rdoc: true
-homepage: http://github.com/whoisjake/dropio_api_ruby
+homepage: http://github.com/dropio/dropio
+licenses: []
+
 post_install_message: 
 rdoc_options: 
 - --line-numbers
@@ -105,7 +117,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
 requirements: []
 
 rubyforge_project: dropio
-rubygems_version: 1.3.0
+rubygems_version: 1.3.5
 signing_key: 
 specification_version: 2
 summary: A Ruby client library for the Drop.io API (http://api.drop.io)

data/lib/dropio/client/mapper.rb

@@ -1,47 +0,0 @@
-class Dropio::Client::Mapper
-  
-  def self.map_drops(response_body)
-    return parse_and_map(Dropio::Drop, response_body)
-  end
-  
-  def self.map_assets(drop, response_body)
-    assets = parse_and_map(Dropio::Asset, response_body)
-    
-    assets.drop = drop if assets.is_a?(Dropio::Asset)
-    assets.each{ |a| a.drop = drop } if assets.is_a?(Array)
-    
-    return assets
-  end
-  
-  def self.map_comments(asset, response_body)
-    comments = parse_and_map(Dropio::Comment, response_body)
-    
-    comments.asset = asset if comments.is_a?(Dropio::Comment)
-    comments.each{ |c| c.asset = asset } if comments.is_a?(Array)
-    
-    return comments
-  end
-  
-  private
-  
-  def self.parse_and_map(model_class, response_body)
-    h = JSON.parse(response_body)
-    
-    # single model
-    return model_class.new(h) if h.is_a?(Hash)
-    
-    # multiple models
-    models = []
-    
-    h.each do |model_hash|
-      if model_hash.is_a?(Hash)
-        model = model_class.new(model_hash) 
-        models << model
-      end
-    end if h.is_a?(Array)
-    
-    return models
-    
-  end
-  
-end

data/lib/dropio/client/multipart_post.rb

@@ -1,35 +0,0 @@
-module Dropio::Client::MultipartPost
-  def multipart_params=(param_hash={})
-    boundary_token = [Array.new(8) {rand(256)}].join
-    self.content_type = "multipart/form-data; boundary=#{boundary_token}"
-    boundary_marker = "--#{boundary_token}\r\n"
-    self.body = param_hash.map { |param_name, param_value|
-      unless param_value.nil?
-        boundary_marker + case param_value
-        when String
-          text_to_multipart(param_name, param_value.to_s)
-        when File
-          file_to_multipart(param_name, param_value)
-        end
-      end
-    }.join('') + "--#{boundary_token}--\r\n"
-  end
-
-  protected
-  def file_to_multipart(key,file)
-    filename = File.basename(file.path)
-    mime_types = MIME::Types.of(filename)
-    mime_type = mime_types.empty? ? "application/octet-stream" : mime_types.first.content_type
-    part = "Content-Disposition: form-data; name=\"#{key}\"; filename=\"#{filename}\"\r\n"
-    part += "Content-Transfer-Encoding: binary\r\n"
-    part += "Content-Type: #{mime_type}\r\n\r\n#{file.read}\r\n"
-  end
-
-  def text_to_multipart(key,value)
-    "Content-Disposition: form-data; name=\"#{key}\"\r\n\r\n#{value}\r\n"
-  end
-end
-
-class Net::HTTP::Post
-  include Dropio::Client::MultipartPost
-end

data/spec/dropio/client/mapper_spec.rb

@@ -1,169 +0,0 @@
-require File.dirname(__FILE__) + '/../../spec_helper'
-
-describe Client::Mapper do
-  it "should parse Drops" do
-    test_drop = Client::Mapper.map_drops <<-RESPONSE
-      {"rss":"http:\/\/drop.io\/k4gpvnty6tw9ho8u3j4w\/b9b2c8f2b8e655679d2fb62b83f8efec4fb4c8af\/test_drop.rss",
-       "voicemail":"646-495-9201 x 02302",
-       "email":"test_drop@drop.io",
-       "admin_token":"A97KQ4Vk7qQU90",
-       "fax":"856-632-1999",
-       "conference":" 218-486-3891 x 915088666",
-       "name":"test_drop",
-       "upload_url":"http:\/\/assets.drop.io\/upload",
-       "guests_can_add":"true",
-       "guests_can_comment":"true",
-       "guests_can_delete":"false",
-       "current_bytes":51488,
-       "max_bytes":104857600.0,
-       "expiration_length":"1_YEAR_FROM_LAST_VIEW",
-       "delete_permission_type":"ALLOWED"}
-    RESPONSE
-    
-    test_drop.should be_an_instance_of(Drop)
-    test_drop.rss.should                    == "http:\/\/drop.io\/k4gpvnty6tw9ho8u3j4w\/b9b2c8f2b8e655679d2fb62b83f8efec4fb4c8af\/test_drop.rss"
-    test_drop.voicemail.should              == "646-495-9201 x 02302"
-    test_drop.email.should                  == "test_drop@drop.io"
-    test_drop.admin_token.should            == "A97KQ4Vk7qQU90"
-    test_drop.fax.should                    == "856-632-1999"
-    test_drop.conference.should             == " 218-486-3891 x 915088666"
-    test_drop.name.should                   == "test_drop"
-    test_drop.upload_url.should             == "http:\/\/assets.drop.io\/upload"
-    test_drop.guests_can_add.should         == "true"
-    test_drop.guests_can_comment.should         == "true"
-    test_drop.guests_can_delete.should         == "false"
-    test_drop.current_bytes.should          == 51488
-    test_drop.max_bytes.should              == 104857600.0
-    test_drop.expiration_length.should      == "1_YEAR_FROM_LAST_VIEW"
-  end
-  
-  it "should parse lists of Drops" do
-    drops = Client::Mapper.map_drops <<-RESPONSE
-      [{"rss":"http:\/\/drop.io\/k4gpvnty6tw9ho8u3j4w\/b9b2c8f2b8e655679d2fb62b83f8efec4fb4c8af\/test_drop.rss",
-        "voicemail":"646-495-9201 x 02302",
-        "email":"test_drop@drop.io",
-        "admin_token":"A97KQ4Vk7qQU90",
-        "fax":"856-632-1999",
-        "conference":" 218-486-3891 x 915088666",
-        "name":"test_drop",
-        "upload_url":"http:\/\/assets.drop.io\/upload",
-        "guests_can_add":"true",
-        "guests_can_comment":"true",
-        "guests_can_delete":"false",
-        "current_bytes":51488,
-        "max_bytes":104857600.0,
-        "expiration_length":"1_YEAR_FROM_LAST_VIEW"},
-       {"current_bytes":178857,
-        "max_bytes":104857600.0,
-        "voicemail":"646-495-9201 x 72025",
-        "admin_token":"8a8vfzfvs2",
-        "email":"0sdcmz7@drop.io",
-        "upload_url":"http:\/\/assets.drop.io\/upload",
-        "guests_can_add":"true",
-        "guests_can_comment":"true",
-        "guests_can_delete":"false",
-        "conference":" 218-486-3891 x 680277944",
-        "name":"0sdcmz7",
-        "expiration_length":"1_YEAR_FROM_LAST_VIEW",
-        "fax":"856-632-1999",
-        "rss":"http:\/\/drop.io\/no9mbevuq3ttmfi6kqen\/3ec4f87d720032bae579cca40740e5e4b267e090\/0sdcmz7.rss"}]
-    RESPONSE
-    
-    drops.should be_an_instance_of(Array)
-    drops.each_should be_an_instance_of(Drop)
-    drops[0].name.should == "test_drop"
-    drops[1].name.should == "0sdcmz7"
-  end
-  
-  it "should parse Assets" do
-    parent_drop = stub(Drop)
-    audio = Client::Mapper.map_assets parent_drop, <<-RESPONSE
-      {"type":"audio",
-       "thumbnail":"http:\/\/drop.io\/download\/public\/vhahbqybv2dx2vdh0z7c\/8c525895399b8eafcd1621b4bd001ee1797e6679\/1c073ae0-7d29-012b-f659-002241261481\/316995b0-7d2e-012b-b8db-002241261481\/",
-       "status":"converted",
-       "filesize":178857,
-       "name":"audio",
-       "file":"http:\/\/drop.io\/download\/48f660ba\/b3c057eb4ce606f2f9f478379ea94ab703416ab3\/1c073ae0-7d29-012b-f659-002241261481\/316995b0-7d2e-012b-b8db-002241261481\/audio.mp3\/audio.mp3",
-       "duration":11,
-       "track_title":"Unknown",
-       "artist":"Unknown",
-       "created_at":"2008/10/15 21:28:55 +0000",
-       "converted":"http:\/\/drop.io\/download\/48f660ba\/26f6688a110f481b169daaa0a023656d4688dac0\/1c073ae0-7d29-012b-f659-002241261481\/316995b0-7d2e-012b-b8db-002241261481\/converted-audio_converted.mp3"}
-    RESPONSE
-    
-    audio.should be_an_instance_of(Asset)
-    audio.drop.should        == parent_drop
-    audio.type.should        == "audio"
-    audio.thumbnail.should   == "http:\/\/drop.io\/download\/public\/vhahbqybv2dx2vdh0z7c\/8c525895399b8eafcd1621b4bd001ee1797e6679\/1c073ae0-7d29-012b-f659-002241261481\/316995b0-7d2e-012b-b8db-002241261481\/"
-    audio.status.should      == "converted"
-    audio.filesize.should    == 178857
-    audio.name.should        == "audio"
-    audio.file.should        == "http:\/\/drop.io\/download\/48f660ba\/b3c057eb4ce606f2f9f478379ea94ab703416ab3\/1c073ae0-7d29-012b-f659-002241261481\/316995b0-7d2e-012b-b8db-002241261481\/audio.mp3\/audio.mp3"
-    audio.duration.should    == 11
-    audio.track_title.should == "Unknown"
-    audio.artist.should      == "Unknown"
-    audio.created_at.should  == "2008/10/15 21:28:55 +0000"
-    audio.converted.should   == "http:\/\/drop.io\/download\/48f660ba\/26f6688a110f481b169daaa0a023656d4688dac0\/1c073ae0-7d29-012b-f659-002241261481\/316995b0-7d2e-012b-b8db-002241261481\/converted-audio_converted.mp3"
-  end
-
-  it "should parse lists of Assets" do
-    parent_drop = stub(Drop)
-    assets = Client::Mapper.map_assets parent_drop, <<-RESPONSE
-      [{"type":"note",
-        "status":"converted",
-        "filesize":109,
-        "name":"about-these-files",
-        "contents":"These files are vital to our mission.&nbsp; <strong>Please<\/strong> do not delete them.<br \/><br \/>Thank you.",
-        "title":"About these files",
-        "created_at":"2008/10/15 21:21:49 +0000"},
-       {"type":"audio",
-        "thumbnail":"http:\/\/drop.io\/download\/public\/vhahbqybv2dx2vdh0z7c\/8c525895399b8eafcd1621b4bd001ee1797e6679\/1c073ae0-7d29-012b-f659-002241261481\/316995b0-7d2e-012b-b8db-002241261481\/",
-        "status":"converted",
-        "filesize":178857,
-        "name":"audio",
-        "file":"http:\/\/drop.io\/download\/48f660ba\/b3c057eb4ce606f2f9f478379ea94ab703416ab3\/1c073ae0-7d29-012b-f659-002241261481\/316995b0-7d2e-012b-b8db-002241261481\/audio.mp3\/audio.mp3",
-        "duration":11,
-        "track_title":"Unknown",
-        "artist":"Unknown",
-        "created_at":"2008/10/15 21:28:55 +0000",
-        "converted":"http:\/\/drop.io\/download\/48f660ba\/26f6688a110f481b169daaa0a023656d4688dac0\/1c073ae0-7d29-012b-f659-002241261481\/316995b0-7d2e-012b-b8db-002241261481\/converted-audio_converted.mp3"}]
-    RESPONSE
-    
-    assets.should be_an_instance_of(Array)
-    assets.each_should be_an_instance_of(Asset)
-    assets[0].name.should == "about-these-files"
-    assets[1].name.should == "audio"
-  end
-
-  it "should parse Comments" do
-    parent_asset = stub(Asset)
-    comment = Client::Mapper.map_comments parent_asset, <<-RESPONSE
-      {"created_at":"2008/09/17 20:47:47 +0000",
-       "id":1,
-       "contents":"This is my comment content."}
-    RESPONSE
-    
-    comment.should be_an_instance_of(Comment)
-    comment.asset.should == parent_asset
-    comment.created_at.should == "2008/09/17 20:47:47 +0000"
-    comment.id.should         == 1
-    comment.contents.should   == "This is my comment content."
-  end
-
-  it "should parse lists of Comments" do
-    parent_asset = stub(Asset)
-    comments = Client::Mapper.map_comments parent_asset, <<-RESPONSE
-      [{"contents":"Really?  Looks like a waste of space to me.  I'm going to delete some of these...",
-        "created_at":"2008/10/15 21:44:24 +0000",
-        "id":1},
-       {"contents":"DON'T DO IT!!",
-        "created_at":"2008/10/15 21:44:36 +0000",
-        "id":2}]
-    RESPONSE
-    
-    comments.should be_an_instance_of(Array)
-    comments.each_should be_an_instance_of(Comment)
-    comments[0].contents.should =~ /waste of space/
-    comments[1].contents.should =~ /DON'T/
-  end
-end