You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@whimsical.apache.org by se...@apache.org on 2020/06/13 16:40:18 UTC

[whimsy] branch master updated: Fix tests to take account of code changes

This is an automated email from the ASF dual-hosted git repository.

sebb pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/whimsy.git


The following commit(s) were added to refs/heads/master by this push:
     new 96a11bf  Fix tests to take account of code changes
96a11bf is described below

commit 96a11bf14f101e5cf162d79de7bb465d2d3f30aa
Author: Sebb <se...@apache.org>
AuthorDate: Sat Jun 13 17:40:08 2020 +0100

    Fix tests to take account of code changes
---
 lib/spec/lib/svn_spec.rb           | 4 +---
 lib/spec/lib/svn_wunderbar_spec.rb | 2 +-
 2 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/lib/spec/lib/svn_spec.rb b/lib/spec/lib/svn_spec.rb
index 5a99f64..cbe6aaa 100644
--- a/lib/spec/lib/svn_spec.rb
+++ b/lib/spec/lib/svn_spec.rb
@@ -149,9 +149,7 @@ describe ASF::SVN do
     end
 
     it "getInfo(nil) should fail" do
-      out, err = ASF::SVN.getInfo(nil)
-      expect(out).to eq(nil)
-      expect(err).to eq('path must not be nil')
+      expect { ASF::SVN.getInfo(nil) }.to raise_error(ArgumentError, 'path must not be nil')
     end
 
 # How to ensure local SVN cached auth is not used?    
diff --git a/lib/spec/lib/svn_wunderbar_spec.rb b/lib/spec/lib/svn_wunderbar_spec.rb
index 0334afe..3afecf4 100644
--- a/lib/spec/lib/svn_wunderbar_spec.rb
+++ b/lib/spec/lib/svn_wunderbar_spec.rb
@@ -104,7 +104,7 @@ describe "ASF::SVN.update" do
       end
     end
 
-    expect(rc).to be(nil) # update does not return a value
+    expect(rc).to be(0)
     expect(out['transcript'].class).to equal(Array)
     # could look for "Checked out revision" and "Update to revision"
     expect(out['transcript'][-1]).to eql('+test') #