feature(api): added file type and size

https://jira.railsc.ru/browse/PC4-16800

Conflicts:
	app/models/apress/amazon_assets/concerns/base_asset.rb
	app/views/apress/amazon_assets/api/v1/assets/_asset.json.jbuilder

app/models/apress/amazon_assets/concerns/base_asset.rb

@@ -56,6 +56,28 @@ def copy_to_remote
           end
         end
 
+        # Public: Возвращает размер файла, либо с амазона, либо локальное.
+        #
+        # Returns Integer.
+        def file_size
+          if remote?
+            remote_file_size
+          else
+            local_file_size
+          end
+        end
+
+        # Public: Возвращает тип файла, либо с амазона, либо локальное.
+        #
+        # Returns String.
+        def file_content_type
+          if remote?
+            remote_content_type
+          else
+            local_content_type
+          end
+        end
+
         # Подготовим имя файла
         #
         # Returns nothing

app/views/apress/amazon_assets/api/v1/assets/_asset.json.jbuilder

@@ -1,3 +1,3 @@
-json.(asset, :id, :origin_file_name)
+json.(asset, :id, :origin_file_name, :file_size, :file_content_type)
 
 json.file asset.file.to_s

docs/api/v1/README.md

@@ -18,9 +18,11 @@ Status: 200
 
 {
   "asset": {
-    "id": 9999,
-    "file": "http://s3.amazonaws.com/pulscen_production/public_assets/000/000/185/2fc7__sports-q-c-1920-1895-3.jpg",
-    "origin_file_name": "sports-q-c-1920-1895-3.jpg"
+    "id": 185,
+    "file": "http://s3.amazonaws.com/pulscen_development/public_assets/000/000/185/2fc7__sports-q-c-1920-1895-3.jpg",
+    "origin_file_name": "sports-q-c-1920-1895-3.jpg",
+    "file_size": 395316,
+    "file_content_type": "image/jpeg"
   }
 }
 ```

spec/models/apress/amazon_assets/concerns/base_asset_shared_examples.rb

@@ -415,5 +415,63 @@ def self.valid_content_types
 
     it { expect(subject.file_name).to eq subject.remote_file_name }
   end
+
+  describe '#file_content_type' do
+    let(:file) { File.new('spec/fixtures/assets/test.png') }
+
+    context 'with both files' do
+      before do
+        subject.local = file
+        subject.remote = file
+      end
+
+      it { expect(subject.file_content_type).to eq 'image/png' }
+    end
+
+    context 'with local file only' do
+      before do
+        subject.local = file
+      end
+
+      it { expect(subject.file_content_type).to eq 'image/png' }
+    end
+
+    context 'with remote file only' do
+      before do
+        subject.local = file
+      end
+
+      it { expect(subject.file_content_type).to eq 'image/png' }
+    end
+  end
+
+  describe '#file_size' do
+    let(:file) { File.new('spec/fixtures/assets/test.png') }
+
+    context 'with both files' do
+      before do
+        subject.local = file
+        subject.remote = file
+      end
+
+      it { expect(subject.file_size).to eq 827 }
+    end
+
+    context 'with local file only' do
+      before do
+        subject.local = file
+      end
+
+      it { expect(subject.file_size).to eq 827 }
+    end
+
+    context 'with remote file only' do
+      before do
+        subject.local = file
+      end
+
+      it { expect(subject.file_size).to eq 827 }
+    end
+  end
 end
 

spec/views/apress/amazon_assets/api/v1/assets/show.json.jbuilder_spec.rb

@@ -12,10 +12,14 @@
             id
             file
             origin_file_name
+            file_size
+            file_content_type
           ),
           id: {type: 'integer'},
           file: {type: 'string'},
-          origin_file_name: {type: 'string'}
+          origin_file_name: {type: 'string'},
+          file_size: {type: 'integer'},
+          file_content_type: {type: 'string'}
         }
       }
     }.with_indifferent_access