Fix when "update snapshots" message is displayed #trivial

[chrismcleod]

The existing check was not working in [email protected]

[coveralls]

Pull Request Test Coverage Report for Build 425

• 0 of 1 (0.0%) changed or added relevant line in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+2.1%) to 84.007%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
src/JestExt.ts	0	1	0.0%

Totals
Change from base Build 418:	2.1%
Covered Lines:	690
Relevant Lines:	806

---

💛 - Coveralls

[connectdotz]

nice catch 👍

This area doesn't have much test coverage, want to take this opportunity to add an unit test? At least for the new pattern if not the whole handler...

[chrismcleod]

I can definitely test the regex. I took about 30 mins to try and figure out how to test that a notification is shown by a vscode extension. I have never developed one myself however, and didn't get very far. Any ideas or links with some information about how to test that?

[chrismcleod]

@connectdotz I added at least one test for the error handler and pattern matching

[connectdotz]

I think what you test is perfectly all right for unit testing. Thanks for taking the time to improve the plugin and its test coverage!