search

ticketmaster / poshspec

Infrastructure Testing DSL running in Pester
MIT License
183 stars 32 forks source link

New test for volumes #14

Closed devblackops closed 8 years ago

devblackops commented 8 years ago

I'm thinking a new test for volumes would be useful. If people agree I'll create a PR implementing it.

describe 'Storage' {
    context 'Capacity' {
        volume 'c' sizeremaining { should begreaterthan 1073741824 }
    }
    context 'Health' {
        volume 'c' operationstatus { should be 'ok' }
        volume 'c' healthstatus { should be 'healthly' }
    }
}
cdhunt commented 8 years ago

I think it would be a good addition.

beaudryj commented 8 years ago

I got a head start on this

devblackops commented 8 years ago

What needs to happen for PR #15 to be completed that implements this?

cdhunt commented 8 years ago

I need @beaudryj to resubmit it with just the one change. Right now the PR contains a bunch of old changes that are causing a conflict.

beaudryj commented 8 years ago

I have been slacking real hard, I'll try to get that in. Sorry Brandon On Tue, Aug 2, 2016 at 7:48 PM Chris Hunt notifications@github.com wrote:

I need @beaudryj https://github.com/beaudryj to resubmit it with just the one change. Right now the PR contains a bunch of old changes that are causing a conflict.

— You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub https://github.com/Ticketmaster/poshspec/issues/14#issuecomment-237085494, or mute the thread https://github.com/notifications/unsubscribe-auth/ACgvXPUjMn4AV0eZe5V_vy1EXgnslM00ks5qb9c4gaJpZM4I5Eqw .

devblackops commented 8 years ago

[cracks whip] @beaudryj :smile:

michaeltlombardi commented 8 years ago

Should this issue be closed by #30?

cdhunt commented 8 years ago

Yup. Thanks, @devblackops and @michaeltlombardi .