Skip to content

Commit a5c8649

Browse files
authored
Merge pull request #80 from Optum/fix/lease-reason-set-to-true
Fix: resolved issue with lease not having LeaseStatusReasons populate…
2 parents 445f98a + 33085c3 commit a5c8649

3 files changed

Lines changed: 9 additions & 3 deletions

File tree

pkg/provision/provisioner.go

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -97,6 +97,7 @@ func (prov *AccountProvision) ActivateAccount(create bool,
9797
PrincipalID: principalID,
9898
ID: leaseID.String(),
9999
LeaseStatus: db.Active,
100+
LeaseStatusReason: db.LeaseActive,
100101
BudgetAmount: budgetAmount,
101102
BudgetCurrency: budgetCurrency,
102103
BudgetNotificationEmails: budgetNotificationEmails,

pkg/provision/provisioner_test.go

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -203,9 +203,10 @@ type testActivateLeaseInput struct {
203203
func TestActivateLease(t *testing.T) {
204204
// Construct test scenarios
205205
lease := &db.RedboxLease{
206-
AccountID: "123",
207-
PrincipalID: "abc",
208-
LeaseStatus: db.Active,
206+
AccountID: "123",
207+
PrincipalID: "abc",
208+
LeaseStatus: db.Active,
209+
LeaseStatusReason: db.LeaseActive,
209210
}
210211
tests := []testActivateLeaseInput{
211212
// Happy Path - Create
@@ -221,6 +222,7 @@ func TestActivateLease(t *testing.T) {
221222
AccountID: "123",
222223
PrincipalID: "abc",
223224
LeaseStatus: db.Active,
225+
LeaseStatusReason: db.LeaseActive,
224226
LastModifiedOn: 456,
225227
LeaseStatusModifiedOn: 789,
226228
},

tests/acceptance/provisioner_test.go

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -60,6 +60,7 @@ func TestProvisioner(t *testing.T) {
6060
require.Equal(t, principalID, result.PrincipalID)
6161
require.Equal(t, acctID, result.AccountID)
6262
require.Equal(t, db.Active, result.LeaseStatus)
63+
require.Equal(t, db.LeaseActive, result.LeaseStatusReason)
6364
require.NotEqual(t, 0, result.CreatedOn)
6465
require.NotEqual(t, 0, result.LastModifiedOn)
6566
require.NotEqual(t, 0, result.LastModifiedOn)
@@ -73,6 +74,8 @@ func TestProvisioner(t *testing.T) {
7374
require.Equal(t, result.AccountID, assgnAfter.AccountID)
7475
require.Equal(t, result.LeaseStatus,
7576
assgnAfter.LeaseStatus)
77+
require.Equal(t, result.LeaseStatusReason,
78+
assgnAfter.LeaseStatusReason)
7679
require.Equal(t, result.CreatedOn, assgnAfter.CreatedOn)
7780
require.Equal(t, result.LastModifiedOn, assgnAfter.LastModifiedOn)
7881
require.Equal(t, result.LeaseStatusModifiedOn, assgnAfter.LeaseStatusModifiedOn)

0 commit comments

Comments
 (0)