Skip to content
This repository has been archived by the owner on Feb 20, 2020. It is now read-only.

Onset! のテスト構築 #27

Open
akarregi opened this issue Aug 20, 2016 · 2 comments
Open

Onset! のテスト構築 #27

akarregi opened this issue Aug 20, 2016 · 2 comments

Comments

@akarregi
Copy link
Contributor

現在の Onset! はテストが実装されていないので、コードの可用性が十分ではありません
テストの追加を提案します。

必要なこと

  • ファイル群の副作用をどうにかする
  • PHPUnit を要求
  • というか Composer 実装によるライブラリの活用

利点

  • テスト開発駆動(= 先任者の知識が活かせる) / テストファースト
  • テストが走ればコードが動くと確認できる
  • 何がどうなればいいのか見当がつく

欠点

  • Composer
  • 現状のコードの大幅な書き換え ― 相応の労力が必要
  • テストベースな考えが必要

以上

@kiridaruma
Copy link
Owner

確かに、現状だと不便なので改善したいですね...
が、テストというのがいまいちわかっていないので、どうしようかといったところです
PHPUnitに関してもいいと思うのですが、かなりコードを書き直さないといけないので、現状は保留で前向きに検討で行こうかなと思います

@kiridaruma
Copy link
Owner

当面の間は見送ろうと思います
(devブランチにて大規模変更を行っていましたが、挫折しました)

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

No branches or pull requests

2 participants