Skip to content

Commit

Permalink
finish federatedinformermanager tests
Browse files Browse the repository at this point in the history
  • Loading branch information
limhawjia committed Jul 12, 2023
1 parent cf4a9a7 commit 666afef
Show file tree
Hide file tree
Showing 3 changed files with 1,140 additions and 194 deletions.
10 changes: 6 additions & 4 deletions pkg/util/informermanager/federatedinformermanager.go
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ type federatedInformerManager struct {
clusterInformer fedcorev1a1informers.FederatedClusterInformer

eventHandlerGenerators []*EventHandlerGenerator
clusterEventHandler []*ClusterEventHandler
clusterEventHandlers []*ClusterEventHandler

clients map[string]dynamic.Interface
connectionMap map[string][]byte
Expand All @@ -53,7 +53,7 @@ func NewFederatedInformerManager(
ftcInformer: ftcInformer,
clusterInformer: clusterInformer,
eventHandlerGenerators: []*EventHandlerGenerator{},
clusterEventHandler: []*ClusterEventHandler{},
clusterEventHandlers: []*ClusterEventHandler{},
clients: map[string]dynamic.Interface{},
connectionMap: map[string][]byte{},
informerManagers: map[string]InformerManager{},
Expand All @@ -73,6 +73,8 @@ func NewFederatedInformerManager(
},
})

ftcInformer.Informer()

return manager
}

Expand Down Expand Up @@ -210,7 +212,7 @@ func (m *federatedInformerManager) AddClusterEventHandler(handler *ClusterEventH
return fmt.Errorf("FederatedInformerManager is already started.")
}

m.clusterEventHandler = append(m.clusterEventHandler, handler)
m.clusterEventHandlers = append(m.clusterEventHandlers, handler)
return nil

}
Expand Down Expand Up @@ -280,7 +282,7 @@ func (m *federatedInformerManager) Start(ctx context.Context) {
return
}

for _, handler := range m.clusterEventHandler {
for _, handler := range m.clusterEventHandlers {
predicate := handler.Predicate
callback := handler.Callback

Expand Down
Loading

0 comments on commit 666afef

Please sign in to comment.