Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0049510 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Openbravo ERP] A. Platform | minor | have not tried | 2022-06-08 11:06 | 2022-07-02 13:33 | |||
Reporter | jlopez | View Status | public | |||||
Assigned To | jlopez | |||||||
Priority | normal | Resolution | fixed | Fixed in Version | PR22Q4 | |||
Status | closed | Fix in branch | Fixed in SCM revision | |||||
Projection | none | ETA | none | Target Version | ||||
OS | Any | Database | Any | Java version | ||||
OS Version | Database version | Ant version | ||||||
Product Version | SCM revision | |||||||
Review Assigned To | ||||||||
Web browser | ||||||||
Modules | Core | |||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0049510: HttpServletRequestMock.getSession(boolean arg0) returns null instead of the active session | |||||||
Description | In jUnit tests you have to use the HttpServletRequestMock class, for example to test Callouts that make use of the active session to transform the request parameters to numeric types with the CalloutInfo.getBigDecimalParameter(String param) method, it is returning null instead of the active session. | |||||||
Steps To Reproduce | Run any jUnit test of a Callout that is retrieving parameters of type BigDecimal with the method CalloutInfo.getBigDecimalParameter(String param). For example: https://gitlab.com/openbravo/devel/pmods/org.openbravo.commercialoperations/-/blob/fr/860/src-test/org/openbravo/commercialoperations/callouts/CheckProductPricesValuesTest.java [^] | |||||||
Proposed Solution | Modify the HttpServletRequestMock.getSession(boolean arg0) method so that it returns the active session. | |||||||
Tags | No tags attached. | |||||||
Attached Files | ||||||||
Relationships [ Relation Graph ] [ Dependency Graph ] | |
Notes | |
(0138151) hgbot (developer) 2022-06-08 11:23 |
Merge Request created: https://gitlab.com/openbravo/product/openbravo/-/merge_requests/616 [^] |
(0138979) hgbot (developer) 2022-07-02 11:53 |
Merge request merged: https://gitlab.com/openbravo/product/openbravo/-/merge_requests/616 [^] |
(0138980) hgbot (developer) 2022-07-02 11:53 |
Directly closing issue as related merge request is already approved. Repository: https://gitlab.com/openbravo/product/openbravo [^] Changeset: 55b773e0a14aa51cbcd140a0a41c5b68cff337b8 Author: Javier López <javier.lopez@precognis.com> Date: 02-07-2022 09:53:19 URL: https://gitlab.com/openbravo/product/openbravo/-/commit/55b773e0a14aa51cbcd140a0a41c5b68cff337b8 [^] Fixes ISSUE-49510: HttpServletRequestMock.getSession(boolean arg0) returns null instead of the active session --- M src-test/src/org/openbravo/test/base/mock/HttpServletRequestMock.java --- |
Issue History | |||
Date Modified | Username | Field | Change |
2022-06-08 11:06 | jlopez | New Issue | |
2022-06-08 11:06 | jlopez | Assigned To | => Triage Platform Base |
2022-06-08 11:06 | jlopez | Modules | => Core |
2022-06-08 11:06 | jlopez | Triggers an Emergency Pack | => No |
2022-06-08 11:16 | jlopez | Issue Monitored: jlopez | |
2022-06-08 11:16 | jlopez | Issue End Monitor: jlopez | |
2022-06-08 11:23 | hgbot | Note Added: 0138151 | |
2022-06-08 11:38 | aferraz | Summary | Using the HttpServletRequestMock class in jUnit tests => HttpServletRequestMock.getSession(boolean arg0) returns null instead of the active session |
2022-07-02 11:53 | hgbot | Resolution | open => fixed |
2022-07-02 11:53 | hgbot | Status | new => closed |
2022-07-02 11:53 | hgbot | Note Added: 0138979 | |
2022-07-02 11:53 | hgbot | Fixed in Version | => PR22Q4 |
2022-07-02 11:53 | hgbot | Note Added: 0138980 | |
2022-07-02 13:33 | aferraz | Assigned To | Triage Platform Base => jlopez |
Copyright © 2000 - 2009 MantisBT Group |