Remove unneeded functions that were enabling testing

Author: davidcauchi

sentinel/sentinel.go

@@ -107,23 +107,6 @@ func (s *Sentinel) Unsubscribe(chainID int64, address common.Address, topic comm
 	return eps.SubscriptionManager().Unsubscribe(address, topic, ch)
 }
 
-// GetService gets the chain poller service for a chain id.
-func (s *Sentinel) GetService(chainID int64) (*chain_poller_service.ChainPollerService, bool) {
-	s.mu.RLock()
-	eps, exists := s.services[chainID]
-	s.mu.RUnlock()
-
-	return eps, exists
-}
-
-// HasServices returns true if there is at least 1 service running.
-func (s *Sentinel) HasServices() bool {
-	s.mu.RLock()
-	defer s.mu.RUnlock()
-
-	return len(s.services) > 0
-}
-
 // Close shuts down all chains and the global registry.
 func (s *Sentinel) Close() {
 	s.mu.Lock()

sentinel/sentinel_test.go

@@ -115,7 +115,7 @@ func TestAddChain_SubscribeUnsubscribeEvent(t *testing.T) {
 	log := createLog(1, common.HexToHash("0x1234"), address, []common.Hash{topic}, []byte("event data"), 0)
 
 	// Retrieve the chain service to access the Subscription Manager.
-	chainService, exists := s.GetService(1)
+	chainService, exists := s.services[1]
 	require.True(t, exists, "Chain service should exist")
 
 	// Broadcast the log.
@@ -175,9 +175,9 @@ func TestAddChains_MultipleConsumers(t *testing.T) {
 	logEvent2 := createLog(3, common.HexToHash("0x2345"), address2, []common.Hash{topic2}, []byte("another log data 2"), 0)
 	logEvent3 := createLog(4, common.HexToHash("0x3456"), address3, []common.Hash{topic3}, []byte("another log data 3"), 0)
 
-	chainService1, exists1 := s.GetService(1)
+	chainService1, exists1 := s.services[1]
 	require.True(t, exists1, "Chain service 1 should exist")
-	chainService2, exists2 := s.GetService(2)
+	chainService2, exists2 := s.services[2]
 	require.True(t, exists2, "Chain service 2 should exist")
 
 	chainService1.SubscriptionMgr.BroadcastLog(createEventKey(address1, topic1), logEvent1)
@@ -259,5 +259,5 @@ func TestAddMultipleChains_CloseSentinel(t *testing.T) {
 	s.Close()
 
 	// Verify that all chains are cleaned up.
-	assert.False(t, s.HasServices(), "All chains should be cleaned up after Close")
+	assert.False(t, len(s.services) > 0, "All chains should be cleaned up after Close")
 }