activegist 0.6.4 → 0.7.0

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA1:
+  metadata.gz: bdc5b895d5ae92cd675a9025e6c6a48f469eb0bf
+  data.tar.gz: 0357e681690985952b7f3092171c0de396395d23
+SHA512:
+  metadata.gz: 4301d242d38735b3a77ab10ddaa8465c3f5ca010182bf6306a901f0b3ee100b3f3fccdb20b4f2d2800b498f0433373b9a5d0bcdbcfaa1eb9359dea5560bfc640
+  data.tar.gz: fdae8a42aa92fb6ff25004cc91ee9135d2e9f2c33952c4e2af9ca6f26ae6fa1a03c87d05f5f7ce8f1c577e48000627f1a6a464989627ca80bce6599eefd1fc47

data/.gitignore

@@ -4,3 +4,4 @@ Gemfile.lock
 pkg/*
 html
 doc
+coverage

data/.travis.yml

@@ -1,8 +1,7 @@
 rvm:
-  - 1.8.7
-  - 1.9.2
-  - 1.9.3
-  # - ree
-  # - jruby
+  - 2.0.0
   - ruby-head
-  - rbx-2.0
+
+matrix:
+  allow_failures:
+    - rvm: ruby-head

data/README.rdoc

@@ -1,4 +1,4 @@
-= ActiveGist! {<img src="https://secure.travis-ci.org/sinisterchipmunk/active-gist.png" />}[http://travis-ci.org/sinisterchipmunk/active-gist]
+= ActiveGist! {<img src="https://secure.travis-ci.org/sinisterchipmunk/active-gist.png" />}[http://travis-ci.org/sinisterchipmunk/active-gist] {<img src="https://codeclimate.com/github/sinisterchipmunk/active-gist.png" />}[https://codeclimate.com/github/sinisterchipmunk/active-gist] {<img src="https://coveralls.io/repos/sinisterchipmunk/active-gist/badge.png" alt="Coverage Status" />}[https://coveralls.io/r/sinisterchipmunk/active-gist]
 
 I needed a Ruby library to perform basic create, read, update and delete operations on Gists. I looked, I saw basically nothing (except hacky, test-less tools), and I decided to roll my own. Here's the result.
 
@@ -6,13 +6,12 @@ ActiveGist is so named because it wraps GitHub's Gist API with a class implement
 
 == Installation
 
-The obligatory installation steps, in case you haven't yet figured out they're the same as any other gem...
+The obligatory installation steps...
 
   gem install activegist
 
 == Usage
 
-  require 'rubygems' # on old rubies
   require 'activegist'
   
   # Set up credentials. A required step, as far as I know.

data/activegist.gemspec

@@ -18,12 +18,13 @@ Gem::Specification.new do |s|
   s.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
   s.require_paths = ["lib"]
 
-  s.add_development_dependency "rspec", '~> 2.8.0'
-  s.add_development_dependency "fakeweb", '~> 1.3.0'
-  s.add_development_dependency 'rake', "~> 0.9.2.2"
+  s.add_development_dependency "rspec", '~> 2.8'
+  s.add_development_dependency "fakeweb", '~> 1.3'
+  s.add_development_dependency 'rake', "~> 0.9"
   s.add_development_dependency 'rdoc', "~> 3.12"
+  s.add_development_dependency 'coveralls'
   
-  s.add_runtime_dependency "activemodel", '~> 3.1'
-  s.add_runtime_dependency "activesupport", '~> 3.1'
-  s.add_runtime_dependency "rest-client", '~> 1.6.7'
+  s.add_runtime_dependency "activemodel", '~> 4'
+  s.add_runtime_dependency "activesupport", '~> 4'
+  s.add_runtime_dependency "rest-client", '~> 1.6'
 end

data/lib/active_gist.rb

@@ -90,10 +90,10 @@ class ActiveGist
     run_callbacks create_or_update_callback do
       run_callbacks :save do
         if new_record?
-          data = as_json(:only => [:description, :public, :files]).delete('active_gist').to_json
+          data = as_json(:only => [:description, :public, :files]).to_json
           response = api.post data, :content_type => 'application/json', :accept => 'application/json'
         else
-          data = as_json(:only => [:description, :files]).delete('active_gist').to_json
+          data = as_json(:only => [:description, :files]).to_json
           response = api[id].patch data, :content_type => 'application/json', :accept => 'application/json'
         end
         self.attributes = JSON.parse response

data/lib/active_gist/attributes.rb

@@ -1,92 +1,53 @@
 module ActiveGist::Attributes
   GIST_ATTRIBUTES = %w(url id description public user files comments
                        html_url git_pull_url git_push_url created_at
-                       forks history updated_at comments_url)
-                       
+                       forks forks_url history updated_at comments_url commits_url)
+
   def self.included(base) #:nodoc:
     base.define_attribute_methods GIST_ATTRIBUTES
   end
-  
-  def forks
-    @forks
-  end
-  
-  def history
-    @history
-  end
-  
-  def url
-    @url
-  end
-  
-  def id
-    @id
-  end
-  
-  def description
-    @description
-  end
-  
+
+  attr_reader :url,
+              :id,
+              :description,
+              :user,
+              :files,
+              :comments,
+              :comments_url,
+              :created_at,
+              :forks,
+              :forks_url,
+              :history,
+              :updated_at,
+              :html_url,
+              :git_pull_url,
+              :git_push_url,
+              :commits_url
+
   def public?
     !!@public
   end
   alias public public?
-  
-  def user
-    @user
-  end
-  
-  def files
-    @files
-  end
-  
-  def comments
-    @comments
-  end
-  
-  def html_url
-    @html_url
-  end
-  
-  def git_pull_url
-    @git_pull_url
-  end
-  
-  def git_push_url
-    @git_push_url
-  end
-  
-  def created_at
-    @created_at
-  end
-  
-  def updated_at
-    @updated_at
-  end
 
-  def comments_url
-    @comments_url
-  end
-  
   def description=(descr)
     description_will_change!
     @description = descr
   end
-  
+
   def public=(pub)
     public_will_change!
     @public = pub
   end
-  
+
   def files=(files)
     files_will_change!
     @files = files
   end
-  
+
   def attributes
     GIST_ATTRIBUTES.inject({}) { |h,k| h[k] = self[k]; h }
   end
-  
+
   def attributes=(attributes)
     attributes.each do |key, value|
       if respond_to?(:"#{key}=")
@@ -104,7 +65,7 @@ module ActiveGist::Attributes
   def [](attribute)
     send attribute.to_sym
   end
-  
+
   def []=(attribute, value)
     send :"#{attribute}=", value
   end

data/lib/active_gist/version.rb

@@ -1,6 +1,6 @@
 class ActiveGist
   module Version
-    MAJOR, MINOR, TINY = 0, 6, 4
+    MAJOR, MINOR, TINY = 0, 7, 0
     STRING = [MAJOR, MINOR, TINY].join '.'
   end
   

data/spec/lib/active_gist_spec.rb

@@ -3,7 +3,7 @@ require 'test/unit'
 
 describe ActiveGist do
   let(:valid_attributes) do { :files => { "file1.txt" => { :content => 'String file contents' } } } end
-    
+
   def expect_request(method, path, data = nil, headers = { :accept => 'application/json' }, return_val = {:id=>1})
     RestClient::Request.should_receive(:execute) do |options|
       expectation = [method, File.join("https://api.github.com", path)]
@@ -11,24 +11,26 @@ describe ActiveGist do
       case data
         when NilClass then expectation << nil; actual << options[:payload]
         when String   then expectation << data; actual << options[:payload]
-        when Hash     then expectation << JSON.parse(data.to_json); actual << JSON.parse(options[:payload])
+        when Hash
+          expectation << JSON.parse(data.to_json)
+          actual << JSON.parse(options[:payload])
         else raise "Don't know how to handle data format: #{data.inspect}"
       end
       options[:headers].each { |k,v| headers[k] = v unless headers.key?(k) }
       expectation << headers
       actual << options[:headers]
-      
+
       actual.should == expectation
       return_val.to_json
     end
   end
-  
+
   it "should not be 'equal' to another object with same id" do
     obj = Object.new
     def obj.id; '1'; end
     ActiveGist.new(:id => '1').should_not == obj
   end
-  
+
   describe "a new gist" do
     it "assign a file and validate" do
       a = ActiveGist.new
@@ -36,21 +38,21 @@ describe ActiveGist do
       a.should be_valid
     end
   end
-  
+
   describe "an existing gist" do
     describe "that is starred" do
       subject { ActiveGist.find(1) }
-      
+
       it "check if it is starred" do
         subject.should be_starred
       end
-      
+
       it "check valid request" do
         subject
         expect_request(:get, "/gists/1/star")
         subject.starred?
       end
-      
+
       describe "unstarring it" do
         it { subject.unstar!; should_not be_starred }
         it "check valid request" do
@@ -60,14 +62,14 @@ describe ActiveGist do
         end
       end
     end
-    
+
     describe "that is not starred" do
       subject { ActiveGist.find(2) }
-      
+
       it "check if it is not starred" do
         subject.should_not be_starred
       end
-      
+
       it "check valid request" do
         subject
         expect_request :get, "/gists/2/star"
@@ -83,31 +85,31 @@ describe ActiveGist do
         end
       end
     end
-    
+
     describe "destroying it" do
       subject { ActiveGist.find 1 }
-      
+
       it "should destroy properly" do
         subject.destroy
         subject.should_not be_persisted
         subject.should_not be_new_record
         subject.should be_destroyed
       end
-      
+
       it "should send correct request" do
         subject
         expect_request :delete, "gists/1"
         subject.destroy
       end
     end
-    
+
     describe "forking it" do
       subject { ActiveGist.find 1 }
-      
+
       it "should return the forked gist" do
         subject.fork.should_not == subject
       end
-      
+
       it "should send the correct request" do
         subject
         expect_request :post, "/gists/1/fork", ""
@@ -115,49 +117,49 @@ describe ActiveGist do
       end
     end
   end
-  
+
   describe "validation" do
     before { subject.valid? }
-    
+
     it "should not validate presence of description" do
       subject.errors[:description].should_not include("can't be blank")
     end
-    
+
     it "should not validate presence of public" do
       # github requires this but we're just going to default it to false
       subject.errors[:public].should_not include("can't be blank")
     end
-    
+
     it "should validate presence of files" do
       subject.errors[:files].should include("can't be blank")
     end
   end
-  
+
   describe "creating a valid gist with arrays and hashes" do
     subject { ActiveGist.create!(valid_attributes) }
-    
+
     it "should return the new gist" do
       subject.id.should == '1'
       subject.url.should == 'https://api.github.com/gists/1'
       subject.should be_public # this would default to false but our fake response returns true
     end
-    
+
     it "should be persisted" do
       subject.should be_persisted
     end
-    
+
     it "should send proper request" do
       expect_request :post, "/gists", valid_attributes.merge(:description => nil, :public => false)
       subject
     end
   end
-  
+
   describe 'changing just files' do
     subject { ActiveGist.find(1) }
     before { subject.files['file1.txt'][:content] = 'updated' }
     it { should_not be_persisted }
   end
-  
+
   describe "changing an existing gist" do
     subject { ActiveGist.find(1) }
     before do
@@ -166,9 +168,9 @@ describe ActiveGist do
       subject.files['deleted.txt'] = nil
       subject.files['file1.txt'][:content] = "updated file contents"
     end
-    
+
     it { should_not be_persisted }
-    
+
     it "should send proper request to save" do
       expect_request :patch, "/gists/1", {
         :description => "updated description",
@@ -176,93 +178,93 @@ describe ActiveGist do
       }
       subject.save!
     end
-    
+
     describe "saving" do
       before { subject.save! }
-      
+
       it "should store the new description" do
         subject.description.should == 'returned updated description'
       end
-      
+
       it "should remove the renamed file's old filename" do
         subject.files.should_not have_key('old_name.txt')
       end
-      
+
       it "should add the renamed file's new filename" do
         subject.files.should have_key('new_name.txt')
       end
-      
+
       it "should remove the deleted file" do
         subject.files.should_not have_key('deleted.txt')
       end
-      
+
       it "should update the modified file contents" do
         subject.files['file1.txt'][:content].should == 'returned updated file contents'
       end
     end
   end
-  
+
   describe "saving a valid gist" do
     subject { ActiveGist.new(valid_attributes) }
     before { subject.save }
-    
+
     it '#save should return true' do
       subject.save.should be_true
     end
-    
+
     it "should be persisted" do
       subject.should be_persisted
     end
   end
-  
+
   describe "creating an invalid gist" do
     it "::create! should raise an error" do
       proc { ActiveGist.create! }.should raise_error
     end
-    
+
     it "::create should return a record that is not persisted" do
       ActiveGist.create.should_not be_persisted
     end
   end
-  
+
   describe "saving an invalid gist" do
     it '#save! should raise an error' do
       proc { ActiveGist.new.save! }.should raise_error
     end
-    
+
     it '#save should return false' do
       ActiveGist.new.save.should be_false
     end
   end
-  
+
   it "should have a count of all gists" do
     ActiveGist.count.should == 3
   end
-  
+
   it "should have a count of public gists" do
     ActiveGist.count(:public).should == 2
   end
-  
+
   it "should have a count of starred gists" do
     ActiveGist.count(:starred).should == 1
   end
-  
+
   it "should return the last gist" do
     ActiveGist.last.id.should == '3'
   end
-  
+
   it "should return the first gist" do
     ActiveGist.first.id.should == '1'
   end
-  
+
   it "should find a gist by id" do
     ActiveGist.find(2).should == ActiveGist.all[1]
-    
+
     # check request
     expect_request(:get, "/gists/2")
     ActiveGist.find(2)
   end
-  
+
   describe "a gist returned by github" do
     subject { ActiveGist.first }
     it { should respond_to(:url) }
@@ -272,24 +274,30 @@ describe ActiveGist do
     it { should respond_to(:user) }
     it { should respond_to(:files) }
     it { should respond_to(:comments) }
+    it { should respond_to(:comments_url) }
+    it { should respond_to(:created_at) }
+    it { should respond_to(:forks) }
+    it { should respond_to(:forks_url) }
+    it { should respond_to(:history) }
+    it { should respond_to(:updated_at) }
     it { should respond_to(:html_url) }
     it { should respond_to(:git_pull_url) }
     it { should respond_to(:git_push_url) }
-    it { should respond_to(:created_at) }
+    it { should respond_to(:commits_url) }
     it { should be_persisted }
   end
-  
+
   describe "fetching all gists" do
     subject { ActiveGist.all }
-    
+
     it { should have(3).gists }
-    
+
     it "should request the gist properly" do
       expect_request(:get, "/gists", nil, {:accept => 'application/json'}, [])
       subject
     end
   end
-  
+
   describe "active model lint tests" do
     include Test::Unit::Assertions
     include ActiveModel::Lint::Tests

data/spec/spec_helper.rb

@@ -1,3 +1,6 @@
+require 'coveralls'
+Coveralls.wear!
+
 $:.unshift File.expand_path('../lib', File.dirname(__FILE__))
 
 require 'active_gist'

data/spec/support/fakeweb.rb

@@ -4,7 +4,11 @@ def json(name)
   File.read(File.expand_path("../fixtures/#{name}.json", File.dirname(__FILE__)))
 end
 
-FakeWeb.allow_net_connect = false
+RSpec.configure do |c|
+  c.before { FakeWeb.allow_net_connect = false }
+  c.after  { FakeWeb.allow_net_connect = true  }
+end
+
 FakeWeb.register_uri(:get, 'https://username:password@api.github.com/gists', :response => json('all_gists'))
 FakeWeb.register_uri(:get, 'https://username:password@api.github.com/gists/public', :response => json('public_gists'))
 FakeWeb.register_uri(:get, 'https://username:password@api.github.com/gists/starred', :response => json('starred_gists'))

metadata

@@ -1,68 +1,60 @@
 --- !ruby/object:Gem::Specification
 name: activegist
 version: !ruby/object:Gem::Version
-  version: 0.6.4
-  prerelease: 
+  version: 0.7.0
 platform: ruby
 authors:
 - Colin MacKenzie IV
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-11-01 00:00:00.000000000 Z
+date: 2013-11-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 2.8.0
+        version: '2.8'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 2.8.0
+        version: '2.8'
 - !ruby/object:Gem::Dependency
   name: fakeweb
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 1.3.0
+        version: '1.3'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 1.3.0
+        version: '1.3'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 0.9.2.2
+        version: '0.9'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 0.9.2.2
+        version: '0.9'
 - !ruby/object:Gem::Dependency
   name: rdoc
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
@@ -70,59 +62,66 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
         version: '3.12'
+- !ruby/object:Gem::Dependency
+  name: coveralls
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: activemodel
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: '3.1'
+        version: '4'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: '3.1'
+        version: '4'
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: '3.1'
+        version: '4'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: '3.1'
+        version: '4'
 - !ruby/object:Gem::Dependency
   name: rest-client
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 1.6.7
+        version: '1.6'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 1.6.7
+        version: '1.6'
 description: Wraps GitHub's Gist API with a class implementing the ActiveModel modules.
   So, it should be pretty familiar to anyone who's ever used models in Ruby on Rails.
 email:
@@ -167,33 +166,26 @@ files:
 - spec/support/fakeweb.rb
 homepage: http://github.com/sinisterchipmunk/active-gist
 licenses: []
+metadata: {}
 post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - '>='
     - !ruby/object:Gem::Version
       version: '0'
-      segments:
-      - 0
-      hash: -3826366150870984350
 required_rubygems_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - '>='
     - !ruby/object:Gem::Version
       version: '0'
-      segments:
-      - 0
-      hash: -3826366150870984350
 requirements: []
 rubyforge_project: activegist
-rubygems_version: 1.8.24
+rubygems_version: 2.0.3
 signing_key: 
-specification_version: 3
+specification_version: 4
 summary: Wraps GitHub's Gist API with an intuitive class based on ActiveModel.
 test_files:
 - spec/fixtures/all_gists.json