Skip to content

Update and rename Edgecloud_LCM_Test.feature to edge-application-mana…#401

Open
JoseMConde wants to merge 4 commits intomainfrom
JoseMConde-patch-3
Open

Update and rename Edgecloud_LCM_Test.feature to edge-application-mana…#401
JoseMConde wants to merge 4 commits intomainfrom
JoseMConde-patch-3

Conversation

@JoseMConde
Copy link
Collaborator

What type of PR is this?

  • documentation

What this PR does / why we need it:

Update and rename Edge Application Management test.feature file.

Which issue(s) this PR fixes:

Fixes #400

Special notes for reviewers:

Changelog input

 release-note

Additional documentation

This section can be blank.

docs

…gement.feature

Update and rename Edge Application Management test.feature file.
@github-actions
Copy link

github-actions bot commented Nov 4, 2025

🦙 MegaLinter status: ✅ SUCCESS

Descriptor Linter Files Fixed Errors Elapsed time
✅ ACTION actionlint 2 0 0.02s
✅ JSON eslint-plugin-jsonc 1 0 0 1.12s
✅ JSON jsonlint 1 0 0.18s
✅ JSON prettier 1 1 0 0.57s
✅ JSON v8r 1 0 1.9s
✅ OPENAPI spectral 1 0 3.26s
✅ REPOSITORY git_diff yes no 0.28s
✅ REPOSITORY secretlint yes no 2.49s
✅ YAML yamllint 1 0 0.75s

See detailed report in MegaLinter reports

MegaLinter is graciously provided by OX Security

@JoseMConde
Copy link
Collaborator Author

@camaraproject/edge-cloud_codeowners any comment or issue about this??

Copy link
Collaborator

@Kevsy Kevsy left a comment

Choose a reason for hiding this comment

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

HI @JoseMConde , significant rework is needed because the the format for .feature files needs to follow the API Testing Guidelines:

There is also the Gherkin lint to check that the files are compliant.

@JoseMConde
Copy link
Collaborator Author

Thanks @Kevsy!! I will take care of this.

@JoseMConde JoseMConde requested a review from Kevsy January 14, 2026 09:39
@JoseMConde
Copy link
Collaborator Author

Thanks @Kevsy for the comment, I have updated the test file.
@camaraproject/edge-cloud_codeowners please, take a look.

Copy link
Collaborator

@Kevsy Kevsy left a comment

Choose a reason for hiding this comment

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

LGTM in terms of following the Test Definition format - and will be checked by updated linters for Fall 26.

An EdgeCLoud LCM expert should check the test logic as I am less familiar with that part 👍

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.

Edge Application Management Test feature file update

2 participants