More attempts to fix CI

Author: kddnewton

lib/syntax_tree/cli.rb

@@ -51,13 +51,15 @@ def handler
       def source
         handler.read(filepath)
       end
+
+      def writable?
+        File.writable?(filepath)
+      end
     end
 
     # An item of work that corresponds to a script content passed via the
     # command line.
     class ScriptItem
-      FILEPATH = :script
-
       attr_reader :source
 
       def initialize(source)
@@ -69,7 +71,30 @@ def handler
       end
 
       def filepath
-        FILEPATH
+        :script
+      end
+
+      def writable?
+        false
+      end
+    end
+
+    # An item of work that correspond to the content passed in via stdin.
+    class STDINItem
+      def handler
+        HANDLERS[".rb"]
+      end
+
+      def filepath
+        :stdin
+      end
+
+      def source
+        $stdin.read
+      end
+
+      def writable?
+        false
       end
     end
 
@@ -196,7 +221,7 @@ def run(item)
 
         source = item.source
         formatted = item.handler.format(source, options.print_width)
-        File.write(filepath, formatted) if item.filepath != :script
+        File.write(filepath, formatted) if item.writable?
 
         color = source == formatted ? Color.gray(filepath) : filepath
         delta = ((Time.now - start) * 1000).round
@@ -386,7 +411,7 @@ def run(argv)
             return 1
           end
 
-        # If we're not reading from stdin and the user didn't supply and
+        # If we're not reading from stdin and the user didn't supply any
         # filepaths to be read, then we exit with the usage message.
         if $stdin.tty? && arguments.empty? && options.scripts.empty?
           warn(HELP)
@@ -403,12 +428,13 @@ def run(argv)
             Dir
               .glob(pattern)
               .each do |filepath|
-                queue << FileItem.new(filepath) if File.file?(filepath)
+                queue << FileItem.new(filepath) if File.readable?(filepath)
               end
           end
+
           options.scripts.each { |script| queue << ScriptItem.new(script) }
         else
-          queue << ScriptItem.new($stdin.read)
+          queue << STDINItem.new
         end
 
         # At the end, we're going to return whether or not this worker ever