This provides an alternative way to mock a module's dependencies that
fixes several issues with proxyquire and proxyquireify [1]. In
particular we have had issues in the past with:

- Third party modules that have global state (eg. angular) causing
  hard-to-debug errors when mocking
- An initial attempt to convert Common JS imports to ES 6 imports ran
  into problems with the transformed code breaking proxyquire

This also provides future-proofing as well in that it enables us to
change the module bundler or test runner without changing how mocking
works. It should also be much more efficient if we make more extensive
use of import mocking as AngularJS is gradually removed from the codebase.

[1] https://github.com/robertknight/babel-plugin-mockable-imports#how-does-this-plugin-differ-from-alternative-approaches

Add annotations:group:GROUP_ID fragment url

- Add an api call to get the linked to group
- Add the linked to group to the featuredGroups list.
- Pass the directLinkedGroup into the filterGroups function.
- Take into account the directLinkedGroup when deciding whether to
filter out the "Public" group or not.
- If there is a directLinkedGroup focus it.

Fix feature flag race condition

Previously, a test that should have failed passed when uri=null, because
none of the test groups had uri patterns to check against. Make the
test groups more realistic by adding in the scopes attribute for at
least one of them.

In cases like: https://hypothes.is/a/rGSnNwCKSuiG3VT__k3VvA where the
sidebar is not being used, the document uri is set to null in the groups
service and passed to the combineGroups function. This means that the
isScopedToUri logic must handle the case where uri=null.

Since the feature flag isn't available until the profile api request
completes, and the profile api request is run asynchronously with the
group api requests, it doesn't make sense to load conditionaly. Instead,
call all the group api requests and filter the groups for the old groups
menu later. In this case, filtering is done in the groups.all
conditionally based on the feature flag.

Since the isScopedToUri is no longer dependent on scope enforcement
and scope enforcement is now taken into account as part of whether
a group is selectable or not, disableOosGroupSelection is no longer
necessary. Group selection can be determined purely from isScopedToUri
and scopes.enforced.

Previously, if the scope was not enforced, isScopedToUri would always
return true. Now, isScopedToUri is true only when the scopes.uri_patterns
match.

getInScopeGroups returns the list of groups for the old groups
menu. Aka: only groups that are in scope are returned.

Revert "Remove feature flag from group service"

Remove feature flag from group service

Convert `<svg-icon>` component to React

Merge pull request #1055 from hypothesis/dependabot/npm_and_yarn/babel-plugin-transform-async-to-promises-0.8.9

Bump babel-plugin-transform-async-to-promises from 0.8.8 to 0.8.9

Bump mocha from 6.1.1 to 6.1.2

Bump enzyme-adapter-preact-pure from 1.10.3 to 1.10.4

Bumps [babel-plugin-transform-async-to-promises](https://github.com/rpetrich/babel-plugin-transform-async-to-promises) from 0.8.8 to 0.8.9.
- [Release notes](https://github.com/rpetrich/babel-plugin-transform-async-to-promises/releases)
- [Commits](https://github.com/rpetrich/babel-plugin-transform-async-to-promises/compare/v0.8.8...v0.8.9)
Signed-off-by: dependabot[bot] <support@dependabot.com>

Bumps [mocha](https://github.com/mochajs/mocha) from 6.1.1 to 6.1.2.
- [Release notes](https://github.com/mochajs/mocha/releases)
- [Changelog](https://github.com/mochajs/mocha/blob/master/CHANGELOG.md)
- [Commits](https://github.com/mochajs/mocha/compare/v6.1.1...v6.1.2)
Signed-off-by: dependabot[bot] <support@dependabot.com>

Bumps [enzyme-adapter-preact-pure](https://github.com/robertknight/enzyme-adapter-preact-pure) from 1.10.3 to 1.10.4.
- [Release notes](https://github.com/robertknight/enzyme-adapter-preact-pure/releases)
- [Changelog](https://github.com/robertknight/enzyme-adapter-preact-pure/blob/master/CHANGELOG.md)
- [Commits](https://github.com/robertknight/enzyme-adapter-preact-pure/commits)
Signed-off-by: dependabot[bot] <support@dependabot.com>

Previously, isScopedToUri took into account enforcement but now
it does not. Thus, isSelectable logic based on scopes.enforced
and isScopedToUri was added to the group-list-section controller
to determine whether a group should be out of scope/not selectable.

Since the feature flag isn't available until the profile api request
completes, and the profile api request is run asynchronously with the
group api requests, it doesn't make sense to load conditionaly. Instead,
call all the group api requests and filter the groups for the old groups
menu later. In this case, filtering is done in the groups.all
conditionally based on the feature flag.

Previously, if the scope was not enforced, isScopedToUri would always
return true. Now, isScopedToUri is true only when the scopes.uri_patterns
match.

getInScopeGroups returns the list of groups for the old groups
menu. Aka: only groups that are in scope are returned.

Convert `<annotation-action-button>` component to React

Bump @babel/preset-env from 7.4.2 to 7.4.3

Bumps [@babel/preset-env](https://github.com/babel/babel) from 7.4.2 to 7.4.3.
- [Release notes](https://github.com/babel/babel/releases)
- [Changelog](https://github.com/babel/babel/blob/master/CHANGELOG.md)
- [Commits](https://github.com/babel/babel/compare/v7.4.2...v7.4.3)
Signed-off-by: dependabot[bot] <support@dependabot.com>

Bump @octokit/rest from 16.22.0 to 16.23.2

Bump @babel/core from 7.4.0 to 7.4.3

Bump enzyme-adapter-preact-pure from 1.10.2 to 1.10.3

Bump preact from 10.0.0-alpha.2 to 10.0.0-alpha.4