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

Attempt to address issue #201 #226

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

gmaynor
Copy link

@gmaynor gmaynor commented Aug 8, 2019

Test classes with namespace that doesn't match assembly name cause TreeView to get out of sync

It seems that this is only related to XUnit tests, and only [Fact] tests.

Aside from changes to 'testResultsFile.ts' and the various test projects, the following changes were made :

  • updated packages to address npm audit findings
  • added a call to sort on initially discovered tests (the tests are sorted when new ones are added, this helps keep the TreeView consistent)

updated packages to address npm audit findings
@gmaynor
Copy link
Author

gmaynor commented Aug 8, 2019

Not sure what the travis build failure is. All tests pass on my machine.

OS: Gentoo Linux x64
DotNet Core SDK: 2.2.300
Node: v11.9.0

Am I missing anything?

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.

1 participant