reivt 1.6.0 → 1.6.1

data/lib/reivt/document.rb

@@ -1,67 +1,67 @@
-# An extension of our main module
-#
-# @author [brwnrclse]
-#
-module Reivt
-  # Blueprint for Document objects
-  #
-  # @author [brwnrclse]
-  #
-  # @!attribute [rw] blob
-  #   @return [String] The contents of the file
-  #
-  # @!attribute [rw] content_type
-  #   @return [String] The type of content within the file
-  #
-  # @!attribute [rw] doc_name
-  #   @return [Array<DocumentClass>] The name of the file
-  #
-  # @!attribute [rw] has_diff
-  #   @return [Boolean] Whether or not the file will have a diff
-  #
-  class Document
-    attr_accessor(:blob, :content_type, :doc_name, :name, :has_diff)
-
-    def initialize(blob, content_type, doc_name, has_diff = false)
-      @blob = blob
-      @content_type = content_type
-      @has_diff = has_diff
-      @doc_name = doc_name
-    end
-
-    # Allows Documents objects to be compared using .eql? using their contents
-    #       and file name.
-    #
-    # @param other [Rev::Document] The Document to compare to
-    #
-    # @return [Boolean] True if equal
-    #                False if not equa
-    #
-    def eql?(other)
-      self == other
-    end
-
-    # Allows Documents objects to be compared using hashes created from a
-    #       logical and (&) between the files contents and name.
-    #
-    # @param other [Rev::Document] The Document to compare to
-    #
-    # @return [Fixnum] The & between file content and name
-    #
-    def hash
-      @blob.hash & @doc_name.hash
-    end
-
-    # Allows Documents objects to be compared using == using their contents
-    #       and file name.
-    #
-    # @param other [Rev::Document] The Document to compare to
-    #
-    # @return [Boolean] True if equal
-    #                   False if not equa
-    #
-    def ==(other)
-      @blob == other.blob && @doc_name == other.doc_name
-    end
-  end
-end
+# An extension of our main module
+#
+# @author [brwnrclse]
+#
+module Reivt
+  # Blueprint for Document objects
+  #
+  # @author [brwnrclse]
+  #
+  # @!attribute [rw] blob
+  #   @return [String] The contents of the file
+  #
+  # @!attribute [rw] content_type
+  #   @return [String] The type of content within the file
+  #
+  # @!attribute [rw] doc_name
+  #   @return [Array<DocumentClass>] The name of the file
+  #
+  # @!attribute [rw] has_diff
+  #   @return [Boolean] Whether or not the file will have a diff
+  #
+  class Document
+    attr_accessor(:blob, :content_type, :doc_name, :name, :has_diff)
+
+    def initialize(blob, content_type, doc_name, has_diff = false)
+      @blob = blob
+      @content_type = content_type
+      @has_diff = has_diff
+      @doc_name = doc_name
+    end
+
+    # Allows Documents objects to be compared using .eql? using their contents
+    #       and file name.
+    #
+    # @param other [Rev::Document] The Document to compare to
+    #
+    # @return [Boolean] True if equal
+    #                False if not equa
+    #
+    def eql?(other)
+      self == other
+    end
+
+    # Allows Documents objects to be compared using hashes created from a
+    #       logical and (&) between the files contents and name.
+    #
+    # @param other [Rev::Document] The Document to compare to
+    #
+    # @return [Fixnum] The & between file content and name
+    #
+    def hash
+      @blob.hash & @doc_name.hash
+    end
+
+    # Allows Documents objects to be compared using == using their contents
+    #       and file name.
+    #
+    # @param other [Rev::Document] The Document to compare to
+    #
+    # @return [Boolean] True if equal
+    #                   False if not equa
+    #
+    def ==(other)
+      @blob == other.blob && @doc_name == other.doc_name
+    end
+  end
+end

data/lib/reivt/exception.rb

@@ -1,48 +1,52 @@
-# An extension of our main module
-#
-# @author [brwnrclse]
-#
-module Reivt
-  # Custom errors for dealing with repos in revit
-  #
-  # @author [bwrnrclse]
-  #
-  class BaemptyException < IOError
-    def initialize(msg = 'Empty File/Repo!')
-      msg << "\n visit this url for more information: https://bitbucket.org/vaemoi/revit/wiki/Exceptions%20-%20Bare%20Repo"
-      super(msg)
-    end
-  end
-
-  # Custom errors for dealing with graphql in revit
-  #
-  # @author [bwrnrclse]
-  #
-  class GraphQLDataException < IOError
-    def initialize(msg = 'GraphQL Data Exception')
-      msg << "\n visit this url for more information: https://bitbucket.org/vaemoi/revit/wiki/Exceptions%20-%20GraphQL%20Data%20Issue"
-      super(msg)
-    end
-  end
-
-  # Custom errors for dealing with graphql in revit
-  #
-  # @author [bwrnrclse]
-  #
-  class GraphQLValidationException < IOError
-    def initialize(msg = 'GraphQL Validation Exception')
-      msg << "\n visit this url for more information: https://bitbucket.org/vaemoi/revit/wiki/Exceptions%20-%20GraphQL%20Validation%20Error"
-      super(msg)
-    end
-  end
-
-  # Custom error for dealing with users who aren't logged in
-  #
-  # @author [firaga]
-  #
-  class LoginException < StandardError
-    def initialize(msg = 'LoginException - run revit login to login.')
-      super(msg)
-    end
-  end
-end
+# An extension of our main module
+#
+# @author [brwnrclse]
+#
+module Reivt
+  # Custom errors for dealing with repos in revit
+  #
+  # @author [bwrnrclse]
+  #
+  class BaemptyException < IOError
+    def initialize(msg = 'Bare or Empty repo passed!')
+      msg = "BaemptyException:\n" + msg + "\nVisit here for more info => " \
+            'link.vaemoi.co/revitbaempty'
+      super(msg)
+    end
+  end
+
+  # Custom errors for dealing with graphql in revit
+  #
+  # @author [bwrnrclse]
+  #
+  class GraphQLDataException < IOError
+    def initialize(msg = 'Error in GraphQL API request!')
+      msg = "GraphQLDataException:\n" + msg + "\nVisit her for more info => " \
+            'link.vaemoi.co/revitgqde'
+      super(msg)
+    end
+  end
+
+  # Custom errors for dealing with graphql in revit
+  #
+  # @author [bwrnrclse]
+  #
+  class GraphQLValidationException < IOError
+    def initialize(msg = 'Error in GraphQL Validation!')
+      msg = "GraphQLValidationException:\n" + msg + "\nVisit here for more " \
+          'info => link.vaemoi.co/revitgqve'
+      super(msg)
+    end
+  end
+
+  # Custom error for dealing with users who aren't logged in
+  #
+  # @author [firaga]
+  #
+  class LoginException < StandardError
+    def initialize
+      msg = 'LoginException: Run ' + Paint['revit login', :yellow] + ' first.'
+      super(msg)
+    end
+  end
+end

data/lib/reivt/util.rb

@@ -1,112 +1,113 @@
-# rubocop:disable Metrics/CyclomaticComplexity, Metrics/AbcSize
-require 'rugged'
-require 'tempfile'
-
-# An extension of our main module
-#
-# @author [brwnrclse]
-#
-module Reivt
-  # A collection of utility functions for use with reivt
-  #
-  # @author [brwnrclse]
-  #
-  module Util
-    # Wrapper for creating a Document object from a path
-    #
-    # @param path [String] Location on the filesystem to access for files
-    #
-    # @param has_diff [Boolean] A flag to tell if the document has a dif
-    #
-    # @param spinner [TTY:Spinner] A spinner for feedback
-    #
-    # @return [Revit::Document] A newly created Document object
-    #
-    def self.doc_from_path(path, has_diff = false)
-      blob = File.read(path).gsub("\r\n", "\n") #gsub replace CRLF line endings and converts to LF
-      doc_name = File.basename(path)
-      content_type = Util.ext_to_lang(File.extname(path))
-      Document.new(blob, content_type, doc_name, has_diff)
-    end
-
-    # Creates a list of documents from files in the passed directory
-    #
-    # @param path [String] Location on the filesystem to access for files
-    #
-    # @return [Array<Reivt::Document>] List of documents from directory
-    #
-    def self.docs_from_dir(path)
-      docs = []
-
-      # Recursively get all file paths in a directory
-      entries = Dir.glob("#{path}/**/*").reject { |entry| Dir.exist?(entry) }
-
-      entries.each do |entry|
-        docs.push(Util.doc_from_path(entry)) if entry != '.' && entry != '..'
-      end
-
-      docs
-    end
-
-    # Creates a list of documents from repo changelist
-    #
-    # @param path [String] Location on the filesystem to access for files
-    #
-    # @return [Array<Reivt::Document>] List of documents from repo commit
-    #
-    def self.docs_from_repo(path)
-      docs = []
-      repo = Rugged::Repository.discover(path)
-
-      if repo.bare? || repo.empty?
-        raise Reivt::BaemptyException, "Bad repo: #{path}"
-      end
-
-      commit = repo.head.target
-      diff = commit.parents.first.diff(commit)
-
-      diff.find_similar!
-      diff.each_delta do |d|
-        file_path = d.new_file[:path]
-        docs.push(Util.doc_from_path("#{path}/#{file_path}", true))
-
-        ofile = repo.lookup(d.old_file[:oid])
-        nfile = repo.lookup(d.new_file[:oid])
-        diff_file = Tempfile.new([File.basename(file_path).to_s, '.diff'])
-
-        diff_file.write(ofile.diff(nfile).to_s)
-        docs.push(Util.doc_from_path(diff_file.path))
-        diff_file.close
-        diff_file.unlink
-      end
-
-      docs
-    end
-
-    # Translates a file extenstion to its corresponding programming language.
-    #
-    # @param extension [String] The file extension to be analyzed
-    #
-    # @return [String] The name of the language corresponding to the extension
-    #
-    def self.ext_to_lang(extension)
-      case extension.downcase
-      when '.rb', '.gemspec' then 'ruby'
-      when '.py', '.pyw', '.pyd', '.py3' then 'python'
-      when '.c', '.cpp', '.h', '.hpp' then 'c/c++'
-      when '.js', '.jsx' then 'javascript'
-      when '.java', '.class' then 'java'
-      when '.json' then 'json'
-      when '.yml', '.yaml' then 'yaml'
-      when '.xml' then 'xml'
-      when '.txt', ' ' then 'plain text'
-      when '.sh', '.zsh', '.bash' then 'shell script'
-      when '.md', '.markdown' then 'markdown'
-      when '.fountain' then 'fountain'
-      when '.diff' then 'diff'
-      else
-        'unknown'
-      end
-    end
-  end
-end
+# rubocop:disable Metrics/CyclomaticComplexity, Metrics/AbcSize
+require 'rugged'
+require 'tempfile'
+
+# An extension of our main module
+#
+# @author [brwnrclse]
+#
+module Reivt
+  # A collection of utility functions for use with reivt
+  #
+  # @author [brwnrclse]
+  #
+  module Util
+    # Wrapper for creating a Document object from a path, removes all carriage
+    #   returns from file contents
+    #
+    # @param path [String] Location on the filesystem to access for files
+    #
+    # @param has_diff [Boolean] A flag to tell if the document has a dif
+    #
+    # @param spinner [TTY:Spinner] A spinner for feedback
+    #
+    # @return [Revit::Document] A newly created Document object
+    #
+    def self.doc_from_path(path, has_diff = false)
+      blob = File.read(path).gsub("\r\n", "\n")
+      doc_name = File.basename(path)
+      content_type = Util.ext_to_lang(File.extname(path))
+      Document.new(blob, content_type, doc_name, has_diff)
+    end
+
+    # Creates a list of documents from files in the passed directory
+    #
+    # @param path [String] Location on the filesystem to access for files
+    #
+    # @return [Array<Reivt::Document>] List of documents from directory
+    #
+    def self.docs_from_dir(path)
+      docs = []
+
+      # Recursively get all file paths in a directory
+      entries = Dir.glob("#{path}/**/*").reject { |entry| Dir.exist?(entry) }
+
+      entries.each do |entry|
+        docs.push(Util.doc_from_path(entry)) if entry != '.' && entry != '..'
+      end
+
+      docs
+    end
+
+    # Creates a list of documents from repo changelist
+    #
+    # @param path [String] Location on the filesystem to access for files
+    #
+    # @return [Array<Reivt::Document>] List of documents from repo commit
+    #
+    def self.docs_from_repo(path)
+      docs = []
+      repo = Rugged::Repository.discover(path)
+
+      if repo.bare? || repo.empty?
+        raise Reivt::BaemptyException, "Bad repo: #{path}"
+      end
+
+      commit = repo.head.target
+      diff = commit.parents.first.diff(commit)
+
+      diff.find_similar!
+      diff.each_delta do |d|
+        file_path = d.new_file[:path]
+        docs.push(Util.doc_from_path("#{path}/#{file_path}", true))
+
+        ofile = repo.lookup(d.old_file[:oid])
+        nfile = repo.lookup(d.new_file[:oid])
+        diff_file = Tempfile.new([File.basename(file_path).to_s, '.diff'])
+
+        diff_file.write(ofile.diff(nfile).to_s)
+        docs.push(Util.doc_from_path(diff_file.path))
+        diff_file.close
+        diff_file.unlink
+      end
+
+      docs
+    end
+
+    # Translates a file extenstion to its corresponding programming language.
+    #
+    # @param extension [String] The file extension to be analyzed
+    #
+    # @return [String] The name of the language corresponding to the extension
+    #
+    def self.ext_to_lang(extension)
+      case extension.downcase
+      when '.rb', '.gemspec' then 'ruby'
+      when '.py', '.pyw', '.pyd', '.py3' then 'python'
+      when '.c', '.cpp', '.h', '.hpp' then 'c/c++'
+      when '.js', '.jsx' then 'javascript'
+      when '.java', '.class' then 'java'
+      when '.json' then 'json'
+      when '.yml', '.yaml' then 'yaml'
+      when '.xml' then 'xml'
+      when '.txt', ' ' then 'plain text'
+      when '.sh', '.zsh', '.bash' then 'shell script'
+      when '.md', '.markdown' then 'markdown'
+      when '.fountain' then 'fountain'
+      when '.diff' then 'diff'
+      else
+        'unknown'
+      end
+    end
+  end
+end

data/lib/reivt/version.rb

@@ -1,3 +1,3 @@
-module Reivt
-  VERSION = '1.6.0'.freeze
-end
+module Reivt
+  VERSION = '1.6.1'.freeze
+end

data/lib/reivt.rb

@@ -1,42 +1,42 @@
-require 'fileutils'
-require 'logger'
-require 'paint'
-require 'pstore'
-
-# The cli tool for rev. Revit process paths passed to it in order to create new
-#       documents to review online. Each path passed is determined to be a file,
-#       directory or git repo. In the case of the first two "Document" objects
-#       are created as a simple wrapper around important info we need to pass to
-#       the Rev API. In the case of the repo, we only use files from the latest
-#       commit (or a passed commit hash) along with diffs for each. The iops are
-#       grouped together and the api calls are made last (aside from creating
-#       the rev). This allows us to have UI feedback (spinnys) that'll keep the
-#       user informed about what's going on
-#
-# @author [brwnrclse]
-#
-module Reivt
-  FileUtils.mkdir_p("#{Dir.home}/.reivt")
-
-  REIVT_STORE = PStore.new("#{Dir.home}/.reivt/reivt.pstore")
-  LOGGER = Logger.new(STDOUT)
-  DEVLOGGER = Logger.new("#{Dir.home}/.reivt/reivt-dev.log")
-
-  DEVLOGGER.datetime_format = '%Y-%m-%d %H:%M:%S '
-  LOGGER.formatter = proc do |severity, _, _, msg|
-    case severity
-    when 'ERROR' then "\n#{Paint[severity, :red]}: #{msg}\n"
-    when 'FATAL' then "\n#{Paint[severity, :red]}L #{msg}\n"
-    else
-      "\n#{msg}\n"
-    end
-  end
-end
-
-require 'reivt/api'
-require 'reivt/auth'
-require 'reivt/cli'
-require 'reivt/document'
-require 'reivt/exception'
-require 'reivt/util'
-require 'reivt/version'
+require 'fileutils'
+require 'logger'
+require 'paint'
+require 'pstore'
+
+# The cli tool for rev. Revit process paths passed to it in order to create new
+#       documents to review online. Each path passed is determined to be a file,
+#       directory or git repo. In the case of the first two "Document" objects
+#       are created as a simple wrapper around important info we need to pass to
+#       the Rev API. In the case of the repo, we only use files from the latest
+#       commit (or a passed commit hash) along with diffs for each. The iops are
+#       grouped together and the api calls are made last (aside from creating
+#       the rev). This allows us to have UI feedback (spinnys) that'll keep the
+#       user informed about what's going on
+#
+# @author [brwnrclse]
+#
+module Reivt
+  FileUtils.mkdir_p("#{Dir.home}/.reivt")
+
+  REIVT_STORE = PStore.new("#{Dir.home}/.reivt/reivt.pstore")
+  LOGGER = Logger.new(STDOUT)
+  DEVLOGGER = Logger.new("#{Dir.home}/.reivt/reivt-dev.log")
+
+  DEVLOGGER.datetime_format = '%Y-%m-%d %H:%M:%S '
+  LOGGER.formatter = proc do |severity, _, _, msg|
+    case severity
+    when 'ERROR' then "\n#{Paint[severity, :red]}: #{msg}\n"
+    when 'FATAL' then "\n#{Paint[severity, :red]}L #{msg}\n"
+    else
+      "\n#{msg}\n"
+    end
+  end
+end
+
+require 'reivt/api'
+require 'reivt/auth'
+require 'reivt/cli'
+require 'reivt/document'
+require 'reivt/exception'
+require 'reivt/util'
+require 'reivt/version'

data/revit.gemspec

@@ -1,44 +1,44 @@
-# rubocop disable Metrics/BlockLength
-# coding: utf-8
-
-lib = File.expand_path('../lib', __FILE__)
-$LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
-require 'reivt/version'
-
-Gem::Specification.new do |spec|
-  spec.name          = 'reivt'
-  spec.version       = Reivt::VERSION
-  spec.authors       = ['vaemoi', 'Barry Harris, Tarik Massac']
-  spec.email         = ['dev@vaemoi.co', 'brwnrclse@vaemoi.co', 'tai@vaemoi.co']
-  spec.summary       = 'NOTICE!!! - we will be moving to revit in a few days ' \
-                        "https://rubygems.org/gems/revit !!! \n\n" \
-                        ' Creating and uploading code reviews to use with rev' \
-                        ' - https://rev.vaemoi.co'
-  spec.homepage      = 'https://rev.vaemoi.co'
-  spec.license       = 'MIT'
-
-  spec.files = `git ls-files -z`.split("\x0").reject do |f|
-    f.match(%r{^(test|spec|features)/})
-  end
-
-  spec.bindir        = 'exe'
-  spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
-  spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
-  spec.require_paths = ['lib']
-  spec.required_ruby_version = '>= 2.3.0'
-
-  spec.add_dependency 'graphql-client', '~> 0.8.0'
-  spec.add_dependency 'thor', '~> 0.19.4'
-  spec.add_dependency 'tty-spinner', '~> 0.4.1'
-
-  spec.add_runtime_dependency 'bcrypt', '~> 3.1', '>= 3.1.11'
-  spec.add_runtime_dependency 'paint', '~> 2.0', '>= 2.0.0'
-  spec.add_runtime_dependency 'sysrandom', '~> 1.0', '>= 1.0.5'
-
-  spec.add_development_dependency 'bundler', '~> 1.13'
-  spec.add_development_dependency 'pry', '~> 0.10.4'
-  spec.add_development_dependency 'rake', '~> 10.0'
-  spec.add_development_dependency 'rspec', '~> 3.5', '>= 3.5.0'
-  spec.add_development_dependency 'rubocop', '~> 0.48.1'
-  spec.add_development_dependency 'webmock', '~> 2.3', '>= 2.3.2'
-end
+# rubocop disable Metrics/BlockLength
+# coding: utf-8
+
+lib = File.expand_path('../lib', __FILE__)
+$LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
+require 'reivt/version'
+
+Gem::Specification.new do |spec|
+  spec.name          = 'reivt'
+  spec.version       = Reivt::VERSION
+  spec.authors       = ['vaemoi', 'Barry Harris, Tarik Massac']
+  spec.email         = ['dev@vaemoi.co', 'brwnrclse@vaemoi.co', 'tai@vaemoi.co']
+  spec.summary       = 'NOTICE!!! - we will be moving to revit in a few days ' \
+                        "https://rubygems.org/gems/revit !!! \n\n" \
+                        ' Creating and uploading code reviews to use with rev' \
+                        ' - https://rev.vaemoi.co'
+  spec.homepage      = 'https://rev.vaemoi.co'
+  spec.license       = 'MIT'
+
+  spec.files = `git ls-files -z`.split("\x0").reject do |f|
+    f.match(%r{^(test|spec|features)/})
+  end
+
+  spec.bindir        = 'exe'
+  spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
+  spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
+  spec.require_paths = ['lib']
+  spec.required_ruby_version = '>= 2.3.0'
+
+  spec.add_dependency 'graphql-client', '~> 0.8.0'
+  spec.add_dependency 'thor', '~> 0.19.4'
+  spec.add_dependency 'tty-spinner', '~> 0.4.1'
+
+  spec.add_runtime_dependency 'bcrypt', '~> 3.1', '>= 3.1.11'
+  spec.add_runtime_dependency 'paint', '~> 2.0', '>= 2.0.0'
+  spec.add_runtime_dependency 'sysrandom', '~> 1.0', '>= 1.0.5'
+
+  spec.add_development_dependency 'bundler', '~> 1.13'
+  spec.add_development_dependency 'pry', '~> 0.10.4'
+  spec.add_development_dependency 'rake', '~> 10.0'
+  spec.add_development_dependency 'rspec', '~> 3.5', '>= 3.5.0'
+  spec.add_development_dependency 'rubocop', '~> 0.48.1'
+  spec.add_development_dependency 'webmock', '~> 2.3', '>= 2.3.2'
+end