Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add unittest #94

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

rkoyama1623-2021
Copy link
Contributor

@rkoyama1623-2021 rkoyama1623-2021 commented Feb 25, 2017

#87 関連のunittestを追加しました.
現状,ローカルで走らせて確認する形で,.travis.yamlを整備すると,travisでも走る予定です.

log_plotter
├── config
├── doc
├── nautilus-scripts
├── src
└── test ★今回追加
    ├── config
    └── data
        ├── jaxon_test_data
        └── sample

テスト用のログデータがあるため,ファイル数が多いです.
ログデータは10行なので,容量は164Kで収まっています.

@kindsenior
Copy link
Owner

rfuのログファイルがない(他のファイルは全てある?)のはわざとですか?

@rkoyama1623-2021
Copy link
Contributor Author

rkoyama1623-2021 commented Feb 26, 2017

元にしたログが古かったため,rfuのログが抜けていました.
新しいログのものに差し替えますので,mergeはお待ちください.

testデータは,別ブランチ(例えばtest-data)にしておくと,
レポジトリが肥大化したとき,テストデータの履歴を削除したりするのが容易になりそうです.

$ git worktree add test/data test-data
$ cd test/data
$ ls 
test_data1 test_data2 ...
$ cd ../
$ python test_all.py

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants