[WIP] oj testでディレクトリ指定できないバグ修正

[fukatani]

/home/ryo/work/github/rust-debug/src/
というディレクトリにおいて
oj test -c ./a.out -d ./testを行うと、

Traceback (most recent call last):
  File "/home/ryo/.virtualenvs/default/bin/oj", line 6, in <module>
    exec(compile(open(__file__).read(), __file__, 'exec'))
  File "/home/ryo/work/github/online-judge-tools/oj", line 6, in <module>
    onlinejudge.implementation.main.main(args=sys.argv[1:])
  File "/home/ryo/work/github/online-judge-tools/onlinejudge/implementation/main.py", line 345, in main
    run_program(namespace, parser=parser)
  File "/home/ryo/work/github/online-judge-tools/onlinejudge/implementation/main.py", line 320, in run_program
    test(args)
  File "/home/ryo/work/github/online-judge-tools/onlinejudge/implementation/command/test.py", line 41, in test
    tests = cutils.construct_relationship_of_files(args.test, args.directory, args.format)
  File "/home/ryo/work/github/online-judge-tools/onlinejudge/implementation/command/utils.py", line 52, in construct_relationship_of_files
    m = match_with_format(directory, format, path.resolve())
  File "/home/ryo/work/github/online-judge-tools/onlinejudge/implementation/command/utils.py", line 27, in match_with_format
    path = path.absolute().relative_to(directory.absolute()).resolve()
  File "/usr/lib/python3.5/pathlib.py", line 1109, in resolve
    s = self._flavour.resolve(self)
  File "/usr/lib/python3.5/pathlib.py", line 330, in resolve
    return _resolve(base, str(path)) or sep
  File "/usr/lib/python3.5/pathlib.py", line 315, in _resolve
    target = accessor.readlink(newpath)
  File "/usr/lib/python3.5/pathlib.py", line 422, in readlink
    return os.readlink(path)
FileNotFoundError: [Errno 2] No such file or directory: '/home/ryo/work/github/rust-debug/src/sample-1.in'

というエラーが発生します。
私がローカルで修正を試みたパッチを添えておきます。
実行環境: Python 3.5、https://atcoder.jp/contests/arc087/tasks/arc087_b の問題をダウンロード

[fukatani]

pathにはcutils.glob_with_format(args.directory, args.format)の結果が入っていて
/home/ryo/work/github/rust-debug/src/test/sample1.inが格納されており、
directoryには/home/ryo/work/github/rust-debug/src/testが入っています。
この２つにrelativeをとると、'sample1.in'になり/home/ryo/work/github/rust-debug/src/sample-1.inをresolveするようにみえました。

[fukatani]

formatがディレクトリを複数含んでいる場合、それに応じてサーチの際のルートディレクトリが変わるので、ベタに数えてカウントアップしてしまいました。
ただ、formatが"./"や"/"(システムのルートディレクトリ)から始まることはあるでしょうか？
-dから相対パスで探すものと仮定してコード書きました。

[kmyk]

ありがとうございます。
str だと分かりにくいので Path にしたときに混ざったバグのようです。
この部分のテストがないのでやらかしてしまいました。気付いてなかったので助かります。

ただCIが型チェックで落ちてるのと、他にもまだバグが残ってるのでまだmergeはできなさそうです。

他のバグとは、ケース名の取得が壊れている (下図) のと、 oj test -c ./a.out -d ./hoge/../test が動かないのとの2点です。
なお、報告されている例外は再現しないのでこれに関しては3.5特有の可能性はあります。

誤:

[*] /home/user/GitHub/online-judge-tools/foo/sample-1
[x] time: 0.003562 sec
[+] AC

正:

[*] sample-1
[x] time: 0.003562 sec
[+] AC

[kmyk]

テストを書きました。 (#235) (CIが落ちるため develop branchにmergeしてます)
これにrebaseした上でCIが通るように修正してもらえればmergeします。

やってもらえるなら任せたいですが、バージョン依存は少ないはずなので続きは私にパスしてもらってもかまいません (せっかく直してもらったのに追加になってしまって申し訳なさがあるため)。どうしますか？

[fukatani]

テスト追加ありがとうございます。
私としてはPR作成が趣味ですし、online_judge_toolsの理解を深めていきたいので、明日まで待ってもらえるならぜひやりたいです 😃

このPRもWIPのつもりで、「こんな感じでしょうか？」というつもり出したので、全然問題ないです。

[kmyk]

それならよかったです。では続きは任せます。

[fukatani]

もともとはpath/toでしたが、そのようなディレクトリが存在しないことがエラーの原因なので、
テストケースを修正しました。

[kmyk]

これ知らなかったです。
Python 3.6のresolve()はいい感じにしてくれるけど3.5はだめ、普通のcatとかlsとかは同じエラーを吐くので、この修正は妥当そう。

[fukatani]

generate-outputのテストも作成中です

[kmyk]

CI通ってるのでひとまずmergeします。

generate-outputの方も壊れてるかもしれませんが、別のプルリクに分けましょう。

[kmyk]

generate-outputのテストも書いてくれる (感謝) とのことなので、これができたらバージョン上げます