Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0049510
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] A. Platformminorhave not tried2022-06-08 11:062022-07-02 13:33
ReporterjlopezView Statuspublic 
Assigned Tojlopez 
PrioritynormalResolutionfixedFixed in VersionPR22Q4
StatusclosedFix in branchFixed in SCM revision
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned To
Web browser
ModulesCore
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0049510: HttpServletRequestMock.getSession(boolean arg0) returns null instead of the active session

DescriptionIn 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 ReproduceRun 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 SolutionModify the HttpServletRequestMock.getSession(boolean arg0) method so that it returns the active session.
TagsNo 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
Powered by Mantis Bugtracker