-
Notifications
You must be signed in to change notification settings - Fork 592
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(nms): Show NGC AMF config for gateways #14243
Conversation
Thanks for opening a PR! 💯
Howto
More infoPlease take a moment to read through the Magma project's
If this is your first Magma PR, also consider reading
|
const nameCell = within(ngcConfig).getByTestId('Name'); | ||
within(nameCell).getByText('amf.example.org'); | ||
const pointerCell = within(ngcConfig).getByTestId('Pointer'); | ||
within(pointerCell).getByText('1F'); | ||
const regionCell = within(ngcConfig).getByTestId('Region ID'); | ||
within(regionCell).getByText('C1'); | ||
const setCell = within(ngcConfig).getByTestId('Set ID'); | ||
within(setCell).getByText('2A1'); | ||
const serviceTypeCell = within(ngcConfig).getByTestId( | ||
'Default Slice Service Type', | ||
); | ||
within(serviceTypeCell).getByText('2'); | ||
const descriptorCell = within(ngcConfig).getByTestId( | ||
'Default Slice Descriptor', | ||
); | ||
within(descriptorCell).getByText('AFAF'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is the category of those items really identical to the testID
?
I mean for the single fields testID
is never set, however you getByTestId
here, where I would have expected a getByText
or getByName
or the like.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's a feature of the data grid that sets the category automatically as test ID for the whole cell (including title and value as children). If I used getByText
, I'd only get the title and not the value that I want to check.
02143db
to
ebd98b1
Compare
Adds a new section to the Gateway Config that shows the NGC AMF configuration. The data is already available in the frontend as part of the gateway data. Also moves the EPC config to the left column to make room for the new config on the right column. Signed-off-by: Sebastian Thomas <sebastian.thomas@tngtech.com>
ebd98b1
to
6bfb295
Compare
Adds a new section to the Gateway Config that shows the NGC AMF configuration. The data is already available in the frontend as part of the gateway data. Also moves the EPC config to the left column to make room for the new config on the right column. Signed-off-by: Sebastian Thomas <sebastian.thomas@tngtech.com>
Summary
Adds a new section to the Gateway Config that shows the NGC AMF configuration. The data is already available in the frontend as part of the gateway data.
Also moves the EPC config to the left column to make room for the new config on the right column.
A new dialog to edit the config will be added in a follow-up PR.
Config with example data:
Empty config:
Test Plan
Additional Information