-
Notifications
You must be signed in to change notification settings - Fork 1
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Send processed usage records to Odoo
- Loading branch information
1 parent
32b2c97
commit 6a93bd1
Showing
9 changed files
with
234 additions
and
52 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,126 @@ | ||
| package odoo_test | ||
|
|
||
| import ( | ||
| "context" | ||
| "errors" | ||
| "io" | ||
| "net/http" | ||
| "net/http/httptest" | ||
| "testing" | ||
|
|
||
| "github.com/go-logr/logr" | ||
| "github.com/stretchr/testify/require" | ||
|
|
||
| "github.com/appuio/appuio-cloud-reporting/pkg/odoo" | ||
| ) | ||
|
|
||
| type mockRoundTripper struct { | ||
| cannedResponse *http.Response | ||
| receivedContent string | ||
| } | ||
|
|
||
| type mockRoundTripperWhichFails struct { | ||
| } | ||
|
|
||
| func (rt *mockRoundTripper) RoundTrip(req *http.Request) (*http.Response, error) { | ||
| body, _ := io.ReadAll(req.Body) | ||
| rt.receivedContent = string(body) | ||
| return rt.cannedResponse, nil | ||
| } | ||
|
|
||
| func (rt *mockRoundTripperWhichFails) RoundTrip(req *http.Request) (*http.Response, error) { | ||
| return nil, errors.New("there was an error") | ||
| } | ||
|
|
||
| func TestOdooRecordsSent(t *testing.T) { | ||
| recorder := httptest.NewRecorder() | ||
| recorder.WriteString("success") | ||
| expectedResponse := recorder.Result() | ||
|
|
||
| mrt := &mockRoundTripper{cannedResponse: expectedResponse} | ||
| client := http.Client{Transport: mrt} | ||
|
|
||
| logger := logr.New(logr.Discard().GetSink()) | ||
| uut := odoo.NewOdooAPIWithClient("https://foo.bar/odoo16/", &client, logger) | ||
|
|
||
| err := uut.SendData(context.Background(), []odoo.OdooMeteredBillingRecord{getOdooRecord()}) | ||
|
|
||
| require.NoError(t, err) | ||
| require.Equal(t, mrt.receivedContent, `{"data":[{"product_id":"my-product","instance_id":"my-instance","item_description":"my-description","item_group_description":"my-group","sales_order_id":"SO00000","unit_id":"my-unit","consumed_units":11.1,"timerange":"my-timerange"}]}`) | ||
| } | ||
|
|
||
| func TestErrorHandling(t *testing.T) { | ||
| mrt := &mockRoundTripperWhichFails{} | ||
| client := http.Client{Transport: mrt} | ||
|
|
||
| logger := logr.New(logr.Discard().GetSink()) | ||
| uut := odoo.NewOdooAPIWithClient("https://foo.bar/odoo16/", &client, logger) | ||
|
|
||
| err := uut.SendData(context.Background(), []odoo.OdooMeteredBillingRecord{getOdooRecord()}) | ||
|
|
||
| require.Error(t, err) | ||
| } | ||
|
|
||
| func TestErrorFromServerRaisesError(t *testing.T) { | ||
| recorder := httptest.NewRecorder() | ||
| recorder.WriteHeader(500) | ||
| recorder.WriteString(`{ | ||
| "arguments": [ | ||
| "data" | ||
| ], | ||
| "code": 500, | ||
| "context": {}, | ||
| "message": "data", | ||
| "name": "builtins.KeyError", | ||
| "traceback": [ | ||
| "Traceback (most recent call last):", | ||
| " File \"/opt/odoo/bin/odoo/http.py\", line 1589, in _serve_db", | ||
| " return service_model.retrying(self._serve_ir_http, self.env)", | ||
| " File \"/opt/odoo/bin/odoo/service/model.py\", line 133, in retrying", | ||
| " result = func()", | ||
| " File \"/opt/odoo/bin/odoo/http.py\", line 1616, in _serve_ir_http", | ||
| " response = self.dispatcher.dispatch(rule.endpoint, args)", | ||
| " File \"/opt/odoo/braintec/ext/muk_rest/core/http.py\", line 295, in dispatch", | ||
| " result = self.request.registry['ir.http']._dispatch(endpoint)", | ||
| " File \"/opt/odoo/bin/addons/website/models/ir_http.py\", line 237, in _dispatch", | ||
| " response = super()._dispatch(endpoint)", | ||
| " File \"/opt/odoo/braintec/ext/muk_rest/models/ir_http.py\", line 160, in _dispatch", | ||
| " response = super()._dispatch(endpoint)", | ||
| " File \"/opt/odoo/addons/monitoring_prometheus/models/ir_http.py\", line 38, in _dispatch", | ||
| " res = super()._dispatch(endpoint)", | ||
| " File \"/opt/odoo/bin/odoo/addons/base/models/ir_http.py\", line 154, in _dispatch", | ||
| " result = endpoint(**request.params)", | ||
| " File \"/opt/odoo/bin/odoo/http.py\", line 697, in route_wrapper", | ||
| " result = endpoint(self, *args, **params_ok)", | ||
| " File \"/opt/odoo/braintec/ext/muk_rest/core/http.py\", line 122, in wrapper", | ||
| " result = func(*args, **kwargs)", | ||
| " File \"/opt/odoo/braintec/vshn/vshn_metered_usage_rest/controllers/metered_usage_rest.py\", line 65, in vshn_send_metered_usage", | ||
| " 'payload': json.dumps(kw['data'], indent=4)", | ||
| "KeyError: 'data'" | ||
| ] | ||
| }`) | ||
| expectedResponse := recorder.Result() | ||
|
|
||
| mrt := &mockRoundTripper{cannedResponse: expectedResponse} | ||
| client := http.Client{Transport: mrt} | ||
|
|
||
| logger := logr.New(logr.Discard().GetSink()) | ||
| uut := odoo.NewOdooAPIWithClient("https://foo.bar/odoo16/", &client, logger) | ||
|
|
||
| err := uut.SendData(context.Background(), []odoo.OdooMeteredBillingRecord{getOdooRecord()}) | ||
|
|
||
| require.Error(t, err) | ||
| } | ||
|
|
||
| func getOdooRecord() odoo.OdooMeteredBillingRecord { | ||
| return odoo.OdooMeteredBillingRecord{ | ||
| ProductID: "my-product", | ||
| UnitID: "my-unit", | ||
| SalesOrderID: "SO00000", | ||
| InstanceID: "my-instance", | ||
| ItemDescription: "my-description", | ||
| ItemGroupDescription: "my-group", | ||
| ConsumedUnits: 11.1, | ||
| Timerange: "my-timerange", | ||
| } | ||
| } |
Oops, something went wrong.