MuranoCLI 3.0.2 → 3.0.4

data/spec/SyncRoot_spec.rb

@@ -1,4 +1,4 @@
-# Last Modified: 2017.08.17 /coding: utf-8
+# Last Modified: 2017.09.12 /coding: utf-8
 # frozen_string_literal: true
 
 # Copyright © 2016-2017 Exosite LLC.
@@ -13,7 +13,7 @@ require 'MrMurano/SyncUpDown'
 require '_workspace'
 
 RSpec.describe MrMurano::SyncRoot do
-  include_context "WORKSPACE"
+  include_context 'WORKSPACE'
 
   after(:example) do
     MrMurano::SyncRoot.instance.reset
@@ -27,7 +27,7 @@ RSpec.describe MrMurano::SyncRoot do
     #       warning: method redefined; discarding old description
     #   /exo/clients/exosite/MuranoCLIs/MuranoCLI+landonb/spec/SyncRoot_spec.rb:30:
     #       warning: previous definition of description was here
-    if !defined?(User)
+    unless defined?(User)
       class User
         def self.description
           %(describe user)
@@ -35,7 +35,7 @@ RSpec.describe MrMurano::SyncRoot do
       end
     end
     MrMurano::SyncRoot.instance.add('user', User, 'U', true)
-    if !defined?(Role)
+    unless defined?(Role)
       class Role
         def self.description
           %(describe role)
@@ -52,58 +52,60 @@ RSpec.describe MrMurano::SyncRoot do
 
     @options = {}
     @options.define_singleton_method(:method_missing) do |mid, *args|
-      if mid.to_s.match(/^(.+)=$/) then
-        self[$1.to_sym] = args.first
+      if mid.to_s =~ /^(.+)=$/
+        self[Regexp.last_match(1).to_sym] = args.first
       else
         self[mid]
       end
     end
   end
 
-  it "has defaults" do
+  it 'has defaults' do
     ret = MrMurano::SyncRoot.instance.bydefault
     expect(ret).to eq(['user'])
   end
 
-  it "iterates on each" do
-    ret=[]
+  it 'iterates on each' do
+    ret = []
     MrMurano::SyncRoot.instance.each { |a, _b, _c, _d| ret << a }
-    expect(ret).to eq(["user", "role"])
+    expect(ret).to eq(%w[user role])
   end
 
-  it "iterates only on selected" do
+  it 'iterates only on selected' do
     @options.role = true
-    ret=[]
+    ret = []
     MrMurano::SyncRoot.instance.each_filtered(@options) { |a, _b, _c, _d| ret << a }
-    expect(ret).to eq(["role"])
+    expect(ret).to eq(['role'])
   end
 
-  it "selects all" do
+  it 'selects all' do
     @options.all = true
     MrMurano::SyncRoot.instance.check_same(@options)
-    expect(@options).to eq({:all=>true, :user=>true, :role=>true})
+    expect(@options).to eq(all: true, user: true, role: true)
   end
 
-  it "selects defaults when none" do
+  it 'selects defaults when none' do
     MrMurano::SyncRoot.instance.check_same(@options)
-    expect(@options).to eq({:user=>true})
+    expect(@options).to eq(user: true)
   end
 
-  it "selects custom defaults when none" do
+  it 'selects custom defaults when none' do
     $cfg['sync.bydefault'] = 'role'
     MrMurano::SyncRoot.instance.check_same(@options)
-    expect(@options).to eq({role: true})
+    expect(@options).to eq(role: true)
   end
 
-  it "builds option params" do
+  it 'builds option params' do
     ret = []
     MrMurano::SyncRoot.instance.each_option do |s, l, d|
       ret << [s, l, d]
     end
-    expect(ret).to eq([
-      ["-U", "--[no-]user", "describe user"],
-      ["-G", "--[no-]role", "describe role"],
-    ])
+    expect(ret).to eq(
+      [
+        ['-U', '--[no-]user', 'describe user'],
+        ['-G', '--[no-]role', 'describe role'],
+      ]
+    )
   end
 end
 

data/spec/SyncUpDown_spec.rb

@@ -1,3 +1,10 @@
+# Last Modified: 2017.09.12 /coding: utf-8
+# frozen_string_literal: true
+
+# Copyright © 2016-2017 Exosite LLC.
+# License: MIT. See LICENSE.txt.
+#  vim:tw=0:ts=2:sw=2:et:ai
+
 require 'MrMurano/verbosing'
 require 'MrMurano/version'
 require 'MrMurano/Config'
@@ -20,9 +27,7 @@ class TSUD
     @project_section = :routes
   end
 
-  def api_id
-    @api_id
-  end
+  attr_reader :api_id
 
   def api_id?
     @valid_api_id
@@ -42,10 +47,10 @@ RSpec::Matchers.define :pathname_globs do |glob|
   end
 end
 
-ITEM_UPDATED_AT="2017-06-24T00:45:15.564Z"
+ITEM_UPDATED_AT = '2017-06-24T00:45:15.564Z'
 
 RSpec.describe MrMurano::SyncUpDown do
-  include_context "WORKSPACE"
+  include_context 'WORKSPACE'
   before(:example) do
     MrMurano::SyncRoot.instance.reset
     $cfg = MrMurano::Config.new
@@ -59,35 +64,37 @@ RSpec.describe MrMurano::SyncUpDown do
     $cfg['application.id'] = 'XYZ'
   end
 
-  context "status" do
-    it "warns with missing directory" do
+  context 'status' do
+    it 'warns with missing directory' do
       t = TSUD.new
       expect(t).to receive(:warning).once.with(/Skipping missing location.*/)
       ret = t.status
       expect(ret).to eq(
-        { toadd: [], todel: [], tomod: [], unchg: [], skipd: [], clash: [] }
+        toadd: [], todel: [], tomod: [], unchg: [], skipd: [], clash: []
       )
     end
 
-    it "finds nothing in empty directory" do
+    it 'finds nothing in empty directory' do
       FileUtils.mkpath(@project_dir + '/tsud')
       t = TSUD.new
       ret = t.status
       expect(ret).to eq(
-        { toadd: [], todel: [], tomod: [], unchg: [], skipd: [], clash: [] }
+        toadd: [], todel: [], tomod: [], unchg: [], skipd: [], clash: []
       )
     end
 
-    it "finds things there but not here" do
+    it 'finds things there but not here' do
       FileUtils.mkpath(@project_dir + '/tsud')
       t = TSUD.new
-      expect(t).to receive(:list).once.and_return([
-        {name: 1},
-        {name: 2},
-        {name: 3},
-      ])
+      expect(t).to receive(:list).once.and_return(
+        [
+          { name: 1 },
+          { name: 2 },
+          { name: 3 },
+        ]
+      )
       ret = t.status
-      expect(ret).to eq({
+      expect(ret).to eq(
         toadd: [],
         todel: [
           {
@@ -104,24 +111,27 @@ RSpec.describe MrMurano::SyncUpDown do
             name: 3,
             synckey: 3,
             synctype: TSUD.description,
-          }],
+          },
+        ],
         tomod: [],
         unchg: [],
         skipd: [],
-        clash: [],
-      })
+        clash: []
+      )
     end
 
-    it "finds things there but not here; asdown" do
+    it 'finds things there but not here; asdown' do
       FileUtils.mkpath(@project_dir + '/tsud')
       t = TSUD.new
-      expect(t).to receive(:list).once.and_return([
-        {name: 1},
-        {name: 2},
-        {name: 3},
-      ])
-      ret = t.status({asdown: true})
-      expect(ret).to eq({
+      expect(t).to receive(:list).once.and_return(
+        [
+          { name: 1 },
+          { name: 2 },
+          { name: 3 },
+        ]
+      )
+      ret = t.status(asdown: true)
+      expect(ret).to eq(
         todel: [],
         toadd: [
           { name: 1, synckey: 1, synctype: TSUD.description },
@@ -131,21 +141,21 @@ RSpec.describe MrMurano::SyncUpDown do
         tomod: [],
         unchg: [],
         skipd: [],
-        clash: [],
-      })
+        clash: []
+      )
     end
 
-    it "finds things here but not there" do
+    it 'finds things here but not there' do
       FileUtils.mkpath(@project_dir + '/tsud')
       FileUtils.touch(@project_dir + '/tsud/one.lua')
       FileUtils.touch(@project_dir + '/tsud/two.lua')
       t = TSUD.new
       expect(t).to receive(:to_remote_item).and_return(
-        {name: 'one.lua'},
-        {name: 'two.lua'},
+        { name: 'one.lua' },
+        name: 'two.lua',
       )
       ret = t.status
-      expect(ret).to match({
+      expect(ret).to match(
         toadd: [
           {
             name: 'one.lua',
@@ -164,25 +174,27 @@ RSpec.describe MrMurano::SyncUpDown do
         tomod: [],
         unchg: [],
         skipd: [],
-        clash: [],
-      })
+        clash: []
+      )
     end
 
-    it "finds things here and there" do
+    it 'finds things here and there' do
       FileUtils.mkpath(@project_dir + '/tsud')
       FileUtils.touch(@project_dir + '/tsud/one.lua')
       FileUtils.touch(@project_dir + '/tsud/two.lua')
       t = TSUD.new
-      expect(t).to receive(:list).once.and_return([
-        {name: 'one.lua'},
-        {name: 'two.lua'},
-      ])
+      expect(t).to receive(:list).once.and_return(
+        [
+          { name: 'one.lua' },
+          { name: 'two.lua' },
+        ]
+      )
       expect(t).to receive(:to_remote_item).and_return(
-        {name: 'one.lua'},
-        {name: 'two.lua'},
+        { name: 'one.lua' },
+        name: 'two.lua',
       )
       ret = t.status
-      expect(ret).to match({
+      expect(ret).to match(
         tomod: [
           {
             name: 'one.lua',
@@ -201,25 +213,27 @@ RSpec.describe MrMurano::SyncUpDown do
         toadd: [],
         unchg: [],
         skipd: [],
-        clash: [],
-      })
+        clash: []
+      )
     end
     it "finds things here and there; but they're the same" do
       FileUtils.mkpath(@project_dir + '/tsud')
       FileUtils.touch(@project_dir + '/tsud/one.lua')
       FileUtils.touch(@project_dir + '/tsud/two.lua')
       t = TSUD.new
-      expect(t).to receive(:list).once.and_return([
-        {name: 'one.lua'},
-        {name: 'two.lua'},
-      ])
+      expect(t).to receive(:list).once.and_return(
+        [
+          { name: 'one.lua' },
+          { name: 'two.lua' },
+        ]
+      )
       expect(t).to receive(:to_remote_item).and_return(
-        {name: 'one.lua'},
-        {name: 'two.lua'},
+        { name: 'one.lua' },
+        name: 'two.lua',
       )
       expect(t).to receive(:docmp).twice.and_return(false)
       ret = t.status
-      expect(ret).to match({
+      expect(ret).to match(
         unchg: [
           {
             name: 'one.lua',
@@ -238,41 +252,41 @@ RSpec.describe MrMurano::SyncUpDown do
         toadd: [],
         tomod: [],
         skipd: [],
-        clash: [],
-      })
+        clash: []
+      )
     end
 
-    it "calls diff" do
+    it 'calls diff' do
       FileUtils.mkpath(@project_dir + '/tsud')
       FileUtils.touch(@project_dir + '/tsud/one.lua')
       t = TSUD.new
       expect(t).to receive(:list).once.and_return([
-        {name: 'one.lua'}
-      ])
+                                                    { name: 'one.lua' },
+                                                  ])
       expect(t).to receive(:to_remote_item).and_return(
-        {name: 'one.lua'}
+        name: 'one.lua'
       )
-      expect(t).to receive(:dodiff).once.and_return("diffed output")
-      ret = t.status({diff: true})
-      expect(ret).to match({
+      expect(t).to receive(:dodiff).once.and_return('diffed output')
+      ret = t.status(diff: true)
+      expect(ret).to match(
         tomod: [
           {
             name: 'one.lua',
             synckey: 'one.lua',
             synctype: TSUD.description,
             local_path: an_instance_of(Pathname),
-            diff: "diffed output",
+            diff: 'diffed output',
           },
         ],
         todel: [],
         toadd: [],
         unchg: [],
         skipd: [],
-        clash: [],
-      })
+        clash: []
+      )
     end
 
-    context "Filtering" do
+    context 'Filtering' do
       before(:example) do
         FileUtils.mkpath(@project_dir + '/tsud/ga')
         FileUtils.mkpath(@project_dir + '/tsud/gb')
@@ -283,113 +297,107 @@ RSpec.describe MrMurano::SyncUpDown do
         FileUtils.touch(@project_dir + '/tsud/five.lua')    # toadd
         FileUtils.touch(@project_dir + '/tsud/ga/six.lua')  # toadd
         @t = TSUD.new
-        expect(@t).to receive(:list).once.and_return([
-          MrMurano::SyncUpDown::Item.new({name: 'eight.lua', updated_at: ITEM_UPDATED_AT}), # todel
-          MrMurano::SyncUpDown::Item.new({name: 'four.lua', updated_at: ITEM_UPDATED_AT}),  # unchg
-          MrMurano::SyncUpDown::Item.new({name: 'one.lua', updated_at: ITEM_UPDATED_AT}),   # tomod
-          MrMurano::SyncUpDown::Item.new({name: 'seven.lua', updated_at: ITEM_UPDATED_AT}), # todel
-          MrMurano::SyncUpDown::Item.new({name: 'three.lua', updated_at: ITEM_UPDATED_AT}), # unchg
-          MrMurano::SyncUpDown::Item.new({name: 'two.lua', updated_at: ITEM_UPDATED_AT}),   # tomod
-        ])
-        expect(@t).to receive(:to_remote_item).
-          with(anything(), pathname_globs('**/one.lua')).
-          and_return(MrMurano::SyncUpDown::Item.new({name: 'one.lua', updated_at: ITEM_UPDATED_AT}))
-        expect(@t).to receive(:to_remote_item).
-          with(anything(), pathname_globs('**/two.lua')).
-          and_return(MrMurano::SyncUpDown::Item.new({name: 'two.lua', updated_at: ITEM_UPDATED_AT}))
-        expect(@t).to receive(:to_remote_item).
-          with(anything(), pathname_globs('**/three.lua')).
-          and_return(MrMurano::SyncUpDown::Item.new({name: 'three.lua', updated_at: ITEM_UPDATED_AT}))
-        expect(@t).to receive(:to_remote_item).
-          with(anything(), pathname_globs('**/four.lua')).
-          and_return(MrMurano::SyncUpDown::Item.new({name: 'four.lua', updated_at: ITEM_UPDATED_AT}))
-        expect(@t).to receive(:to_remote_item).
-          with(anything(), pathname_globs('**/five.lua')).
-          and_return(MrMurano::SyncUpDown::Item.new({name: 'five.lua', updated_at: ITEM_UPDATED_AT}))
-        expect(@t).to receive(:to_remote_item).
-          with(anything(), pathname_globs('**/six.lua')).
-          and_return(MrMurano::SyncUpDown::Item.new({name: 'six.lua', updated_at: ITEM_UPDATED_AT}))
-
-        expect(@t).to receive(:docmp).with(have_attributes({name: 'one.lua'}),anything()).and_return(true)
-        expect(@t).to receive(:docmp).with(have_attributes({name: 'two.lua'}),anything()).and_return(true)
-        expect(@t).to receive(:docmp).with(have_attributes({name: 'three.lua'}),anything()).and_return(false)
-        expect(@t).to receive(:docmp).with(have_attributes({name: 'four.lua'}),anything()).and_return(false)
+        expect(@t).to receive(:list).once.and_return(
+          [
+            MrMurano::SyncUpDown::Item.new(name: 'eight.lua', updated_at: ITEM_UPDATED_AT), # todel
+            MrMurano::SyncUpDown::Item.new(name: 'four.lua', updated_at: ITEM_UPDATED_AT),  # unchg
+            MrMurano::SyncUpDown::Item.new(name: 'one.lua', updated_at: ITEM_UPDATED_AT),   # tomod
+            MrMurano::SyncUpDown::Item.new(name: 'seven.lua', updated_at: ITEM_UPDATED_AT), # todel
+            MrMurano::SyncUpDown::Item.new(name: 'three.lua', updated_at: ITEM_UPDATED_AT), # unchg
+            MrMurano::SyncUpDown::Item.new(name: 'two.lua', updated_at: ITEM_UPDATED_AT),   # tomod
+          ]
+        )
+        expect(@t).to receive(:to_remote_item)
+          .with(anything, pathname_globs('**/one.lua'))
+          .and_return(MrMurano::SyncUpDown::Item.new(name: 'one.lua', updated_at: ITEM_UPDATED_AT))
+        expect(@t).to receive(:to_remote_item)
+          .with(anything, pathname_globs('**/two.lua'))
+          .and_return(MrMurano::SyncUpDown::Item.new(name: 'two.lua', updated_at: ITEM_UPDATED_AT))
+        expect(@t).to receive(:to_remote_item)
+          .with(anything, pathname_globs('**/three.lua'))
+          .and_return(MrMurano::SyncUpDown::Item.new(name: 'three.lua', updated_at: ITEM_UPDATED_AT))
+        expect(@t).to receive(:to_remote_item)
+          .with(anything, pathname_globs('**/four.lua'))
+          .and_return(MrMurano::SyncUpDown::Item.new(name: 'four.lua', updated_at: ITEM_UPDATED_AT))
+        expect(@t).to receive(:to_remote_item)
+          .with(anything, pathname_globs('**/five.lua'))
+          .and_return(MrMurano::SyncUpDown::Item.new(name: 'five.lua', updated_at: ITEM_UPDATED_AT))
+        expect(@t).to receive(:to_remote_item)
+          .with(anything, pathname_globs('**/six.lua'))
+          .and_return(MrMurano::SyncUpDown::Item.new(name: 'six.lua', updated_at: ITEM_UPDATED_AT))
+
+        expect(@t).to receive(:docmp).with(have_attributes(name: 'one.lua'), anything).and_return(true)
+        expect(@t).to receive(:docmp).with(have_attributes(name: 'two.lua'), anything).and_return(true)
+        expect(@t).to receive(:docmp).with(have_attributes(name: 'three.lua'), anything).and_return(false)
+        expect(@t).to receive(:docmp).with(have_attributes(name: 'four.lua'), anything).and_return(false)
       end
 
-      it "Returns all with no filter" do
+      it 'Returns all with no filter' do
         ret = @t.status
-        expect(ret).to match({
+        expect(ret).to match(
           unchg: [
             have_attributes(
-              {
-                name: 'four.lua',
-                synckey: 'four.lua',
-                synctype: TSUD.description,
-                local_path: pathname_globs('**/four.lua'),
-              }),
+              name: 'four.lua',
+              synckey: 'four.lua',
+              synctype: TSUD.description,
+              local_path: pathname_globs('**/four.lua')
+            ),
             have_attributes(
-              {
-                name: 'three.lua',
-                synckey: 'three.lua',
-                synctype: TSUD.description,
-                local_path: pathname_globs('**/three.lua'),
-              }),
+              name: 'three.lua',
+              synckey: 'three.lua',
+              synctype: TSUD.description,
+              local_path: pathname_globs('**/three.lua')
+            ),
           ],
           toadd: [
             have_attributes(
-              {
-                name: 'five.lua',
-                synckey: 'five.lua',
-                synctype: TSUD.description,
-                local_path: pathname_globs('**/five.lua'),
-              }),
+              name: 'five.lua',
+              synckey: 'five.lua',
+              synctype: TSUD.description,
+              local_path: pathname_globs('**/five.lua')
+            ),
             have_attributes(
-              {
-                name: 'six.lua',
-                synckey: 'six.lua',
-                synctype: TSUD.description,
-                local_path: pathname_globs('**/six.lua'),
-              }),
+              name: 'six.lua',
+              synckey: 'six.lua',
+              synctype: TSUD.description,
+              local_path: pathname_globs('**/six.lua')
+            ),
           ],
           todel: [
             have_attributes(
-              {
-                name: 'eight.lua',
-                synckey: 'eight.lua',
-                synctype: TSUD.description,
-              }),
+              name: 'eight.lua',
+              synckey: 'eight.lua',
+              synctype: TSUD.description
+            ),
             have_attributes(
-              {
-                name: 'seven.lua',
-                synckey: 'seven.lua',
-                synctype: TSUD.description,
-              }),
+              name: 'seven.lua',
+              synckey: 'seven.lua',
+              synctype: TSUD.description
+            ),
           ],
           tomod: [
             have_attributes(
-              {
-                name: 'one.lua',
-                synckey: 'one.lua',
-                synctype: TSUD.description,
-                local_path: pathname_globs('**/one.lua'),
-              }),
+              name: 'one.lua',
+              synckey: 'one.lua',
+              synctype: TSUD.description,
+              local_path: pathname_globs('**/one.lua')
+            ),
             have_attributes(
-              {
-                name: 'two.lua',
-                synckey: 'two.lua',
-                synctype: TSUD.description,
-                local_path: pathname_globs('**/two.lua'),
-              }),
+              name: 'two.lua',
+              synckey: 'two.lua',
+              synctype: TSUD.description,
+              local_path: pathname_globs('**/two.lua')
+            ),
           ],
           skipd: [],
-          clash: [],
-        })
+          clash: []
+        )
       end
 
-      it "Finds local path globs" do
+      it 'Finds local path globs' do
         ret = @t.status({}, ['**/ga/*.lua'])
-        expect(ret).to match({
-          unchg: [ ],
+        expect(ret).to match(
+          unchg: [],
           toadd: [
             have_attributes(
               name: 'six.lua',
@@ -398,7 +406,7 @@ RSpec.describe MrMurano::SyncUpDown do
               local_path: an_instance_of(Pathname)
             ),
           ],
-          todel: [ ],
+          todel: [],
           tomod: [
             have_attributes(
               name: 'two.lua',
@@ -408,25 +416,25 @@ RSpec.describe MrMurano::SyncUpDown do
             ),
           ],
           skipd: [],
-          clash: [],
-        })
+          clash: []
+        )
       end
 
-      it "Finds nothing with specific matcher" do
+      it 'Finds nothing with specific matcher' do
         ret = @t.status({}, ['#foo'])
-        expect(ret).to match({
+        expect(ret).to match(
           unchg: [],
           toadd: [],
           todel: [],
           tomod: [],
           skipd: [],
-          clash: [],
-        })
+          clash: []
+        )
       end
 
-      it "gets all the details" do
-        ret = @t.status({unselected: true})
-        expect(ret).to match({
+      it 'gets all the details' do
+        ret = @t.status(unselected: true)
+        expect(ret).to match(
           unchg: [
             have_attributes(
               name: 'four.lua',
@@ -434,7 +442,7 @@ RSpec.describe MrMurano::SyncUpDown do
               synctype: TSUD.description,
               selected: true,
               local_path: pathname_globs('**/four.lua')
-              ),
+            ),
             have_attributes(
               name: 'three.lua',
               synckey: 'three.lua',
@@ -490,92 +498,96 @@ RSpec.describe MrMurano::SyncUpDown do
             ),
           ],
           skipd: [],
-          clash: [],
-        })
+          clash: []
+        )
       end
     end
   end
 
-  context "localitems" do
+  context 'localitems' do
     before(:example) do
       FileUtils.mkpath('tsud')
       FileUtils.touch('tsud/one.lua')
       FileUtils.touch('tsud/two.lua')
       @t = TSUD.new
     end
-    it "finds local items" do
+    it 'finds local items' do
       expect(@t).to receive(:to_remote_item).and_return(
-        {name: 'one.lua'},
-        {name: 'two.lua'},
+        { name: 'one.lua' },
+        name: 'two.lua',
       )
       ret = @t.localitems(Pathname.new(@project_dir + '/tsud').realpath)
-      expect(ret).to match([
-        {
-          name: 'one.lua',
-          local_path: an_instance_of(Pathname)
-        },
-        {
-          name: 'two.lua',
-          local_path: an_instance_of(Pathname)
-        },
-      ])
+      expect(ret).to match(
+        [
+          {
+            name: 'one.lua',
+            local_path: an_instance_of(Pathname),
+          },
+          {
+            name: 'two.lua',
+            local_path: an_instance_of(Pathname),
+          },
+        ]
+      )
     end
 
-    it "takes an array from to_remote_item" do
+    it 'takes an array from to_remote_item' do
       expect(@t).to receive(:to_remote_item).and_return(
         [
-          {name: 'one:1'},
-          {name: 'one:2'},
+          { name: 'one:1' },
+          { name: 'one:2' },
         ],
         [
-          {name: 'two:1'},
-          {name: 'two:2'},
+          { name: 'two:1' },
+          { name: 'two:2' },
         ],
       )
       ret = @t.localitems(Pathname.new(@project_dir + '/tsud').realpath)
-      expect(ret).to match([
-        {
-          name: 'one:1',
-          local_path: an_instance_of(Pathname),
-        },
-        {
-          name: 'one:2',
-          local_path: an_instance_of(Pathname),
-        },
-        {
-          name: 'two:1',
-          local_path: an_instance_of(Pathname),
-        },
-        {
-          name: 'two:2',
-          local_path: an_instance_of(Pathname),
-        },
-      ])
+      expect(ret).to match(
+        [
+          {
+            name: 'one:1',
+            local_path: an_instance_of(Pathname),
+          },
+          {
+            name: 'one:2',
+            local_path: an_instance_of(Pathname),
+          },
+          {
+            name: 'two:1',
+            local_path: an_instance_of(Pathname),
+          },
+          {
+            name: 'two:2',
+            local_path: an_instance_of(Pathname),
+          },
+        ]
+      )
     end
   end
 
-  context "download" do
-    it "defaults to :id if @itemkey missing" do
+  context 'download' do
+    it 'defaults to :id if @itemkey missing' do
       FileUtils.mkpath(@project_dir + '/tsud')
       FileUtils.touch(@project_dir + '/tsud/one.lua')
       lp = Pathname.new(@project_dir + '/tsud/one.lua').realpath
       t = TSUD.new
-      expect(t).to receive(:fetch).once.with(1).and_yield("foo")
-      t.download(lp, {id: 1, updated_at: ITEM_UPDATED_AT})
+      expect(t).to receive(:fetch).once.with(1).and_yield('foo')
+      t.download(lp, id: 1, updated_at: ITEM_UPDATED_AT)
     end
   end
 
-  context "doing diffs" do
+  context 'doing diffs' do
     before(:example) do
       FileUtils.mkpath(@project_dir + '/tsud')
       @t = TSUD.new
       @scpt = Pathname.new(@project_dir + '/tsud/one.lua')
-      @scpt.open('w'){|io| io << %{-- fake lua\nreturn 0\n}}
+      @scpt.open('w') { |io| io << %(-- fake lua\nreturn 0\n) }
       @scpt = @scpt.realpath
     end
 
-    it "nothing when same." do
-      expect(@t).to receive(:fetch).and_yield(%{-- fake lua\nreturn 0\n})
+    it 'nothing when same.' do
+      expect(@t).to receive(:fetch).and_yield(%(-- fake lua\nreturn 0\n))
       ret = @t.dodiff(
         {
           name: 'one.lua',
@@ -584,15 +596,15 @@ RSpec.describe MrMurano::SyncUpDown do
         },
         nil
       )
-      if Gem.win_platform? then
+      if Gem.win_platform?
         expect(ret).to match(/FC: no differences encountered/)
       else
         expect(ret).to eq('')
       end
     end
 
-    it "something when different." do
-      expect(@t).to receive(:fetch).and_yield(%{-- fake lua\nreturn 2\n})
+    it 'something when different.' do
+      expect(@t).to receive(:fetch).and_yield(%(-- fake lua\nreturn 2\n))
       ret = @t.dodiff(
         {
           name: 'one.lua',
@@ -604,8 +616,8 @@ RSpec.describe MrMurano::SyncUpDown do
       expect(ret).not_to eq('')
     end
 
-    it "uses script in item" do
-      script = %{-- fake lua\nreturn 2\n}
+    it 'uses script in item' do
+      script = %(-- fake lua\nreturn 2\n)
       expect(@t).to receive(:fetch).and_yield(script)
       ret = @t.dodiff(
         {
@@ -616,7 +628,7 @@ RSpec.describe MrMurano::SyncUpDown do
         },
         nil
       )
-      if Gem.win_platform? then
+      if Gem.win_platform?
         expect(ret).to match(/FC: no differences encountered/)
       else
         expect(ret).to eq('')
@@ -624,56 +636,61 @@ RSpec.describe MrMurano::SyncUpDown do
     end
   end
 
-  context "syncup" do
+  context 'syncup' do
     before(:example) do
       FileUtils.mkpath(@project_dir + '/tsud')
       @t = TSUD.new
     end
 
-    it "removes" do
-      expect(@t).to receive(:list).once.and_return([
-        {name: 1},
-        {name: 2},
-        {name: 3},
-      ])
+    it 'removes' do
+      expect(@t).to receive(:list).once.and_return(
+        [
+          { name: 1 },
+          { name: 2 },
+          { name: 3 },
+        ]
+      )
       expect(@t).to receive(:remove).exactly(3).times
-      @t.syncup({delete: true})
+      @t.syncup(delete: true)
     end
 
-    it "creates" do
+    it 'creates' do
       FileUtils.touch(@project_dir + '/tsud/one.lua')
       FileUtils.touch(@project_dir + '/tsud/two.lua')
 
-      expect(@t).to receive(:upload).twice.with(kind_of(Pathname), kind_of(MrMurano::SyncUpDown::Item), false)
-      @t.syncup({create: true})
+      expect(@t).to receive(:upload).twice
+                                    .with(kind_of(Pathname), kind_of(MrMurano::SyncUpDown::Item), false)
+      @t.syncup(create: true)
     end
 
-    it "updates" do
+    it 'updates' do
       FileUtils.touch(@project_dir + '/tsud/one.lua')
       FileUtils.touch(@project_dir + '/tsud/two.lua')
-      expect(@t).to receive(:list).once.and_return([
-        MrMurano::SyncUpDown::Item.new({name: 'one.lua', updated_at: ITEM_UPDATED_AT}),
-        MrMurano::SyncUpDown::Item.new({name: 'two.lua', updated_at: ITEM_UPDATED_AT})
-      ])
+      expect(@t).to receive(:list).once.and_return(
+        [
+          MrMurano::SyncUpDown::Item.new(name: 'one.lua', updated_at: ITEM_UPDATED_AT),
+          MrMurano::SyncUpDown::Item.new(name: 'two.lua', updated_at: ITEM_UPDATED_AT),
+        ]
+      )
 
       expect(@t).to receive(:upload).twice.with(
         kind_of(Pathname), kind_of(MrMurano::SyncUpDown::Item), true
       )
       expect(@t).to receive(:to_remote_item).and_return(
-        {name: 'one.lua'},
-        {name: 'two.lua'},
+        { name: 'one.lua' },
+        name: 'two.lua',
       )
-      @t.syncup({update: true})
+      @t.syncup(update: true)
     end
   end
 
-  context "syncdown" do
+  context 'syncdown' do
     before(:example) do
       FileUtils.mkpath(@project_dir + '/tsud')
       @t = TSUD.new
     end
 
-    it "removes" do
+    it 'removes' do
       FileUtils.touch(@project_dir + '/tsud/one.lua')
       FileUtils.touch(@project_dir + '/tsud/two.lua')
 
@@ -682,11 +699,13 @@ RSpec.describe MrMurano::SyncUpDown do
       expect(FileTest.exist?(@project_dir + '/tsud/two.lua')).to be false
     end
 
-    it "creates" do
-      expect(@t).to receive(:list).once.and_return([
-        MrMurano::SyncUpDown::Item.new({name: 'one.lua', updated_at: ITEM_UPDATED_AT}),
-        MrMurano::SyncUpDown::Item.new({name: 'two.lua', updated_at: ITEM_UPDATED_AT})
-      ])
+    it 'creates' do
+      expect(@t).to receive(:list).once.and_return(
+        [
+          MrMurano::SyncUpDown::Item.new(name: 'one.lua', updated_at: ITEM_UPDATED_AT),
+          MrMurano::SyncUpDown::Item.new(name: 'two.lua', updated_at: ITEM_UPDATED_AT),
+        ]
+      )
 
       expect(@t).to receive(:fetch).twice.and_yield("--foo\n")
       @t.syncdown(create: true)
@@ -694,18 +713,20 @@ RSpec.describe MrMurano::SyncUpDown do
       expect(FileTest.exist?(@project_dir + '/tsud/two.lua')).to be true
     end
 
-    it "updates" do
+    it 'updates' do
       FileUtils.touch(@project_dir + '/tsud/one.lua')
       FileUtils.touch(@project_dir + '/tsud/two.lua')
-      expect(@t).to receive(:list).once.and_return([
-        MrMurano::SyncUpDown::Item.new({name: 'one.lua', updated_at: ITEM_UPDATED_AT}),
-        MrMurano::SyncUpDown::Item.new({name: 'two.lua', updated_at: ITEM_UPDATED_AT})
-      ])
+      expect(@t).to receive(:list).once.and_return(
+        [
+          MrMurano::SyncUpDown::Item.new(name: 'one.lua', updated_at: ITEM_UPDATED_AT),
+          MrMurano::SyncUpDown::Item.new(name: 'two.lua', updated_at: ITEM_UPDATED_AT),
+        ]
+      )
 
       expect(@t).to receive(:fetch).twice.and_yield("--foo\n")
       expect(@t).to receive(:to_remote_item).and_return(
-        MrMurano::SyncUpDown::Item.new({name: 'one.lua', updated_at: ITEM_UPDATED_AT}),
-        MrMurano::SyncUpDown::Item.new({name: 'two.lua', updated_at: ITEM_UPDATED_AT})
+        MrMurano::SyncUpDown::Item.new(name: 'one.lua', updated_at: ITEM_UPDATED_AT),
+        MrMurano::SyncUpDown::Item.new(name: 'two.lua', updated_at: ITEM_UPDATED_AT)
       )
       @t.syncdown(update: true)
       expect(FileTest.exist?(@project_dir + '/tsud/one.lua')).to be true
@@ -748,5 +769,3 @@ RSpec.describe MrMurano::SyncUpDown do
 #  end
 end
 
-#  vim: set ai et sw=2 ts=2 :
-