dech 0.0.2 → 0.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 30f639c446e2de69ab0c1d076d46737877cc41e6
-  data.tar.gz: 6b016294b7c598b0862980f6d34c7f803aabb4df
+  metadata.gz: e75dc0ac05d00fe019e35c471e4d8bf99d987499
+  data.tar.gz: c373618bf25961474e9812a1c204ffa74dc34419
 SHA512:
-  metadata.gz: 2da66aeff6363ee32ec6458ebb5093661849531da215f5bb27ea07378b1323a45d0fc85a083a0915c33aaf58053216040c4c9edc7dfcd78850fd6ba7f58b684e
-  data.tar.gz: cecbbbb78092e15d48c53a2615905332d291099a51e5630e35d03de3c04e89dbf4c1730dc88e5e1ff5c542fd33e93fd3f5acd0fbcde18188f3a15edad739b1f8
+  metadata.gz: a19d46d53d061d34999c4f2506a797ade35cc7f148fbf0dc36d7ec3830460017db6013128a788eac0c89456cf6272a4c371d65ade4f5fbab0bd3cf43bd9c24a5
+  data.tar.gz: bd39eb4c5f5775afc549ee875a696b2761e335e8be352997a4b6be2cd6645561d792fad4ae56a7cd1147a0122e94378c64ebf1e64af12ca7fb4e4c6cee367d0e

data/lib/dech.rb

@@ -1,6 +1,7 @@
 # coding: utf-8
 
 require "dech/version"
+require "dech/csvio"
 require "dech/price_uploader"
 
 module Dech

data/lib/dech/csvio.rb

@@ -0,0 +1,15 @@
+# coding: utf-8
+
+require 'csv'
+require 'stringio'
+
+module Dech
+  class CSVIO < StringIO
+    class << self
+      def generate(args={})
+        csv_string = CSV.generate{|csv| yield(csv) if block_given? }
+        self.new(csv_string.encode(args[:encoding] || Encoding::Windows_31J))
+      end
+    end
+  end
+end

data/lib/dech/price_uploader.rb

@@ -1,6 +1,7 @@
 # coding: utf-8
 
 require "dech/price_uploader/ponpare"
+require "dech/price_uploader/dena"
 
 module Dech
   module PriceUploader

data/lib/dech/price_uploader/dena.rb

@@ -0,0 +1,10 @@
+# coding: utf-8
+
+require "dech/price_uploader/dena/ftp"
+
+module Dech
+  module PriceUploader
+    module Dena
+    end
+  end
+end

data/lib/dech/price_uploader/dena/ftp.rb

@@ -0,0 +1,65 @@
+# coding: utf-8
+
+require 'dech/csvio'
+require 'net/ftp'
+
+module Dech
+  module PriceUploader
+    module Dena
+      class FTP
+        HEADERS = %w(Code exhibittype Price KtaiPrice)
+
+        attr_accessor :username, :host, :path
+
+        def initialize(args={})
+          @products = args[:products] || []
+          @username = args[:username]
+          @password = args[:password]
+          @host     = args[:host] || "bcmaster1.dena.ne.jp"
+          @path     = args[:path] || "/data.csv"
+        end
+
+        def ready?
+          true
+        end
+
+        def csv
+          Dech::CSVIO.generate do |csv|
+            csv << HEADERS
+            @products.each do |product|
+              csv << [product[:id], "MX", product[:price], product[:price]]
+            end
+          end
+        end
+
+        def save_csv_as(filename)
+          FileUtils.mkdir_p(File.dirname(filename))
+          File.open(filename, "w:windows-31j") do |file|
+            file << csv.string
+          end
+        end
+
+        def upload!
+          ftp_connection{|ftp| ftp.storlines("STOR #{@path}", csv) }
+          true
+        end
+
+        def upload
+          ready? && upload!
+        end
+
+        private
+
+        def ftp_connection(&block)
+          ftp = Net::FTP.new
+          ftp.connect(@host)
+          ftp.login(@username, @password)
+
+          yield(ftp)
+        ensure
+          ftp.close
+        end
+      end
+    end
+  end
+end

data/lib/dech/price_uploader/ponpare/ftps.rb

@@ -1,8 +1,7 @@
 # coding: utf-8
 
-require "net/ftp"
+require "dech/csvio"
 require "double_bag_ftps"
-require "csv"
 
 module Dech
   module PriceUploader
@@ -25,14 +24,12 @@ module Dech
         end
 
         def csv
-          csv_string = CSV.generate do |csv|
+          Dech::CSVIO.generate do |csv|
             csv << HEADERS
             @products.each do |product|
               csv << ["u", product[:id].to_s.downcase, product[:price]]
             end
           end
-
-          StringIO.new(csv_string.encode(Encoding::Windows_31J))
         end
 
         def save_csv_as(filename)

data/lib/dech/version.rb

@@ -1,5 +1,5 @@
 # coding: utf-8
 
 module Dech
-  VERSION = "0.0.2"
+  VERSION = "0.0.3"
 end

data/spec/dech/csvio_spec.rb

@@ -0,0 +1,44 @@
+# coding: utf-8
+
+require "spec_helper"
+
+describe Dech::CSVIO do
+  describe ".generate" do
+    describe "class" do
+      it "should return an instance of StringIO" do
+        actual = Dech::CSVIO.generate
+        expect(actual).to be_is_a(StringIO)
+      end
+
+      it "should return an instance of Dech::CSVIO" do
+        actual = Dech::CSVIO.generate
+        expect(actual).to be_is_a(Dech::CSVIO)
+      end
+    end
+
+    describe "encoding" do
+      it "should have windows-31j external_encoding as default" do
+        actual = Dech::CSVIO.generate
+        expect(actual.external_encoding).to eq(Encoding::Windows_31J)
+      end
+
+      it "should have given external_encoding" do
+        actual = Dech::CSVIO.generate(encoding: Encoding::UTF_8)
+        expect(actual.external_encoding).to eq(Encoding::UTF_8)
+      end
+    end
+
+    describe "headers" do
+      before do
+        @headers = %w(a b c d e)
+        io = Dech::CSVIO.generate{|csv| csv << @headers }
+        @csv = CSV.new(io, headers: true)
+        @csv.readlines
+      end
+
+      it "should be able to set headers" do
+        expect(@csv.headers).to eq(@headers)
+      end
+    end
+  end
+end

data/spec/dech/price_uploader/dena/ftp_spec.rb

@@ -0,0 +1,111 @@
+# coding: utf-8
+
+require "spec_helper"
+
+describe Dech::PriceUploader::Dena::FTP do
+  let(:dech) {
+    Dech::PriceUploader::Dena::FTP.new(
+      products: [{id: "PRODUCT-CODE", price: 9800}],
+      username: "username",
+      password: "password",
+      host:     "example.com"
+    )
+  }
+
+  let(:ftp) {
+    ftp = double("ftp")
+    allow(ftp).to receive(:passive=)
+    allow(ftp).to receive(:connect)
+    allow(ftp).to receive(:login)
+    allow(ftp).to receive(:close)
+    ftp
+  }
+
+  describe "initialize" do
+    context "given no args" do
+      it "should create an instance successfully" do
+        expect(dech).to be_an_instance_of(Dech::PriceUploader::Dena::FTP)
+      end
+    end
+
+    context "given some args" do
+      it "should create an instance successfully" do
+        expect(dech).to be_an_instance_of(Dech::PriceUploader::Dena::FTP)
+      end
+    end
+  end
+
+  describe "#ready?" do
+    it "should be false" do
+      expect(dech.ready?).to be true
+    end
+  end
+
+  describe "#csv" do
+    headers = {
+      "Code"        => String,
+      "exhibittype" => /\AMX\Z/,
+      "KtaiPrice"   => /\d+/
+    }
+
+    describe "headers" do
+      let(:csv){ CSV.new(dech.csv, headers: true) }
+
+      headers.each_key.map{|h| h.encode(Encoding::Windows_31J) }.each do |header|
+        it "should have '#{header}' header" do
+          csv.readlines
+          expect(csv.headers).to be_include(header)
+        end
+      end
+    end
+
+    describe "columns" do
+      let(:csv){ CSV.new(dech.csv, headers: true).read }
+
+      headers.each do |header, type|
+        it "should have #{type} in '#{header}'" do
+          expect(csv[header.encode(Encoding::Windows_31J)]).to be_all{|c| type === c }
+        end
+      end
+    end
+
+    describe "encoding" do
+      let(:io){ dech.csv }
+
+      it "should have windows-31j as external_encoding" do
+        expect(io.external_encoding).to eq(Encoding::Windows_31J)
+      end
+    end
+  end
+
+  describe "#save_csv_as" do
+    let(:filename){ "tmp/#{Time.now.strftime("%Y%m%d_%H%M%S_%N")}.csv" }
+
+    it "should save CSV file as given name" do
+      dech.save_csv_as(filename)
+      expect(Dir.glob("tmp/*")).to be_include(filename)
+    end
+
+    it "should save CSV file in Shift_JIS" do
+      dech.save_csv_as(filename)
+      CSV.open(filename, "r:windows-31j:utf-8", headers: true) do |csv|
+        expect{csv.readlines}.not_to raise_error
+        expect(csv.headers).to eq(Dech::PriceUploader::Dena::FTP::HEADERS)
+      end
+    end
+  end
+
+  describe "#upload" do
+    context "server is ready" do
+      before do
+        allow(ftp).to receive(:nlst).and_return([])
+        expect(ftp).to receive(:storlines)
+        expect(Net::FTP).to receive(:new).and_return(ftp).at_least(:once)
+      end
+
+      it "should upload CSV file to the path on FTP server" do
+        expect(dech.upload).to be true
+      end
+    end
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: dech
 version: !ruby/object:Gem::Version
-  version: 0.0.2
+  version: 0.0.3
 platform: ruby
 authors:
 - OSA Shunsuke
@@ -97,10 +97,15 @@ files:
 - Rakefile
 - dech.gemspec
 - lib/dech.rb
+- lib/dech/csvio.rb
 - lib/dech/price_uploader.rb
+- lib/dech/price_uploader/dena.rb
+- lib/dech/price_uploader/dena/ftp.rb
 - lib/dech/price_uploader/ponpare.rb
 - lib/dech/price_uploader/ponpare/ftps.rb
 - lib/dech/version.rb
+- spec/dech/csvio_spec.rb
+- spec/dech/price_uploader/dena/ftp_spec.rb
 - spec/dech/price_uploader/ponpare/ftps_spec.rb
 - spec/dech_spec.rb
 - spec/spec_helper.rb
@@ -129,6 +134,8 @@ signing_key:
 specification_version: 4
 summary: Utilities gem for e-commerce mall in Japan.
 test_files:
+- spec/dech/csvio_spec.rb
+- spec/dech/price_uploader/dena/ftp_spec.rb
 - spec/dech/price_uploader/ponpare/ftps_spec.rb
 - spec/dech_spec.rb
 - spec/spec_helper.rb