Makes a slight modification to the arrangement of

TestRemoteRelationProviderRoleConsumingSide around the arg copying for
macaroon comparison. We observed panics in CI testing.

worker/firewaller/firewaller_test.go

@@ -1038,18 +1038,15 @@ func (s *InstanceModeSuite) TestRemoteRelationProviderRoleConsumingSide(c *gc.C)
 			c.Check(version, gc.Equals, 0)
 			c.Check(id, gc.Equals, "")
 			c.Check(request, gc.Equals, "WatchEgressAddressesForRelations")
-			expected := params.RemoteEntityArgs{
-				Args: []params.RemoteEntityArg{{
-					Token: relToken,
-				}},
-			}
-			// Extract macaroons so we can compare them separately
-			// (as they can't be compared using DeepEquals due to 'UnmarshaledAs')
+
 			rArgs := arg.(params.RemoteEntityArgs)
-			newMacs := rArgs.Args[0].Macaroons
-			rArgs.Args[0].Macaroons = nil
-			apitesting.MacaroonEquals(c, newMacs[0], mac)
-			c.Check(arg, gc.DeepEquals, expected)
+			c.Assert(rArgs.Args, gc.HasLen, 1)
+			c.Assert(rArgs.Args[0].Macaroons, gc.HasLen, 1)
+
+			apitesting.MacaroonEquals(c, rArgs.Args[0].Macaroons[0], mac)
+
+			c.Check(rArgs.Args[0].Token, gc.Equals, relToken)
+
 			c.Assert(result, gc.FitsTypeOf, &params.StringsWatchResults{})
 			*(result.(*params.StringsWatchResults)) = params.StringsWatchResults{
 				Results: []params.StringsWatchResult{{StringsWatcherId: "1"}},