{"action":"created","issue":{"url":"https://api.github.com/repos/puppetlabs/puppet-server/issues/520","labels_url":"https://api.github.com/repos/puppetlabs/puppet-server/issues/520/labels{/name}","comments_url":"https://api.github.com/repos/puppetlabs/puppet-server/issues/520/comments","events_url":"https://api.github.com/repos/puppetlabs/puppet-server/issues/520/events","html_url":"https://github.com/puppetlabs/puppet-server/pull/520","id":72285299,"number":520,"title":"(SERVER-390) Acceptance test for foreground subcmd","user":{"login":"jpinsonault","id":137581,"avatar_url":"https://avatars.githubusercontent.com/u/137581?v=3","gravatar_id":"","url":"https://api.github.com/users/jpinsonault","html_url":"https://github.com/jpinsonault","followers_url":"https://api.github.com/users/jpinsonault/followers","following_url":"https://api.github.com/users/jpinsonault/following{/other_user}","gists_url":"https://api.github.com/users/jpinsonault/gists{/gist_id}","starred_url":"https://api.github.com/users/jpinsonault/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jpinsonault/subscriptions","organizations_url":"https://api.github.com/users/jpinsonault/orgs","repos_url":"https://api.github.com/users/jpinsonault/repos","events_url":"https://api.github.com/users/jpinsonault/events{/privacy}","received_events_url":"https://api.github.com/users/jpinsonault/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":9,"created_at":"2015-04-30T21:24:22Z","updated_at":"2015-05-08T18:08:49Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/puppetlabs/puppet-server/pulls/520","html_url":"https://github.com/puppetlabs/puppet-server/pull/520","diff_url":"https://github.com/puppetlabs/puppet-server/pull/520.diff","patch_url":"https://github.com/puppetlabs/puppet-server/pull/520.patch"},"body":"The test starts up puppetserver with the `foreground` subcommand, kills it after 60s, and examines the output for some expected text.\r\n\r\nI tested it with beaker like so:\r\n\r\n```bash\r\nexport PACKAGE_BUILD_VERSION=2.0.0\r\nexport PUPPET_VERSION=1.0.0\r\n\r\nbundle exec beaker \\\r\n  --debug \\\r\n  --root-keys \\\r\n  --repo-proxy \\\r\n  --preserve-hosts always \\\r\n  --type foss \\\r\n  --pre-suite acceptance/suites/pre_suite/foss \\\r\n  --config acceptance/config/beaker/jenkins/el-6-x86_64-mdca.cfg \\\r\n  --tests acceptance/suites/tests/010-puppetserver-cli/subcommand/foreground.rb \\\r\n  --helper acceptance/lib/helper.rb \\\r\n  --options-file acceptance/config/beaker/options.rb \\\r\n  --load-path acceptance/lib\r\n```"},"comment":{"url":"https://api.github.com/repos/puppetlabs/puppet-server/issues/comments/100315977","html_url":"https://github.com/puppetlabs/puppet-server/pull/520#issuecomment-100315977","issue_url":"https://api.github.com/repos/puppetlabs/puppet-server/issues/520","id":100315977,"user":{"login":"camlow325","id":5652737,"avatar_url":"https://avatars.githubusercontent.com/u/5652737?v=3","gravatar_id":"","url":"https://api.github.com/users/camlow325","html_url":"https://github.com/camlow325","followers_url":"https://api.github.com/users/camlow325/followers","following_url":"https://api.github.com/users/camlow325/following{/other_user}","gists_url":"https://api.github.com/users/camlow325/gists{/gist_id}","starred_url":"https://api.github.com/users/camlow325/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/camlow325/subscriptions","organizations_url":"https://api.github.com/users/camlow325/orgs","repos_url":"https://api.github.com/users/camlow325/repos","events_url":"https://api.github.com/users/camlow325/events{/privacy}","received_events_url":"https://api.github.com/users/camlow325/received_events","type":"User","site_admin":false},"created_at":"2015-05-08T18:08:49Z","updated_at":"2015-05-08T18:08:49Z","body":":+1: other than the one cosmetic spelling thing.  I guess I'm okay with the removal of the `is_pe?` check for determining the name of the service to use since we don't run these tests in PE today.  Would need to revisit that if we ever do in the future, though."}}