Skip to content

add tagging functionality to pester describes#35

Merged
scottmuc merged 1 commit intopester:masterfrom
rismoney:add_tagging
Jan 13, 2013
Merged

add tagging functionality to pester describes#35
scottmuc merged 1 commit intopester:masterfrom
rismoney:add_tagging

Conversation

@rismoney
Copy link
Copy Markdown
Contributor

This is an initial attempt to add tagging functionality to pester. I am new to pester in general, so if I am going about this wrong let me know.

Do changes to the code typically require their own pester testing?

cli usage:
pester <specific test> -Tags "<tag1><space><optionaltag2>"

describe usage:
Describe "Add-Numbers" -Tags 'adder','calculator' {...}

cli usage:
pester <specific test> -Tags "<tag1><space><optionaltag2>"

describe usage:
Describe "Add-Numbers" -Tags 'adder','calculator' {...}
@mwrock
Copy link
Copy Markdown
Member

mwrock commented Nov 29, 2012

This is really nice. We do test pester with pester. Also dont forget to add some documentation to the comment help. Thanks!

@rismoney
Copy link
Copy Markdown
Contributor Author

Thanks! btw- i think you have a bug in your last commit in here... i left you a msg.

@rismoney
Copy link
Copy Markdown
Contributor Author

rismoney commented Jan 8, 2013

do I need to do anything to get this merged? ;)

@scottmuc
Copy link
Copy Markdown
Contributor

scottmuc commented Jan 8, 2013

You just did... I just needed a reminder and a nudge :-)

I'll be doing some work on Pester this weekend (yay! Another weekend where
I'm not booked solid).

I want to introduce high level acceptance tests to ensure the growth of
features in Pester don't begin to break things.

On Tue, Jan 8, 2013 at 10:28 PM, Rich Siegel notifications@github.comwrote:

do I need to do anything to get this merged? ;)


Reply to this email directly or view it on GitHubhttps://github.com//pull/35#issuecomment-12005901.

Scott Muc
ThoughtWorks Canada
Suite 1100, 600 - 6th Avenue SW
Calgary, AB T2P 0S5

@scottmuc scottmuc merged commit 348079d into pester:master Jan 13, 2013
@scottmuc
Copy link
Copy Markdown
Contributor

Thanks, merged and tested it. Seems to be working great.

bin\pester -Tags "Example"

The command above does what I expect it to do. I definitely feel that as the API gets more flags and configuration we'll need more tests around this. The current test suite isn't giving me the confidence that everything works if the tests pass. Maybe that's because I haven't gotten my hands dirty much with the codebase lately.

Anyways, thank you for reminding me to take a look at this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants