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

test(gatsby-transformer-asciidoc): init some tests #23611

Merged
merged 1 commit into from
May 5, 2020

Conversation

Slashgear
Copy link
Contributor

Description

  • add test on asciidoc transformer
  • mock asciidoctor usage because it was quite complicate to make it work in the jest context
  • we could complete with more test in a second time to keep this PR small

Documentation

Related Issues

@Slashgear Slashgear requested a review from a team as a code owner April 29, 2020 21:04
@pieh pieh self-assigned this May 5, 2020
Copy link
Contributor

@pieh pieh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's get this in. Thanks for doing work on adding tests for those plugins that don't have any!

I'll merge the PR, but for potential follow up - I have some question

mock asciidoctor usage because it was quite complicate to make it work in the jest context

That's unfortunate, could you expand on what kind of problems you saw?

@pieh pieh merged commit 7046230 into gatsbyjs:master May 5, 2020
@Slashgear
Copy link
Contributor Author

@pieh I got some strange error in the console when I tried it. Forgot to save them somewhere.
It may be cause by jest context.

How about this on #23477

@pieh
Copy link
Contributor

pieh commented May 5, 2020

Will check this that PR when I'm back in front of computer. Reviews on mobile are less than ideal ;)

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.

2 participants