From 9d02fc12595a695ab5d96ecb45c0521ce20fff0a Mon Sep 17 00:00:00 2001
From: Erik Michaels-Ober <sferik@gmail.com>
Date: Wed, 22 Jun 2011 15:43:42 -0700
Subject: [PATCH] Implement Gems::Client#dependencies

---
 gems.gemspec                    |   3 ++-
 lib/gems/client.rb              |  10 ++++++++++
 lib/gems/connection.rb          |   9 +++++----
 lib/gems/request.rb             |   8 ++++----
 spec/fixtures/dependencies.json | Bin 25535 -> 18574 bytes
 spec/gems/client_spec.rb        |  14 +++++++++++++-
 6 files changed, 34 insertions(+), 10 deletions(-)

diff --git a/gems.gemspec b/gems.gemspec
index 4054190..4e9bdcb 100644
--- a/gems.gemspec
+++ b/gems.gemspec
@@ -10,9 +10,10 @@ Gem::Specification.new do |gem|
   gem.add_development_dependency 'webmock', '~> 1.6'
   gem.add_development_dependency 'yard', '~> 0.7'
   gem.add_runtime_dependency 'faraday', '~> 0.6.1'
-  gem.add_runtime_dependency 'faraday_middleware', '~> 0.6.3'
+  gem.add_runtime_dependency 'faraday_middleware', '~> 0.6.5'
   gem.add_runtime_dependency 'hashie', '~> 1.0.0'
   gem.add_runtime_dependency 'multi_json', '~> 1.0.3'
+  gem.add_runtime_dependency 'multi_xml', '~> 0.2.2'
 
   gem.authors       = ["Erik Michaels-Ober"]
   gem.email         = ['sferik@gmail.com']
diff --git a/lib/gems/client.rb b/lib/gems/client.rb
index 90ac0f6..f9d7690 100644
--- a/lib/gems/client.rb
+++ b/lib/gems/client.rb
@@ -50,5 +50,15 @@ def versions(gem, options={})
     def downloads(gem, version, options={})
       get("/api/v1/versions/#{gem}-#{version}/downloads.json", options)
     end
+
+    # Returns an array of hashes for all versions of given gems
+    #
+    # @param gems [Array] A list of gem names
+    # @return [Array]
+    # @example
+    #   Gems.dependencies 'rails', 'thor'
+    def dependencies(*gems)
+      get("/api/v1/dependencies", {:gems => gems.join(',')}, :marshal)
+    end
   end
 end
diff --git a/lib/gems/connection.rb b/lib/gems/connection.rb
index 130aa42..df9573a 100644
--- a/lib/gems/connection.rb
+++ b/lib/gems/connection.rb
@@ -4,9 +4,8 @@ module Gems
   module Connection
     private
 
-    def connection
+    def connection(format=:json)
       options = {
-        :headers => {'Accept' => 'application/json'},
         :ssl => {:verify => false},
         :url => 'https://rubygems.org',
       }
@@ -15,8 +14,10 @@ def connection
         connection.use Faraday::Request::UrlEncoded
         connection.use Faraday::Response::RaiseError
         connection.use Faraday::Response::Mashify
-        connection.use Faraday::Response::ParseJson
-        connection.adapter(Faraday.default_adapter)
+        connection.use Faraday::Response::ParseXml if :xml == format
+        connection.use Faraday::Response::ParseJson if :json == format
+        connection.use Faraday::Response::ParseMarshal if :marshal == format
+        connection.adapter Faraday.default_adapter
       end
     end
   end
diff --git a/lib/gems/request.rb b/lib/gems/request.rb
index 9d20e91..fab0b52 100644
--- a/lib/gems/request.rb
+++ b/lib/gems/request.rb
@@ -1,13 +1,13 @@
 module Gems
   module Request
-    def get(path, options={})
-      request(:get, path, options)
+    def get(path, options={}, format=:json)
+      request(:get, path, options, format)
     end
 
     private
 
-    def request(method, path, options)
-      response = connection.send(method) do |request|
+    def request(method, path, options, format)
+      response = connection(format).send(method) do |request|
         request.url(path, options)
       end
       response.body
diff --git a/spec/fixtures/dependencies.json b/spec/fixtures/dependencies.json
index 1d784950896631e29cc14dfe9b0dc5ab98ef8c92..5d67781e168bdfbdcd2316385ac556699be35999 100644
GIT binary patch
literal 18574
zcmb_k+fH0Z7R^I631IBlqj|UZfFrtTFxYZ_#7}^G6e~8M1~M8&N`8Izr7pX6tzGTY
zJ+Hm0>a4XdtLoIj{_*EefB$*#{^;)c+s%4&a(FhKPFL@bAHH5c{ri6N?c`wdeDmM;
z2Y1)s)+dLX>n~p)-@pF6epugqUf<n*SwDVy@#&|N7dOv$pTE-dqrZNAyO>Vtr(+)b
zL<3G<e0(d5(vPpNZ=b&0-+d#91{>g?4E(X)tRL^6H@9mG$sC4m?>C<<@JbhZxW4_}
z{`^LldVGF(xZgY#<j;c-Kb*W2BGcyf{KG#~*@vH!st-SJSMp0z!V7e29OxA}xK-sq
ztIR=8okd%Pt9=!&MhY1mC@gbJ-V9L&1BDF53RhVnRT!mU4~1y~tgwPmVTC~sE7&Ql
z5K@@XtdM$;6rSyy!!xWftxh0RSYe>Bf}O$&A%zLe3RUnP3ey5uVFjVW3Il}|>=afA
zDNJZqsDc;!=5T=(rq#zeEOSvGGT14s%OZsd6@^F3>1<jok-Q_o>iFYZozTg#h~;B{
zZ_{KEN7F(Fo9o}#C&#~je0w@wPG>nUnkGnm>s}@{mKNxQIB-yT%RINuOI_H;!7b|q
zTT~Rv4t|N!vQ=H$kS{Ykzf&{dRrxtZkpZk4^RrWuk(?(^+FmZ77U<MS$SUX+Ik+u7
z2U=wga_TJFD%_(IRYa+#k}vZx1_uhu+>$ppH&Dp)Vuj0A;a(odMNoVUDP(Y<u*^jY
z0}ss4j1?}jLhac-6s84ml~xcctT3q33U)cH5K@>>QTPjaJe{^~)=$^FX?(+H=CVYm
zvV*<OZa}NFgRRyMbZT2%^oo1F+s8fMbsCegG2qo1sMTcvtH}K9v}7dL<>~Z}8fPzW
zkp((6$saQ46*;(7<v^><K~9}TTZOw-rCG1|gZCCzIPjJXhYAC4$@5}`>Mc@<7yDG=
zMVGr&Bn<ex8K_lc0GluKvs03hnx~6>D^V8c)TAV4&?|CqtIB~^nS-1<i?#~)szeqM
zR$>MR3d>x-%jLO&LY@~ZG?lp9jY+D7{<g#bUX_8`oEgBXF+V#c8L4?X=P>WoQz`GV
zKvant^or12pQk+6Ds%IkI*W=zag5z=MN*{7y-o2#19(*if_d`yJZ8L8k`Ym%@0Bn~
zqxZ_v0EK#f6=cLl)&AZRw}_)?q2RspbXx6OU9vz_T^aOwNOh3mR+0m49312XTeMc!
zZud1Qzx{2L0lX>$wTcX2)tH~1l8jUp+fPNudXJ57vdcSDDYf(n^sZ$fsDtMK9pL>c
zuT*uwCTZUChdTO}NsnM4$VZt|{L9P%I(R(Jhwd%f?Wf3+N8Qd}7$lq}1-U^Pu{`YW
zZ2~OfC|U@0`ypRtU41WUija)&I2*uHHb1wF4d4hH1o3?-&z~MP>+j`E?O*0}yd7w{
zJE-OFK&Qc>NQdljuvoso>ig26;PH1M27;<Db9~kNYBNhYbs3TFmG4me4P(0#y`k$%
zetqWA6Cd2`hcybA@aA<;jx6Yfp}9qz16_XYvQDI(C_<0(U{rTa&KjNU8dc}3s<V^X
zVqRs7w#C?DR%44ei3SvfbV0(&;R}-ON3WN17kxD?+eFGeL|LWN*uh?AH=tG8!B%Vs
zI=wBfdk;GUykSO`ovaC7D$7Eo*AnvMOH~@caWn{WjaTv*c0agbBGRwb{w`%wmk^Iy
z`W|aj;ZY~V!$^(nJ@AGK2}z-Wmr@)S1}uF)-bbIuJJM!K$QRyY4EVyU3+BrJ;9JIf
zz6@Z#j3B<Ip|_uQb3-q#XtnBCP1E$p-VPo%beUhYb@OdRI?%~)aqnT<H>4t&-|qs+
z{dUwmmEob}G9OE3IF!ubNV0yLcvWqC>(-=S5XrS0!PKjD4ML>R%W>4BUarSGn`qXy
zV*N~~wSlW$!|R(t1Hu~{Za@Ysoz(qhe94#r97Thm1|)-{2cfKld}(}_0eOl79bYrP
z<*NYVt4P52Y=V1#$d|@rz6x}F&G?qD0*J37A>VoNJ&WS20N`8l;rTLv`7(m|9-U4L
zIvhm>@h|-|u>Y)Y9@m>czUUv1olfU+o^P9yN`gO&fJ6Jjj<=K-X?#yv4&{Ziy5eeD
zoJ-Fs=kT}@aK4B+^NHAnnK{GP@dbT2XN{OMPth;~&LnDPoG&VeVtx{5<<N098rwKu
z4LGy&4mi(`GcO2g<5I+QoV8?HizbZ=`XeolCXTZ&XDY~28q3*XHWb=0YuGx<8vT|s
z-D)hPl|s(!O11^h(_hVSIWMQVVSX2thK3Dr?#nrIj^jMyOq=M;ayD_Zan8-&an|so
zIMb9M&Wc2Frmy9*IhVM3&I|y~WqfHtjWd-;$eEFdGx^?3&RPOp3lagLU^&|oK!K*h
z3`i?tNncEQ{NsXNP4KYGn|9#OdIW@|*-#MaVl*z&bjAruGZK>yckYmM*^AriSWFD`
zq_sRzq^X@k(%$acNQcLxkTg6Xad|L+NJlkI{S=b+-jPWk@*h?~>o}mb+5*tG0}V~X
zE8X>Q$Xqb<l^!8B&#OuQ!dadU5A4y?SCSa?^!{AVd^wKHOP^lO0qJuu|CQ$~@&EtF
zLpzDo>rv0ECxw!E6nol#hw7m*%zZrUBMH71OB{UI;&UIb8j*TUmwUXPTVHVQ12>Nd
z=n1e7zcwBspyyUdfNGxzpv@$HJm(TUKI%NJm-z9sS}#(sK}PEJ1dJcgMGIRm+?8Yo
z1wfwU4c@=3=UqP9mCgpc60fDC)$xi~ikYB{4LF<LXgO=uHrW)rg4HaD1{$5gZJQ})
zIz{W12y=!WGvr*h7|)pj#JNmxyFYap&A^-)v2mtTbma_<X~?;xdVkK*iA5Vd<P6<x
z$T>K$=$Q${xrprI%uDSMXXrpf&cS)5zkt#nL^n{1tmIg>17~PkL(b)J;A;b)v6l7L
zs*5wv*nxBST>zd7xogW!zvqm-ZYdX*&W4k3?8KRjXr|X0>uNcJGovum-w|isxlo+#
zo}Rs~_{Q8<_XS<R7VGky8R&ZuZuX)^H~Sqp&-FUB2e1$7u7)^AzHuJzgX-LY*HKf?
zu8qL!`ioU{2XhnadXVmFf*zz7T|!#U`Wk%5nV#!)#s++CVBbKu*Q(}R90k_~s|h*J
z^*Un%o-_6h{m_AXkY`}dtUu&D*Xx#%*Y#gPb8QrJ>}msl2c@@Z$pGX`z7>0&|DGjZ
zTWCB5pzA<chydGxetaQpfIh}~rI2rUrVIE&m*b~+mrJzDc+Zys%$JdnZ|HFW-(py{
zikUA1Aiib1=gR=*%SgyK^ooElHw3H)?-VJ%WxVIh0Ord`$T#$ffG<1|aeTc?B)(<5
z=ZifeuN3g5x1wnOfP90uDB8TG5xaUY<6FK8p!h0c`O-HMeIMb+%<DHAkLy7Jz_*O|
ze4~ETl|sJr`i*wSh%W<xZyE3TGJxaDNWhm)G_&d<I04bm*dNUJmahVcuOcm9{g>79
zcS-a3>cI^7Lw{nEelz1+z6v0|iXguF3BYd32T#A<Z|MI4`JsHLTTRwL5N-_-;r2j?
za2qg$+Xh3pHH;o_|4a|z_7BMrK8_Eal1zc3`Q!M|X?Dx}aeU~lW6S(;eA)wh&!4J&
vxIX#$5T`HkTkvuG>2-I@{5bw{lN!?(c+2eg;W9hsE^sVeVEgCuCG!6RamaH!

literal 25535
zcmc(n-EP}P5QXpi6oRh{LXv-C1xQ{b$c2AY)M#uOvYnzR+Gp<$DVsCroF$ohsN8xF
zcex+W%<L{{efj(2!^{2kcK_+~$!W7}E}Q+$`sCMN7K@LM->#m2+wJd5%l_s1ukMcz
zSNB`{!`0olr|vJe+sEz0?e^j3Zu|7<bGdNw<;%(S%fs!r?f&G~#mT>)7prD@^40n*
zZSAGcMQd%J7X6rSf~%Y7yWPWm8KLW-D*C?NZ=ZHA`<v}xUnaZR?QaJ~yNAcCn?D96
z-?vXMkB__kGpWA*Qzj=su6_c@RZqGO5=1qSLdGCLq(FkC0<x$WWU)?h<Z(g1jydQ^
zDPoXSnFlFF5u}t5$TyV+dA)tUIzGy7)KQp1rg2t;2vo=zsSru12OVQwHXn{0<R8WY
z90Ux}?m0XTlX!e2h~TK1khKa4>nD_#5AB#?!dza2NP^?O1UwcIK$a+oovH@%xS?F0
zX}KLp<r*Z2Y9NIST&M<&bz0n01*9x5%jV+9;d>E7j{$-}!4r}Q4g}~vXcoYVLb>ju
zHRN$uhuxSX;VVQETo+KUOHo@Oarlb92%zeRR)>d<Y9NmtzIHnha>9Kqs)2;QG)NFB
zkRVZR>~2bdJU?>yo{xnu2oyXaiQx+Zga*w5SW!q=Eyv@A?>QTaT_;2m9QVbp6MYdt
z)enX5p(6$I?8vS=8*?296g(k`t^)x=gJ$8nqCi;d=CoNQB)Zp3?YH}fz6UCUR`d%9
z)gBF{6~QVf*<bz9o=1HCw6NopQ-)$^9%nxx6uU78c=b0HMAA(vgm4!j2kh}zhYNh2
z3f*N*r$-LO@;v?WfpZxM6g(k`p$GyDMQ9eGSQHA@94C9jXM5{0Tp&=mge2ku0pfyY
zfvYGKu7t!z2cg5bU@gS%0)fIMBoP+~5EnEHTt$Jn+AO$ye=JrLy*okXbIrr(5Se|Z
zQSFAV=b{xM5us!}(c2Z9`Iu~mVyhc_kO*AQKuAD5oMIx5=5b%j<9pvP)NHb8yN6l(
zR|zc^C5mPd{RlkV4+M)CZ5)$TwS-9e!OSxlU)?PQtSa+BACZ{o12Yd@AN+Rc05ebf
z=dv2ft!-uXW{2ta@}V3Nz0KU-d&QSmrho#&SzCxCKkiE~WD#*>2^h17Mw$Gz>HR_K
z9O!Yn`l506xH;3m-QC+GQK_J=>NCF9D3tMjDkQlib_3X(gWT<6yRq0b>k(Ua0|!jX
zm_nwlSrH;oA!DXOB(WZJ81j}@FW>J^<(#hT0cuhZi^nAz1{WG62zd@h7%v#@1y+j^
z)~i(}UH@ZXqzdQ=P0Vs`K4Cnth>}f%mGH{0#tGi6#fY;zCPAbF%?ySF<Q=D&h9lt>
zNDYU}Q6^y=B#1bmnQ@SSa5%*{jy`ASo+})3_A0EGc`0G}4mS4@nZG}X4gMBp*@*rk
z!fJmyXT!ZgK!(mRXZMZi3$<mlX;$udQmrbC#Dry<kQp$4?C2*#k3dyJ8n(CiKwrVf
zL02dH{*oA%YzEfdeT{1@cQwOCRh=s96x68FZjG9%S$XAVc{<_6QY|UM`<n=?8#@@w
z&rUSQh?at2CQytg#dT9IdB3+b4(l3b8~5u%&<9X=YvFptLDz#3po=LuPMd@ky~bf(
zhv~38NZ@hM^@u|V$Q^EkX*gzc$LW|mgdpMocQFnU5Dupp$FXcyDaSY6a_P_Q_0x9$
z{qFS>*JX3=50E>Nj~Dw&2aZvE`dIPxk;sa#hj?j^MS710B4^D?yn1etB<)Rh4~*o3
zXN*YPnM22{H;EyEW<+v{0|+Fpi)ly-$DEw1k+}O&E}<z!mk?4!vYCJ+EFIiS(W*;U
z<w2wz4QnLsel!wd=3PQ@#Yo_K;gVV^X`>n!b|r4sbDD%^6l;yG{svPCc?gNLnJ`o{
zfuvS!&MoyzBr}@i1EvB-0uMRH%_UwhkU$3sNXqtLzPO>M9Fagd;Ot5!A|#dxhg}jB
zA-&dIQa(GG6A3&+MnVFgO3HQ>C+v?1OB%FHIEKV2CLt+lcRnOEB}76qih)y7g`1DL
zh-|WEMOdn>$~MLI5tv-qK2{`?v-0=ZDo)rubHyn{{O+0~r!aw%C76(cm^j5mCP~YA
zRfRXh3wmSXYNTKiQi1e|Wy9gv30uO6Op<ndnu)7{+9xFN=|p7<$S0N#$C$V=rZBO(
zkQ6%oYP~%tr3cujio-c7J>Y)2MFAx#Oy<n?*PIA0v(9^b+G}t24BbeRf#V<_<LzXE
z<6$!8+e$GTE4lMLnx8Y9w7LaG6MN~`xH{N~q6Um+>+`p`n)qzeD$WniCin<5XFJ~z
zcoS!nRx5sRHi5jHP4FttMjlga%4fNAdF=u-aL5Ct>R~(OfpAPQbv`rY0XiR`*|@ln
zPbb9Q#MvaB$y1IVdp5yIayEg-gxPc@Dk(?fi5S#ByJ0_NE0E>TtIu1V;5GBL&9pN0
zZfq5&yf&qkryC<Mk*jgLskE}iNGSUu5=tn&rP~QpJ0TG^0VMSV+n&YXy2OyA<s*dz
z${QmgL2yafZ(&@t&@7+6)yGajNSq?Lq&<sKNYV<ZLgIEyDsX7Z<PtZX3KAHC4uC=_
zn1&>+3M(YEA?r)EJ>>ENEk0vH0_`GjV2BJPvu)TcD&1a5y96dgNTB^VNYd)3a*0d!
z8VNCjTmtO^2`S#6OOn2Q;%&XT#C>|Ckq|RL0`12@Vr6c&bqUo_tT>@ScFY12Y9}ON
zU1>c~E(v9NN_w@NKO+(-7>tAjfCSnF5>i0m{0}5E7GY6PV#T4gC!|+u7f7fk5fTS&
znoDLZ!r{bH<&wegAWANxc7cRy5+QNW07+@#rP?~{I~kQ;t@E(7<iH^T1P-(dB%~lD
z4%##%GZx`B@9HpcprTOVK)XOf3PR$bO+zwc5nhiK;ZqiXOQ4DiB&2vBl2RqJim=_R
z|9G|QdLV41pa&wFBM&JEI6yJ(5W66$04p3W&2wVy#s*#y2VBoMNI*E8Vj7O5;;V4D
z6t8iRAmV^##z6wY;S|$wByIH+4wuU`4iZEh(9AeUKscOY8jhq2qi{g-k0sZFg#-}?
zG&2qo5Dur9f+Ouvk#US1DVA-6e>>FK$K#-8#32NLBParn_DdvOlG!cImP)N_$drEd
z(Txp)hy$)?93&tdPB9I~Y^juv&@otqAmVU&(9FX)NI*E8VhRr1d8O=hIao$Eb)|Xe
z7*RAM4!RyT{?G;BD9d;b7TZV8goC!I=nf(9?x1GGAq3!#pcuzd7GY;)tvt%(`S;zv
zQ^{Y3e!9DV{8oN=<GcNea`|m&`vsr3q_BT_+}<1-?9`=i2tTUi{(igb^{z7fR`lhN
z^3YZIrpWz^)9X)D<CAZvI-!crGu8Q~RMtu2XT2964?pVNO=s~jKDRHDPC4ty>Z-(8
zR#&CAWp!0fL36rl|5ZJ!tMZ@ASzVP-&g!a!N>*2C7PSMfw7W{PX!}`m7OOOi+Ua5v
zE33k3U8PNMWj_~@gT4HNR9;tUF=*d&<V?^$TF&Yz@1C+iZPK3JCe2bC`;D;wWzMg^
F{0ApXMoRzy

diff --git a/spec/gems/client_spec.rb b/spec/gems/client_spec.rb
index 76e75c6..138d97b 100644
--- a/spec/gems/client_spec.rb
+++ b/spec/gems/client_spec.rb
@@ -41,7 +41,7 @@
     end
   end
 
-  describe "#dependencies" do
+  describe "#downloads" do
     before do
       stub_get("/api/v1/versions/coulda-0.6.3/downloads.json").to_return(:body => fixture("downloads.json"))
     end
@@ -52,4 +52,16 @@
       downloads["2011-11-01"].should == 0
     end
   end
+
+  describe "#dependencies" do
+    before do
+      stub_get("/api/v1/dependencies?gems=rails,thor").to_return(:body => fixture("dependencies.json"))
+    end
+
+    it "should return an array of hashes for all versions of given gems" do
+      dependencies = @client.dependencies 'rails', 'thor'
+      a_get("/api/v1/dependencies?gems=rails,thor").should have_been_made
+      dependencies.first.number.should == "3.0.9"
+    end
+  end
 end