Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Console & LiveHtmlReporter show different results #127

Closed
coridrew opened this issue Jan 24, 2014 · 4 comments
Closed

Console & LiveHtmlReporter show different results #127

coridrew opened this issue Jan 24, 2014 · 4 comments
Labels

Comments

@coridrew
Copy link

I created a context with 9 tests. 2 were failing. The console shows test 6 & 9 as failing tests. The LiveHtmlReporter seems to start over / get confused when the 1st test is hit. The report ends up showing 8 passing tests. I emailed screenshots & details.

@lefthandedgoat
Copy link
Owner

I figured it out. The single ticks in test names was causing the javascript string to end and be malformed. I added code to escape that stuff properly and it should work now. It was also affecting passing tests. I will get a new build out after lunch. Thanks!

Note, that this will require your project that hosts your canopy tests to use the full .Net 4.0/4.5 project and not the client profile. I will add that to the release notes.

@lefthandedgoat
Copy link
Owner

Cori,

I won't be doing a new build just yet. I am trying to squeeze one more fix in and I didn't finish it tonight. F# has a really cool feature for strings so you don't have to escape which should provide a temporary work around for this problem.

let literalString = """ I can type whatever I want including " and other characters that would normally need to be escaped """

Just put your tests names in triple quotes and you can convert your single ticks to normal quotes and it won't blow up the javascript. I hope to have these other changes done tomorrow and a new build out tomorrow night. Sorry for any inconvenience.

@lefthandedgoat
Copy link
Owner

Fixed in canopy 0.9.3 which is now available on github

Thanks!

@lefthandedgoat
Copy link
Owner

Re-opening because it seems that I didn't escape stuff in the right places like context and maybe puts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants