Skip to content

sort tests - #1004

Merged
joshbruce merged 5 commits into
markedjs:masterfrom
UziTech:sort-tests
Jan 7, 2018
Merged

sort tests#1004
joshbruce merged 5 commits into
markedjs:masterfrom
UziTech:sort-tests

Conversation

@UziTech

Copy link
Copy Markdown
Member

Sorting tests alphabetically for more deterministic tests.

@joshbruce this was supposed to go with #1002

@joshbruce

Copy link
Copy Markdown
Member

@UziTech: Interesting. This is alphabetizing the console output, yeah? Just to make sure I'm understanding...then I should probably go to sleep.

@UziTech

Copy link
Copy Markdown
MemberAuthor

And alphabetizing the way the tests are ran. So we can run tests in a specific order if we need to.

@joshbruce

Copy link
Copy Markdown
Member

Cool. Do we have a way to run a single test case? Just curious.

@UziTech

Copy link
Copy Markdown
MemberAuthor

No we don't. Do you want me to set that up?

@joshbruce

Copy link
Copy Markdown
Member

@UziTech: If it's not difficult, it would probably be helpful - not that it takes that long for our tests to run, of course...we just have a lot and might be easier for someone adding a new feature to test their single piece of new functionality, you know?

@UziTech

UziTech commented Jan 7, 2018

Copy link
Copy Markdown
MemberAuthor

I added running only certain tests by specifying a glob pattern

node test --glob gfm*

that will only run tests that start with 'gfm'

that glob also works with --bench to only bench certain tests

node test --bench --glob gfm*

@joshbruce

Copy link
Copy Markdown
Member

Thanks, brother!

@joshbruce
joshbruce merged commit ef80689 into markedjs:masterJan 7, 2018
@joshbruce

Copy link
Copy Markdown
Member

#746

@UziTech
UziTech deleted the sort-tests branch April 5, 2018 14:38
zhenalexfan pushed a commit to zhenalexfan/MarkdownHan that referenced this pull request Nov 8, 2021
Sign up for freeto 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.

2 participants

@UziTech@joshbruce