redd 0.8.0.pre.1 → 0.8.0.pre.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: bc08060b2e7111adfd18a83c7d23e464f20a92dc
-  data.tar.gz: 79897c294f7f612e90acee7184b91a4982d54cb6
+  metadata.gz: 69d6cf992f71356b1db86a000bdc3be4508e351c
+  data.tar.gz: 01d0b098fb7c26a91e34ff001310417c93e3bd74
 SHA512:
-  metadata.gz: 0ccfa94a4923e3748931c6ca5e5515f23c10bd23d979c1ef79e5f3f62a87c78ff66ed01f72033d95d3da3e8ca0e47cf9ebd412d758d3b751d3d935545abd787c
-  data.tar.gz: f2881316388b4b3b1183aebdc867184aa64750c57ac29cfb6b17532ef099f0c4de309f064f3160a3eb670ee4d70d91cee335367b9c43a34c5f70eb3254781743
+  metadata.gz: 4019ce12ee1cfac8927642bfdd327e4221835842a230ad73bce35b0ce3e1f0445523bf5a796aeca182b63c7061861ba1a28a587fe8814594d6d89531ece2c3a6
+  data.tar.gz: 292775ce9121b4de029ee49e451417508b64524940043569ebe53b807387fac0b1539a7cf5e37a13d494c83267421a1789a75e00ec96cadbaa2cf01128c0f9b7

data/.rubocop.yml

@@ -4,6 +4,10 @@ AllCops:
 Metrics/LineLength:
   Max: 100
 
+# Dumb, but also brand new.
+Style/NumericPredicate:
+  Enabled: false
+
 # Allows for raising Redd::ResponseErrors with a Response object.
 Style/RaiseArgs:
   Enabled: false

data/TODO.md

@@ -1,11 +1,7 @@
 # v0.8.0 checklist
 
-- [ ] Make v0.8.0 feature-complete with v0.7.x
-  - [ ] `Messageable`
-  - [ ] `MoreComments` and expansion
 - [ ] make rubocop fail on error
 - [ ] unduplicate duplicated code
 - [ ] Write tests (oh boy)
-- [ ] More comprehensive error checking
-- [ ] A `FrontPage` object
+- [ ] abstract out listing methods?
 - [ ] Add `#==` and `#to_s` methods to models

data/bin/console

@@ -37,7 +37,7 @@ server.mount_proc '/authenticate' do |_, res|
       state: '0',
       redirect_uri: 'http://localhost:8000/redirect',
       'scope': %w(identity read subscribe privatemessages wikiread submit vote edit modposts history
-                  modflair)
+                  modflair modconfig)
     )
   )
 end

data/lib/redd/api_client.rb

@@ -64,37 +64,21 @@ module Redd
       @unmarshaller.unmarshal(object)
     end
 
-    def model(verb, path, params = {})
+    def model(verb, path, options = {})
       # XXX: make unmarshal explicit in methods?
-      unmarshal(send(verb, path, params).body)
-    end
-
-    private
-
-    # Makes sure a valid access is present, raising an error if nil
-    def ensure_access_is_valid
-      # Authenticate first if auto_login is enabled
-      authenticate if @access.nil? && @auto_login
-      # Refresh access if auto_refresh is enabled
-      refresh if @access.expired? && @auto_refresh
-      # Fuck it, panic
-      raise 'client access is nil, try calling #authenticate' if @access.nil?
-    end
-
-    def connection
-      super.auth("Bearer #{@access.access_token}")
+      unmarshal(request(verb, path, options).body)
     end
 
     # Makes a request, ensuring not to break the rate limit by sleeping.
     # @see Client#request
-    def request(verb, path, params: {}, form: {})
+    def request(verb, path, raw: false, params: {}, **options)
       # Make sure @access is populated by a valid access
       ensure_access_is_valid
       # Setup base API params and make request
       api_params = { api_type: 'json', raw_json: 1 }.merge(params)
-      response = @rate_limiter.after_limit { super(verb, path, params: api_params, form: form) }
+      response = @rate_limiter.after_limit { super(verb, path, params: api_params, **options) }
       # Check for errors in the returned response
-      response_error = @error_handler.check_error(response)
+      response_error = @error_handler.check_error(response, raw: raw)
       raise response_error unless response_error.nil?
       # All done, return the response
       @failures = 0
@@ -105,5 +89,21 @@ module Redd
       warn "Redd got a #{e.class.name} error (#{e.message}), retrying..."
       retry
     end
+
+    private
+
+    # Makes sure a valid access is present, raising an error if nil
+    def ensure_access_is_valid
+      # Authenticate first if auto_login is enabled
+      authenticate if @access.nil? && @auto_login
+      # Refresh access if auto_refresh is enabled
+      refresh if @access.expired? && @auto_refresh
+      # Fuck it, panic
+      raise 'client access is nil, try calling #authenticate' if @access.nil?
+    end
+
+    def connection
+      super.auth("Bearer #{@access.access_token}")
+    end
   end
 end

data/lib/redd/client.rb

@@ -24,6 +24,19 @@ module Redd
       @user_agent = user_agent
     end
 
+    # Make an HTTP request.
+    # @param verb [:get, :post, :put, :patch, :delete] the HTTP verb to use
+    # @param path [String] the path relative to the endpoint
+    # @param options [Hash] the request parameters
+    # @option options [Hash] :params the parameters to supply with the url
+    # @option options [Hash] :form the parameters to supply in the body
+    # @option options [Hash] :body the direct body contents
+    # @return [Response] the response
+    def request(verb, path, options = {})
+      response = connection.request(verb, path, **options)
+      Response.new(response.status.code, response.headers, response.body.to_s)
+    end
+
     # Make a GET request.
     # @param path [String] the path relative to the endpoint
     # @param options [Hash] the parameters to supply
@@ -73,16 +86,5 @@ module Redd
                           .headers('User-Agent' => @user_agent)
                           .timeout(:per_operation, write: 5, connect: 5, read: 5)
     end
-
-    # Make an HTTP request.
-    # @param verb [:get, :post, :put, :patch, :delete] the HTTP verb to use
-    # @param path [String] the path relative to the endpoint
-    # @param params [Hash] the parameters to supply with the url
-    # @param form [Hash] the parameters to supply in the body
-    # @return [Response] the response
-    def request(verb, path, params: {}, form: {})
-      response = connection.request(verb, path, params: params, form: form)
-      Response.new(response.status.code, response.headers, response.body.to_s)
-    end
   end
 end

data/lib/redd/error.rb

@@ -1,6 +1,17 @@
 # frozen_string_literal: true
 
 module Redd
+  # An error with the API.
+  class APIError < StandardError
+    attr_reader :response, :name
+
+    def initialize(response)
+      @response = response
+      @name, message = response.body[:json][:errors][0]
+      super(message)
+    end
+  end
+
   # Represents an error from reddit returned in a response.
   class ResponseError < StandardError
     attr_accessor :response
@@ -14,6 +25,9 @@ module Redd
   # An error with Redd, probably (let me know!)
   class BadRequest < ResponseError; end
 
+  # Whatever it is, you're not allowed to do it.
+  class Forbidden < ResponseError; end
+
   # You don't have the correct scope to do this.
   class InsufficientScope < ResponseError; end
 

data/lib/redd/models/basic_model.rb

@@ -61,15 +61,15 @@ module Redd
       # @param include_private [Boolean] whether to also include private methods
       # @return [Boolean] whether the method is handled by method_missing
       def respond_to_missing?(method_name, include_private = false)
-        attribute?(method_name) || attribute?(depredicate(method_name)) || super
+        @attributes.key?(method_name) || @attributes.key?(depredicate(method_name)) || super
       end
 
       # Return an attribute or raise a NoMethodError if it doesn't exist.
       # @param method_name [Symbol] the name of the attribute
       # @return [Object] the result of the attribute check
       def method_missing(method_name, *args, &block)
-        return get_attribute(method_name) if attribute?(method_name)
-        return get_attribute(depredicate(method_name)) if attribute?(depredicate(method_name))
+        return get_attribute(method_name) if @attributes.key?(method_name)
+        return get_attribute(depredicate(method_name)) if @attributes.key?(depredicate(method_name))
         super
       end
 
@@ -113,12 +113,6 @@ module Redd
         # Fetch the attribute, raising a KeyError if it doesn't exist.
         @attributes.fetch(name)
       end
-
-      # @param name [Symbol] the name of the attribute to check
-      # @return [Boolean] whether the attribute exists
-      def attribute?(name)
-        @attributes.key?(name)
-      end
     end
   end
 end

data/lib/redd/models/comment.rb

@@ -28,8 +28,10 @@ module Redd
       # @option hash [String] :id the comment's id (e.g. abc123)
       # @return [Comment]
       def self.from_response(client, hash)
+        # FIXME: listings can be empty... (for some reason)
+
         # Ensure we have the comment's id.
-        id = hash.fetch(:id, hash.fetch(:name).tr('t1_', ''))
+        id = hash.fetch(:id) { hash.fetch(:name).tr('t1_', '') }
 
         # If we have the link_id, we can load the listing with replies.
         if hash.key?(:link_id)
@@ -46,6 +48,12 @@ module Redd
           c.get('/api/info', id: "t1_#{id}").body[:data][:children][0][:data]
         end
       end
+
+      private
+
+      def after_initialize
+        @attributes[:replies] = [] if !@attributes.key?(:replies) || @attributes[:replies] == ''
+      end
     end
   end
 end

data/lib/redd/models/lazy_model.rb

@@ -40,6 +40,23 @@ module Redd
         super
       end
 
+      # Checks whether an attribute is supported by method_missing. Since we don't know whether an
+      # attribute exists until we load it, we have to respond true until we load it.
+      # @param method_name [Symbol] the method name or attribute to check
+      # @param include_private [Boolean] whether to also include private methods
+      # @return [Boolean] whether the method is handled by method_missing
+      def respond_to_missing?(method_name, include_private = false)
+        @definitely_fully_loaded ? super : true
+      end
+
+      # Return an attribute or raise a NoMethodError if it doesn't exist.
+      # @param method_name [Symbol] the name of the attribute
+      # @return [Object] the result of the attribute check
+      def method_missing(method_name, *args, &block)
+        ensure_fully_loaded unless @attributes.key?(method_name)
+        super
+      end
+
       private
 
       # Make sure the model is loaded at least once.
@@ -49,12 +66,7 @@ module Redd
 
       # Gets the attribute and loads it if it may be available from the response.
       def get_attribute(name)
-        ensure_fully_loaded unless @attributes.key?(name)
-        super
-      end
-
-      # Checks whether an attribute exists, loading it first if necessary.
-      def attribute?(name)
+        # XXX: Replace get_attribute calls with simple method calls?
         ensure_fully_loaded unless @attributes.key?(name)
         super
       end

data/lib/redd/models/listing.rb

@@ -16,7 +16,7 @@ module Redd
         new(client, hash)
       end
 
-      %i([] each empty? length size).each do |method_name|
+      %i([] each empty? first last).each do |method_name|
         define_method(method_name) do |*args, &block|
           get_attribute(:children).public_send(method_name, *args, &block)
         end

data/lib/redd/models/more_comments.rb

@@ -1,10 +1,31 @@
 # frozen_string_literal: true
 
-require_relative 'lazy_model'
+require_relative 'basic_model'
 
 module Redd
   module Models
-    class MoreComments < LazyModel
+    # An object that represents a bunch of comments that need to be expanded.
+    class MoreComments < BasicModel
+      # Expand the object's children into a listing of Comments and MoreComments.
+      # @param link [String] the fullname of the submission the object belongs to
+      # @param sort [String] the sort order of the submission
+      # @return [Listing<Comment, MoreComments>] the expanded children
+      def expand(link:, sort: 'best')
+        @client.model(
+          :get, '/api/morechildren',
+          link_id: link,
+          children: get_attribute(:children).join(','),
+          sort: sort
+        )
+      end
+
+      # Keep expanding until all top-level MoreComments are converted to comments
+      def recursive_expand(link:, sort: 'best')
+        # FIXME: this returns a flattened listing of comments and doesn't preserve the structure
+        expand(link: link, sort: sort).flat_map do |o|
+          o.is_a?(MoreComments) && o.count > 0 ? o.recursive_expand(link: link, sort: sort) : [o]
+        end
+      end
     end
   end
 end

data/lib/redd/models/searchable.rb

@@ -0,0 +1,35 @@
+# frozen_string_literal: true
+
+module Redd
+  module Models
+    # Applied to {Session} for site-wide and {Subreddit} for subreddit-specific search.
+    module Searchable
+      # Search reddit.
+      # @see https://www.reddit.com/wiki/search
+      #
+      # @param query [String] the search query
+      # @param params [Hash] the search params
+      # @option params [:cloudsearch, :lucene, :plain] :syntax the query's syntax
+      # @option params [String] :after return results after the given fullname
+      # @option params [String] :before return results before the given fullname
+      # @option params [Integer] :count the number of items already seen in the listing
+      # @option params [1..100] :limit the maximum number of things to return
+      # @option params [:hour, :day, :week, :month, :year, :all] :time the time period to restrict
+      #   search results by
+      # @option params [:relevance, :hot, :top, :new, :comments] :sort the sort order of results
+      # @option params [String] :restrict_to restrict by subreddit (prefer {Subreddit#search})
+      # @return [Listing<Comment, Submission>] the search results
+      def search(query, **params)
+        params[:q] = query if query
+        params[:t] = params.delete(:time) if params.key?(:time)
+        if params[:restrict_to]
+          subreddit = params.delete(:restrict_to)
+          params[:restrict_sr] = true
+          @client.model(:get, "/r/#{subreddit}/search", params)
+        else
+          @client.model(:get, '/search', params)
+        end
+      end
+    end
+  end
+end

data/lib/redd/models/session.rb

@@ -1,11 +1,14 @@
 # frozen_string_literal: true
 
 require_relative 'lazy_model'
+require_relative 'searchable'
 
 module Redd
   module Models
     # The starter class.
     class Session < BasicModel
+      include Searchable
+
       # @return [FrontPage] the user's front page
       def front_page
         FrontPage.new(@client)
@@ -45,7 +48,7 @@ module Redd
       # Get submissions or comments by their fullnames.
       # @param fullnames [String, Array<String>] one or an array of fullnames (e.g. t3_abc1234)
       # @return [Listing<Submission, Comment>]
-      def get(fullnames)
+      def from_ids(fullnames)
         # XXX: Could we use better methods for t1_ and t3_?
         @client.model(:get, '/api/info', id: Array(fullnames).join(','))
       end

data/lib/redd/models/submission.rb

@@ -25,16 +25,68 @@ module Redd
       def self.from_response(client, hash)
         link_id = hash.fetch(:id)
         new(client, hash) do |c|
-          # `details` is a pair (2-element array):
-          #   - details[0] is a one-item listing containing the submission
-          #   - details[1] is listing of comments
-          details = c.get("/comments/#{link_id}").body
-          comments = details[1][:data][:children].map do |comment_object|
-            Comment.from_response(c, comment_object[:data])
-          end
-          details[0][:data][:children][0][:data].merge(comments: comments)
+          # `data` is a pair (2-element array):
+          #   - data[0] is a one-item listing containing the submission
+          #   - data[1] is listing of comments
+          data = c.get("/comments/#{link_id}").body
+          data[0][:data][:children][0][:data].merge(comments: c.unmarshal(data[1]))
         end
       end
+
+      # Mark the link as "Not Suitable For Work".
+      def mark_as_nsfw
+        @client.get('/api/marknsfw', id: get_attribute(:name))
+        @attributes[:over_18] = true
+      end
+
+      # No longer mark the link as "Not Suitable For Work".
+      def unmark_as_nsfw
+        @client.get('/api/unmarknsfw', id: get_attribute(:name))
+        @attributes[:over_18] = false
+      end
+
+      # Mark the link as a spoiler.
+      def mark_as_spoiler
+        @client.get('/api/spoiler', id: get_attribute(:name))
+        @attributes[:spoiler] = true
+      end
+
+      # No longer mark the link as a spoiler.
+      def unmark_as_spoiler
+        @client.get('/api/unspoiler', id: get_attribute(:name))
+        @attributes[:spoiler] = false
+      end
+
+      # Set the submission to "contest mode" (comments are randomly sorted)
+      def enable_contest_mode
+        @client.post('/api/set_contest_mode', id: get_attribute(:name), state: true)
+      end
+
+      # Disable the "contest mode".
+      def disable_contest_mode
+        @client.post('/api/set_contest_mode', id: get_attribute(:name), state: false)
+      end
+
+      # Set the submission as the sticky post of the subreddit.
+      # @param slot [1, 2] which "slot" to place the sticky on
+      def make_sticky(slot: nil)
+        @client.post('/api/set_subreddit_sticky', id: get_attribute(:name), num: slot, state: true)
+      end
+
+      # Unsticky the post from the subreddit.
+      def remove_sticky
+        @client.post('/api/set_subreddit_sticky', id: get_attribute(:name), state: false)
+      end
+
+      # Prevent users from commenting on the link (and hide it as well).
+      def lock
+        @client.post('/api/lock', id: get_attribute(:name))
+      end
+
+      # Allow users to comment on the link again.
+      def unlock
+        @client.post('/api/unlock', id: get_attribute(:name))
+      end
     end
   end
 end

data/lib/redd/models/subreddit.rb

@@ -2,6 +2,7 @@
 
 require_relative 'lazy_model'
 require_relative 'messageable'
+require_relative 'searchable'
 require_relative '../utilities/stream'
 
 module Redd
@@ -9,6 +10,24 @@ module Redd
     # A subreddit.
     class Subreddit < LazyModel
       include Messageable
+      include Searchable
+
+      # A list of keys that are required by #modify_settings.
+      SETTINGS_KEYS = %i(allow_images allow_top collapse_deleted_comments comment_score_hide_mins
+                         description exclude_banned_modqueue header-title hide_ads lang link_type
+                         over_18 public_description public_traffic show_media show_media_preview
+                         spam_comments spam_links spam_selfposts spoilers_enabled submit_link_label
+                         submit_text submit_text_label suggested_comment_sort theme_sr
+                         theme_sr_update title type wiki_edit_age wiki_edit_karma wikimode).freeze
+
+      # A mapping from keys returned by #settings to keys required by #modify_settings
+      SETTINGS_MAP = {
+        subreddit_type: :type,
+        language: :lang,
+        content_options: :link_type,
+        default_set: :allow_top,
+        header_hover_text: :'header-title'
+      }.freeze
 
       # Make a Subreddit from its name.
       # @option hash [String] :display_name the subreddit's name
@@ -38,6 +57,15 @@ module Redd
         WikiPage.from_response(@client, title: title, subreddit: self)
       end
 
+      # Search a subreddit.
+      # @param query [String] the search query
+      # @param params [Hash] refer to {Searchable} to see search parameters
+      # @see Searchable#search
+      def search(query, **params)
+        restricted_params = { restrict_to: get_attribute(:display_name) }.merge(params)
+        super(query, restricted_params)
+      end
+
       # @!group Listings
 
       # Get the appropriate listing.
@@ -54,7 +82,7 @@ module Redd
       # @return [Listing<Submission, Comment>]
       def listing(sort, **params)
         params[:t] = params.delete(:time) if params.key?(:time)
-        @client.model(:get, "/r/#{get_attribute(:display_name)}/#{sort}.json", params)
+        @client.model(:get, "/r/#{get_attribute(:display_name)}/#{sort}", params)
       end
 
       # @!method hot(**params)
@@ -83,7 +111,7 @@ module Redd
       #
       # @return [Listing<Submission, Comment>]
       def moderator_listing(type, **params)
-        @client.model(:get, "/r/#{get_attribute(:display_name)}/about/#{type}.json", params)
+        @client.model(:get, "/r/#{get_attribute(:display_name)}/about/#{type}", params)
       end
 
       # @!method reports(**params)
@@ -97,6 +125,38 @@ module Redd
         define_method(type) { |**params| moderator_listing(type, **params) }
       end
 
+      # @!endgroup
+      # @!group Relationship Listings
+
+      # Get the appropriate relationship listing.
+      # @param type [:banned, :muted, :wikibanned, :contributors, :wikicontributors, :moderators]
+      #   the type of listing
+      # @param params [Hash] a list of params to send with the request
+      # @option params [String] :after return results after the given fullname
+      # @option params [String] :before return results before the given fullname
+      # @option params [Integer] :count the number of items already seen in the listing
+      # @option params [1..100] :limit the maximum number of things to return
+      # @option params [String] :user find a specific user
+      #
+      # @return [Array<Hash>]
+      def relationship_listing(type, **params)
+        # TODO: add methods to determine if a certain user was banned/muted/etc
+        user_list = @client.get("/r/#{get_attribute(:display_name)}/about/#{type}", params).body
+        user_list[:data][:children]
+      end
+
+      # @!method banned(**params)
+      # @!method muted(**params)
+      # @!method wikibanned(**params)
+      # @!method contributors(**params)
+      # @!method wikicontributors(**params)
+      # @!method moderators(**params)
+      #
+      # @see #relationship_listing
+      %i(banned muted wikibanned contributors wikicontributors moderators).each do |type|
+        define_method(type) { |**params| relationship_listing(type, **params) }
+      end
+
       # @!endgroup
 
       # Stream newly submitted posts.
@@ -196,6 +256,50 @@ module Redd
       def unsubscribe
         subscribe(action: :unsub)
       end
+
+      # Get the subreddit's CSS.
+      # @return [String, nil] the stylesheet or nil if no stylesheet exists
+      def stylesheet
+        url = @client.get("/r/#{get_attribute(:display_name)}/stylesheet").headers['location']
+        HTTP.get(url).body.to_s
+      rescue Redd::NotFound
+        nil
+      end
+
+      # Edit the subreddit's stylesheet.
+      # @param text [String] the updated CSS
+      # @param reason [String] the reason for modifying the stylesheet
+      def update_stylesheet(text, reason: nil)
+        params = { op: 'save', stylesheet_contents: text }
+        params[:reason] = reason if reason
+        @client.post("/r/#{get_attribute(:display_name)}/api/subreddit_stylesheet", params)
+      end
+
+      # @return [Hash] the subreddit's settings
+      def settings
+        @client.get("/r/#{get_attribute(:display_name)}/about/edit").body[:data]
+      end
+
+      # Modify the subreddit's settings.
+      # @param params [Hash] the settings to change
+      # @see https://www.reddit.com/dev/api#POST_api_site_admin
+      def modify_settings(**params)
+        full_params = settings.merge(params)
+        full_params[:sr] = get_attribute(:name)
+        SETTINGS_MAP.each { |src, dest| full_params[dest] = full_params.delete(src) }
+        @client.post('/api/site_admin', full_params)
+      end
+
+      private
+
+      def add_relationship(**params)
+        # FIXME: add public methods
+        @client.post("/r/#{get_attribute(:display_name)}/api/friend", params)
+      end
+
+      def remove_relationship(**params)
+        @client.post("/r/#{get_attribute(:display_name)}/api/unfriend", params)
+      end
     end
   end
 end

data/lib/redd/models/user.rb

@@ -34,6 +34,20 @@ module Redd
         super(to: get_attribute(:name), subject: subject, text: text, from: from)
       end
 
+      # Add the user as a friend.
+      # @param note [String] a note for the friend
+      def friend(note = nil)
+        name = get_attribute(:name)
+        body = JSON.generate(note ? { name: name, note: note } : { name: name })
+        @client.request(:put, "/api/v1/me/friends/#{name}", body: body)
+      end
+
+      # Unfriend the user.
+      def unfriend
+        name = get_attribute(:name)
+        @client.request(:delete, "/api/v1/me/friends/#{name}", raw: true, form: { id: name })
+      end
+
       # Get the appropriate listing.
       # @param type [:overview, :submitted, :comments, :liked, :disliked, :hidden, :saved, :gilded]
       #   the type of listing to request

data/lib/redd/utilities/error_handler.rb

@@ -5,10 +5,10 @@ require_relative '../error'
 module Redd
   module Utilities
     # Handles response errors in API responses.
-    # TODO: handle [:json][:errors] array
     class ErrorHandler
       HTTP_ERRORS = {
         400 => Redd::BadRequest,
+        403 => Redd::Forbidden,
         404 => Redd::NotFound,
         500 => Redd::ServerError,
         502 => Redd::ServerError,
@@ -21,14 +21,19 @@ module Redd
         'invalid_token' => Redd::InvalidAccess
       }.freeze
 
-      def check_error(response)
-        return HTTP_ERRORS[response.code].new(response) if HTTP_ERRORS.key?(response.code)
-        if response.code == 403
+      def check_error(response, raw:)
+        # TODO: deal with errors of type { fields:, explanation:, message:, reason: }
+        if !raw && response.body[:json] && response.body[:json][:errors] &&
+           !response.body[:json][:errors].empty?
+          Redd::APIError.new(response)
+        elsif HTTP_ERRORS.key?(response.code)
+          HTTP_ERRORS[response.code].new(response)
+        elsif response.code == 401
           AUTHORIZATION_ERRORS.each do |key, klass|
-            return klass.new(response) if response.headers['www-authenticate'].include?(key)
+            auth_header = response.headers['www-authenticate']
+            return klass.new(response) if auth_header && auth_header.include?(key)
           end
         end
-        nil
       end
     end
   end

data/lib/redd/utilities/stream.rb

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require 'enumerator'
-
 module Redd
   module Utilities
     # A forward-expading listing of items that can be enumerated forever.

data/lib/redd/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Redd
-  VERSION = '0.8.0.pre.1'
+  VERSION = '0.8.0.pre.2'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: redd
 version: !ruby/object:Gem::Version
-  version: 0.8.0.pre.1
+  version: 0.8.0.pre.2
 platform: ruby
 authors:
 - Avinash Dwarapu
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2017-02-25 00:00:00.000000000 Z
+date: 2017-02-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: http
@@ -163,6 +163,7 @@ files:
 - lib/redd/models/postable.rb
 - lib/redd/models/private_message.rb
 - lib/redd/models/replyable.rb
+- lib/redd/models/searchable.rb
 - lib/redd/models/session.rb
 - lib/redd/models/submission.rb
 - lib/redd/models/subreddit.rb