7709: Use assert_nil instead of assert_equal nil.
authorTom Clegg <tom@curoverse.com>
Tue, 28 Mar 2017 17:24:57 +0000 (13:24 -0400)
committerTom Clegg <tom@curoverse.com>
Tue, 28 Mar 2017 17:24:57 +0000 (13:24 -0400)
services/api/test/functional/arvados/v1/virtual_machines_controller_test.rb
services/api/test/unit/workflow_test.rb

index 9b805af8e38b7095cdddafd67a5f18c63a7e5237..626d7f04b05184514d9a96ab64bd56574d89db11 100644 (file)
@@ -59,8 +59,8 @@ class Arvados::V1::VirtualMachinesControllerTest < ActionController::TestCase
     get :logins, id: vm.uuid
     assert_response :success
     assert_equal 1, json_response['items'].length
-    assert_equal nil, json_response['items'][0]['public_key']
-    assert_equal nil, json_response['items'][0]['authorized_key_uuid']
+    assert_nil json_response['items'][0]['public_key']
+    assert_nil json_response['items'][0]['authorized_key_uuid']
     assert_equal u.uuid, json_response['items'][0]['user_uuid']
     assert_equal 'bobblogin', json_response['items'][0]['username']
   end
index c7c528828d5887d62c62a8077ad4f476719da5c9..29ca09bdab2d200fc9f0f99ead3ec1f6f64cb85b 100644 (file)
@@ -85,8 +85,8 @@ class WorkflowTest < ActiveSupport::TestCase
     definition = "more: etc"
     w.update_attributes!(definition: definition)
     w.reload
-    assert_equal nil, w.name
-    assert_equal nil, w.description
+    assert_nil w.name
+    assert_nil w.description
 
     # Workflow name and desc set using definition yaml should be cleared
     # if definition yaml is cleared
@@ -96,8 +96,8 @@ class WorkflowTest < ActiveSupport::TestCase
     definition = nil
     w.update_attributes!(definition: definition)
     w.reload
-    assert_equal nil, w.name
-    assert_equal nil, w.description
+    assert_nil w.name
+    assert_nil w.description
 
     # Workflow name and desc should be set to provided custom values
     definition = "name: test name 3\ndescription: test desc 3\nother: some more"