run ldap tests on travis #2875
run ldap tests on travis #2875
Conversation
|
I've prepared a stage to preview changes. Open stage or view logs. |
|
You are missing a description. In the description you could for example explain why you removed the One consequence I know about: http://docs.travis-ci.com/user/workers/container-based-infrastructure/ But there are probably more consequences... |
|
I'd be loath to move off the Travis container infrastructure. |
|
If only your PR descriptions were as expressive as your tweets... https://twitter.com/marutosijp/status/590166516395413505 |
… that depends on it to rspec/factory girl." This needs in LDAP test. https://github.com/opf/openproject/blob/v4.0.9/test/unit/ldap_auth_source_test.rb#L49 This reverts commit e55c976.
|
|
And still you fail to see which other consequences this has (e.g. it is no longer possible to run on travis containerized infrastructure). And you point me to a comment that points someone else to a commit message that points to the log output of travis. You see the layers of indirection here? I can already see you pointing someone else to your comment here, just to add another layer of indirection. |
|
This contains #2882 entirely, please see the discussion there for my suggestion on fixing the test instead of reintroducing the fixture. |
No description provided.