GetSummary should raise

lib/campaign_monitor/campaign.rb

@@ -193,17 +193,19 @@ def method_missing(m, *args)
       end
     end
 
-    # Calls Campaign.GetSummary.  Often you probably should just use Campaign#summary
-    # It will return true if successful and false if not.
+    # Calls Campaign.GetSummary.  OYou probably should just use Campaign#summary which caches results
+    # It will raise ApiError if an error occurs
     # Campaign#result will have the result of the API call
     #
     # Example
     #   @camp=@client.campaigns.first
-    #   @camp.GetSummary
+    #   @camp.GetSummary["Clicks"]
     def GetSummary
-      @result=Result.new(cm_client.Campaign_GetSummary('CampaignID' => self.id))
-      @summary=parse_summary(@result.raw) if @result.success?
-      @result.success?
+      handle_response(cm_client.Campaign_GetSummary('CampaignID' => self.id)) do |response|
+        @result=Result.new(response)
+        @summary=parse_summary(@result.raw)
+      end
+      @summary
     end
 
     # Convenience method for accessing summary details of a campaign

test/campaign_test.rb

@@ -37,9 +37,10 @@ def test_bracket_lookup_for_nonexistant
     assert_not_nil @campaign
     assert_equal 12345, @campaign.id
     assert_raises( CampaignMonitor::ApiError ) { @campaign.lists }
+    assert_raises( CampaignMonitor::ApiError ) { @campaign.GetSummary }
   end
 
-  def test_braket_lookup_for_existing
+  def test_bracket_lookup_for_existing
     camp=CampaignMonitor::Campaign[@campaign.id]
     assert_not_nil camp
     camp.lists