-
Notifications
You must be signed in to change notification settings - Fork 35
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
B21576 int add order types for sc too #14446
base: integrationTesting
Are you sure you want to change the base?
B21576 int add order types for sc too #14446
Conversation
…-add_order_types_for_sc_too
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.
I also see a client test failing that maybe be related to your changes
await userEvent.selectOptions(screen.getByLabelText('Orders type'), ORDERS_TYPE.STUDENT_TRAVEL); | ||
|
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.
I think you may be missing the checks to see if the radio buttons are checked and disabled.
I think this would be needed to test if the buttons were disabled and then re-enabled.
await userEvent.selectOptions(screen.getByLabelText('Orders type'), ORDERS_TYPE.STUDENT_TRAVEL);
const hasDependentsYes = screen.getByLabelText('Yes');
const hasDependentsNo = screen.getByLabelText('No');
await waitFor(() => {
expect(hasDependentsYes).toBeChecked();
expect(hasDependentsYes).toBeDisabled();
expect(hasDependentsNo).toBeDisabled();
});
await userEvent.selectOptions(screen.getByLabelText('Orders type'), ORDERS_TYPE.EARLY_RETURN_OF_DEPENDENTS); | ||
|
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.
Same missing checks here
Also noting the concern here that edit orders page doesn't have an option to choose "has dependent" for office users. This means if SC chooses |
Agility ticket
B21576
Summary
In B21575, 2 new order types were added for the customer: Student Travel, Early Return of Dependents. B21576 adds these new order types for SC and TOO. When the SC or TOO edits orders, they will be able to select the new order types and "has dependents" will update to "yes" on the backend. In addition, the new order types will display in the "Orders" section of the "Move Details Page" for the SC or TOO. And, finally, when SC creates a move, the "Orders" page will behave exactly like the customer version of this page.
How to test
In the .envrc.local file, add the following to turn on the feature flag:
export FEATURE_FLAG_ENABLE_ALASKA=true
restart server and client
=====
Create a move with a HHG shipment and submit it.
=====
Test: SC: edit orders, Student Travel
=====
Test: SC: edit orders, Early Return of Dependents
=====
Test: SC: edit orders, Select neither Early Return of Dependents nor Student Travel
=====
Test: SC: create move
=====
Test: turn off feature flag
In the .envrc.local file, add the following to turn on the feature flag:
export FEATURE_FLAG_ENABLE_ALASKA=false
restart server and client
ANDI Screenshots
SC: move, edit orders
SC: move, orders section
TOO: move, orders section
SC: create move