added test for skip delete event when resources are not added in test_config

[akankshakumari393]

ISSUE TYPE

• Bug fix Pull Request

SUMMARY

Added skip delete event test case that should be skipped if event occurred for a resource which is not added in test configuration.
Modified AllowedEventKindsMap to removed the v1/services resource

Fixes test case - Validate that delete events are skipped when an event occurs for a resource which is not added in resource_config of issue #354

[PrasadG193]

@A-kanksh-a should we wrap the conditions to check if an event is allowed in functions and use that in tests?
e.g

utils.AllowedEventKindsMap[utils.EventKind{
				Resource:  resource,
				Namespace: "all",
				EventType: config.DeleteEvent}] ||
				utils.AllowedEventKindsMap[utils.EventKind{
					Resource:  resource,
					Namespace: test.Namespace,
					EventType: config.DeleteEvent}]

[akankshakumari393]

@A-kanksh-a should we wrap the conditions to check if an event is allowed in functions and use that in tests?
e.g

utils.AllowedEventKindsMap[utils.EventKind{
				Resource:  resource,
				Namespace: "all",
				EventType: config.DeleteEvent}] ||
				utils.AllowedEventKindsMap[utils.EventKind{
					Resource:  resource,
					Namespace: test.Namespace,
					EventType: config.DeleteEvent}]

@PrasadG193 will do the changes.

[PrasadG193]

@A-kanksh-a I meant to add this function in pkg/utils and use it in controller.go as well

[akankshakumari393]

yes we can do that.

[PrasadG193]

Please check if eventMap is nil before accessing the fields

[chetanpdeshmukh]

Hello @A-kanksh-a ,
Thanks again for the PR!

Thank you for contributing to BotKube. Could you please fill out this form, so we can send the well deserved awesome swags 🙂

Team BotKube