Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/1316#issuecomment-206751818
No, you are right. I no longer have that 'hardware' so I can rerun the test
but as I stated earlier the test was basing itself on an exception not being
throw at the time I did the test and therefore not as useful as would be
expected. The test has improved but I have no longer access to the virtualised
environment in which I ran it.
However; this is a test, not production code. What do you want from the
test other then the test writer being able to use it to verify ACS? Unless
testcode can harm production environments I'd say merge @swill @wilderrodrigues
@remibergsma @bhaisaab
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
