browse-everything 0.7.1 → 0.8.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: c0537c853279c1686cb6da696a32c72ece3ecb9d
-  data.tar.gz: 79ba0379386ab732db182cffc6f2d45415574bc9
+  metadata.gz: 534d8093e331f6d7cdb1267d9daeb07028ea429c
+  data.tar.gz: 113173c876354894d06d02e5bb87e8860ef12021
 SHA512:
-  metadata.gz: 60207418ff9b7fd8d73e5f566dca07f0f1abb2b906d28a73785d629d4bc05ab1958dd0036148720738a663f11936b4f0f094a5719a77e3293e2c007434f882f0
-  data.tar.gz: 8beddddc6d2bb71cd29247e1e9bdf7c4d326cd276531950e3875e17239011a88e095754af3512a69e5c0c3d94fdafa8ad13440eb2e35e114d55f62c7b3f210ce
+  metadata.gz: bf4801b32d5b067c2a5a17d366070a1dbaf111d6a7a92c5351f4bfbed898c42bd687a8e4b2884241b110d49228c8f05a918b12f53522b54ea23ffbb2f3c9414f
+  data.tar.gz: 20d1e90a20b62a30d3e5748add97a1fc8b7209a7b81d382d57fa7fc74dd2b5c0693f8493749f497d87af1c0ea0fadd8b552609dc60db55fa5a1e805e559dbbca

data/HISTORY.md

@@ -1,3 +1,7 @@
+### 0.8.0 (2015-02-27)
+- Add `max_upload_file_size` option to configuration
+- Disable selection of files that are larger than `max_upload_file_size`
+
 ### 0.7.1 (2014-12-23)
 - Rails 4.2 support
 

data/app/views/browse_everything/_file.html.erb

@@ -1,11 +1,27 @@
 <% unless file.relative_parent_path? %>
+
+  <% if file.container? || provider.config[:max_upload_file_size].blank? #never disable a folder or if no maximum is set
+     disabled = false
+   else
+     max_size = provider.config[:max_upload_file_size].to_i
+     disabled = file.size > max_size
+   end
+  %>
+
 <tr role="row" tabindex="-1" data-ev-location="<%= file.location %>" data-tt-id="<%=path%>" data-tt-parent-id="<%=parent%>" data-tt-branch="<%=file.container? ? 'true' : 'false'%>">
   <td role="gridcell" class="<%=file.container? ? 'ev-container' : 'ev-file'%> ev-file-name">
-      <%= link_to browse_everything_engine.contents_path(provider_name,file.id), {:class=>'ev-link'} do %>
+    <% if disabled %>
+      <span title="<%= t('browse_everything.size_disabled', max_size: number_to_human_size(max_size)) %>"class="<%=file.container? ? 'folder' : 'file'%>" aria-hidden="true">
+      <%= file.name %>
+      </span>
+      <span class="sr-only"><%= file.container? ? ', folder' : ', file' %> </span>
+    <% else %>
+      <%= link_to browse_everything_engine.contents_path(provider_name,file.id), :class=>'ev-link' do %>
         <span class="<%=file.container? ? 'folder' : 'file'%>" aria-hidden="true"/>
         <%= file.name %>
         <span class="sr-only"><%= file.container? ? ', folder' : ', file' %> </span>
       <% end %>
+    <% end %>
   </td>
   <td role="gridcell" class="ev-file-size">
     <%= number_to_human_size(file.size).sub(/Bytes/,'bytes') %>

data/config/locales/en_browse_everything.yml

@@ -11,3 +11,4 @@ en:
     modal_form:
       submit: "Submit"
       cancel: "Cancel"
+    size_disabled: "This file can not be uploaded, since it is larger than the maximum file size %{max_size}"

data/lib/browse_everything/version.rb

@@ -1,3 +1,3 @@
 module BrowseEverything
-  VERSION = "0.7.1"
+  VERSION = "0.8.0"
 end

data/spec/spec_helper.rb

@@ -1,5 +1,6 @@
 require File.expand_path("config/environment", ENV['RAILS_ROOT'] || File.expand_path("../internal", __FILE__))
 require 'rspec'
+require 'rspec/rails'
 require 'rspec/its'
 require 'webmock/rspec'
 require 'simplecov'

data/spec/views/browse_everything/_file.html.erb_spec.rb

@@ -0,0 +1,46 @@
+require File.expand_path('../../../spec_helper',__FILE__)
+
+describe 'browse_everything/_file.html.erb', type: :view do
+
+  let(:file)  {
+                BrowseEverything::FileEntry.new(
+                     'file_id_01234', 'my_provider:/location/pa/th/file.m4v',
+                     'file.m4v', 1024*1024*1024, Time.now, false
+                   )
+               }
+  let(:provider) { double("provider") }
+  let(:page) { Capybara::Node::Simple.new(rendered) }
+
+
+  before do
+    allow(view).to receive(:file).and_return(file)
+    allow(view).to receive(:provider).and_return(provider)
+    allow(view).to receive(:path).and_return("path")
+    allow(view).to receive(:parent).and_return("parent")
+    allow(view).to receive(:provider_name).and_return("my provider")
+    allow(provider).to receive(:config).and_return(config)
+    render
+  end
+
+  context "file not too big" do
+    let(:config) { { max_upload_file_size: (5*1024*1024*1024) } }
+    it "should draw link" do
+      expect(page).to have_selector("a.ev-link")
+    end
+  end
+
+  context "max not configured" do
+    let(:config) { { } }
+    it "should draw link" do
+      expect(page).to have_selector("a.ev-link")
+    end
+  end
+
+  context "file too big" do
+    let(:config) { { max_upload_file_size: 1024 } }
+    it "should draw link" do
+      expect(page).not_to have_selector("a.ev-link")
+    end
+  end
+
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: browse-everything
 version: !ruby/object:Gem::Version
-  version: 0.7.1
+  version: 0.8.0
 platform: ruby
 authors:
 - Carolyn Cole
@@ -13,7 +13,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-12-23 00:00:00.000000000 Z
+date: 2015-02-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -360,7 +360,7 @@ files:
 - app/.DS_Store
 - app/assets/javascripts/browse_everything.js
 - app/assets/javascripts/browse_everything/behavior.js.coffee
-- app/assets/stylesheets/browse_everything.css.scss
+- app/assets/stylesheets/browse_everything.scss
 - app/assets/stylesheets/jquery.treetable.theme.browse.css
 - app/controllers/browse_everything_controller.rb
 - app/helpers/bootstrap_version_helper.rb
@@ -401,7 +401,6 @@ files:
 - spec/fixtures/vcr_cassettes/dropbox.yml
 - spec/fixtures/vcr_cassettes/retriever.yml
 - spec/spec_helper.rb
-- spec/spec_helper.rb.orig
 - spec/support/app/controllers/file_handler_controller.rb
 - spec/support/app/views/file_handler/index.html.erb
 - spec/support/app/views/file_handler/main.html.erb
@@ -414,6 +413,7 @@ files:
 - spec/unit/file_system_spec.rb
 - spec/unit/retriever_spec.rb
 - spec/unit/sky_drive_spec.rb
+- spec/views/browse_everything/_file.html.erb_spec.rb
 - tasks/ci.rake
 - vendor/assets/javascripts/jquery.treetable.js
 - vendor/assets/stylesheets/jquery.treetable.css
@@ -451,7 +451,6 @@ test_files:
 - spec/fixtures/vcr_cassettes/dropbox.yml
 - spec/fixtures/vcr_cassettes/retriever.yml
 - spec/spec_helper.rb
-- spec/spec_helper.rb.orig
 - spec/support/app/controllers/file_handler_controller.rb
 - spec/support/app/views/file_handler/index.html.erb
 - spec/support/app/views/file_handler/main.html.erb
@@ -464,4 +463,4 @@ test_files:
 - spec/unit/file_system_spec.rb
 - spec/unit/retriever_spec.rb
 - spec/unit/sky_drive_spec.rb
-has_rdoc: 
+- spec/views/browse_everything/_file.html.erb_spec.rb

data/spec/spec_helper.rb.orig

@@ -1,47 +0,0 @@
-require File.expand_path("config/environment", ENV['RAILS_ROOT'] || File.expand_path("../internal", __FILE__))
-<<<<<<< HEAD
-require 'rspec/rails'
-require 'webmock/rspec'
-=======
-require 'simplecov'
-SimpleCov.start do
-  add_filter "/spec/"
-end
-
-module BrowserConfigHelper
-  def url_options
-    {
-      protocol: 'http://',
-      host: 'browse-everything.example.edu'
-    }
-  end
-
-  def stub_configuration
-    BrowseEverything.configure({
-      "file_system" => {
-        home: File.expand_path('../fixtures/file_system',__FILE__)
-      },
-      "box" => {
-        client_id: "BoxClientId",
-        client_secret: "BoxClientSecret"
-      },
-      "drop_box" => { 
-        app_key: "DropBoxAppKey", 
-        app_secret: "DropBoxAppSecret"
-      },
-      "google_drive" => {
-        client_id: "GoogleClientId",
-        client_secret: "GoogleClientSecret"
-      },
-      "sky_drive" => {
-        client_id: "SkyDriveClientId",
-        client_secret: "SkyDriveClientSecret"
-      }
-    })
-  end
-
-  def unstub_configuration
-    BrowseEverything.configure(nil)
-  end
-end
->>>>>>> Add coverage and configuration stubs