(maint) Simplify how noop events are checked and handled
(PUP-900) Don't process refresh events for noop'ed resources

When resources were processing queued events, the event callbacks would
be invoked if any of the events were not noop regardless of the state of
the resource in question. This means that a resource that had `noop =>
true` would have the refresh action triggered on refresh, bypassing the
noop statement.

This commit changes the refresh behavior to ensure that resources that
are set to noop are not refreshed.
Merge pull request #3091 from adrienthebo/issue/master/pup-900-noop-refresh

Issue/master/pup 900 noop refresh
(PUP-2560) Convert acceptance test for #7117 to unit test

The acceptance test for Redmine issur #7117 assumed that we would always
be able to reach the inventory the service and blindly did an HTTP
request against that to verify that directives in auth.conf could be
restricted to an environment. This is a very heavyweight way to test
this functionality, and it turns out that removing the inventory service
broke this test anyways. This commit adds a unit test to verify the
behavior of network rights with environments and removes the acceptance
test in favor of the unit test.
(maint) Actually test expected rights behavior

A number of tests were spuriously passing because we were doing a blind
check for truthiness, and `expect(:dunno).to be_true` will pass. In
addition we were checking a negative test that could never fail because
we were passing unmatchable arguments. This commit corrects the tests to
verify the correct behavior.