Skip to content

テストを追加#46

Merged
anqooqie merged 1 commit intomasterfrom
add_test
Oct 13, 2015
Merged

テストを追加#46
anqooqie merged 1 commit intomasterfrom
add_test

Conversation

@anqooqie
Copy link
Copy Markdown
Contributor

@anqooqie anqooqie commented Oct 8, 2015

テストの実行方法

  1. 立ち上げているChromeがあれば一度終了させる
  2. Macの場合、 open /Applications/Google\ Chrome.app --args --allow-file-access-from-files でChromeを起動(Windowsの場合は多分、コマンドプロンプトで "C:\Program Files (x86)\Google\Chrome\Application\chrome.exe" --allow-file-access-from-files
  3. chrome_extension/spec/SpecRunner.htmlをChromeで開く

参考にしたページ

Chrome Extension の CI が出来るようになるまで(導入編)

その他

とりあえずの叩き台なのでテストケースは2つしか書いてません
よくわからないところがあれば遠慮なく質問してください

@anqooqie
Copy link
Copy Markdown
Contributor Author

anqooqie commented Oct 8, 2015

前回の打ち合わせではMocha + chai.jsでテスト書くって言いましたが、再検討の結果Jasmineを使うことになりました
既に勉強し始めてた人がいたらごめんなさい

@anqooqie anqooqie self-assigned this Oct 8, 2015
@anqooqie
Copy link
Copy Markdown
Contributor Author

テストケースの追加は別のプルリクでやるつもりなのでもうコードレビューしてよいです

@dulltz
Copy link
Copy Markdown
Contributor

dulltz commented Oct 11, 2015

👍

@chamstorage
Copy link
Copy Markdown
Contributor

LGTM

anqooqie added a commit that referenced this pull request Oct 13, 2015
@anqooqie anqooqie merged commit c11e3b4 into master Oct 13, 2015
@anqooqie anqooqie deleted the add_test branch October 13, 2015 09:07
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.

3 participants