From afc4b5b77f9200d914323567d0c291aecab46ab0 Mon Sep 17 00:00:00 2001 From: KonstanceH Date: Mon, 1 Jul 2024 16:29:02 +0000 Subject: [PATCH 01/10] removed query line that adds closeout complete --- pkg/services/order/order_fetcher.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pkg/services/order/order_fetcher.go b/pkg/services/order/order_fetcher.go index 0547c9f1e49..7cd76f07867 100644 --- a/pkg/services/order/order_fetcher.go +++ b/pkg/services/order/order_fetcher.go @@ -175,7 +175,7 @@ func (f orderFetcher) ListOrders(appCtx appcontext.AppContext, officeUserID uuid if *params.NeedsPPMCloseout { query.InnerJoin("ppm_shipments", "ppm_shipments.shipment_id = mto_shipments.id"). LeftJoin("transportation_offices as closeout_to", "closeout_to.id = moves.closeout_office_id"). - Where("ppm_shipments.status IN (?)", models.PPMShipmentStatusWaitingOnCustomer, models.PPMShipmentStatusNeedsCloseout, models.PPMShipmentStatusCloseoutComplete). + Where("ppm_shipments.status IN (?)", models.PPMShipmentStatusWaitingOnCustomer, models.PPMShipmentStatusNeedsCloseout). Where("service_members.affiliation NOT IN (?)", models.AffiliationNAVY, models.AffiliationMARINES, models.AffiliationCOASTGUARD) } else { query.LeftJoin("ppm_shipments", "ppm_shipments.shipment_id = mto_shipments.id"). From ae5f1fdd18609c4bee19478dc10a437588dd36d3 Mon Sep 17 00:00:00 2001 From: KonstanceH Date: Mon, 8 Jul 2024 19:33:00 +0000 Subject: [PATCH 02/10] adding in tests --- pkg/services/order/order_fetcher_test.go | 492 +++++++---------------- 1 file changed, 140 insertions(+), 352 deletions(-) diff --git a/pkg/services/order/order_fetcher_test.go b/pkg/services/order/order_fetcher_test.go index 83f3a68b44c..480087bcf34 100644 --- a/pkg/services/order/order_fetcher_test.go +++ b/pkg/services/order/order_fetcher_test.go @@ -633,9 +633,88 @@ func (suite *OrderServiceSuite) TestListOrdersUSMCGBLOC() { }) } +func getMoveNeedsServiceCounseling(suite *OrderServiceSuite, showMove bool, affiliation models.ServiceMemberAffiliation) models.Move { + nonCloseoutMove := factory.BuildMove(suite.DB(), []factory.Customization{ + { + Model: models.Move{ + Status: models.MoveStatusNeedsServiceCounseling, + Show: &showMove, + }, + }, + { + Model: models.ServiceMember{ + Affiliation: &affiliation, + }, + }, + }, nil) + + return nonCloseoutMove +} + +func getSubmittedMove(suite *OrderServiceSuite, showMove bool, affiliation models.ServiceMemberAffiliation) models.Move { + move := factory.BuildMove(suite.DB(), []factory.Customization{ + { + Model: models.Move{ + Status: models.MoveStatusSUBMITTED, + Show: &showMove, + }, + }, + { + Model: models.ServiceMember{ + Affiliation: &affiliation, + }, + }, + }, nil) + return move +} + +func buildPPMShipmentNeedsCloseout(suite *OrderServiceSuite, move models.Move) models.PPMShipment { + ppm := factory.BuildMinimalPPMShipment(suite.DB(), []factory.Customization{ + { + Model: models.PPMShipment{ + Status: models.PPMShipmentStatusNeedsCloseout, + }, + }, + { + Model: move, + LinkOnly: true, + }, + }, nil) + return ppm +} + +func buildPPMShipmentDraft(suite *OrderServiceSuite, move models.Move) models.PPMShipment { + ppm := factory.BuildMinimalPPMShipment(suite.DB(), []factory.Customization{ + { + Model: models.PPMShipment{ + Status: models.PPMShipmentStatusDraft, + }, + }, + { + Model: move, + LinkOnly: true, + }, + }, nil) + return ppm +} + +func buildPPMShipmentCloseoutComplete(suite *OrderServiceSuite, move models.Move) models.PPMShipment { + ppm := factory.BuildMinimalPPMShipment(suite.DB(), []factory.Customization{ + { + Model: models.PPMShipment{ + Status: models.PPMShipmentStatusCloseoutComplete, + }, + }, + { + Model: move, + LinkOnly: true, + }, + }, nil) + return ppm +} + func (suite *OrderServiceSuite) TestListOrdersPPMCloseoutForArmyAirforce() { orderFetcher := NewOrderFetcher() - showMove := true var session auth.Session @@ -649,85 +728,15 @@ func (suite *OrderServiceSuite) TestListOrdersPPMCloseoutForArmyAirforce() { IDToken: "fake_token", AccessToken: "fakeAccessToken", } + + move := getMoveNeedsServiceCounseling(suite, true, models.AffiliationARMY) + buildPPMShipmentNeedsCloseout(suite, move) - army := models.AffiliationARMY - move := factory.BuildMove(suite.DB(), []factory.Customization{ - { - Model: models.Move{ - Status: models.MoveStatusNeedsServiceCounseling, - Show: &showMove, - }, - }, - { - Model: models.ServiceMember{ - Affiliation: &army, - }, - }, - }, nil) - factory.BuildMinimalPPMShipment(suite.DB(), []factory.Customization{ - { - Model: models.PPMShipment{ - Status: models.PPMShipmentStatusNeedsCloseout, - }, - }, - { - Model: move, - LinkOnly: true, - }, - }, nil) - // Moves that are not ready for closeout should not show in this queue - af := models.AffiliationAIRFORCE - afMove := factory.BuildMove(suite.DB(), []factory.Customization{ - { - Model: models.Move{ - Status: models.MoveStatusNeedsServiceCounseling, - Show: &showMove, - }, - }, - { - Model: models.ServiceMember{ - Affiliation: &af, - }, - }, - }, nil) - factory.BuildMinimalPPMShipment(suite.DB(), []factory.Customization{ - { - Model: models.PPMShipment{ - Status: models.PPMShipmentStatusDraft, - }, - }, - { - Model: afMove, - LinkOnly: true, - }, - }, nil) - // Coast guard moves should not show up in our office user's closeout queue - cg := models.AffiliationCOASTGUARD - cgMove := factory.BuildMove(suite.DB(), []factory.Customization{ - { - Model: models.Move{ - Status: models.MoveStatusNeedsServiceCounseling, - Show: &showMove, - }, - }, - { - Model: models.ServiceMember{ - Affiliation: &cg, - }, - }, - }, nil) + afMove := getMoveNeedsServiceCounseling(suite, true, models.AffiliationAIRFORCE) + buildPPMShipmentDraft(suite, afMove) - factory.BuildMinimalPPMShipment(suite.DB(), []factory.Customization{ - { - Model: models.PPMShipment{ - Status: models.PPMShipmentStatusNeedsCloseout, - }, - }, - { - Model: cgMove, - LinkOnly: true, - }, - }, nil) + cgMove := getMoveNeedsServiceCounseling(suite, true, models.AffiliationCOASTGUARD) + buildPPMShipmentNeedsCloseout(suite, cgMove) params := services.ListOrderParams{PerPage: models.Int64Pointer(9), Page: models.Int64Pointer(1), NeedsPPMCloseout: models.BoolPointer(true), Status: []string{string(models.MoveStatusNeedsServiceCounseling)}} moves, _, err := orderFetcher.ListOrders(suite.AppContextWithSessionForTest(&session), officeUserSC.ID, ¶ms) @@ -748,58 +757,12 @@ func (suite *OrderServiceSuite) TestListOrdersPPMCloseoutForArmyAirforce() { AccessToken: "fakeAccessToken", } - // PPM moves that need closeout should not show up in counseling queue - army := models.AffiliationARMY - closeoutMove := factory.BuildMove(suite.DB(), []factory.Customization{ - { - Model: models.Move{ - Status: models.MoveStatusNeedsServiceCounseling, - Show: &showMove, - }, - }, - { - Model: models.ServiceMember{ - Affiliation: &army, - }, - }, - }, nil) - factory.BuildMinimalPPMShipment(suite.DB(), []factory.Customization{ - { - Model: models.PPMShipment{ - Status: models.PPMShipmentStatusNeedsCloseout, - }, - }, - { - Model: closeoutMove, - LinkOnly: true, - }, - }, nil) + closeoutMove := getMoveNeedsServiceCounseling(suite, true, models.AffiliationARMY) + buildPPMShipmentCloseoutComplete(suite, closeoutMove) + // PPM moves that are not in one of the closeout statuses - airforce := models.AffiliationAIRFORCE - nonCloseoutMove := factory.BuildMove(suite.DB(), []factory.Customization{ - { - Model: models.Move{ - Status: models.MoveStatusNeedsServiceCounseling, - Show: &showMove, - }, - }, - { - Model: models.ServiceMember{ - Affiliation: &airforce, - }, - }, - }, nil) - factory.BuildMinimalPPMShipment(suite.DB(), []factory.Customization{ - { - Model: models.PPMShipment{ - Status: models.PPMShipmentStatusDraft, - }, - }, - { - Model: nonCloseoutMove, - LinkOnly: true, - }, - }, nil) + nonCloseoutMove := getMoveNeedsServiceCounseling(suite, true, models.AffiliationAIRFORCE) + buildPPMShipmentDraft(suite, nonCloseoutMove) params := services.ListOrderParams{PerPage: models.Int64Pointer(9), Page: models.Int64Pointer(1), NeedsPPMCloseout: models.BoolPointer(false), Status: []string{string(models.MoveStatusNeedsServiceCounseling)}} moves, _, err := orderFetcher.ListOrders(suite.AppContextWithSessionForTest(&session), officeUserSC.ID, ¶ms) @@ -812,62 +775,17 @@ func (suite *OrderServiceSuite) TestListOrdersPPMCloseoutForArmyAirforce() { func (suite *OrderServiceSuite) TestListOrdersPPMCloseoutForNavyCoastGuardAndMarines() { orderFetcher := NewOrderFetcher() - showMove := true suite.Run("returns Navy order for NAVY office user when there's a ppm shipment in closeout", func() { - navy := models.AffiliationNAVY // It doesn't matter what the Origin GBLOC is for the move. Only the navy // affiliation matters for SC who are tied to the NAVY GBLOC. - move := factory.BuildMove(suite.DB(), []factory.Customization{ - { - Model: models.Move{ - Status: models.MoveStatusSUBMITTED, - Show: &showMove, - }, - }, - { - Model: models.ServiceMember{ - Affiliation: &navy, - }, - }, - }, nil) - factory.BuildMinimalPPMShipment(suite.DB(), []factory.Customization{ - { - Model: models.PPMShipment{ - Status: models.PPMShipmentStatusNeedsCloseout, - }, - }, - { - Model: move, - LinkOnly: true, - }, - }, nil) - cg := models.AffiliationCOASTGUARD - cgMove := factory.BuildMove(suite.DB(), []factory.Customization{ - { - Model: models.Move{ - Status: models.MoveStatusSUBMITTED, - Show: &showMove, - }, - }, - { - Model: models.ServiceMember{ - Affiliation: &cg, - }, - }, - }, nil) - factory.BuildMinimalPPMShipment(suite.DB(), []factory.Customization{ - { - Model: models.PPMShipment{ - Status: models.PPMShipmentStatusNeedsCloseout, - }, - }, - { - Model: cgMove, - LinkOnly: true, - }, - }, nil) + move := getSubmittedMove(suite, true, models.AffiliationNAVY) + buildPPMShipmentNeedsCloseout(suite, move) + + cgMove := getSubmittedMove(suite, true, models.AffiliationCOASTGUARD) + buildPPMShipmentNeedsCloseout(suite, cgMove) + officeUserSC := factory.BuildOfficeUserWithRoles(suite.DB(), []factory.Customization{ { Model: models.TransportationOffice{ @@ -894,58 +812,14 @@ func (suite *OrderServiceSuite) TestListOrdersPPMCloseoutForNavyCoastGuardAndMar }) suite.Run("returns TVCB order for TVCB office user when there's a ppm shipment in closeout", func() { - marines := models.AffiliationMARINES // It doesn't matter what the Origin GBLOC is for the move. Only the marines // affiliation matters for SC who are tied to the TVCB GBLOC. - move := factory.BuildMove(suite.DB(), []factory.Customization{ - { - Model: models.Move{ - Status: models.MoveStatusSUBMITTED, - Show: &showMove, - }, - }, - { - Model: models.ServiceMember{ - Affiliation: &marines, - }, - }, - }, nil) - factory.BuildMinimalPPMShipment(suite.DB(), []factory.Customization{ - { - Model: models.PPMShipment{ - Status: models.PPMShipmentStatusNeedsCloseout, - }, - }, - { - Model: move, - LinkOnly: true, - }, - }, nil) - army := models.AffiliationARMY - nonMarineMove := factory.BuildMove(suite.DB(), []factory.Customization{ - { - Model: models.Move{ - Status: models.MoveStatusSUBMITTED, - Show: &showMove, - }, - }, - { - Model: models.ServiceMember{ - Affiliation: &army, - }, - }, - }, nil) - factory.BuildMinimalPPMShipment(suite.DB(), []factory.Customization{ - { - Model: models.PPMShipment{ - Status: models.PPMShipmentStatusNeedsCloseout, - }, - }, - { - Model: nonMarineMove, - LinkOnly: true, - }, - }, nil) + move := getSubmittedMove(suite, true, models.AffiliationMARINES) + buildPPMShipmentNeedsCloseout(suite, move) + + nonMarineMove := getSubmittedMove(suite, true, models.AffiliationARMY) + buildPPMShipmentNeedsCloseout(suite, nonMarineMove) + officeUserSC := factory.BuildOfficeUserWithRoles(suite.DB(), []factory.Customization{ { Model: models.TransportationOffice{ @@ -972,58 +846,14 @@ func (suite *OrderServiceSuite) TestListOrdersPPMCloseoutForNavyCoastGuardAndMar }) suite.Run("returns coast guard order for USCG office user when there's a ppm shipment in closeout and filters out non coast guard moves", func() { - cg := models.AffiliationCOASTGUARD // It doesn't matter what the Origin GBLOC is for the move. Only the coast guard // affiliation matters for SC who are tied to the USCG GBLOC. - move := factory.BuildMove(suite.DB(), []factory.Customization{ - { - Model: models.Move{ - Status: models.MoveStatusSUBMITTED, - Show: &showMove, - }, - }, - { - Model: models.ServiceMember{ - Affiliation: &cg, - }, - }, - }, nil) - factory.BuildMinimalPPMShipment(suite.DB(), []factory.Customization{ - { - Model: models.PPMShipment{ - Status: models.PPMShipmentStatusNeedsCloseout, - }, - }, - { - Model: move, - LinkOnly: true, - }, - }, nil) - army := models.AffiliationARMY - armyMove := factory.BuildMove(suite.DB(), []factory.Customization{ - { - Model: models.Move{ - Status: models.MoveStatusSUBMITTED, - Show: &showMove, - }, - }, - { - Model: models.ServiceMember{ - Affiliation: &army, - }, - }, - }, nil) - factory.BuildMinimalPPMShipment(suite.DB(), []factory.Customization{ - { - Model: models.PPMShipment{ - Status: models.PPMShipmentStatusNeedsCloseout, - }, - }, - { - Model: armyMove, - LinkOnly: true, - }, - }, nil) + move := getSubmittedMove(suite, true, models.AffiliationCOASTGUARD) + buildPPMShipmentNeedsCloseout(suite, move) + + armyMove := getSubmittedMove(suite, true, models.AffiliationARMY) + buildPPMShipmentNeedsCloseout(suite, armyMove) + officeUserSC := factory.BuildOfficeUserWithRoles(suite.DB(), []factory.Customization{ { Model: models.TransportationOffice{ @@ -1049,32 +879,9 @@ func (suite *OrderServiceSuite) TestListOrdersPPMCloseoutForNavyCoastGuardAndMar }) suite.Run("Filters out moves with PPM shipments not in the status of NeedsApproval", func() { - cg := models.AffiliationCOASTGUARD + cgMoveInWrongStatus := getSubmittedMove(suite, true, models.AffiliationCOASTGUARD) + buildPPMShipmentCloseoutComplete(suite, cgMoveInWrongStatus) - cgMoveInWrongStatus := factory.BuildMove(suite.DB(), []factory.Customization{ - { - Model: models.Move{ - Status: models.MoveStatusSUBMITTED, - Show: &showMove, - }, - }, - { - Model: models.ServiceMember{ - Affiliation: &cg, - }, - }, - }, nil) - factory.BuildMinimalPPMShipment(suite.DB(), []factory.Customization{ - { - Model: models.PPMShipment{ - Status: models.PPMShipmentStatusCloseoutComplete, - }, - }, - { - Model: cgMoveInWrongStatus, - LinkOnly: true, - }, - }, nil) officeUserSC := factory.BuildOfficeUserWithRoles(suite.DB(), []factory.Customization{ { Model: models.TransportationOffice{ @@ -1091,21 +898,8 @@ func (suite *OrderServiceSuite) TestListOrdersPPMCloseoutForNavyCoastGuardAndMar }) suite.Run("Filters out moves with no PPM shipment", func() { - cg := models.AffiliationCOASTGUARD + moveWithHHG := getSubmittedMove(suite, true, models.AffiliationCOASTGUARD) - moveWithHHG := factory.BuildMove(suite.DB(), []factory.Customization{ - { - Model: models.Move{ - Status: models.MoveStatusSUBMITTED, - Show: &showMove, - }, - }, - { - Model: models.ServiceMember{ - Affiliation: &cg, - }, - }, - }, nil) factory.BuildMTOShipment(suite.DB(), []factory.Customization{ { Model: models.MTOShipment{ @@ -1465,6 +1259,27 @@ func (suite *OrderServiceSuite) TestListOrdersWithSortOrder() { }) } +func getTransportationOffice(suite *OrderServiceSuite, name string) models.TransportationOffice { + trasportationOffice := factory.BuildTransportationOffice(suite.DB(), []factory.Customization{ + { + Model: models.TransportationOffice{ + Name: name, + }, + }}, nil) + return trasportationOffice +} + +func getPPMShipmentWithCloseoutOfficeNeedsCloseout(suite *OrderServiceSuite, closeoutOffice models.TransportationOffice) models.PPMShipment { + ppm := factory.BuildPPMShipmentThatNeedsCloseout(suite.DB(), nil, []factory.Customization{ + { + Model: closeoutOffice, + LinkOnly: true, + Type: &factory.TransportationOffices.CloseoutOffice, + }, + }) + return ppm +} + func (suite *OrderServiceSuite) TestListOrdersNeedingServicesCounselingWithPPMCloseoutColumnsSort() { defaultShipmentPickupPostalCode := "90210" setupTestData := func() models.OfficeUser { @@ -1549,32 +1364,11 @@ func (suite *OrderServiceSuite) TestListOrdersNeedingServicesCounselingWithPPMCl suite.Run("Sort by PPM closeout location", func() { officeUser := setupTestData() - locationA := factory.BuildTransportationOffice(suite.DB(), []factory.Customization{ - { - Model: models.TransportationOffice{ - Name: "A", - }, - }}, nil) - ppmShipmentA := factory.BuildPPMShipmentThatNeedsCloseout(suite.DB(), nil, []factory.Customization{ - { - Model: locationA, - LinkOnly: true, - Type: &factory.TransportationOffices.CloseoutOffice, - }, - }) - locationB := factory.BuildTransportationOffice(suite.DB(), []factory.Customization{ - { - Model: models.TransportationOffice{ - Name: "B", - }, - }}, nil) - ppmShipmentB := factory.BuildPPMShipmentThatNeedsCloseout(suite.DB(), nil, []factory.Customization{ - { - Model: locationB, - LinkOnly: true, - Type: &factory.TransportationOffices.CloseoutOffice, - }, - }) + locationA := getTransportationOffice(suite, "A") + ppmShipmentA := getPPMShipmentWithCloseoutOfficeNeedsCloseout(suite, locationA) + + locationB := getTransportationOffice(suite, "B") + ppmShipmentB := getPPMShipmentWithCloseoutOfficeNeedsCloseout(suite, locationB) // Sort by closeout location (ascending) moves, _, err := orderFetcher.ListOrders(suite.AppContextWithSessionForTest(&session), officeUser.ID, &services.ListOrderParams{ @@ -1737,13 +1531,7 @@ func (suite *OrderServiceSuite) TestListOrdersNeedingServicesCounselingWithPPMCl Model: models.TransportationOffice{Gbloc: "KKFA"}, }, }, nil) - ppmShipmentNeedsCloseout := factory.BuildPPMShipmentThatNeedsCloseout(suite.DB(), nil, []factory.Customization{ - { - Model: closeoutOffice, - LinkOnly: true, - Type: &factory.TransportationOffices.CloseoutOffice, - }, - }) + ppmShipmentNeedsCloseout := getPPMShipmentWithCloseoutOfficeNeedsCloseout(suite, closeoutOffice) ppmShipmentWaitingOnCustomer := factory.BuildPPMShipmentWaitingOnCustomer(suite.DB(), nil, []factory.Customization{ { Model: closeoutOffice, From 2f34db3d127a54373285d83af8f011975987a4e4 Mon Sep 17 00:00:00 2001 From: KonstanceH Date: Mon, 8 Jul 2024 20:04:55 +0000 Subject: [PATCH 03/10] linter change --- pkg/services/order/order_fetcher_test.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pkg/services/order/order_fetcher_test.go b/pkg/services/order/order_fetcher_test.go index 480087bcf34..00ca4bc79e0 100644 --- a/pkg/services/order/order_fetcher_test.go +++ b/pkg/services/order/order_fetcher_test.go @@ -728,7 +728,7 @@ func (suite *OrderServiceSuite) TestListOrdersPPMCloseoutForArmyAirforce() { IDToken: "fake_token", AccessToken: "fakeAccessToken", } - + move := getMoveNeedsServiceCounseling(suite, true, models.AffiliationARMY) buildPPMShipmentNeedsCloseout(suite, move) From d5b0644181ea7a7473c3872a74ce33ba1715ecb3 Mon Sep 17 00:00:00 2001 From: KonstanceH Date: Mon, 8 Jul 2024 20:21:49 +0000 Subject: [PATCH 04/10] adding missing spaces --- pkg/services/order/order_fetcher_test.go | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/pkg/services/order/order_fetcher_test.go b/pkg/services/order/order_fetcher_test.go index 00ca4bc79e0..0454f0d5701 100644 --- a/pkg/services/order/order_fetcher_test.go +++ b/pkg/services/order/order_fetcher_test.go @@ -879,6 +879,7 @@ func (suite *OrderServiceSuite) TestListOrdersPPMCloseoutForNavyCoastGuardAndMar }) suite.Run("Filters out moves with PPM shipments not in the status of NeedsApproval", func() { + cgMoveInWrongStatus := getSubmittedMove(suite, true, models.AffiliationCOASTGUARD) buildPPMShipmentCloseoutComplete(suite, cgMoveInWrongStatus) @@ -898,8 +899,8 @@ func (suite *OrderServiceSuite) TestListOrdersPPMCloseoutForNavyCoastGuardAndMar }) suite.Run("Filters out moves with no PPM shipment", func() { + moveWithHHG := getSubmittedMove(suite, true, models.AffiliationCOASTGUARD) - factory.BuildMTOShipment(suite.DB(), []factory.Customization{ { Model: models.MTOShipment{ From 951efb923b9f92c381ea78ee5fbecbeebba4c930 Mon Sep 17 00:00:00 2001 From: KonstanceH Date: Mon, 8 Jul 2024 20:27:47 +0000 Subject: [PATCH 05/10] remove extra space --- pkg/services/order/order_fetcher_test.go | 1 - 1 file changed, 1 deletion(-) diff --git a/pkg/services/order/order_fetcher_test.go b/pkg/services/order/order_fetcher_test.go index 0454f0d5701..3ef8fc0f87b 100644 --- a/pkg/services/order/order_fetcher_test.go +++ b/pkg/services/order/order_fetcher_test.go @@ -712,7 +712,6 @@ func buildPPMShipmentCloseoutComplete(suite *OrderServiceSuite, move models.Move }, nil) return ppm } - func (suite *OrderServiceSuite) TestListOrdersPPMCloseoutForArmyAirforce() { orderFetcher := NewOrderFetcher() From c907b7dc9b0bd88a9ad3b9eb0016f15676d64d3b Mon Sep 17 00:00:00 2001 From: KonstanceH Date: Mon, 8 Jul 2024 20:30:51 +0000 Subject: [PATCH 06/10] fixing line spacing --- pkg/services/order/order_fetcher_test.go | 1 + 1 file changed, 1 insertion(+) diff --git a/pkg/services/order/order_fetcher_test.go b/pkg/services/order/order_fetcher_test.go index 3ef8fc0f87b..f0ee5cae801 100644 --- a/pkg/services/order/order_fetcher_test.go +++ b/pkg/services/order/order_fetcher_test.go @@ -762,6 +762,7 @@ func (suite *OrderServiceSuite) TestListOrdersPPMCloseoutForArmyAirforce() { // PPM moves that are not in one of the closeout statuses nonCloseoutMove := getMoveNeedsServiceCounseling(suite, true, models.AffiliationAIRFORCE) buildPPMShipmentDraft(suite, nonCloseoutMove) + params := services.ListOrderParams{PerPage: models.Int64Pointer(9), Page: models.Int64Pointer(1), NeedsPPMCloseout: models.BoolPointer(false), Status: []string{string(models.MoveStatusNeedsServiceCounseling)}} moves, _, err := orderFetcher.ListOrders(suite.AppContextWithSessionForTest(&session), officeUserSC.ID, ¶ms) From 0210cf64eff9bb451abb3b184d32dd8dfea24e86 Mon Sep 17 00:00:00 2001 From: KonstanceH Date: Mon, 8 Jul 2024 20:33:26 +0000 Subject: [PATCH 07/10] contd --- pkg/services/order/order_fetcher_test.go | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/pkg/services/order/order_fetcher_test.go b/pkg/services/order/order_fetcher_test.go index f0ee5cae801..b9a9fab6748 100644 --- a/pkg/services/order/order_fetcher_test.go +++ b/pkg/services/order/order_fetcher_test.go @@ -762,7 +762,7 @@ func (suite *OrderServiceSuite) TestListOrdersPPMCloseoutForArmyAirforce() { // PPM moves that are not in one of the closeout statuses nonCloseoutMove := getMoveNeedsServiceCounseling(suite, true, models.AffiliationAIRFORCE) buildPPMShipmentDraft(suite, nonCloseoutMove) - + params := services.ListOrderParams{PerPage: models.Int64Pointer(9), Page: models.Int64Pointer(1), NeedsPPMCloseout: models.BoolPointer(false), Status: []string{string(models.MoveStatusNeedsServiceCounseling)}} moves, _, err := orderFetcher.ListOrders(suite.AppContextWithSessionForTest(&session), officeUserSC.ID, ¶ms) @@ -779,7 +779,6 @@ func (suite *OrderServiceSuite) TestListOrdersPPMCloseoutForNavyCoastGuardAndMar suite.Run("returns Navy order for NAVY office user when there's a ppm shipment in closeout", func() { // It doesn't matter what the Origin GBLOC is for the move. Only the navy // affiliation matters for SC who are tied to the NAVY GBLOC. - move := getSubmittedMove(suite, true, models.AffiliationNAVY) buildPPMShipmentNeedsCloseout(suite, move) From 3e8208d5c65e2d51d935612c4f1f8486135bc125 Mon Sep 17 00:00:00 2001 From: KonstanceH Date: Mon, 8 Jul 2024 20:37:49 +0000 Subject: [PATCH 08/10] .. --- pkg/services/order/order_fetcher_test.go | 2 -- 1 file changed, 2 deletions(-) diff --git a/pkg/services/order/order_fetcher_test.go b/pkg/services/order/order_fetcher_test.go index b9a9fab6748..b6f8dfda7bf 100644 --- a/pkg/services/order/order_fetcher_test.go +++ b/pkg/services/order/order_fetcher_test.go @@ -878,7 +878,6 @@ func (suite *OrderServiceSuite) TestListOrdersPPMCloseoutForNavyCoastGuardAndMar }) suite.Run("Filters out moves with PPM shipments not in the status of NeedsApproval", func() { - cgMoveInWrongStatus := getSubmittedMove(suite, true, models.AffiliationCOASTGUARD) buildPPMShipmentCloseoutComplete(suite, cgMoveInWrongStatus) @@ -898,7 +897,6 @@ func (suite *OrderServiceSuite) TestListOrdersPPMCloseoutForNavyCoastGuardAndMar }) suite.Run("Filters out moves with no PPM shipment", func() { - moveWithHHG := getSubmittedMove(suite, true, models.AffiliationCOASTGUARD) factory.BuildMTOShipment(suite.DB(), []factory.Customization{ { From 0f5b449b91ac7f9c23f1e1424ab638e6475bec71 Mon Sep 17 00:00:00 2001 From: KonstanceH Date: Mon, 8 Jul 2024 20:48:16 +0000 Subject: [PATCH 09/10] last one --- pkg/services/order/order_fetcher_test.go | 2 ++ 1 file changed, 2 insertions(+) diff --git a/pkg/services/order/order_fetcher_test.go b/pkg/services/order/order_fetcher_test.go index b6f8dfda7bf..b9a9fab6748 100644 --- a/pkg/services/order/order_fetcher_test.go +++ b/pkg/services/order/order_fetcher_test.go @@ -878,6 +878,7 @@ func (suite *OrderServiceSuite) TestListOrdersPPMCloseoutForNavyCoastGuardAndMar }) suite.Run("Filters out moves with PPM shipments not in the status of NeedsApproval", func() { + cgMoveInWrongStatus := getSubmittedMove(suite, true, models.AffiliationCOASTGUARD) buildPPMShipmentCloseoutComplete(suite, cgMoveInWrongStatus) @@ -897,6 +898,7 @@ func (suite *OrderServiceSuite) TestListOrdersPPMCloseoutForNavyCoastGuardAndMar }) suite.Run("Filters out moves with no PPM shipment", func() { + moveWithHHG := getSubmittedMove(suite, true, models.AffiliationCOASTGUARD) factory.BuildMTOShipment(suite.DB(), []factory.Customization{ { From ed24664808ad7f301fcfedf4adb00fd662ce4928 Mon Sep 17 00:00:00 2001 From: KonstanceH Date: Mon, 8 Jul 2024 20:57:58 +0000 Subject: [PATCH 10/10] linter change --- pkg/services/order/order_fetcher_test.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/pkg/services/order/order_fetcher_test.go b/pkg/services/order/order_fetcher_test.go index b9a9fab6748..e44ce4730bf 100644 --- a/pkg/services/order/order_fetcher_test.go +++ b/pkg/services/order/order_fetcher_test.go @@ -878,7 +878,7 @@ func (suite *OrderServiceSuite) TestListOrdersPPMCloseoutForNavyCoastGuardAndMar }) suite.Run("Filters out moves with PPM shipments not in the status of NeedsApproval", func() { - + cgMoveInWrongStatus := getSubmittedMove(suite, true, models.AffiliationCOASTGUARD) buildPPMShipmentCloseoutComplete(suite, cgMoveInWrongStatus) @@ -898,7 +898,7 @@ func (suite *OrderServiceSuite) TestListOrdersPPMCloseoutForNavyCoastGuardAndMar }) suite.Run("Filters out moves with no PPM shipment", func() { - + moveWithHHG := getSubmittedMove(suite, true, models.AffiliationCOASTGUARD) factory.BuildMTOShipment(suite.DB(), []factory.Customization{ {