asset_cloud 2.7.0 → 2.7.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8573a0d845bd6022d3540cbb6e28027637bea815896060a6637ed70402fb0614
-  data.tar.gz: df5f502bd556e2414ee896327ba18c54ce679faa2a2ebcf5b50a630803454719
+  metadata.gz: 568dd22b66b6a00beacf0a8d099cad040bcddac422b3932b946573d823e67291
+  data.tar.gz: 8e97aa129cc5c6617722a6f5212ff51562cee437ba22476ceb63d7ad9dc62d25
 SHA512:
-  metadata.gz: 7302dd7d0e4e4ca392b344c530d3ebd3e610c25e5e6aa45658582944e2333ca66371e511e911f93e38f471815fb184a7d2c52d38dadfdda172a33ff7ff90e6ec
-  data.tar.gz: 91a7fcfaf2dddbb1b855d6b25cadd563c414e09a625ceafdaaecf68faf07d4b89a6e80b89ab9cd2f264098f3ab9f1f387cf08f8ec8bcde19b67179045989bf40
+  metadata.gz: 3cde8568299bb0b63c31ce9affa4880bcc7aa1940a63d1541210289f25dc7766528846d110a9b00ae7d1a412bc5bb2e695294b4bff851fb7aa0cc090856ed2bb
+  data.tar.gz: 821d7f7f2fe4c37df691d52b40217168638ff2d2d5cc3ee2dd056ec231e30f44584b1dbd538dc98d167d07d1a29ac0948bbde798d1b7f1870324f593090ecf88

data/.github/workflows/ci.yml

@@ -0,0 +1,42 @@
+name: CI
+
+on: [push, pull_request]
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    name: Ruby ${{ matrix.ruby }}
+    strategy:
+      fail-fast: false
+      matrix:
+        ruby: ["2.5", "2.6", "2.7", "3.0", "3.1", "3.2"]
+    steps:
+
+    - name: Check out code
+      uses: actions/checkout@v3
+
+    - name: Set up Ruby ${{ matrix.ruby }}
+      uses: ruby/setup-ruby@v1
+      with:
+        ruby-version: ${{ matrix.ruby }}
+        bundler-cache: true
+
+    - name: Tests
+      run: bundle exec rspec
+
+  lint:
+    runs-on: ubuntu-latest
+    name: RuboCop
+    steps:
+
+    - name: Check out code
+      uses: actions/checkout@v3
+
+    - name: Set up Ruby
+      uses: ruby/setup-ruby@v1
+      with:
+        ruby-version: '3.2'
+        bundler-cache: true
+
+    - name: RuboCop
+      run: bundle exec rubocop

data/.github/workflows/cla.yml

@@ -0,0 +1,22 @@
+name: Contributor License Agreement (CLA)
+
+on:
+  pull_request_target:
+    types: [opened, synchronize]
+  issue_comment:
+    types: [created]
+
+jobs:
+  cla:
+    runs-on: ubuntu-latest
+    if: |
+      (github.event.issue.pull_request 
+        && !github.event.issue.pull_request.merged_at
+        && contains(github.event.comment.body, 'signed')
+      ) 
+      || (github.event.pull_request && !github.event.pull_request.merged)
+    steps:
+      - uses: Shopify/shopify-cla-action@v1
+        with:
+          github-token: ${{ secrets.GITHUB_TOKEN }}
+          cla-token: ${{ secrets.CLA_TOKEN }}

data/.rubocop.yml

@@ -1,3 +1,5 @@
 inherit_gem:
   rubocop-shopify: rubocop.yml
-inherit_from: .rubocop_todo.yml
+
+AllCops:
+  TargetRubyVersion: 2.5

data/Gemfile

@@ -1,6 +1,8 @@
-source 'https://rubygems.org'
+# frozen_string_literal: true
 
-gem 'aws-sdk-s3', '>= 1.60.2', require: false
-gem 'google-cloud-storage'
+source "https://rubygems.org"
+
+gem "aws-sdk-s3", ">= 1.60.2", require: false
+gem "google-cloud-storage"
 
 gemspec

data/History.md

@@ -1,5 +1,13 @@
 # Asset Cloud Version History
 
+## Version 2.7.2, 2023-04-20
+
+* Swap the order of operations for checking UUID and asset exist logic in free key locator (https://github.com/Shopify/asset_cloud/pull/83)
+
+## Version 2.7.1, 2022-03-18
+
+* Fix incorrect invocation of callbacks defined in external classes (https://github.com/Shopify/asset_cloud/issues/71)
+
 ## Version 2.7.0, 2020-07-15
 
 * Add `Asset#write!` which raises on validation failure (https://github.com/Shopify/asset_cloud/pull/67)

data/README.rdoc

@@ -34,13 +34,12 @@ With GCS Remote test:
 
 Releasing is handled by Shopify's internal Shipit server.
 
-After merging a change, a separate PR should be opened to update `History.md` and bump the version
-(following [Semantic Versioning](https://semver.org/))
+After merging a change, a separate PR should be opened to update +History.md+ and bump the version
+(following {Semantic Versioning}[https://semver.org]).
 
 When merged, the new release will be automatically tagged and deployed to rubygems.org.
 
-Note that the automatic process does not create a new [release](https://github.com/Shopify/asset_cloud/releases) on
-GitHub.
+Note that the automatic process does not create a new {release}[https://github.com/Shopify/asset_cloud/releases] on GitHub.
 
 == Copyright
 

data/Rakefile

@@ -1,28 +1,30 @@
-require 'bundler/gem_tasks'
-require 'rspec'
+# frozen_string_literal: true
 
-require 'rake'
-require 'rake/testtask'
-require 'rdoc/task'
-require 'rspec/core/rake_task'
-require 'rubocop/rake_task'
+require "bundler/gem_tasks"
+require "rspec"
 
-desc 'Default: run unit tests and style checks.'
+require "rake"
+require "rake/testtask"
+require "rdoc/task"
+require "rspec/core/rake_task"
+require "rubocop/rake_task"
+
+desc "Default: run unit tests and style checks."
 task default: [:spec, :rubocop]
 
 desc "Run all spec examples"
 RSpec::Core::RakeTask.new do |t|
-  t.pattern = 'spec/**/*_spec.rb'
-  t.rspec_opts = ['--color']
+  t.pattern = "spec/**/*_spec.rb"
+  t.rspec_opts = ["--color"]
 end
 
-desc 'Generate documentation for the asset_cloud plugin.'
+desc "Generate documentation for the asset_cloud plugin."
 Rake::RDocTask.new(:rdoc) do |rdoc|
-  rdoc.rdoc_dir = 'rdoc'
-  rdoc.title    = 'AssetCloud'
-  rdoc.options << '--line-numbers' << '--inline-source'
-  rdoc.rdoc_files.include('README')
-  rdoc.rdoc_files.include('lib/**/*.rb')
+  rdoc.rdoc_dir = "rdoc"
+  rdoc.title    = "AssetCloud"
+  rdoc.options << "--line-numbers" << "--inline-source"
+  rdoc.rdoc_files.include("README")
+  rdoc.rdoc_files.include("lib/**/*.rb")
 end
 
 RuboCop::RakeTask.new

data/asset_cloud.gemspec

@@ -1,29 +1,30 @@
 # -*- encoding: utf-8 -*-
+# frozen_string_literal: true
 
+require "English"
 Gem::Specification.new do |s|
-  s.name = %q{asset_cloud}
-  s.version = "2.7.0"
+  s.name = "asset_cloud"
+  s.version = "2.7.2"
 
-  s.authors = %w(Shopify)
-  s.summary = %q{An abstraction layer around arbitrary and diverse asset stores.}
-  s.description = %q{An abstraction layer around arbitrary and diverse asset stores.}
+  s.authors = ["Shopify"]
+  s.summary = "An abstraction layer around arbitrary and diverse asset stores."
+  s.description = "An abstraction layer around arbitrary and diverse asset stores."
 
-  s.required_ruby_version = '>= 2.5.0'
+  s.required_ruby_version = ">= 2.5.0"
 
-  s.email = %q{developers@shopify.com}
-  s.homepage = %q{http://github.com/Shopify/asset_cloud}
-  s.require_paths = %w(lib)
+  s.email = "developers@shopify.com"
+  s.homepage = "http://github.com/Shopify/asset_cloud"
+  s.require_paths = ["lib"]
 
-  s.files = `git ls-files`.split($/)
-  s.test_files = s.files.grep(%r{^(test|spec|features)/})
+  s.files = %x(git ls-files).split($INPUT_RECORD_SEPARATOR)
 
-  s.add_dependency 'activesupport'
+  s.add_dependency("activesupport")
 
-  s.metadata['allowed_push_host'] = "https://rubygems.org"
+  s.metadata["allowed_push_host"] = "https://rubygems.org"
 
-  s.add_development_dependency 'rspec'
-  s.add_development_dependency 'rake'
-  s.add_development_dependency 'pry'
-  s.add_development_dependency 'pry-byebug'
-  s.add_development_dependency 'rubocop-shopify'
+  s.add_development_dependency("pry")
+  s.add_development_dependency("pry-byebug")
+  s.add_development_dependency("rake")
+  s.add_development_dependency("rspec")
+  s.add_development_dependency("rubocop-shopify")
 end

data/dev.yml

@@ -2,7 +2,7 @@ name: asset-cloud
 
 up:
   - ruby:
-      version: 2.5.0
+      version: 3.2.2
   - bundler
 
 commands:

data/lib/asset_cloud/asset.rb

@@ -1,5 +1,6 @@
-module AssetCloud
+# frozen_string_literal: true
 
+module AssetCloud
   class AssetError < StandardError
   end
 
@@ -8,8 +9,9 @@ module AssetCloud
 
   class Asset
     include Comparable
-    attr_accessor :key, :value, :cloud, :metadata, :new_asset
+    attr_accessor :key, :cloud, :new_asset
     attr_reader   :extensions
+    attr_writer   :value, :metadata
 
     def initialize(cloud, key, value = nil, metadata = Metadata.non_existing)
       @new_asset = true
@@ -27,10 +29,12 @@ module AssetCloud
       yield self if block_given?
     end
 
-    def self.at(cloud, key, value = nil, metadata = nil, &block)
-      file = self.new(cloud, key, value, metadata,  &block)
-      file.new_asset = false
-      file
+    class << self
+      def at(cloud, key, value = nil, metadata = nil, &block)
+        file = new(cloud, key, value, metadata, &block)
+        file.new_asset = false
+        file
+      end
     end
 
     def <=>(other)
@@ -42,11 +46,11 @@ module AssetCloud
     end
 
     def relative_key
-      @key.split("/",2).last
+      @key.split("/", 2).last
     end
 
     def relative_key_without_ext
-      relative_key.gsub(/\.[^.]+$/,"")
+      relative_key.gsub(/\.[^.]+$/, "")
     end
 
     def dirname
@@ -58,7 +62,7 @@ module AssetCloud
     end
 
     def format
-      extname.sub('.', '')
+      extname.sub(".", "")
     end
 
     def basename
@@ -122,7 +126,7 @@ module AssetCloud
     end
 
     def store!
-      store or raise(AssetNotSaved, "Validation failed: #{errors.join(', ')}")
+      store || raise(AssetNotSaved, "Validation failed: #{errors.join(", ")}")
     end
 
     def to_param
@@ -134,11 +138,11 @@ module AssetCloud
     end
 
     def url(options = {})
-      cloud.url_for key, options
+      cloud.url_for(key, options)
     end
 
     def bucket_name
-      @key.split('/').first
+      @key.split("/").first
     end
 
     def bucket
@@ -169,7 +173,7 @@ module AssetCloud
     end
 
     def method_missing(method, *args)
-      if extension = @extensions.find { |e| e.respond_to?(method) }
+      if (extension = @extensions.find { |e| e.respond_to?(method) })
         extension.public_send(method, *args)
       else
         super

data/lib/asset_cloud/asset_extension.rb

@@ -1,9 +1,17 @@
+# frozen_string_literal: true
+
 module AssetCloud
   class AssetExtension
     class AssetMismatch < StandardError
     end
-    def store; true; end
-    def delete; true; end
+
+    def store
+      true
+    end
+
+    def delete
+      true
+    end
 
     include AssetCloud::Callbacks
     include AssetCloud::Validations
@@ -11,31 +19,35 @@ module AssetCloud
     callback_methods :store, :delete, :validate
 
     attr_reader :asset
-    delegate :add_error, :to => :asset
+
+    delegate :add_error, to: :asset
 
     class_attribute :extnames
 
-    def self.applies_to(*args)
-      extnames = args.map do |arg|
-        arg = arg.to_s.downcase
-        arg = ".#{arg}" unless arg.starts_with?('.')
-        arg
+    class << self
+      def applies_to(*args)
+        extnames = args.map do |arg|
+          arg = arg.to_s.downcase
+          arg = ".#{arg}" unless arg.starts_with?(".")
+          arg
+        end
+        self.extnames = extnames
       end
-      self.extnames = extnames
-    end
 
-    def self.applies_to_asset?(asset)
-      extnames = self.extnames || []
-      extnames.each do |extname|
-        return true if asset.key.downcase.ends_with?(extname)
+      def applies_to_asset?(asset)
+        extnames = self.extnames || []
+        extnames.each do |extname|
+          return true if asset.key.downcase.ends_with?(extname)
+        end
+        false
       end
-      false
     end
 
     def initialize(asset)
       unless self.class.applies_to_asset?(asset)
         raise AssetMismatch, "Instances of #{self.class.name} cannot be applied to asset #{asset.key.inspect}"
       end
+
       @asset = asset
     end
   end