docs(samples): Update previous month logic to avoid zero-index bug in…

… webhook_prebuilt_telecom sample (#479)

* fix: Update previous month logic to avoid zer-index bug

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>

Dialogflow-CX/requirements.txt

@@ -1,2 +1,3 @@
 google-cloud-dialogflow-cx==1.14.0
 Flask==2.2.2
+python-dateutil==2.8.2

Dialogflow-CX/webhook_prebuilt_telecom.py

@@ -263,6 +263,7 @@ def cxPrebuiltAgentsTelecom(request):
 # based on today's date
 def get_date_details(bill_state):
     from datetime import date
+    from dateutil.relativedelta import relativedelta
 
     monthNames = [
         "January",
@@ -280,13 +281,15 @@ def get_date_details(bill_state):
     ]
     today = date.today()
     # index starts with 0
-    first_month_name = monthNames[today.month - 1]
+    first_month_name = monthNames[(today.month - 1)]
     firstDay = today.replace(day=1)
     first_day_str = str(firstDay)
 
-    last_month_name = monthNames[today.month - 2]
-    last_month_first_day_str = str(today.replace(day=1, month=(today.month - 1)))
-    second_last_month_name = monthNames[today.month - 3]
+    last_month_name = monthNames[(today.month - 1) - 1]
+    last_month_first_day_str = str(
+        today.replace(day=1, month=(today - relativedelta(months=1)).month)
+    )
+    second_last_month_name = monthNames[(today.month - 1) - 2]
     if bill_state == "current":
         return [first_month_name, first_day_str, last_month_name]
     else:

Dialogflow-CX/webhook_prebuilt_telecom_test.py

@@ -45,7 +45,6 @@ def test_detect_customeranomaly_current(app):
 
     with app.test_request_context(json=request):
         res = cxPrebuiltAgentsTelecom(flask.request)
-        print(res)
         assert res["sessionInfo"]["parameters"]["anomaly_detect"] == "true"
         assert res["sessionInfo"]["parameters"]["total_bill"] == 1054.34
         assert res["sessionInfo"]["parameters"]["first_month"] == str(
@@ -73,12 +72,11 @@ def test_detect_customeranomaly_other(app):
 
     with app.test_request_context(json=request):
         res = cxPrebuiltAgentsTelecom(flask.request)
-        print(res)
         today = date.today()
         assert res["sessionInfo"]["parameters"]["anomaly_detect"] == "false"
         assert res["sessionInfo"]["parameters"]["total_bill"] == 1054.34
         assert res["sessionInfo"]["parameters"]["first_month"] == str(
-            today.replace(day=1, month=(today.month - 1))
+            today.replace(day=1, month=1 + ((today.month - 2) % 12))
         )
 
 
@@ -98,7 +96,6 @@ def test_validate_phoneline(app):
 
     with app.test_request_context(json=request):
         res = cxPrebuiltAgentsTelecom(flask.request)
-        print(res)
         assert res["sessionInfo"]["parameters"]["phone_line_verified"] == "true"
 
 
@@ -118,7 +115,6 @@ def test_invalid_phoneline(app):
 
     with app.test_request_context(json=request):
         res = cxPrebuiltAgentsTelecom(flask.request)
-        print(res)
         assert res["sessionInfo"]["parameters"]["phone_line_verified"] == "false"
 
 
@@ -138,7 +134,6 @@ def test_invalid_phoneline2(app):
 
     with app.test_request_context(json=request):
         res = cxPrebuiltAgentsTelecom(flask.request)
-        print(res)
         assert res["sessionInfo"]["parameters"]["domestic_coverage"] == "true"
 
 
@@ -156,7 +151,6 @@ def test_cruiseplan_coverage(app):
 
     with app.test_request_context(json=request):
         res = cxPrebuiltAgentsTelecom(flask.request)
-        print(res)
         assert res["sessionInfo"]["parameters"]["port_is_covered"] == "true"
 
 
@@ -174,7 +168,6 @@ def test_cruiseplan_notcovered(app):
 
     with app.test_request_context(json=request):
         res = cxPrebuiltAgentsTelecom(flask.request)
-        print(res)
         assert res["sessionInfo"]["parameters"]["port_is_covered"] == "false"
 
 
@@ -192,7 +185,6 @@ def test_international_coverage1(app):
 
     with app.test_request_context(json=request):
         res = cxPrebuiltAgentsTelecom(flask.request)
-        print(res)
         assert res["sessionInfo"]["parameters"]["coverage"] == "both"
 
 
@@ -210,7 +202,6 @@ def test_international_coverage2(app):
 
     with app.test_request_context(json=request):
         res = cxPrebuiltAgentsTelecom(flask.request)
-        print(res)
         assert res["sessionInfo"]["parameters"]["coverage"] == "monthly_only"
 
 
@@ -228,7 +219,6 @@ def test_international_coverage3(app):
 
     with app.test_request_context(json=request):
         res = cxPrebuiltAgentsTelecom(flask.request)
-        print(res)
         assert res["sessionInfo"]["parameters"]["coverage"] == "neither"
 
 
@@ -246,7 +236,6 @@ def test_cheapest_plan1(app):
 
     with app.test_request_context(json=request):
         res = cxPrebuiltAgentsTelecom(flask.request)
-        print(res)
         assert res["sessionInfo"]["parameters"]["monthly_cost"] == 70
         assert res["sessionInfo"]["parameters"]["daily_cost"] == 400
         assert res["sessionInfo"]["parameters"]["suggested_plan"] == "monthly"
@@ -266,7 +255,6 @@ def test_cheapest_plan2(app):
 
     with app.test_request_context(json=request):
         res = cxPrebuiltAgentsTelecom(flask.request)
-        print(res)
         assert res["sessionInfo"]["parameters"]["monthly_cost"] == 70
         assert res["sessionInfo"]["parameters"]["daily_cost"] == 200
         assert res["sessionInfo"]["parameters"]["suggested_plan"] == "monthly"
@@ -286,7 +274,6 @@ def test_cheapest_plan3(app):
 
     with app.test_request_context(json=request):
         res = cxPrebuiltAgentsTelecom(flask.request)
-        print(res)
         assert res["sessionInfo"]["parameters"]["monthly_cost"] == 70
         assert res["sessionInfo"]["parameters"]["daily_cost"] == 50
         assert res["sessionInfo"]["parameters"]["suggested_plan"] == "daily"
@@ -306,7 +293,6 @@ def test_cheapest_plan4(app):
 
     with app.test_request_context(json=request):
         res = cxPrebuiltAgentsTelecom(flask.request)
-        print(res)
         assert res["sessionInfo"]["parameters"]["suggested_plan"] == "null"