Add NewAmfRanDefault function to properly initialize log

[gab-arrobo]

No description provided.

[Copilot]

Pull request overview

This PR introduces a standard constructor for context.AmfRan that pre-initializes its logger, then updates NGAP/GMM tests and dispatcher logic to use that constructor so ran.Log is not nil.

Changes:

• Add context.NewAmfRanDefault() to create an AmfRan with Log initialized to logger.NgapLog.
• Update multiple tests to construct RAN contexts via NewAmfRanDefault() instead of ad-hoc &context.AmfRan{...}.
• Update ngap.DispatchLb to use NewAmfRanDefault() in the “GnbIpAddr but no GnbId” path, and simplify error logging in DispatchNgapMsg to use ran.Log directly.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
context/amf_ran.go	Adds NewAmfRanDefault() to ensure AmfRan.Log is initialized by default.
ngap/dispatcher.go	Uses the new constructor in one dispatch path; simplifies logging by removing fallback logger branches.
ngap/dispatcher_test.go	Updates dispatcher tests to use NewAmfRanDefault() instead of a noop zap logger.
ngap/handler_test.go	Replaces direct AmfRan allocations (with zap noop loggers) with NewAmfRanDefault().
ngap/message/build_test.go	Switches test RAN creation to NewAmfRanDefault() and sets AnType explicitly.
gmm/handler_test.go	Updates GMM tests to construct AmfRan via NewAmfRanDefault() and then set required fields.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated no new comments.

[Copilot]

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated no new comments.