test: remove stale TODO

Author: jbedard

examples/npm_deps/BUILD.bazel

@@ -62,9 +62,6 @@ js_test(
         ":node_modules/@rollup/plugin-commonjs",
         ":node_modules/mobx-react",
         ":node_modules/react",
-        # TODO: fix this test so we test a peer dependency without requiring it as a direct
-        # dependency https://github.com/aspect-build/rules_js/issues/92
-        ":node_modules/rollup",
     ],
     entry_point = "peer_test.js",
 )

examples/npm_deps/peer_test.js

@@ -1,9 +1,10 @@
 const cjs = require('@rollup/plugin-commonjs/package.json')
-const rollup = require('rollup')
 const assert = require('assert')
 const react = require('react')
 const mobx = require('mobx-react/package.json')
 assert.equal(react.version, '17.0.2')
 assert.equal(mobx.version, '7.3.0')
 assert.equal(cjs.version, '21.1.0')
-assert.equal(rollup.VERSION, '2.70.2')
+
+// Ensure the main package with peer dependencies works too
+require('@rollup/plugin-commonjs')