Merge pull request #3 from oudirichi/master

create language directory when language does not exist

lib/cbx_loco/loco_adapter.rb

@@ -166,7 +166,6 @@ def self.import
             api_ext = fmt[:api_ext]
             tag = CbxLoco.asset_tag i18n_file[:id], i18n_file[:name]
 
-            print "Importing \"#{language}\" #{tag} assets... "
 
             api_params = { filter: tag, order: :id }
             case i18n_file[:format]
@@ -180,6 +179,10 @@ def self.import
 
             translations = get "export/locale/#{language}.#{api_ext}", api_params, false
 
+            dirname = File.dirname(file_path)
+            create_directory(dirname) unless File.directory?(dirname)
+
+            print "Importing \"#{language}\" #{tag} assets... "
             f = File.new file_path, "w:UTF-8"
             f.write translations.force_encoding("UTF-8")
             f.close
@@ -203,10 +206,24 @@ def self.import
 
     private
 
+    def self.create_directory(path)
+      print "Creating \"#{path}\" folder... "
+
+      FileUtils.mkdir_p(path)
+      puts "Done!".colorize(:green)
+
+      print "Creating \".keep\" file... "
+      file_path = File.join path, ".keep"
+      FileUtils.touch(file_path)
+
+      puts "Done!".colorize(:green)
+    end
+
     def self.print_error(message)
       puts "\n\n" + message.colorize(:red).bold
     end
 
+
     def self.run_event(event_name)
       CbxLoco.configuration.tasks.with_indifferent_access
 

lib/cbx_loco/version.rb

@@ -1,3 +1,3 @@
 module CbxLoco
-  VERSION = "1.0.0"
+  VERSION = "1.0.1"
 end

spec/cbx_loco/loco_adapter_spec.rb

@@ -19,24 +19,6 @@ def create_files
       f = File.new file_path, "w:UTF-8"
       f.write @fake_translations[i18n_file[:id]].force_encoding("UTF-8")
       f.close
-
-      # create dst dir
-      @fake_languages.each do |language|
-        case i18n_file[:format]
-        when :gettext
-          file_path = CbxLoco.file_path fmt[:path], language, [i18n_file[:name], fmt[:dst_ext]].join(".")
-        when :yaml
-          file_path = CbxLoco.file_path fmt[:path], [i18n_file[:name], language, fmt[:dst_ext]].join(".")
-        end
-
-        dirname = File.dirname(file_path)
-        unless File.directory?(dirname)
-          FileUtils.mkdir_p(dirname)
-          file_path = CbxLoco.file_path dirname, ".keep"
-          f = File.new file_path, "w:UTF-8"
-          f.close
-        end
-      end
     end
   end
 
@@ -315,6 +297,29 @@ def delete_files
       CbxLoco::LocoAdapter.import
     end
 
+    context "when locale folder does not exist" do
+      let!(:mock_file) { double }
+      let(:folder_to_create) { "./locale/en" }
+      let(:keep_file) { File.join(folder_to_create, ".keep") }
+
+      before do
+        allow(File).to receive(:directory?).and_return(true)
+        allow(File).to receive(:directory?).with(folder_to_create).and_return(false)
+        allow(FileUtils).to receive(:mkdir_p).with(folder_to_create)
+        allow(FileUtils).to receive(:touch)
+      end
+
+      it "should create the folder" do
+        CbxLoco::LocoAdapter.import
+        expect(FileUtils).to have_received(:mkdir_p).with(folder_to_create)
+      end
+
+      it "should create .keep file" do
+        CbxLoco::LocoAdapter.import
+        expect(FileUtils).to have_received(:touch).with(keep_file)
+      end
+    end
+
     it "should write API return in language files" do
       CbxLoco::LocoAdapter.import
 

spec/cbx_loco/version_spec.rb

@@ -2,6 +2,6 @@
 
 describe CbxLoco::VERSION do
   it "should return the version number" do
-    expect(CbxLoco::VERSION).to eq "1.0.0"
+    expect(CbxLoco::VERSION).to eq "1.0.1"
   end
 end