daniellmb / markdownsharp

Automatically exported from code.google.com/p/markdownsharp
1 stars 0 forks source link

Embed test cases as resources and drive tests with data #19

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
With the attached patch, NUnit is upgraded to version 2.5.3 (documentation
added as well to help intellisense) and all the file-based tests are
converted to data driven tests via the TestCaseSourceAttribute. See the
following for documentation on the attribute:

http://nunit.com/index.php?p=testCaseSource&r=2.5

All the test files are also included as embedded resources instead of being
addressed directly in the file system, alleviating the need for every
developer to configure the physical path to the "mdtest-1.1" directory.

The patch also adds an exclude pattern to the "mdtest-1.1" directory such
that files generated during execution of the MarkdownSharpTests console
application won't be included in source control.

Original issue reported on code.google.com by asbjornu on 5 Jan 2010 at 1:13

Attachments:

GoogleCodeExporter commented 9 years ago
I like this :)

I've never been convinced that adding const to declarations in unit tests adds 
a lot of 
value, but it doesn't really subtract any either (IMO), so whatever.  ReSharper 
likes 
to complain about that kind of thing iirc.

But I love the end to end test changes.

Original comment by kevin.ba...@gmail.com on 6 Jan 2010 at 1:11

GoogleCodeExporter commented 9 years ago
Thanks, I'm glad you like it. The adding of 'const' is just a Resharper 
knee-jerk 
reaction and can easily be ignored.

Original comment by asbjornu on 6 Jan 2010 at 7:58

GoogleCodeExporter commented 9 years ago
ok, I folded this in. (but the const string seems like unnecessary verbosity, 
so I
didn't include that..)

Original comment by wump...@gmail.com on 6 Jan 2010 at 11:39