pexels 0.2.1 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1fa972e637620e24f2466be1fa4dea4c68f54e0deae50a40b86adfd6148f6a23
-  data.tar.gz: b9892dde37db133e3d1e94113a9f00b7a74c21e4a6ac760bd226fcb1160b9d69
+  metadata.gz: fbc208fe2d27e3dca7c1b94e0e2a3901334d06b9cf3460d952a44aea437bd82e
+  data.tar.gz: f4796400297cf53c41b0740eb57ae9b70efd697018d20c17c0e91f92c9030dc6
 SHA512:
-  metadata.gz: c823ada102ccde2f46e9fc8c3d1f82cc8c8914423558ac17f3bcd4168ba1008d83c8f49a479389cd2f1eea2f19b3096eac92cd6d2262c4507439cff5d7450088
-  data.tar.gz: 29c54e8c7413658efd570604ffe08a024c4af08865d9c3bf3002dcf74c9474d7c4e025924ab239974ce3c2e8ceb3b917ab68ce52084c95f5222b0bbadcdaf023
+  metadata.gz: e050b1e25f4f66b370f1acd2695af58305cd221e7c7d9a67cf4f989bdc3959af172aaebc65b545507992af0a6d24594e19601f316f5b1e2145ebbbc59eae46ce
+  data.tar.gz: 5a334832f860fcbb21dde178bdd871491d70efdeccfa5515309186596fcd1884b4ca3884bd6a6574343daa40623524ce20137366096460a015372a09365020aa

data/CHANGES.md

@@ -1,5 +1,9 @@
 # Change log
 
+## 0.3.0
+* Add support for photo and video search with filters.
+* Added `avg_color` attribute to `Photo` object.
+
 ## 0.2.1
 * Added `type`, `photo?` and `video?` helper methods to `Photo` and `Video` classes.
 

data/lib/pexels.rb

@@ -25,6 +25,7 @@ end
 require_relative 'pexels/client'
 require_relative 'pexels/client/request'
 require_relative 'pexels/client/response'
+require_relative 'pexels/client/search_filters'
 require_relative 'pexels/client/collections'
 require_relative 'pexels/client/photos'
 require_relative 'pexels/client/videos'

data/lib/pexels/client/photos.rb

@@ -1,38 +1,56 @@
-class Pexels::Client::Photos
+module Pexels
+  class Client
+    class Photos
+      include SearchFilters
 
-  def initialize(client)
-    @client = client
-  end
+      def initialize(client)
+        @client = client
+      end
 
-  def [](id)
-    response = @client.request("#{Pexels.api_version}/photos/#{id}")
-    Pexels::Photo.new(response.body)
-  end
-  alias_method :find, :[]
-
-  def search(query, per_page: 15, page: 1, locale: 'en-US')
-    response = @client.request(
-      "#{Pexels.api_version}/search",
-      params: {
-        query: query,
-        per_page: per_page,
-        page: page,
-        locale: locale
-      }
-    )
-
-    Pexels::PhotoSet.new(response)
-  end
+      def [](id)
+        response = @client.request("#{Pexels.api_version}/photos/#{id}")
+        Pexels::Photo.new(response.body)
+      end
+      alias_method :find, :[]
+
+      def search(query, per_page: 15, page: 1, locale: 'en-US', orientation: nil, size: nil, color: nil)
+        validate_search_params(orientation, size, color)
+
+        response = @client.request(
+          "#{Pexels.api_version}/search",
+          params: {
+            query: query,
+            per_page: per_page,
+            page: page,
+            locale: locale,
+            orientation: orientation,
+            size: size,
+            color: color
+          }.compact
+        )
+
+        Pexels::PhotoSet.new(response)
+      end
+
+      def curated(per_page: 15, page: 1)
+        response = @client.request(
+          "#{Pexels.api_version}/curated",
+          params: {
+            per_page: per_page,
+            page: page
+          }
+        )
+
+        Pexels::PhotoSet.new(response)
+      end
 
-  def curated(per_page: 15, page: 1)
-    response = @client.request(
-      "#{Pexels.api_version}/curated",
-      params: {
-        per_page: per_page,
-        page: page,
-      }
-    )
+      private
 
-    Pexels::PhotoSet.new(response)
+      def validate_search_params(orientation, size, color)
+        validate_orientation(orientation) &&
+          validate_size(size) &&
+          validate_color(color)
+      end
+    end
   end
 end

data/lib/pexels/client/search_filters.rb

@@ -0,0 +1,31 @@
+module Pexels
+  class Client
+    module SearchFilters
+      ORIENTATIONS = %w(portrait landscape square).freeze
+      SIZES = %w(small medium large).freeze
+      COLORS = %w(red orange yellow green turquoise blue violet pink brown black gray white).freeze
+
+      def validate_orientation(orientation)
+        return true unless orientation
+        return true if ORIENTATIONS.include?(orientation.to_s)
+
+        raise ArgumentError, "`orientation` must be one of #{ORIENTATIONS.join(', ')}."
+      end
+
+      def validate_size(size)
+        return true unless size
+        return true if SIZES.include?(size.to_s)
+
+        raise ArgumentError, "`size` must be one of #{SIZES.join(', ')}."
+      end
+
+      def validate_color(color)
+        return true unless color
+        return true if COLORS.include?(color.to_s)
+        return true if color.to_s =~ /\A#?(?:[0-9a-f]{3}){1,2}\z/i
+
+        raise ArgumentError, "`color` must be one of #{COLORS.join(', ')} or a valid hex code."
+      end
+    end
+  end
+end

data/lib/pexels/client/videos.rb

@@ -1,37 +1,53 @@
-class Pexels::Client::Videos
+module Pexels
+  class Client
+    class Videos
+      include SearchFilters
 
-  def initialize(client)
-    @client = client
-  end
+      def initialize(client)
+        @client = client
+      end
 
-  def [](id)
-    response = @client.request("/videos/videos/#{id}")
-    Pexels::Video.new(response.body)
-  end
-  alias_method :find, :[]
-
-  def search(query, per_page: 15, page: 1)
-    response = @client.request(
-      '/videos/search',
-      params: {
-        query: query,
-        per_page: per_page,
-        page: page,
-      }
-    )
-
-    Pexels::VideoSet.new(response)
-  end
+      def [](id)
+        response = @client.request("/videos/videos/#{id}")
+        Pexels::Video.new(response.body)
+      end
+      alias_method :find, :[]
+
+      def search(query, per_page: 15, page: 1, orientation: nil, size: nil)
+        validate_search_params(orientation, size)
+
+        response = @client.request(
+          '/videos/search',
+          params: {
+            query: query,
+            per_page: per_page,
+            page: page,
+            orientation: orientation,
+            size: size
+          }.compact
+        )
+
+        Pexels::VideoSet.new(response)
+      end
+
+      def popular(per_page: 15, page: 1)
+        response = @client.request(
+          '/videos/popular',
+          params: {
+            per_page: per_page,
+            page: page,
+          }
+        )
+
+        Pexels::VideoSet.new(response)
+      end
 
-  def popular(per_page: 15, page: 1)
-    response = @client.request(
-      '/videos/popular',
-      params: {
-        per_page: per_page,
-        page: page,
-      }
-    )
+      private
 
-    Pexels::VideoSet.new(response)
+      def validate_search_params(orientation, size)
+        validate_orientation(orientation) &&
+          validate_size(size)
+      end
+    end
   end
 end

data/lib/pexels/photo.rb

@@ -4,8 +4,8 @@ class Pexels::Photo
               :height,
               :url,
               :user,
-              :src
-
+              :src,
+              :avg_color
 
   def initialize(attrs)
     @id = attrs.fetch('id')
@@ -18,6 +18,7 @@ class Pexels::Photo
       url: attrs.fetch('photographer_url')
     )
     @src = attrs.fetch('src')
+    @avg_color = attrs.fetch('avg_color')
 
   rescue KeyError => exception
     raise Pexels::MalformedAPIResponseError.new(exception)

data/lib/pexels/version.rb

@@ -1,3 +1,3 @@
 module Pexels
-  VERSION = '0.2.1'
+  VERSION = '0.3.0'
 end

data/test/collection_test.rb

@@ -74,10 +74,9 @@ class TestCollections < Minitest::Test
   end
 
   def test_invalid_get_collection
-    @client.collections['this-is-not-a-valid-id']
-    raise 'This should not happen'
-  rescue StandardError => exception
-    assert exception.is_a? Pexels::APIError
-    assert_equal exception.message, 'Not Found'
+    error = assert_raises(Pexels::APIError) do
+      @client.collections['this-is-not-a-valid-id']
+    end
+    assert error.message, 'Not Found'
   end
 end

data/test/photo_test.rb

@@ -60,6 +60,7 @@ class TestPhoto < Minitest::Test
     assert_equal photo.user.url, @photo.user.url
     assert_equal photo.user.id, @photo.user.id
     assert_equal photo.src, @photo.src
+    assert_equal photo.avg_color, @photo.avg_color
 
     assert photo.photo?
     assert_equal photo.type, 'Photo'
@@ -67,10 +68,43 @@ class TestPhoto < Minitest::Test
   end
 
   def test_invalid_get_photo
-    photo = @client.photos['this-is-not-a-valid-id']
-    raise 'This should not happen'
-    rescue StandardError => exception
-      assert exception.is_a? Pexels::APIError
-      assert_equal exception.message, 'Not Found'
+    error = assert_raises(Pexels::APIError) do
+      @client.photos['this-is-not-a-valid-id']
+    end
+    assert_equal 'Not Found', error.message
+  end
+
+  def test_invalid_orientation
+    error = assert_raises(ArgumentError) do
+      @client.photos.search('dog', orientation: 'foo')
+    end
+    assert_match '`orientation` must be one of', error.message
+  end
+
+  def test_invalid_size
+    error = assert_raises(ArgumentError) do
+      @client.photos.search('dog', size: 'foo')
+    end
+    assert_match '`size` must be one of', error.message
+  end
+
+  def test_invalid_color
+    error = assert_raises(ArgumentError) do
+      @client.photos.search('dog', color: 'foo')
+    end
+    assert_match '`color` must be one of', error.message
+  end
+
+  def test_invalid_color_hex
+    error = assert_raises(ArgumentError) do
+      @client.photos.search('dog', color: '#gggggg')
+    end
+    assert_match '`color` must be one of', error.message
+  end
+
+  def test_search_filters
+    search_result = @client.photos.search('dog', color: '#FF0000', size: :large, orientation: :square)
+    assert_kind_of Pexels::PhotoSet, search_result
+    assert search_result.photos.any?
   end
 end

data/test/video_test.rb

@@ -74,10 +74,29 @@ class TestVideo < Minitest::Test
   end
 
   def test_invalid_get_video
-    video = @client.videos['this-is-not-a-valid-id']
-    raise 'This should not happen'
-    rescue StandardError => exception
-      assert exception.is_a? Pexels::APIError
-      assert_equal exception.message, 'Not Found'
+    error = assert_raises(Pexels::APIError) do
+      @client.videos['this-is-not-a-valid-id']
+    end
+    assert_equal 'Not Found', error.message
+  end
+
+  def test_invalid_orientation
+    error = assert_raises(ArgumentError) do
+      @client.photos.search('dog', orientation: 'foo')
+    end
+    assert_match '`orientation` must be one of', error.message
+  end
+
+  def test_invalid_size
+    error = assert_raises(ArgumentError) do
+      @client.photos.search('dog', size: 'foo')
+    end
+    assert_match '`size` must be one of', error.message
+  end
+
+  def test_search_filters
+    search_result = @client.videos.search('cat', size: :medium, orientation: :square)
+    assert_kind_of Pexels::VideoSet, search_result
+    assert search_result.videos.any?
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: pexels
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.3.0
 platform: ruby
 authors:
 - Pexels dev team
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-04-15 00:00:00.000000000 Z
+date: 2021-04-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: requests
@@ -44,6 +44,7 @@ files:
 - lib/pexels/client/photos.rb
 - lib/pexels/client/request.rb
 - lib/pexels/client/response.rb
+- lib/pexels/client/search_filters.rb
 - lib/pexels/client/videos.rb
 - lib/pexels/collection.rb
 - lib/pexels/collection_media_set.rb