sync_issues 0.0.1 → 0.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 851be84063a71f5f66d9f4433ef9e412b81ed1a4
-  data.tar.gz: 525105d38ebbb876c30e49c37b5b0e3ebbde1470
+  metadata.gz: cd67c4c0bb708d7b393d620da1a325d0af3cb1bc
+  data.tar.gz: bc62f99652b3bc6ca0c01c1cf157a0687a3e9bf3
 SHA512:
-  metadata.gz: ba04b87e9d73cc9ffd051750c9738a9bc7ec8a7edea7b759274fd89aaf3d970fbeed37bbbb5dd74f31c73da855681a9f4d862229d3c4198fbfa3d22ef8bfcf27
-  data.tar.gz: 4606a46c96a4468bc805cb9f68f6c1dd4fd87e20834410c9166e576a28d57c6d8e565df336ecdc8fd2405d27a27561ac45835409900610557062fc750697fcf2
+  metadata.gz: f1be77f29065be111b5334bf17f0833298ce09865953cf5d841b0f5ccea60ab5d3e6e984b455ff09d02b90307147defa5db20b462cb5ef1c1ece0bb6511c5c61
+  data.tar.gz: 2b93b1d43ed7d83adfb0caf8576ed09c0d74e8ed3c8a50649b7a967b48b1d4098df1d50b4cc3e3de8c5c032899b32bb2a82142c1c688d486b5fa92a7763d6a51

data/lib/sync_issues/comparison.rb

@@ -0,0 +1,37 @@
+require_relative 'error'
+
+module SyncIssues
+  # Comparison represents differences between Issues (local and GitHub)
+  class Comparison
+    attr_reader :changed, :content, :title
+
+    def initialize(issue, github_issue)
+      @changed = []
+      @content = github_issue.body
+      @title = github_issue.title
+      compare(issue, github_issue)
+    end
+
+    def changed?
+      !@changed.empty?
+    end
+
+    private
+
+    def compare(issue, github_issue)
+      unless issue.new_title.nil?
+        @changed << 'title'
+        @title = issue.new_title
+      end
+      unless content_matches?(issue.content, github_issue.body)
+        @changed << 'body'
+        @content = issue.content
+      end
+    end
+
+    def content_matches?(first, second)
+      second.delete!("\r")
+      first.gsub(/\[x\]/, '[ ]') == second.gsub(/\[x\]/, '[ ]')
+    end
+  end
+end

data/lib/sync_issues/github.rb

@@ -25,9 +25,8 @@ module SyncIssues
       raise Error, 'repository not found'
     end
 
-    def update_issue(repository, github_issue, issue)
-      @client.update_issue(repository.full_name, github_issue.number,
-                           issue.new_title || issue.title, issue.content)
+    def update_issue(repository, issue_number, title, content)
+      @client.update_issue(repository.full_name, issue_number, title, content)
     end
 
     private

data/lib/sync_issues/synchronizer.rb

@@ -1,3 +1,4 @@
+require_relative 'comparison'
 require_relative 'error'
 require_relative 'parser'
 require 'English'
@@ -85,13 +86,13 @@ module SyncIssues
     end
 
     def update_issue(repository, issue, github_issue)
-      changed = []
-      changed << 'title' unless issue.new_title.nil?
-      changed << 'body' unless issue.content == github_issue.body
-      return if changed.empty?
+      comparison = Comparison.new(issue, github_issue)
+      return unless comparison.changed?
 
-      puts "Updating #{changed.join(', ')} on ##{github_issue.number}"
-      SyncIssues.github.update_issue(repository, github_issue, issue)
+      changed = comparison.changed.join(', ')
+      puts "Updating #{changed} on ##{github_issue.number}"
+      SyncIssues.github.update_issue(repository, github_issue.number,
+                                     comparison.title, comparison.content)
     end
   end
 end

data/lib/sync_issues/version.rb

@@ -1,4 +1,4 @@
 # SyncIssues
 module SyncIssues
-  VERSION = '0.0.1'.freeze
+  VERSION = '0.1.0'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sync_issues
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.1.0
 platform: ruby
 authors:
 - Bryce Boe
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-02-02 00:00:00.000000000 Z
+date: 2016-02-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: docopt
@@ -66,6 +66,7 @@ files:
 - bin/sync_issues
 - lib/sync_issues.rb
 - lib/sync_issues/command.rb
+- lib/sync_issues/comparison.rb
 - lib/sync_issues/error.rb
 - lib/sync_issues/github.rb
 - lib/sync_issues/issue.rb