Merge branch '8784-dir-listings'

[arvados.git] / services / api / test / unit / app_version_test.rb

diff --git a/services/api/test/unit/app_version_test.rb b/services/api/test/unit/app_version_test.rb
index e8c8374ed8f1c83e428195c0955169df8ae88b2f..dd880042702e7692e4689ddde59578e4f581c5ea 100644 (file)
--- a/services/api/test/unit/app_version_test.rb
+++ b/services/api/test/unit/app_version_test.rb
@@ -1,3 +1,7 @@
+# Copyright (C) The Arvados Authors. All rights reserved.
+#
+# SPDX-License-Identifier: AGPL-3.0
+
 require 'test_helper'
 
 class AppVersionTest < ActiveSupport::TestCase
@@ -8,7 +12,7 @@ class AppVersionTest < ActiveSupport::TestCase
 
   test 'invoke git processes only on first call' do
     AppVersion.expects(:git).
-      with("status", "-s").once.
+      with("status", "--porcelain").once.
       yields " M services/api/README\n"
     AppVersion.expects(:git).
       with("log", "-n1", "--format=%H").once.
@@ -20,16 +24,19 @@ class AppVersionTest < ActiveSupport::TestCase
     end
   end
 
-  test 'override with configuration' do
+  test 'override with configuration "foobar"' do
     Rails.configuration.source_version = 'foobar'
     assert_equal 'foobar', AppVersion.hash
+  end
+
+  test 'override with configuration false' do
     Rails.configuration.source_version = false
     assert_not_equal 'foobar', AppVersion.hash
   end
 
   test 'override with file' do
     path = Rails.root.join 'git-commit.version'
-    assert(!File.exists?(path),
+    assert(!File.exist?(path),
            "Packaged version file found in source tree: #{path}")
     begin
       File.open(path, 'w') do |f|