Skip to content
This repository has been archived by the owner on Jul 28, 2020. It is now read-only.

Integration Tests (InSpec) #29

Closed
arm4b opened this issue Nov 14, 2016 · 6 comments
Closed

Integration Tests (InSpec) #29

arm4b opened this issue Nov 14, 2016 · 6 comments
Assignees
Labels
Milestone

Comments

@arm4b
Copy link
Member

arm4b commented Nov 14, 2016

Partially adresses #26

Serverspec/Inspec Integration tests would be nice.

There are already some examples to re-use from the:
https://github.com/StackStorm/st2-packages/blob/master/rake/spec/default/60-st2_all-services-ok_spec.rb

@arm4b arm4b added the feature label Nov 14, 2016
@arm4b arm4b modified the milestone: v0.5.0 Nov 15, 2016
@arm4b arm4b self-assigned this Nov 15, 2016
This was referenced Nov 15, 2016
@arm4b arm4b changed the title Integration Tests Integration Tests (InSpec) Nov 17, 2016
@jvrplmlmn
Copy link
Contributor

My vote goes for InSpec over Serverspec, because it opens the possibility of reusing the tests for auditing.

@jvrplmlmn
Copy link
Contributor

I got started with InSpec tests for the 'stackstorm::user' recipe, see #47

@arm4b
Copy link
Member Author

arm4b commented Nov 26, 2016

Some thoughts from #47:

I think in future it would be fine to Test installations with custom settings: somehow re-use initial attributes passed by user OR another approach, - read /etc/st2/st2.conf settings like [system_user] and more.

So the Integration Test could be self-sufficient and applied not only just for stackstorm::bundle with all default parameters but for custom installations.

In future I expect users to run InSpec tests for troubleshooting when they experience any Issues with StackStorm Installation.

@arm4b
Copy link
Member Author

arm4b commented Nov 26, 2016

On related note, it may be nice to configure the following directory structure for tests:

  • test/unit/... - ChefSpec
  • test/integration/... - InSpec
    This will make things more straightforward.

@shortdudey123
Copy link
Contributor

FWIW, test/unit/... is not typically the standard path. Some cookbooks do it, but most are in the current place (spec/...)

@arm4b
Copy link
Member Author

arm4b commented Oct 29, 2018

Done in #61 #53 #52 #51

@arm4b arm4b closed this as completed Oct 29, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

3 participants