coupler 0.0.4-java → 0.0.6-java

data/test/helper.rb

@@ -12,6 +12,8 @@ require 'mocha'
 require 'rack/test'
 require 'rack/flash'
 require 'rack/flash/test'
+require 'capybara'
+require 'capybara/dsl'
 require 'nokogiri'
 require 'timecop'
 require 'tempfile'
@@ -21,11 +23,9 @@ require 'table_maker'
 require 'sequel'
 require 'sequel/extensions/schema_dumper'
 require 'forgery'
-require 'ruby-debug'
 
 dir = File.dirname(__FILE__)
 $LOAD_PATH.unshift(dir)
-require 'table_sets'
 
 # set here and in the rake environment task
 ENV['COUPLER_ENV'] = 'test'
@@ -38,16 +38,19 @@ Coupler::Base.set(:sessions, false) # workaround
 Coupler::Base.set(:environment, :test)
 Coupler::Database.instance.migrate!
 
+#Capybara.register_driver :selenium_chrome do |app|
+  #Capybara::Driver::Selenium.new(app, :browser => :chrome)
+#end
+Capybara.javascript_driver = :selenium
+Capybara.app = Coupler::Base
+
 module Coupler
   module Test
     class Base < ::Test::Unit::TestCase
-      include Rack::Test::Methods
+      include Coupler
+      include Coupler::Models
       @@test_config = YAML.load(ERB.new(File.read(File.join(File.dirname(__FILE__), 'config.yml'))).result(binding))
 
-      def app
-        Coupler::Base
-      end
-
       def setup
         #@_original_connection_count = connection_count
         @_database = Coupler::Database.instance
@@ -57,13 +60,6 @@ module Coupler
         end
       end
 
-      #def run(*args, &block)
-        #Sequel::Model.db.transaction do
-          #super
-          #raise Sequel::Error::Rollback
-        #end
-      #end
-
       def teardown
         if @_tmpdirs
           @_tmpdirs.each { |t| FileUtils.rm_rf(t) }
@@ -124,10 +120,27 @@ module Coupler
     end
 
     class UnitTest < Base; end
+    class IntegrationTest < Base; end
 
-    class IntegrationTest < Base
-      include Coupler
-      include Coupler::Models
+    class FunctionalTest < Base
+      include Capybara::DSL
+
+      def app
+        Coupler::Base
+      end
+
+      def teardown
+        super
+        Capybara.reset_sessions!
+        Capybara.use_default_driver
+      end
+
+      def setup
+        if self.class.get_attribute(@method_name, :javascript)
+          Capybara.current_driver = Capybara.javascript_driver
+        end
+        super
+      end
     end
   end
 end
@@ -154,6 +167,3 @@ class Array
     end
   end
 end
-
-require 'factory_girl'
-Factory.find_definitions

data/test/integration/test_transformation.rb

@@ -37,7 +37,12 @@ class TestTransformation < Coupler::Test::IntegrationTest
 
     adapter_test(adapter, "accepts nested attributes for result field") do
       adapter_setup(adapter)
-      transformer = Factory(:transformer, :code => %{value}, :result_type => 'same')
+      transformer = Transformer.create({
+        :name => "noop",
+        :code => %{value},
+        :allowed_types => %w{string integer datetime},
+        :result_type => 'same'
+      })
       field = @resource.fields_dataset[:name => 'first_name']
       transformation = Transformation.create({
         :transformer => transformer,

data/test/unit/models/test_common_model.rb

@@ -1,7 +1,7 @@
 require 'helper'
 
-module Coupler
-  module Models
+module CouplerUnitTests
+  module ModelTests
     class TestCommonModel < Coupler::Test::UnitTest
       def self.startup
         super

data/test/unit/models/test_comparison.rb

@@ -1,7 +1,7 @@
 require 'helper'
 
-module Coupler
-  module Models
+module CouplerUnitTests
+  module ModelTests
     class TestComparison < Coupler::Test::UnitTest
       def sequel_expr(*args)
         if args.length == 1
@@ -559,7 +559,7 @@ module Coupler
 
       test "blocking?" do
         field = stub("field", :name => 'first_name') {
-          stubs(:[]).with(:type).returns('string')
+          stubs(:[]).with(:final_type).returns('string')
         }
         Field.stubs(:[]).with(:id => 1).returns(field)
         comparison = new_comparison({
@@ -572,11 +572,11 @@ module Coupler
 
       test "cross_match?" do
         field_1 = stub("field 1", :id => 1, :name => 'ssn_1', :resource_id => 1) {
-          stubs(:[]).with(:type).returns('string')
+          stubs(:[]).with(:final_type).returns('string')
         }
         Field.stubs(:[]).with(:id => 1).returns(field_1)
         field_2 = stub("field 2", :id => 2, :name => 'ssn_2', :resource_id => 1) {
-          stubs(:[]).with(:type).returns('string')
+          stubs(:[]).with(:final_type).returns('string')
         }
         Field.stubs(:[]).with(:id => 2).returns(field_2)
         comparison = new_comparison({
@@ -589,11 +589,11 @@ module Coupler
 
       test "does not allow two fields of different types" do
         field_1 = stub("field 1", :name => 'first_name') {
-          stubs(:[]).with(:type).returns('string')
+          stubs(:[]).with(:final_type).returns('string')
         }
         Field.stubs(:[]).with(:id => 1).returns(field_1)
         field_2 = stub("field 2", :name => 'age') {
-          stubs(:[]).with(:type).returns('integer')
+          stubs(:[]).with(:final_type).returns('integer')
         }
         Field.stubs(:[]).with(:id => 2).returns(field_2)
         comparison = new_comparison({
@@ -605,7 +605,7 @@ module Coupler
       end
 
       test "does not allow non-equality comparisons for fields" do
-        field = stub("field", :name => 'first_name') { stubs(:[]).with(:type).returns('string') }
+        field = stub("field", :name => 'first_name') { stubs(:[]).with(:final_type).returns('string') }
         Field.stubs(:[]).with(:id => 1).returns(field)
         comparison = new_comparison({
           :lhs_type => 'field', :raw_lhs_value => 1, :lhs_which => 1,

data/test/unit/models/test_connection.rb

@@ -1,7 +1,7 @@
 require 'helper'
 
-module Coupler
-  module Models
+module CouplerUnitTests
+  module ModelTests
     class TestConnection < Coupler::Test::UnitTest
       test "sequel model" do
         assert_equal ::Sequel::Model, Connection.superclass

data/test/unit/models/test_field.rb

@@ -1,7 +1,7 @@
 require 'helper'
 
-module Coupler
-  module Models
+module CouplerUnitTests
+  module ModelTests
     class TestField < Coupler::Test::UnitTest
       def new_field(attrs = {})
         f = Field.new({

data/test/unit/models/test_import.rb

@@ -1,7 +1,7 @@
 require 'helper'
 
-module Coupler
-  module Models
+module CouplerUnitTests
+  module ModelTests
     class TestImport < Coupler::Test::UnitTest
       def new_import(attribs = {})
         values = {
@@ -67,6 +67,11 @@ module Coupler
         assert !import.has_headers
       end
 
+      test "requires name" do
+        import = new_import(:name => nil)
+        assert !import.valid?
+      end
+
       test "requires field names" do
         import = new_import(:data => fixture_file_upload('no-headers.csv'))
         assert_nil import.field_names
@@ -97,8 +102,8 @@ module Coupler
 
       test "requires unused resource name" do
         import = new_import
-        @project.resources_dataset.stubs(:count).returns(1)
-        assert !import.valid?
+        import.expects(:validates_unique).with([:project_id, :name]).returns(false)
+        import.valid?
       end
 
       test "dataset" do

data/test/unit/models/test_job.rb

@@ -1,7 +1,7 @@
 require 'helper'
 
-module Coupler
-  module Models
+module CouplerUnitTests
+  module ModelTests
     class TestJob < Coupler::Test::UnitTest
       def new_job(attribs = {})
         j = Job.new(attribs)

data/test/unit/models/test_matcher.rb

@@ -1,7 +1,7 @@
 require 'helper'
 
-module Coupler
-  module Models
+module CouplerUnitTests
+  module ModelTests
     class TestMatcher < Coupler::Test::UnitTest
       def new_matcher(attribs = {})
         values = {

data/test/unit/models/test_project.rb

@@ -1,7 +1,7 @@
 require 'helper'
 
-module Coupler
-  module Models
+module CouplerUnitTests
+  module ModelTests
     class TestProject < Coupler::Test::UnitTest
       test "sequel model" do
         assert_equal ::Sequel::Model, Project.superclass

data/test/unit/models/test_resource.rb

@@ -1,7 +1,7 @@
 require 'helper'
 
-module Coupler
-  module Models
+module CouplerUnitTests
+  module ModelTests
     class TestResource < Coupler::Test::UnitTest
       def setup
         super

data/test/unit/models/test_result.rb

@@ -1,7 +1,7 @@
 require 'helper'
 
-module Coupler
-  module Models
+module CouplerUnitTests
+  module ModelTests
     class TestResult < Coupler::Test::UnitTest
       def new_result(attribs = {})
         values = {

data/test/unit/models/test_scenario.rb

@@ -1,7 +1,7 @@
 require 'helper'
 
-module Coupler
-  module Models
+module CouplerUnitTests
+  module ModelTests
     class TestScenario < Coupler::Test::UnitTest
       def setup
         super

data/test/unit/models/test_transformation.rb

@@ -1,7 +1,7 @@
 require 'helper'
 
-module Coupler
-  module Models
+module CouplerUnitTests
+  module ModelTests
     class TestTransformation < Coupler::Test::UnitTest
       def new_transformation(attribs = {})
         values = {

data/test/unit/models/test_transformer.rb

@@ -1,7 +1,7 @@
 require 'helper'
 
-module Coupler
-  module Models
+module CouplerUnitTests
+  module ModelTests
     class TestTransformer < Coupler::Test::UnitTest
       def new_transformer(attribs = {})
         values = {
@@ -183,6 +183,16 @@ module Coupler
       #def test_should_handle_empty_values
         #pend
       #end
+
+      test "doesn't escape backslashes too much" do
+        code = %{value =~ /\d/ ? "YES" : "NO"}
+        transformer = new_transformer({
+          :allowed_types => %w{string},
+          :result_type => 'string', :code => code
+        }).save!
+        t = Transformer[:id => transformer.id]
+        assert_equal code, t.code
+      end
     end
   end
 end

data/test/unit/test_base.rb

@@ -1,24 +1,11 @@
 require 'helper'
 
-module Coupler
+module CouplerUnitTests
   class TestBase < Coupler::Test::UnitTest
     def test_subclasses_sinatra_base
       assert_equal Sinatra::Base, Coupler::Base.superclass
     end
 
-    def test_index_when_no_projects
-      get "/"
-      assert last_response.ok?
-      assert_match /Getting Started/, last_response.body
-    end
-
-    def test_redirect_when_projects_exist
-      project = Factory(:project)
-      get "/"
-      assert last_response.redirect?
-      assert_equal "http://example.org/projects", last_response['location']
-    end
-
     def test_db_path
       env = Base.settings.environment
       begin

data/test/unit/test_data_uploader.rb

@@ -1,6 +1,6 @@
 require 'helper'
 
-module Coupler
+module CouplerUnitTests
   class TestDataUploader < Coupler::Test::UnitTest
     def test_carrierwave_subclass
       assert_equal CarrierWave::Uploader::Base, DataUploader.superclass

data/test/unit/test_database.rb

@@ -1,6 +1,6 @@
 require 'helper'
 
-module Coupler
+module CouplerUnitTests
   class TestDatabase < Coupler::Test::UnitTest
     def setup
       super

data/test/unit/test_helpers.rb

@@ -1,6 +1,6 @@
 require 'helper'
 
-module Coupler
+module CouplerUnitTests
   class TestHelpers < Coupler::Test::UnitTest
     include ::Coupler::Helpers
 
@@ -50,7 +50,7 @@ module Coupler
       result = form_tag_for(obj, :base_url => "/projects")
       assert_equal %{<form action="/projects" method="post">}, result
 
-      obj = Factory(:project)
+      obj = Project.create(:name => 'foo')
       result = form_tag_for(obj, :base_url => "/projects")
       assert_equal %{<form action="/projects/#{obj.id}" method="post"><div style="display: none;"><input type="hidden" name="_method" value="put" /></div>}, result
     end

data/test/unit/test_import_buffer.rb

@@ -1,47 +1,49 @@
 require 'helper'
 
-class TestImportBuffer < Coupler::Test::UnitTest
-  include Coupler
+module CouplerUnitTests
+  class TestImportBuffer < Coupler::Test::UnitTest
+    include Coupler
 
-  def setup
-    super
-    @database = stub('database', :run => nil)
-    @dataset = stub('dataset', {
-      :insert_sql => "INSERT INTO \"FOO\" (\"BAR\") VALUES",
-      :db => @database,
-      :first_source_alias => :foo
-    })
-  end
+    def setup
+      super
+      @database = stub('database', :run => nil)
+      @dataset = stub('dataset', {
+        :insert_sql => "INSERT INTO \"FOO\" (\"BAR\") VALUES",
+        :db => @database,
+        :first_source_alias => :foo
+      })
+    end
 
-  test "single insert" do
-    @dataset.expects(:insert_sql).with([:bar], Sequel::LiteralString.new("VALUES")).returns("INSERT INTO \"FOO\" (\"BAR\") VALUES")
-    @dataset.expects(:literal).with([123]).returns("(123)")
-    @database.expects(:run).with("INSERT INTO \"FOO\" (\"BAR\") VALUES (123)")
-    buffer = ImportBuffer.new([:bar], @dataset)
-    buffer.add({:bar => 123})
-    buffer.flush
-  end
+    test "single insert" do
+      @dataset.expects(:insert_sql).with([:bar], Sequel::LiteralString.new("VALUES")).returns("INSERT INTO \"FOO\" (\"BAR\") VALUES")
+      @dataset.expects(:literal).with([123]).returns("(123)")
+      @database.expects(:run).with("INSERT INTO \"FOO\" (\"BAR\") VALUES (123)")
+      buffer = ImportBuffer.new([:bar], @dataset)
+      buffer.add({:bar => 123})
+      buffer.flush
+    end
 
-  test "multiple insert" do
-    @dataset.expects(:literal).with([123]).returns("(123)")
-    @dataset.expects(:literal).with([456]).returns("(456)")
-    @database.expects(:run).with("INSERT INTO \"FOO\" (\"BAR\") VALUES (123), (456)")
-    buffer = ImportBuffer.new([:bar], @dataset)
-    buffer.add({:bar => 123})
-    buffer.add({:bar => 456})
-    buffer.flush
-  end
+    test "multiple insert" do
+      @dataset.expects(:literal).with([123]).returns("(123)")
+      @dataset.expects(:literal).with([456]).returns("(456)")
+      @database.expects(:run).with("INSERT INTO \"FOO\" (\"BAR\") VALUES (123), (456)")
+      buffer = ImportBuffer.new([:bar], @dataset)
+      buffer.add({:bar => 123})
+      buffer.add({:bar => 456})
+      buffer.flush
+    end
 
-  test "max query size / auto-flush" do
-    size = ImportBuffer::MAX_QUERY_SIZE - 50
-    str = "x" * size
+    test "max query size / auto-flush" do
+      size = ImportBuffer::MAX_QUERY_SIZE - 50
+      str = "x" * size
 
-    # bar is a string this time
-    @dataset.expects(:literal).twice.with([str]).returns("(#{str})")
-    @database.expects(:run).twice
-    buffer = ImportBuffer.new([:bar], @dataset)
-    buffer.add({:bar => str})
-    buffer.add({:bar => str})
-    buffer.flush
+      # bar is a string this time
+      @dataset.expects(:literal).twice.with([str]).returns("(#{str})")
+      @database.expects(:run).twice
+      buffer = ImportBuffer.new([:bar], @dataset)
+      buffer.add({:bar => str})
+      buffer.add({:bar => str})
+      buffer.flush
+    end
   end
 end