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

Add automated testing with Travis-CI or similar #108

Closed
gschneider-r7 opened this issue Feb 9, 2015 · 4 comments · Fixed by #111
Closed

Add automated testing with Travis-CI or similar #108

gschneider-r7 opened this issue Feb 9, 2015 · 4 comments · Fixed by #111
Labels

Comments

@gschneider-r7
Copy link
Contributor

We should be able to run at least some automated testing with Travis-CI or similar service. This will require building out all of the tests.

We can also have automated testing against an actual Nexpose console internally. We could potentially publish pass/fail results of that to github.

This should enable fully automated testing of any pull requests in the future.

@erran
Copy link
Contributor

erran commented Feb 11, 2015

I suggest that we have two levels of tests.

  1. Using a web mocking tool such as vcr/vcr for integration tests.
    1. This provides the ability to store requests as fixtures and run the same tests on Travis for stable API versions.
    2. VCR has the ability to "re-run" tapes which can be used to perform the live requests on a internal system.
  2. Unit tests which can use convential mocking with RSpec mocks.

@gschneider-r7 ping me about this as I'd be interested in working on this. I know in Austin it would help the Metasploit and ControlsInsight teams as we have dependencies on the Nexpose gem.

@erran
Copy link
Contributor

erran commented Feb 11, 2015

/cc @lsanchez-r7

@erran-r7 erran-r7 mentioned this issue Feb 11, 2015
9 tasks
@gschneider-r7
Copy link
Contributor Author

Your suggestion is pretty spot on with what I was thinking. 👍

@lsanchez-r7
Copy link

👍

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

Successfully merging a pull request may close this issue.

3 participants