Skip to content

Lowercase test error messages in nfregistration tests - #362

Merged
gab-arrobo merged 1 commit into
omec-project:mainfrom
gab-arrobo:lowercase-uniform
Oct 21, 2026
Merged

gab-arrobo merged 1 commit into
omec-project:mainfrom
gab-arrobo:lowercase-uniform

Conversation

@gab-arrobo

Copy link
Copy Markdown
Contributor

No description provided.

Signed-off-by: Arrobo, Gabriel <gabriel.arrobo@intel.com>
@gab-arrobo
gab-arrobo requested review from a team and a balanced review from Copilot October 21, 2026 17:05

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The changes only adjust diagnostic capitalization and introduce no identified issues.

No open issues 🎉

What changed in this PR

Standardizes diagnostic message capitalization in network-function registration and polling tests.

Changes:

  • Changes selected messages from “Expected” to “expected”.
  • Preserves test logic and formatting arguments.
File Description
polling/​nf_configuration_test.go Lowercases selected polling-test failure messages.
nfregistration/​nf_registration_test.go Lowercases selected registration-test failure messages.

Review effort: Balanced

@andybavier andybavier left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@gab-arrobo
gab-arrobo merged commit 834630f into omec-project:main Oct 21, 2026
12 checks passed
@gab-arrobo
gab-arrobo deleted the lowercase-uniform branch October 21, 2026 17:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants