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

fixed base_url_auto. #180

Merged
merged 2 commits into from
Jun 22, 2012
Merged

fixed base_url_auto. #180

merged 2 commits into from
Jun 22, 2012

Conversation

smallstyle
Copy link
Member

新規にtDiaryを導入した際にRedirectされるURLが,例えば,http://example.com:3000./ とされて正常にRedirectされないので,正しいURLが指定されるように修正しました.

@tdtds
Copy link
Member

tdtds commented Jun 20, 2012

あー、あれってそういうバグだったのか(笑)。
ChangeLogも書いてください。そしたら取り込みます。

@smallstyle
Copy link
Member Author

ChangeLogを追加しました.

tdtds added a commit that referenced this pull request Jun 22, 2012
@tdtds tdtds merged commit 98ea3d2 into tdiary:master Jun 22, 2012
@tdtds
Copy link
Member

tdtds commented Jun 22, 2012

取り込みました。thanks!

@hsbt
Copy link
Member

hsbt commented Jun 23, 2012

この変更以降、リファラ関連のテストが落ちるようになったのですが心当たりありますか? @smallstyle

http://ci.hsbt.org/job/tdiary-core-1.9.3/lastCompletedBuild/testReport/

@smallstyle
Copy link
Member Author

base_urlが空文字で返されるため,リンク元除外リストにおける自分の日記のURLのルールが"^"となるためでした.すみません.

修正して手元の環境でテストを走らせてみましたが,まだテストが通りません.

テスト時に自分の日記の base_url が http://www.example.com/ となるため,リファラのテストで期待しているhttp://www.example.com/ に関するリファラは除外リストに一致するため,リファラが存在せずテストが失敗します.

これまでは自分の日記の base_url は http://www.example.com./ となっていたためテストが通っていたと考えます.

任意のリファラを指定してテストする方法があれば教えてもらえませんか?

@hsbt
Copy link
Member

hsbt commented Jun 23, 2012

任意のリファラを変えるというのが中々難しかったので、テスト用の tdiary.conf から自分の日記の url は除外するようにしてテストを通るようにしておきました。今までおかしなURLを使っていたため、自分の日記の URL が記録できていたということなので結果的には良かったと思っております。

ただ、自分の日記を除外リストに入れるのは標準的な挙動なので、可能なら Capybara のリファラを変えたテストにしたいです。

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