> +/**
> + * Tests parsing and Guice wiring of StackApi
> + */
> +@Test(groups = "live", testName = "StackApiLiveTest")
> +public class StackApiLiveTest extends BaseHeatApiLiveTest {
> +
> + public static final String TEMPLATE_URL =
> "http://10.5.5.121/Installs/cPaaS/YAML/simple_stack.yaml";;
> + protected String stackName = System.getProperty("user.name").replace('.',
> '-').toLowerCase();
> +
> + public void testList() {
> + for (String region : api.getConfiguredRegions()) {
> + StackApi stackApi = api.getStackApi(region);
> +
> + List<Stack> stacks = stackApi.list();
> +
> + assertThat(stacks).isNotNull();
I mean, since the api method defines [this
fallback](https://github.com/jclouds/jclouds-labs-openstack/pull/188/files#diff-667f635b0c1e376a047737c966c674dfR60),
the returned list will never be null, so this test is testing nothing. It
would be better to assert that the list is not empty, and if possible, to
verify that its content is the expected one.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/188/files#r28672664
