go_import 3.0.1 → 3.0.2

data/bin/go-import

@@ -3,6 +3,8 @@
 require "thor"
 require "go_import"
 
+RUNNER_DIR = ".go_import"
+
 class GoImportCommandLine < Thor
 
     desc "about", "About go-import"
@@ -43,51 +45,65 @@ class GoImportCommandLine < Thor
         end
     end
 
-    desc "run", "Executes the current project and create a go.xml file. Existing go.xml will be overwritten, use --output to specify a different filename"
+    desc "run", "Executes the current project and create a go.zip file with data and files. Existing go.zip will be overwritten, use --output to specify a different filename."
     option(:output,
            :desc => "Name of the file where the converted source will be saved. This file should be sent to LIME Go. If the file already exist it will be replaced.",
            :type => :string,
            :required => false)
     def run_import()
-        # run is a Thor reserved word and cant be used as a method name
-
-        runner_dir = ".go_import"
-
-        if Dir.exists?(runner_dir) == false
-            puts "This doesnt look like a go-import project. Are you in the right directory or did you mess with the '#{runner_dir}' folder?"
-            return
-        end
-
-        runner_file = File.expand_path('./.go_import/runner.rb', Dir.pwd)
-        if File.exists?(runner_file) == false
-            puts "I can't run this project. Did you mess with the '#{runner_dir}' folder?"
+        if !is_valid_goimport_project?
             return
         end
 
+        runner_file = File.expand_path("./#{RUNNER_DIR}/runner.rb", Dir.pwd)
         require(runner_file)
-
-        # the source must implement the convert_source method that
-        # returns an instance of GoImport::RootModel
         model = convert_source()
 
-        go_data_filename = options.output.nil? == true ? "go.xml" : options.output
+        if model.documents.files.length > 0 && (!defined?(FILES_FOLDER) || FILES_FOLDER.empty?())
+            puts "WARNING: It looks like you are importing files but FILES_FOLDER has not been set in your converter.rb."
+            puts "WARNING: FILES_FOLDER should be set unless you are only importing files with absolute paths."
+        end
 
-        error = model.sanity_check
-        if error.empty?
-            validation_errors = model.validate
-
-            if validation_errors.empty?
-                model.serialize_to_file(go_data_filename)
-                puts "Source has been been converted into '#{go_data_filename}'."
-            else
-                puts "Source could not be converted due to"
-                puts validation_errors
-            end
+        is_ok, error_msg = can_be_serialized?(model)
+        if is_ok
+            go_data_zip = options.output.nil? == true ? "go.zip" : options.output
+            model.save_to_zip(go_data_zip)
+            puts "Source has been been converted into '#{go_data_zip}'."
         else
             puts "Source could not be converted due to"
-            puts error
+            puts error_msg
+        end
+    end
+
+    private
+    def can_be_serialized?(rootmodel)
+        is_ok = false
+        error = rootmodel.sanity_check
+        if error.empty?
+            error = rootmodel.validate
+
+            if error.empty?
+                is_ok = true
+            end
+        end
+
+        return [is_ok, error]
+    end
+
+    private
+    def is_valid_goimport_project?()
+        if Dir.exists?(RUNNER_DIR) == false
+            puts "This doesnt look like a go-import project. Are you in the right directory or did you mess with the '#{RUNNER_DIR}' folder?"
+            return false
+        end
+
+        runner_file = File.expand_path("./#{RUNNER_DIR}/runner.rb", Dir.pwd)
+        if File.exists?(runner_file) == false
+            puts "I can't run this project. Did you mess with the '#{RUNNER_DIR}' folder?"
+            return false
         end
 
+        return true
     end
 
     private

data/lib/go_import/model/documents.rb

@@ -3,15 +3,14 @@ module GoImport
     #  This class is the container for all documents, ie links and
     #  files.
     class Documents
-        # *** TODO: add files when supported by the backend.
-
         include SerializeHelper
 
-        attr_accessor :links
+        attr_reader :links, :files
 
         def serialize_variables
             [
-             {:id => :links, @type => :links}
+             {:id => :links, @type => :links},
+             {:id => :files, @type => :files}
             ]
         end
 
@@ -21,6 +20,7 @@ module GoImport
 
         def initialize
             @links = []
+            @files = []
         end
 
         def add_link(link)
@@ -34,10 +34,10 @@ module GoImport
 
             if (!link.integration_id.nil? && link.integration_id.length > 0) &&
                     find_link_by_integration_id(link.integration_id) != nil
-                raise AlreadyAddedError, "Already added a link with integration_id #{link.integration_id}"
+                raise AlreadyAddedError, "Already added a link with integration_id '#{link.integration_id}'."
             end
 
-            @links.push(link)
+            @links.push link
 
             return link
         end
@@ -47,5 +47,30 @@ module GoImport
                 link.integration_id == integration_id
             end
         end
+
+        def add_file(file)
+            @files = [] if @files == nil
+
+            if file.nil?
+                return nil
+            end
+
+            file = File.new(file) if !file.is_a?(File)
+
+            if (!file.integration_id.nil? && file.integration_id.length > 0) &&
+                    find_file_by_integration_id(file.integration_id) != nil
+                raise AlreadyAddedError, "Already added a file with integration_id '#{file.integration_id}'."
+            end
+
+            @files.push file
+
+            return file
+        end
+
+        def find_file_by_integration_id(integration_id)
+            return @files.find do |file|
+                file.integration_id == integration_id
+            end
+        end
     end
 end

data/lib/go_import/model/file.rb

@@ -0,0 +1,112 @@
+require 'pathname'
+require_relative '../serialize_helper'
+
+# Note that we name this class File and ruby alread have a File class.
+# To refrence to this
+
+module GoImport
+    class File
+        include SerializeHelper
+        attr_accessor :id, :integration_id, :path, :description
+
+        attr_reader :organization, :created_by, :deal
+
+        attr_writer :name
+
+        # zip_path is used internally when the file is stored in the
+        # zip file that is sent to LIME Go. You should not modify this
+        # property
+        attr_accessor :location_in_zip_file
+
+        def initialize(opt = nil)
+            if !opt.nil?
+                serialize_variables.each do |myattr|
+                    val = opt[myattr[:id]]
+                    instance_variable_set("@" + myattr[:id].to_s, val) if val != nil
+                end
+            end
+        end
+
+        def serialize_name
+            "File"
+        end
+
+        def serialize_variables
+            [ :id, :integration_id, :path, :name, :description, :location_in_zip_file ].map {
+                |p| {
+                    :id => p,
+                    :type => :string
+                }
+            } +
+                [
+                 { :id => :created_by, :type => :coworker_reference },
+                 { :id => :organization, :type => :organization_reference },
+                 { :id => :deal, :type => :deal_reference }
+                ]
+        end
+
+        def name
+            if @name.nil? || @name.empty?
+                if !@path.nil?
+                    return Pathname.new(path).basename.to_s
+                end
+            end
+
+            return @name
+        end
+
+        def has_relative_path?()
+            return Pathname.new(@path).relative?
+        end
+
+        def organization=(org)
+            @organization = OrganizationReference.from_organization(org)
+        end
+
+        def deal=(deal)
+            @deal = DealReference.from_deal(deal)
+        end
+
+        def created_by=(coworker)
+            @created_by = CoworkerReference.from_coworker(coworker)
+        end
+
+        def validate
+            error = String.new
+
+            if @path.nil? || @path.empty?
+                error = "Path is required for file.\n"
+            else
+                if has_relative_path?()
+                    if defined?(FILES_FOLDER) && !FILES_FOLDER.empty?()
+                        root_folder = FILES_FOLDER
+                    else
+                        root_folder = Dir.pwd
+                    end
+
+                    if !::File.exists?("#{root_folder}/#{@path}")
+                        error = "#{error}Can't find file '#{root_folder}/#{@path}'.\n"
+                    end
+                else
+                    if !::File.exists?(@path)
+                        error = "#{error}Can't find file '#{@path}'.\n"
+                    end
+                end
+            end
+
+            if @created_by.nil?
+                error = "#{error}Created_by is required for file.\n"
+            end
+
+            if @organization.nil? && @deal.nil?
+                error = "#{error}A file must have either an organization or a deal.\n"
+            end
+
+            if !@organization.nil? && !@deal.nil?
+                error = "#{error}A file can't be attached to both an organization and a deal."
+            end
+
+            return error
+        end
+    end
+end

data/lib/go_import/model/rootmodel.rb

@@ -1,4 +1,8 @@
 # encoding: utf-8
+
+require 'zip'
+require 'securerandom'
+
 module GoImport
     # The root model for Go import. This class is the container for everything else.
     class RootModel
@@ -213,6 +217,11 @@ module GoImport
             return @documents.add_link(link)
         end
 
+        def add_file(file)
+            @documents = Documents.new if @documents == nil
+
+            return @documents.add_file(file)
+        end
 
         def find_coworker_by_integration_id(integration_id)
             return @coworkers.find do |coworker|
@@ -329,6 +338,13 @@ module GoImport
                 end
             end
 
+            @documents.files.each do |file|
+                validation_message = file.validate
+                if !validation_message.empty?
+                    error = "#{error}\n#{validation_message}"
+                end
+            end
+
             return error.strip
         end
 
@@ -339,6 +355,55 @@ module GoImport
             SerializeHelper::serialize_variables_rexml(elem, self)
         end
 
+        # @!visibility private
+        # zip-filename is the name of the zip file to create
+        def save_to_zip(zip_filename)
+            puts "Trying to save to '#{zip_filename}'..."
+            # saves the model to a zipfile that contains xml data and
+            # document files.
+
+            if ::File.exists?(zip_filename)
+                ::File.delete zip_filename
+            end
+
+            Zip::File.open(zip_filename, Zip::File::CREATE) do |zip_file|
+                puts "Trying to add files to zip..."
+                # We must add files first to the zip file since we
+                # will set each file's location_in_zip_file when the
+                # zip file is created.
+
+                if defined?(FILES_FOLDER) && !FILES_FOLDER.empty?()
+                    puts "Files with relative path are imported from '#{FILES_FOLDER}'."
+                    root_folder = FILES_FOLDER
+                else
+                    puts "Files with relative path are imported from the current folder (#{Dir.pwd})."
+                    root_folder = Dir.pwd
+                end
+
+                # 1) files/ - a folder with all files referenced from
+                # the source.
+                documents.files.each do |file|
+                    # we dont need to check that the file exists since
+                    # we assume that rootmodel.validate has been
+                    # called before save_to_zip.
+                    if file.has_relative_path?
+                        file.location_in_zip_file = "files/#{file.path}"
+                        zip_file.add(file.location_in_zip_file, "#{root_folder}/#{file.path}")
+                    else
+                        file.location_in_zip_file = "files/__abs/#{SecureRandom.uuid}/#{::File.basename(file.path).to_s}"
+                        zip_file.add(file.location_in_zip_file, file.path)
+                    end
+                end
+
+                # 2) go.xml - with all data from source
+                puts "Trying to add organizations, persons, etc to zip..."
+                go_data_file = Tempfile.new('go')
+                serialize_to_file(go_data_file)
+                zip_file.add('go.xml', go_data_file)
+                go_data_file.unlink
+            end
+        end
+
         private
         # returns all items from the object array with duplicate integration ids.
         # To get all organizations with the same integration_id use

data/lib/go_import/phone_helper.rb

@@ -4,7 +4,7 @@ module GoImport
     # The PhoneHelper helps you parse and format phone number strings
     # into pretty looking numbers.
     class PhoneHelper
-        GlobalPhone.db_path = File.join(File.dirname(__FILE__), 'global_phone.json')
+        GlobalPhone.db_path = ::File.join(::File.dirname(__FILE__), 'global_phone.json')
         GlobalPhone.default_territory_name = :se
 
         # Sets the country code used during parsning. The default is

data/lib/go_import/serialize_helper.rb

@@ -25,8 +25,8 @@ module GoImport
                         if (raw_var.respond_to?(:serialize_variables))
                             SerializeHelper::serialize_variables_rexml(element, raw_var)
                         elsif (raw_var.is_a?(Array))
-                            raw_var.each do |raw_var_elem| 
-                                SerializeHelper::serialize_rexml(element, raw_var_elem) 
+                            raw_var.each do |raw_var_elem|
+                                SerializeHelper::serialize_rexml(element, raw_var_elem)
                             end
                         else
                             element.text = raw_var.to_s.encode('UTF-8')
@@ -71,7 +71,7 @@ module GoImport
 
         # @!visibility private
         def self.serialize_to_file(file, obj)
-            File.open(file, 'w') do |f|
+            ::File.open(file, 'w') do |f|
                 f.write(SerializeHelper::serialize(obj))
             end
         end

data/lib/go_import/source.rb

@@ -54,13 +54,13 @@ module GoImport
         private
         def copy_source_to_folder(source_name, project_name)
             puts "Trying to create project '#{project_name}' from source '#{source_name}'..."
-            FileUtils.cp_r File.expand_path(source_name, @path), project_name
+            FileUtils.cp_r ::File.expand_path(source_name, @path), project_name
         end
 
         private
         def install_gems_for_project(project_name)
             puts "Trying to verify that all required gems are installed..."
-            Dir.chdir(File.expand_path(project_name, Dir.pwd)) do
+            Dir.chdir(::File.expand_path(project_name, Dir.pwd)) do
                 exec_but_dont_show_unless_error('bundle install --verbose')
             end
         end

data/sources/easy/.go_import/runner.rb

@@ -17,7 +17,7 @@ def convert_source
     puts "Trying to convert LIME Easy source to LIME Go..."
 
     if !make_sure_database_has_been_exported
-        puts "You must export KONTAKT.mdb to the #{EXPORT_FOLDER} folder."
+        puts "ERROR: You must export KONTAKT.mdb to the #{EXPORT_FOLDER} folder."
         raise
     end
 
@@ -34,7 +34,7 @@ def convert_source
     # start with these since they are referenced
     # from everywhere....
     process_rows COWORKER_FILE do |row|
-        coworkers[row['userIndex']] = row['userId']
+        coworkers[row['idUser']] = row['PowerSellUserID']
         rootmodel.add_coworker(converter.to_coworker(row))
     end
 
@@ -77,6 +77,11 @@ def convert_source
         rootmodel.add_note(converter.to_deal_note(row, coworkers, rootmodel))
     end
 
+    # company documents
+    process_rows ORGANIZATION_DOCUMENT_FILE do |row|
+        rootmodel.add_file(converter.to_organization_document(row, coworkers, rootmodel))
+    end
+
     return rootmodel
 end
 

data/sources/easy/converter.rb

@@ -26,10 +26,14 @@ require 'go_import'
 # 2) Modify this file (the to_* methods) according to your customer's
 # KONTAKT.mdb and wishes.
 #
-# 3) Run easy-to-go.bat in a command prompt.
+# 3) Run go-import run
 #
-# 4) Upload go.xml to LIME Go. First test your import on staging and
+# 4) Upload go.zip to LIME Go. First test your import on staging and
 # when your customer has approved the import, run it on production.
+#
+# You will get a WARNING from 'go-import run' about FILES_FOLDER has
+# not been set. You can ignore the warning since documents are
+# exported with an absolute path from LIME Easy.
 class Converter
     # Turns a user from the User.txt Easy Export file into
     # a go_import coworker.
@@ -262,6 +266,20 @@ class Converter
         return nil
     end
 
+    def to_organization_document(row, coworkers, rootmodel)
+        file = GoImport::File.new()
+
+        file.integration_id = row['PowerSellDocumentID']
+        file.path = row['Path']
+        file.name = row['Comment']
+
+        coworker_id = coworkers[row['idUser-Created']]
+        file.created_by = rootmodel.find_coworker_by_integration_id(coworker_id)
+        file.organization = rootmodel.find_organization_by_integration_id(row['PowerSellCompanyID'])
+
+        return file
+    end
+
     # Turns a row from the Easy exported Project-History.txt file into
     # a go_import model that is used to generate xml
     # Uses coworkers hash to lookup coworkers to connect

data/sources/excel/.go_import/runner.rb

@@ -9,20 +9,26 @@ def convert_source
 
     converter = Converter.new
 
-    # *** TODO:
-    #
-    # Modify the name of the sheets. Or add/remove sheets based on
-    # your Excel file.
-
     # First we read each sheet from the excel file into separate
     # variables
+
+    if defined?(EXCEL_FILE) && !EXCEL_FILE.empty?()
+        if !::File.exists?(EXCEL_FILE)
+            puts "ERROR: Cant find excel file to import: '#{EXCEL_FILE}'."
+            raise
+        end
+    else
+        puts "ERROR: You must set EXCEL_FILE in converter.rb to point to the excel file that should be imported."
+        raise
+    end
+
     excel_workbook = GoImport::ExcelHelper.Open(EXCEL_FILE)
 
     if defined?(COWORKER_SHEET)
         if excel_workbook.has_sheet?(COWORKER_SHEET)
             coworker_rows = excel_workbook.rows_for_sheet COWORKER_SHEET
         else
-            puts "Warning: can't find sheet '#{COWORKER_SHEET}'"
+            puts "WARNING: can't find sheet '#{COWORKER_SHEET}'"
         end
     end
 
@@ -30,7 +36,7 @@ def convert_source
         if excel_workbook.has_sheet?(ORGANIZATION_SHEET)
             organization_rows = excel_workbook.rows_for_sheet ORGANIZATION_SHEET
         else
-            puts "Warning: can't find sheet '#{ORGANIZATION_SHEET}'"
+            puts "WARNING: can't find sheet '#{ORGANIZATION_SHEET}'"
         end
     end
 
@@ -38,7 +44,7 @@ def convert_source
         if excel_workbook.has_sheet?(PERSON_SHEET)
             person_rows = excel_workbook.rows_for_sheet PERSON_SHEET
         else
-            puts "Warning: can't find sheet '#{PERSON_SHEET}'"
+            puts "WARNING: can't find sheet '#{PERSON_SHEET}'"
         end
     end
 
@@ -46,7 +52,7 @@ def convert_source
         if excel_workbook.has_sheet?(DEAL_SHEET)
             deal_rows = excel_workbook.rows_for_sheet DEAL_SHEET
         else
-            puts "Warning: can't find sheet '#{DEAL_SHEET}'"
+            puts "WARNING: can't find sheet '#{DEAL_SHEET}'"
         end
     end
 
@@ -54,7 +60,15 @@ def convert_source
         if excel_workbook.has_sheet?(NOTE_SHEET)
             note_rows = excel_workbook.rows_for_sheet NOTE_SHEET
         else
-            puts "Warning: can't find sheet '#{NOTE_SHEET}'"
+            puts "WARNING: can't find sheet '#{NOTE_SHEET}'"
+        end
+    end
+
+    if defined?(FILE_SHEET)
+        if excel_workbook.has_sheet?(FILE_SHEET)
+            file_rows = excel_workbook.rows_for_sheet FILE_SHEET
+        else
+            puts "WARNING: can't find sheet '#{FILE_SHEET}'"
         end
     end
 
@@ -111,6 +125,13 @@ def convert_source
         end
     end
 
+    if defined?(file_rows) && !file_rows.nil?
+        puts "Trying to convert files..."
+        file_rows.each do |row|
+            rootmodel.add_file(converter.to_file(row, rootmodel))
+        end
+    end
+
     return rootmodel
 end
 

data/sources/excel/converter.rb

@@ -16,6 +16,7 @@ ORGANIZATION_SHEET = "Företag"
 PERSON_SHEET = "Kontaktperson"
 DEAL_SHEET = "Affär"
 NOTE_SHEET = "Anteckningar"
+FILE_SHEET = "Dokument"
 
 # Then you need to modify the script below according to the TODO
 # comments.
@@ -24,6 +25,9 @@ NOTE_SHEET = "Anteckningar"
 # command:
 # go-import run
 
+# If you want to include any file in the import.
+FILES_FOLDER = "D:\\go-import\\file-test\\files"
+
 class Converter
     def configure(rootmodel)
         # *** TODO: Add custom field to your rootmodel here. Custom fields
@@ -127,4 +131,16 @@ class Converter
 
         return note
     end
+
+    def to_file(row, rootmodel)
+        file = GoImport::File.new()
+
+        file.organization = rootmodel.find_organization_by_integration_id(row['Företag'])
+        file.created_by = rootmodel.find_coworker_by_integration_id(row['Skapad Av'])
+        file.name = row['Namn']
+        file.description = row['Kommentar']
+        file.path = row['Path']
+
+        return file
+    end
 end

data/spec/address_spec.rb

@@ -1,4 +1,4 @@
-require 'spec_helper'
+require_relative 'spec_helper'
 require 'go_import'
 
 describe GoImport::Address do
@@ -29,7 +29,7 @@ describe GoImport::Address do
             let (:parse_result){
                 address = GoImport::Address.new
                 line = "CZ-140 00 PRAHA 4"
-                address.parse_zip_and_address_se(line)          
+                address.parse_zip_and_address_se(line)
             }
             it "should be nil" do
                 parse_result.should == nil
@@ -39,11 +39,11 @@ describe GoImport::Address do
             let (:parse_result){
                 address = GoImport::Address.new
                 line = "0511  HELSINKI"
-                address.parse_zip_and_address_se(line)          
+                address.parse_zip_and_address_se(line)
             }
             it "should be nil" do
                 parse_result.should == nil
             end
         end
     end
-end
+end

data/spec/documents_spec.rb

@@ -33,5 +33,32 @@ describe "Documents" do
         documents.find_link_by_integration_id("123").url.should eq "http://dropbox.com"
     end
 
+    it "can add a new file" do
+        # given
+        file = GoImport::File.new
+        file.integration_id = "123key"
+        file.path = "k:\kontakt\databas\dokument"
+
+        # when
+        documents.add_file file
+
+        # then
+        documents.find_file_by_integration_id("123key").path.should eq "k:\kontakt\databas\dokument"
+        documents.files.length.should eq 1
+    end
+
+    it "will not add a new file with a file with the same integration_id already exists" do
+        # given
+        documents.add_file({ :integration_id => "123", :path => "c:\file-1.doc"})
+        documents.files.length.should eq 1
+
+        # when, then
+        expect {
+            documents.add_file({ :integration_id => "123", :path => "c:\file-2.doc"})
+        }.to raise_error(GoImport::AlreadyAddedError)
+        documents.files.length.should eq 1
+        documents.find_file_by_integration_id("123").path.should eq "c:\file-1.doc"
+    end
+
 end
 

data/spec/file_spec.rb

@@ -0,0 +1,140 @@
+require "spec_helper"
+require 'go_import'
+
+describe "File" do
+    let ("file") {
+        GoImport::File.new
+    }
+
+    it "is valid when it has path, created_by and organization" do
+        # given
+        file.path = "spec\\sample_data\\offert.docx"
+        file.created_by = GoImport::CoworkerReference.new( { :integration_id => "123" } )
+        file.organization = GoImport::OrganizationReference.new( { :integration_id => "456" } )
+
+        # when, then
+        file.validate.should eq ""
+    end
+
+    it "is valid when it has name, path, created_by and deal" do
+        # given
+        file.name = "Offert"
+        file.path = "spec\\sample_data\\offert.docx"
+        file.created_by = GoImport::CoworkerReference.new( { :integration_id => "123" } )
+        file.deal = GoImport::DealReference.new( { :integration_id => "456" } )
+
+        # when, then
+        file.validate.should eq ""
+    end
+
+    it "is not valid when it has path and deal" do
+        # must have a created_by
+        # given
+        file.path = "c:\mydocs\deal.xls"
+        file.deal = GoImport::DealReference.new({ :integration_id => "456", :heading => "The new deal" })
+
+        # when, then
+        file.validate.length.should be > 0
+    end
+
+    it "is not valid when it has path and created_by" do
+        # must have an deal or organization
+        # given
+        file.path = "c:\mydocs\deal.xls"
+        file.created_by = GoImport::CoworkerReference.new( { :integration_id => "123", :heading => "billy bob" } )
+
+        # when, then
+        file.validate.length.should be > 0
+    end
+
+    it "is not valid when it has deal and created_by" do
+        # must have a path
+        # given
+        file.created_by = GoImport::CoworkerReference.new( { :integration_id => "123", :heading => "billy bob" } )
+        file.deal = GoImport::DealReference.new({ :integration_id => "456", :heading => "The new deal" })
+
+        # when, then
+        file.validate.length.should be > 0
+    end
+
+    it "knows when a path is not relative" do
+        # given
+        file.path = "c:\files\myfile.doc"
+
+        # when, then
+        file.has_relative_path?().should eq false
+    end
+
+    it "knows when a path is relative" do
+        # given
+        file.path = "files/myfile.doc"
+
+        # when, then
+        file.has_relative_path?().should eq true
+    end
+
+
+    it "will use filename from path as name if name set not explicit" do
+        # given
+        file.path = "some/files/myfile.docx"
+        file.name = ""
+
+        # when, then
+        file.name.should eq 'myfile.docx'
+    end
+
+    it "will use name as name if name is set explicit" do
+        # given
+        file.path = "some/files/myfile.docx"
+        file.name = "This is a filename"
+
+        # when, then
+        file.name.should eq 'This is a filename'
+    end
+
+    it "will not have a name if name is not set or path is empty" do
+        # given
+        file.path = ""
+        file.name = ""
+
+        # when, then
+        file.name.should eq ''
+    end
+
+    it "will auto convert org to org.ref during assignment" do
+        # given
+        org = GoImport::Organization.new({:integration_id => "123", :name => "Beagle Boys!"})
+
+        # when
+        file.organization = org
+
+        # then
+        file.organization.is_a?(GoImport::OrganizationReference).should eq true
+    end
+
+    it "will auto convert deal to deal.ref during assignment" do
+        # given
+        deal = GoImport::Deal.new({:integration_id => "123" })
+        deal.name = "The new deal"
+
+        # when
+        file.deal = deal
+
+        # then
+        file.deal.is_a?(GoImport::DealReference).should eq true
+    end
+
+    it "will auto convert coworker to coworker.ref during assignment" do
+        # given
+        coworker = GoImport::Coworker.new({:integration_id => "123" })
+        coworker.parse_name_to_firstname_lastname_se "Billy Bob"
+
+        # when
+        file.created_by = coworker
+
+        # then
+        file.created_by.is_a?(GoImport::CoworkerReference).should eq true
+    end
+
+
+end

data/spec/link_spec.rb

@@ -101,6 +101,4 @@ describe "Link" do
         # then
         link.created_by.is_a?(GoImport::CoworkerReference).should eq true
     end
-
-
 end

data/spec/rootmodel_spec.rb

@@ -238,6 +238,20 @@ describe "RootModel" do
         rootmodel.documents.links.length.should eq 1
     end
 
+    it "will add a new file" do
+        # given
+        file = GoImport::File.new
+        file.integration_id = "123key"
+        file.path = "k:\kontakt\databas\dokument"
+
+        # when
+        rootmodel.add_file file
+
+        # then
+        rootmodel.documents.find_file_by_integration_id("123key").path.should eq "k:\kontakt\databas\dokument"
+        rootmodel.documents.files.length.should eq 1
+    end
+
     it "will not add a new organizations when the organizations is already added (same integration id)" do
         # given
         rootmodel.add_note({

data/spec/spec_helper.rb

@@ -4,7 +4,7 @@
 
 # Requires supporting ruby files with custom matchers and macros, etc,
 # in spec/support/ and its subdirectories.
-Dir[File.join(File.dirname(File.absolute_path(__FILE__)),"support/**/*.rb")].each { |f| require f }
+Dir[File.join(File.dirname(File.absolute_path(__FILE__)), "support/**/*.rb")].each { |f| require f }
 
 RSpec.configure do |config|
     # ## Mock Framework

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: go_import
 version: !ruby/object:Gem::Version
-  version: 3.0.1
+  version: 3.0.2
   prerelease: 
 platform: ruby
 authors:
@@ -12,7 +12,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-09-10 00:00:00.000000000 Z
+date: 2014-09-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: iso_country_codes
@@ -185,6 +185,7 @@ files:
 - lib/go_import/model/deal_status_reference.rb
 - lib/go_import/model/deal_status_setting.rb
 - lib/go_import/model/documents.rb
+- lib/go_import/model/file.rb
 - lib/go_import/model/link.rb
 - lib/go_import/model/note.rb
 - lib/go_import/model/note_classification.rb
@@ -223,6 +224,11 @@ files:
 - sources/excel/.gitignore
 - sources/excel/.go_import/runner.rb
 - sources/excel/converter.rb
+- sources/excel/files/avtal.docx
+- sources/excel/files/more/avtal.docx
+- sources/excel/files/more/offert-2.pdf
+- sources/excel/files/offert-2.docx
+- sources/excel/files/offert.docx
 - sources/excel/Gemfile
 - sources/excel/sample-data.xlsx
 - sources/VISMA/.gitignore
@@ -241,6 +247,7 @@ files:
 - spec/deal_spec.rb
 - spec/deal_status_reference_spec.rb
 - spec/documents_spec.rb
+- spec/file_spec.rb
 - spec/helpers/csv_helper_spec.rb
 - spec/helpers/email_helper_spec.rb
 - spec/helpers/phone_helper_spec.rb
@@ -287,6 +294,7 @@ test_files:
 - spec/deal_spec.rb
 - spec/deal_status_reference_spec.rb
 - spec/documents_spec.rb
+- spec/file_spec.rb
 - spec/helpers/csv_helper_spec.rb
 - spec/helpers/email_helper_spec.rb
 - spec/helpers/phone_helper_spec.rb