-
Notifications
You must be signed in to change notification settings - Fork 201
Running the tests locally #61
Comments
Thanks for the bug report! |
Drat, I was hoping you did! After two brief forays into the kitchen-inspec source, I'm confident it's not supported, and I don't really think it's worthwhile to submit a change to core if we can work around it. Therefore we're left with the rather kludgy solution of committing symlinks to version control: dev-sec/ssh-baseline#55 Looks like I was a bit hasty in suggesting the dual-suites approach for supporting multiple Ansible versions, but at present I still think it's worth maintaining 1.9.x and 2.x suites since many folks are dragging their feet on migrating (not that I blame them). Hopefully in a few months it'll be realistic to require v2 for the role, and the 1.9 suite can be removed entirely. |
I don't really like the symlink workaround either so I asked to let the user change the search-path. |
@conorsch and @rndmh3ro In test-kitchen, the path is always determined by the suite name e.g in your case, the suite name is The symlink solution is not the right approach going forward, instead I started to transform I started the work in
In addition I already fixed test-kitchen test-kitchen/test-kitchen#944 to make kitchen-docker work with kitchen-inspec, which is required to use docker, ansible and travis at the same time. Solution going forward:
|
@rndmh3ro Tested with the VirtualBox backend—works great! |
Perhaps I'm missing something obvious, but I'm unable to run the inspec test suites locally with virtualbox hosts. Creating and converging is fine, but the inspec tests are skipped altogether:
This is likely a regression introduced by #56, since the integration suites are no longer called "default". Changing the test dir seems to fix it:
Possible solutions are renaming the inspec test directory, which I don't like, or adding a
verifier
parameter to the suite blocks to point to the appropriate tests (since the tests are shared across all versions of ansible).The text was updated successfully, but these errors were encountered: