Tests for remote visualization export + import

spec/factories/external_data_imports.rb

@@ -4,10 +4,7 @@ FactoryGirl.define do
 
   # Requires data_import_id (because DataImport links a user)
   factory :external_data_import_with_external_source, class: Carto::ExternalDataImport do
-    before(:create) do |external_data_import|
-      external_source = FactoryGirl.create(:external_source)
-      external_data_import.external_source_id = external_source.id
-    end
+    external_source
   end
 
 end

spec/factories/external_sources.rb

@@ -3,11 +3,7 @@ require_relative '../../app/models/carto/external_source'
 FactoryGirl.define do
 
   factory :external_source, class: Carto::ExternalSource do
-    before(:create) do |external_source|
-      visualization = FactoryGirl.create(:carto_visualization)
-      external_source.visualization_id = visualization.id
-    end
-
+    association :visualization, factory: :carto_visualization, type: 'remote'
     import_url 'http://www.wadus.com'
     rows_counted 1
     size 1

spec/services/carto/user_metadata_export_service_spec.rb

@@ -27,6 +27,15 @@ describe Carto::UserMetadataExportService do
 
     CartoDB::GeocoderUsageMetrics.new(@user.username).incr(:geocoder_here, :success_responses)
 
+    # Convert @table_visualization into a common data imported table
+    sync = FactoryGirl.create(:carto_synchronization, user: @user)
+    @table_visualization.update_attributes!(synchronization: sync)
+    data_import = FactoryGirl.create(:data_import, user: @user, synchronization_id: sync.id, table_id: @table.id)
+    @table.update_attributes!(data_import: data_import)
+    edi = FactoryGirl.create(:external_data_import_with_external_source, data_import: data_import, synchronization: sync)
+    @remote_visualization = edi.external_source.visualization
+    @remote_visualization.update_attributes!(user: @user)
+
     @user.reload
   end
 
@@ -35,6 +44,7 @@ describe Carto::UserMetadataExportService do
     $users_metadata.DEL(gum.send(:user_key_prefix, :geocoder_here, :success_responses, DateTime.now))
 
     destroy_full_visualization(@map, @table, @table_visualization, @visualization)
+    @remote_visualization.destroy
     @tiled_layer.destroy
     @asset.destroy
     @user.destroy