fix(dev-env): remove test env const check

[naxoc]

All Submissions:

• Have you followed the Newspack Contributing guideline?
• Does your code follow the WordPress' coding standards and VIP Go coding standards?
• Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Changes proposed in this Pull Request:

This is a tiny PR to get rid of the check for IS_TEST_ENV. There are other ways to suppres the popups, so it is not necessary. It was messing with me because I needed that constant set for testing something else :)

Let's see if tests still work.

How to test the changes in this Pull Request:

1. Not much to test – take a look at the code

Other information:

• Have you added an explanation of what your changes do and why you'd like us to include them?
• Have you written new tests for your changes, as applicable?
• Have you successfully ran tests with your changes locally?

[naxoc]

Hm. Not sure I understand why the tests fail. They don't locally. It looks like the dist-folder is not present on Circle. I'll get back to this.

[dkoo]

Strange how the tests run fine locally. Might have something to do with our CI jobs?

[naxoc]

This is not super high priority, but I would like to figure out the discrepancy. It looks like the tests are run before the JS stuff is built on Circle. I took a look at the config and it looks very similar to the one on newspack-plugin for instance 🤔