Skip to content
This repository has been archived by the owner on Sep 24, 2019. It is now read-only.

(do not merge) (qa-3110) spike rototiller create integration suite2 #80

Open
wants to merge 3 commits into
base: stable
Choose a base branch
from

Conversation

er0ck
Copy link
Contributor

@er0ck er0ck commented Jun 22, 2017

@samwoods1 @zreichert that's the last i'll probably spend on this effort. totally stuck and frustrated.

rake is strange. or the way we interface with it is.
i can't get this to put anything on stdout.

this theoretically should be the same as previous attempt, i just re-implemented most of RototillerTask

if you feel like looking at this and helping me figure out WHY it doesn't work, that'd be awesome.

Eric Thompson added 3 commits June 21, 2017 17:18
* implement an integration test helper set
  * this creates a helper that can run a single task in rspec
* we could create another helper for rake -T and rake -D, etc, maybe
  * or it might make sense to just keep testing that stuff in acceptance
* this doesn't work
  * it appears to not run the commands, not clear why

[skip ci]
* this one reaches more into rototiller and should do the right thing
  * i'm convinced something (rototiller) is obscuring stdout in this
  setup

[skip ci]
@er0ck
Copy link
Contributor Author

er0ck commented Jun 27, 2017

ok, so skip ci didn't work here
fun

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

Successfully merging this pull request may close these issues.

1 participant