Project

General

Profile

バグ #616

mikutterのweb image loaderのtestが環境依存になっている

Added by コスモ 虹川 over 6 years ago. Updated over 2 years ago.

Status:
終了
Priority:
通常
Assignee:
-
Target version:
プラグイン名:
ブランチ:
クラッシュする:

Description

具体的にはこの部分

diff --git a/test/core/test_web_image_loader.rb b/test/core/test_web_image_loade
r.rb
index fbdca34..f6c4857 100644
--- a/test/core/test_web_image_loader.rb
+++ b/test/core/test_web_image_loader.rb
@@ -150,7 +150,7 @@ class TC_GtkWebImageLoader < Test::Unit::TestCase
       WebMock.stub_request(:get, url).to_return(File.open(ICON_TEST){ |io| io.r
ead })
       localpath = Gdk::WebImageLoader.local_path(url)
     }.join
-    assert_equal("/home/toshi/.mikutter/tmp/e9183b9265dcf0728fceceb07444e8c1.pn
g.png", localpath)
+    assert_equal("#{ENV["HOME"]}/.mikutter/tmp/e9183b9265dcf0728fceceb07444e8c1
.png.png", localpath)
   end

   must "is local path" do

~/.mikutterの指す位置はアカウントごとに異なっていいはずです。


Files

0001-use-environment-variable-in-test.patch (978 Bytes) 0001-use-environment-variable-in-test.patch コスモ 虹川, 2013-10-13 01:00
mikutter_run_test.rtf (12.2 KB) mikutter_run_test.rtf コスモ 虹川, 2014-07-26 14:59
#1

Updated by toshi_a 初音 over 6 years ago

  • Tracker changed from 環境対応 to バグ

仰るとおりです。ただし、mikutterには.mikutterの位置を指定する機能が有りますが、テストではこれを指定する手段がありません。なので、提案されたパッチとほぼ同等の変更になると思います。

#2

Updated by コスモ 虹川 over 6 years ago

承知しました

#3

Updated by toshi_a 初音 almost 6 years ago

  • Status changed from 新規 to 様子見
  • プラグイン名 deleted (test_image_loader.rb)

遅くなりましたが対応しました。念の為、近いうちに他の環境で試してみてうまくいけば対応完了とします。

#4

Updated by コスモ 虹川 almost 6 years ago

こちらの環境でテストが失敗しない事を確認しました。
問題なさそうです。
OSX Marvericks 10.9.4
ruby 2.1.2p95
mikutter (hotfix-3.0 latest) rev. 4720482e05f119a90389fac4ada43ebbabc551e2

#5

Updated by toshi_a 初音 over 2 years ago

  • Status changed from 様子見 to 終了

Also available in: Atom PDF