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

Added {suitename} as possible injection value for classNameTemplate #138

Merged
merged 1 commit into from Jul 24, 2020

Conversation

@dtom90
Copy link
Contributor

dtom90 commented Jul 24, 2020

Allows the user to create a classNameTemplate which contains {suitename}

This was the main case I was looking for, but could also be done for suiteNameTemplate and titleNameTemplate if it seems helpful.

Also added a few tests for classNameTemplate and corrected a link in index.js

@dtom90 dtom90 force-pushed the dtom90:suitename-injection-value branch from ebd865a to 1410bb8 Jul 24, 2020
@palmerj3
Copy link
Collaborator

palmerj3 commented Jul 24, 2020

Sweet! Looks good to me.

I'll push a new release this weekend.

@palmerj3 palmerj3 merged commit cd5c96f into jest-community:master Jul 24, 2020
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@dtom90
Copy link
Contributor Author

dtom90 commented Jul 24, 2020

Awesome, thanks!

@palmerj3
Copy link
Collaborator

palmerj3 commented Jul 26, 2020

This is now published as part of 11.1.0.

Thanks!

@dtom90
Copy link
Contributor Author

dtom90 commented Jul 27, 2020

All working in our build environment. Thanks again for the quick merge & release!

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

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.