Error when committing without "author"

In the documentation author doesn’t appear as required for commit but this error is thrown:

models.js:440 Uncaught (in promise) TypeError: Cannot read property '_named' of undefined
    at isNamedSection (models.js:440)
    at GitConfig.get (models.js:460)
    at config (commands.js:135)

The error goes away if I specify author.

My guess is that all commits must have an author so it should be marked as required.

[wmhilton]

I guess it’s not required because you can also set it using config.

That’s exactly it. I think the fix will actually be to throw a better error. I’m guessing it’s throwing that error because it can’t find a “user” section in the config. @TomasHubelbauer actually has a PR to fix a problem in the config parser, so once I merge that, I’ll fix this, and then the config should just return undefined instead of throwing an error, and then I’ll add a check to make sure that author.name && author.email are not undefined, and if THAT is the case, throw a nice “Author name and email must be specified as an argument or in the .git/config file” error.

[wmhilton]

Yeah, I’ve noticed that on my laptop, but not on my desktop. (I develop on Windows 10 as my primary OS.) I think it just “happens” to work in Git Bash because it creates a virtual link from /tmp to %TEMP%.

I’ll drop a PR at https://github.com/Thinkmill/jest-fixtures/pull/4

@TomasHubelbauer temporarily you can try npm install https://github.com/wmhilton-contrib/jest-fixtures#win32 If that works, then we can switch the devDependency to point at the fork, or if the PR gets merged and published then we can just update the version.

edit: I ran into one more problem running the tests on my laptop, so I finally made PR #119 which should fix these issues