[Remote suggestions] Clean up variation params in the status service

components/ntp_snippets/remote/remote_suggestions_status_service_unittest.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/ntp_snippets/remote/remote_suggestions_status_service.h"
 
 #include <memory>
 
 #include "base/memory/ptr_util.h"
 #include "components/ntp_snippets/features.h"
@@ skipping 22 matching lines @@
   std::unique_ptr<RemoteSuggestionsStatusService> MakeService() {
     return base::MakeUnique<RemoteSuggestionsStatusService>(
         utils_.fake_signin_manager(), utils_.pref_service());
   }
 
  protected:
   test::RemoteSuggestionsTestUtils utils_;
   variations::testing::VariationParamsManager params_manager_;
 };
 
-TEST_F(RemoteSuggestionsStatusServiceTest, SigninNeededIfSpecifiedByParam) {
-  // Specify by the parameter that signin is required.
-  params_manager_.SetVariationParamsWithFeatureAssociations(
-      ntp_snippets::kStudyName, {{"fetching_requires_signin", "true"}},
-      {ntp_snippets::kArticleSuggestionsFeature.name});
-
-  auto service = MakeService();
-
-  // The default test setup is signed out.
-  EXPECT_EQ(RemoteSuggestionsStatus::SIGNED_OUT_AND_DISABLED,
-            service->GetStatusFromDeps());
-
-  // Once signed in, we should be in a compatible state.
-  utils_.fake_signin_manager()->SignIn("foo@bar.com");
-  EXPECT_EQ(RemoteSuggestionsStatus::ENABLED_AND_SIGNED_IN,
-            service->GetStatusFromDeps());
-}
-
 TEST_F(RemoteSuggestionsStatusServiceTest, NoSigninNeeded) {
   auto service = MakeService();
 
   // By default, no signin is required.
   EXPECT_EQ(RemoteSuggestionsStatus::ENABLED_AND_SIGNED_OUT,
             service->GetStatusFromDeps());
 
   // One can still sign in.
   utils_.fake_signin_manager()->SignIn("foo@bar.com");
   EXPECT_EQ(RemoteSuggestionsStatus::ENABLED_AND_SIGNED_IN,
@@ skipping 12 matching lines @@
   EXPECT_EQ(RemoteSuggestionsStatus::EXPLICITLY_DISABLED,
             service->GetStatusFromDeps());
 
   // The other dependencies shouldn't matter anymore.
   utils_.fake_signin_manager()->SignIn("foo@bar.com");
   EXPECT_EQ(RemoteSuggestionsStatus::EXPLICITLY_DISABLED,
             service->GetStatusFromDeps());
 }
 
 }  // namespace ntp_snippets