jakewendt-simply_documents 1.3.4

data/generators/simply_documents/templates/autotest_simply_documents.rb

@@ -0,0 +1,2 @@
+#	From `script/generate simply_documents` ...
+require 'simply_documents/autotest'

data/generators/simply_documents/templates/functional/documents_controller_test.rb

@@ -0,0 +1,220 @@
+#require File.dirname(__FILE__) + '/../../test_helper'
+require 'test_helper'
+
+class SimplyDocuments::DocumentsControllerTest < ActionController::TestCase
+	tests DocumentsController
+
+	ASSERT_ACCESS_OPTIONS = {
+		:model => 'Document',
+		:actions => [:new,:create,:edit,:update,:destroy,:index],
+		:method_for_create => :factory_create,
+		:attributes_for_create => :factory_attributes
+	}
+
+	def factory_create
+		Factory(:document)
+	end
+	def factory_attributes
+		Factory.attributes_for(:document)
+	end
+
+	assert_access_with_https
+	assert_access_with_login({
+		:logins => [:super_user,:admin,:editor]})
+
+	assert_no_access_with_http 
+	assert_no_access_with_login({ 
+		:logins => [:interviewer,:reader,:active_user] })
+	assert_no_access_without_login
+
+	assert_no_access_with_login(
+		:attributes_for_create => nil,
+		:method_for_create => nil,
+		:actions => nil,
+		:suffix => " and invalid id",
+		:login => :superuser,
+		:redirect => :documents_path,
+		:edit => { :id => 0 },
+		:update => { :id => 0 },
+		:destroy => { :id => 0 }
+	)
+
+%w( super_user admin editor ).each do |cu|
+
+
+##	still only privacy filter is based on "may_maintain_pages"
+##	which isn't really gonna work
+#	test "should get redirect to public s3 document with #{cu} login" do
+#		Document.any_instance.stubs(:s3_public?).returns(true)
+#		document = Factory(:document, :document_file_name => 'bogus_file_name')
+#		assert !File.exists?(document.document.path)
+#		login_as send(cu)
+#		get :show, :id => document.id
+#		assert_redirected_to document.document.url
+#	end
+#
+#	test "should get redirect to private s3 document with #{cu} login" do
+#		Document.any_instance.stubs(:s3_private?).returns(true)
+#		document = Factory(:document, :document_file_name => 'bogus_file_name')
+#		assert !File.exists?(document.document.path)
+#		login_as send(cu)
+#		get :show, :id => document.id
+#		assert_redirected_to document.s3_url
+#	end
+
+#	Add may_download_document
+
+#	test "should get redirect to public s3 document with #{cu} login" do
+#		Document.any_instance.stubs(:s3_public?).returns(true)
+#		document = Factory(:document, :document_file_name => 'bogus_file_name')
+#		assert !File.exists?(document.document.path)
+#		login_as send(cu)
+#		get :show, :id => document.id
+#		assert_redirected_to document.document.url
+#	end
+
+	test "should get redirect to private s3 document with #{cu} login" do
+		Document.has_attached_file :document, {
+			:s3_headers => {
+				'x-amz-storage-class' => 'REDUCED_REDUNDANCY' },
+			:s3_permissions => :private,
+			:storage => :s3,
+			:s3_protocol => 'https',
+			:s3_credentials => "#{Rails.root}/config/s3.yml",
+			:bucket => 'ccls',
+			:path => "documents/:id/:filename"
+		}
+
+		#	Since the REAL S3 credentials are only in production
+		#	Bad credentials make exists? return true????
+		Rails.stubs(:env).returns('production')
+		document = Factory(:document, :document_file_name => 'bogus_file_name')
+		assert !document.document.exists?
+		assert !File.exists?(document.document.path)
+
+		AWS::S3::S3Object.stubs(:exists?).returns(true)
+
+		login_as send(cu)
+		get :show, :id => document.id
+		assert_response :redirect
+		assert_match %r{\Ahttp(s)?://s3.amazonaws.com/ccls/documents/\d+/bogus_file_name\.\?AWSAccessKeyId=\w+&Expires=\d+&Signature=.+\z}, @response.redirected_to
+	end
+
+
+
+
+
+
+	test "should NOT download document with nil document and #{cu} login" do
+		document = Factory(:document)
+		assert document.document.path.blank?
+		login_as send(cu)
+		get :show, :id => document.id
+		assert_redirected_to preview_document_path(document)
+		assert_not_nil flash[:error]
+	end
+
+	test "should NOT download document with no document and #{cu} login" do
+		document = Factory(:document, :document_file_name => 'bogus_file_name')
+		assert !File.exists?(document.document.path)
+		login_as send(cu)
+		get :show, :id => document.id
+		assert_redirected_to preview_document_path(document)
+		assert_not_nil flash[:error]
+	end
+
+	test "should NOT download nonexistant document with #{cu} login" do
+		assert !File.exists?('some_fake_file_name.doc')
+		login_as send(cu)
+		get :show, :id => 'some_fake_file_name',:format => 'doc'
+		assert_redirected_to documents_path
+		assert_not_nil flash[:error]
+	end
+
+	test "should preview document with document and #{cu} login" do
+		document = Factory(:document)
+		login_as send(cu)
+		get :preview, :id => document.id
+		assert_response :success
+		assert_nil flash[:error]
+	end
+
+	test "should download document by id with document and #{cu} login" do
+		document = Document.create!(Factory.attributes_for(:document, 
+			:document => File.open(File.dirname(__FILE__) + 
+				'/../../assets/edit_save_wireframe.pdf')))
+		login_as send(cu)
+		get :show, :id => document.reload.id
+		assert_nil flash[:error]
+		assert_not_nil @response.headers['Content-disposition'].match(
+			/attachment;.*pdf/)
+		document.destroy
+	end
+
+	test "should download document by name with document and #{cu} login" do
+		document = Document.create!(Factory.attributes_for(:document, 
+			:document => File.open(File.dirname(__FILE__) + 
+				'/../../assets/edit_save_wireframe.pdf')))
+		login_as send(cu)
+		get :show, :id => 'edit_save_wireframe',
+			:format => 'pdf'
+		assert_nil flash[:error]
+		assert_not_nil @response.headers['Content-disposition'].match(
+			/attachment;.*pdf/)
+		document.destroy
+	end
+
+	test "should NOT create invalid document with #{cu} login" do
+		login_as send(cu)
+		assert_no_difference('Document.count') do
+			post :create, :document => {}
+		end
+		assert_not_nil flash[:error]
+		assert_template 'new'
+		assert_response :success
+	end
+
+	test "should NOT update invalid document with #{cu} login" do
+		login_as send(cu)
+		put :update, :id => Factory(:document).id, 
+			:document => { :title => "a" }
+		assert_not_nil flash[:error]
+		assert_template 'edit'
+		assert_response :success
+	end
+
+end
+
+%w( interviewer reader active_user ).each do |cu|
+
+	test "should NOT preview document with #{cu} login" do
+		document = Factory(:document)
+		login_as send(cu)
+		get :preview, :id => document.id
+		assert_redirected_to root_path
+		assert_not_nil flash[:error]
+	end
+
+	test "should NOT download document with #{cu} login" do
+		document = Factory(:document)
+		login_as send(cu)
+		get :show, :id => document.id
+		assert_redirected_to root_path
+		assert_not_nil flash[:error]
+	end
+
+end
+
+	test "should NOT preview document without login" do
+		document = Factory(:document)
+		get :preview, :id => document.id
+		assert_redirected_to_login
+	end
+
+	test "should NOT download document without login" do
+		document = Factory(:document)
+		get :show, :id => document.id
+		assert_redirected_to_login
+	end
+
+end

data/generators/simply_documents/templates/migrations/add_attachments_document_to_document.rb

@@ -0,0 +1,19 @@
+class AddAttachmentsDocumentToDocument < ActiveRecord::Migration
+	def self.up
+		add_column :documents, :document_file_name, :string
+		add_column :documents, :document_content_type, :string
+		add_column :documents, :document_file_size, :integer
+		add_column :documents, :document_updated_at, :datetime
+
+		add_index :documents, :document_file_name, :unique => true
+	end
+
+	def self.down
+		remove_index :documents, :document_file_name
+
+		remove_column :documents, :document_file_name
+		remove_column :documents, :document_content_type
+		remove_column :documents, :document_file_size
+		remove_column :documents, :document_updated_at
+	end
+end

data/generators/simply_documents/templates/migrations/create_documents.rb

@@ -0,0 +1,19 @@
+class CreateDocuments < ActiveRecord::Migration
+	def self.up
+		create_table :documents do |t|
+			t.references :owner
+			t.string :title, :null => false
+			t.text   :abstract
+			t.boolean :shared_with_all, 
+				:default => false, :null => false
+			t.boolean :shared_with_select, 
+				:default => false, :null => false
+			t.timestamps
+		end
+		add_index :documents, :owner_id
+	end
+
+	def self.down
+		drop_table :documents
+	end
+end

data/generators/simply_documents/templates/migrations/polymorphicize_document_owner.rb

@@ -0,0 +1,13 @@
+class PolymorphicizeDocumentOwner < ActiveRecord::Migration
+	def self.up
+		add_column :documents, :owner_type, :string
+		remove_index :documents, :owner_id
+		add_index :documents, [:owner_id,:owner_type]
+	end
+
+	def self.down
+		remove_index :documents, [:owner_id,:owner_type]
+		add_index :documents, :owner_id
+		remove_column :documents, :owner_type_string
+	end
+end

data/generators/simply_documents/templates/simply_documents.rake

@@ -0,0 +1,6 @@
+#	From `script/generate simply_documents` ...
+unless Gem.source_index.find_name('jakewendt-simply_documents').empty?
+	gem 'jakewendt-simply_documents'
+	require 'simply_documents'
+	require 'simply_documents/test_tasks'
+end

data/generators/simply_documents/templates/stylesheets/documents.css

@@ -0,0 +1,4 @@
+form.new_document,
+form.edit_document {
+	display:inline-block;
+}

data/generators/simply_documents/templates/unit/document_test.rb

@@ -0,0 +1,18 @@
+#require File.dirname(__FILE__) + '/../../test_helper'
+require 'test_helper'
+
+class SimplyDocuments::DocumentTest < ActiveSupport::TestCase
+
+	assert_should_require(:title)
+	assert_should_require_attribute_length(:title, :minimum => 4)
+	assert_should_belong_to(:owner,:class_name => 'User')
+
+	test "should create document" do
+		assert_difference 'Document.count' do
+			object = create_object
+			assert !object.new_record?, 
+				"#{object.errors.full_messages.to_sentence}"
+		end
+	end
+
+end

data/lib/jakewendt-simply_documents.rb

@@ -0,0 +1 @@
+require 'simply_documents'

data/lib/simply_documents.rb

@@ -0,0 +1,54 @@
+require 'active_record'
+require 'active_support'
+require 'ruby_extension'
+require 'simply_helpful'
+require 'gravatar'
+#require 'calnet_authenticated'
+require 'simply_authorized'
+require 'acts_as_list'
+module SimplyDocuments
+#	predefine namespace
+end
+require 'simply_documents/owner'
+
+#	This doesn't seem necessary
+%w{models controllers}.each do |dir|
+	path = File.expand_path(File.join(File.dirname(__FILE__), '../app', dir))
+	ActiveSupport::Dependencies.autoload_paths << path
+	ActiveSupport::Dependencies.autoload_once_paths << path
+end
+
+require 'action_controller'	#	loads HTML
+HTML::WhiteListSanitizer.allowed_attributes.merge(%w(
+	id class style
+))
+
+if defined?(Rails) && Rails.env == 'test' && Rails.class_variable_defined?("@@configuration")
+	require 'active_support/test_case'
+	require 'factory_girl'
+	require 'simply_documents/factories'
+#	else
+#		running a rake task
+end
+
+if RUBY_PLATFORM =~ /java/i
+	require 'simply_documents/file_utils_extension'
+end
+
+require 'paperclip'
+if defined? ::Paperclip::Glue
+	ActiveRecord::Base.send(:include, ::Paperclip::Glue)
+else
+	ActiveRecord::Base.send(:include, ::Paperclip)
+end
+
+ActionController::Routing::Routes.add_configuration_file(
+	File.expand_path(
+		File.join(
+			File.dirname(__FILE__), '../config/routes.rb')))
+
+ActionController::Base.view_paths <<
+	File.expand_path(
+		File.join(
+			File.dirname(__FILE__), '../app/views'))
+

data/lib/simply_documents/autotest.rb

@@ -0,0 +1,26 @@
+class Autotest::Rails
+
+#
+#	Need both the mapping and the extra files
+#
+	def run_with_simply_documents
+		add_exception %r%config/%
+		add_exception %r%versions/%
+		add_exception %r%\.git/%
+		self.extra_files << File.expand_path(File.join(
+				File.dirname(__FILE__),'/../../test/unit/documents/'))
+
+		self.extra_files << File.expand_path(File.join(
+				File.dirname(__FILE__),'/../../test/functional/documents/'))
+
+		add_mapping( 
+			%r{^#{File.expand_path(File.join(File.dirname(__FILE__),'/../../test/'))}/(unit|functional)/documents/.*_test\.rb$}
+			) do |filename, _|
+			filename
+		end
+		run_without_simply_documents
+	end
+	alias_method_chain :run, :simply_documents
+
+
+end

data/lib/simply_documents/factories.rb

@@ -0,0 +1,4 @@
+Factory.define :document do |f|
+	f.sequence(:title) { |n| "Title#{n}" }
+#	f.sequence(:document_file_name) { |n| "document_file_name#{n}" }
+end

data/lib/simply_documents/file_utils_extension.rb

@@ -0,0 +1,18 @@
+#	Needed for Paperclip gem 2.3.3
+#	http://jira.codehaus.org/browse/JRUBY-3381
+#	http://github.com/thoughtbot/paperclip/issues/issue/193
+#	Errno::EACCES: Permission denied - /var/folders/kV/kV5XVPtqE9uZBCjn3z6vmk+++TM/-Tmp-/stream,19661,34729.pdf or /Users/jakewendt/github_repo/jakewendt/ucb_ccls_buffler/development/documents/2/edit_save_wireframe.pdf
+FileUtils.module_eval do
+  class << self
+    alias_method :built_in_mv, :mv
+
+    def mv(src, dest, options = {})
+      begin
+        built_in_mv(src, dest, options)
+      rescue Errno::EACCES
+        cp(src, dest)
+        rm(src)
+      end
+    end
+  end 
+end unless FileUtils.methods.include?('built_in_mv')

data/lib/simply_documents/owner.rb

@@ -0,0 +1,12 @@
+module SimplyDocuments::Owner
+	def self.included(base)
+		base.extend(PrepMethod)
+	end
+	module PrepMethod
+		def document_owner(*args)
+			options = args.extract_options!
+			has_many :documents, :as => :owner
+		end
+	end
+end
+ActiveRecord::Base.send(:include,SimplyDocuments::Owner)

data/lib/simply_documents/tasks.rb

@@ -0,0 +1 @@
+Dir["#{File.dirname(__FILE__)}/../tasks/**/*.rake"].sort.each { |ext| load ext }

data/lib/simply_documents/test_tasks.rb

@@ -0,0 +1,56 @@
+module SimplyDocuments;end
+namespace :test do
+	namespace :units do
+		Rake::TestTask.new(:simply_documents => "db:test:prepare") do |t|
+			t.pattern = File.expand_path(File.join(
+				File.dirname(__FILE__),'/../../test/unit/documents/*_test.rb'))
+			t.libs << "test"
+			t.verbose = true
+		end
+	end
+	namespace :functionals do
+		Rake::TestTask.new(:simply_documents => "db:test:prepare") do |t|
+#			t.pattern = File.expand_path(File.join(
+#				File.dirname(__FILE__),'/../../test/functional/documents/*_test.rb'))
+			t.libs << "test"
+			t.verbose = true
+			#	this enables user override
+			#	used to work without this, but now it doesn't???!?!?
+			t.test_files = [
+				File.expand_path(File.join(File.dirname(__FILE__),
+					'/../../test/functional/documents/*_test.rb')),
+				File.expand_path(File.join(Rails.root,
+					'/test/functional/documents/*_test.rb'))
+			]
+		end
+	end
+end
+Rake::Task['test:functionals'].prerequisites.unshift(
+	"test:functionals:simply_documents" )
+Rake::Task['test:units'].prerequisites.unshift(
+	"test:units:simply_documents" )
+
+#	I thought of possibly just including this file
+#	but that would make __FILE__ different.
+#	Hmmm
+
+#
+#	used in simply_helpful's rake test:coverage to run gem's 
+#		tests in the context of the application
+#
+@gem_test_dirs ||= []
+#@gem_test_dirs << File.expand_path(File.join(File.dirname(__FILE__),
+#	'/../../test/unit/documents/'))
+#@gem_test_dirs << File.expand_path(File.join(File.dirname(__FILE__),
+#	'/../../test/functional/documents/'))
+
+#
+#	More flexible. Find all test files, pick out their dir, uniq 'em and add.
+#
+Dir.glob( File.expand_path(File.join(File.dirname(__FILE__),
+	'/../../test/*/documents/*_test.rb'))
+).collect{|f|
+	File.dirname(f)
+}.uniq.each{ |dir|
+	@gem_test_dirs << dir
+}