Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0045740 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Openbravo ERP] A. Platform | minor | have not tried | 2021-01-18 09:59 | 2021-01-18 11:40 | |||
Reporter | shuehner | View Status | public | |||||
Assigned To | shuehner | |||||||
Priority | normal | Resolution | fixed | Fixed in Version | PR21Q2 | |||
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 | 0045740: RuntimeModelTest.testIsParent/testisParent2 hides details of wrong defined columns under isDebug | |||||||
Description | Those test cases show following output when they detect a wrongly defined column: expected:<0> but was:<1> Details about which column that is about is not shown at all (unless log4j debug level is enabled). That makes it very impractical to find and fix the problem. Instead info from debug output should be directly added to the assertion. | |||||||
Steps To Reproduce | - | |||||||
Proposed Solution | Instead info from debug output should be directly added to the assertion to show similar to this: java.lang.AssertionError: 1 columns set as *isParent* and are *primitive type*: [C_Order - EM_OBGM_c_bpartner_taxid] expected:<0> but was:<1> at org.junit.Assert.fail(Assert.java:88) at org.junit.Assert.failNotEquals(Assert.java:834) at org.junit.Assert.assertEquals(Assert.java:645) at org.openbravo.test.model.RuntimeModelTest.testIsParent2(RuntimeModelTest.java:296) at | |||||||
Tags | No tags attached. | |||||||
Attached Files | ||||||||
Relationships [ Relation Graph ] [ Dependency Graph ] | |
Notes | |
(0125489) hgbot (developer) 2021-01-18 10:25 |
Merge Request created: https://gitlab.com/openbravo/product/openbravo/-/merge_requests/267 [^] |
(0125495) hgbot (developer) 2021-01-18 11:40 |
Directly closing issue as related merge request is already approved. Repository: https://gitlab.com/openbravo/product/openbravo [^] Changeset: ad7f74cabc4bf4a1517b2d4f47124ccc6bd59889 Author: Stefan Huehner <stefan.huehner@openbravo.com> Date: 2021-01-18T10:20:38+01:00 URL: https://gitlab.com/openbravo/product/openbravo/-/commit/ad7f74cabc4bf4a1517b2d4f47124ccc6bd59889 [^] Fixes ISSUE-45740. Improve test case output for better unterstanding. Testcase did not show location of the problem in its output but only behind log4j.isDebug. Move details of the problem directly into the assert output for immediate feedback to the developer. --- M src-test/src/org/openbravo/test/model/RuntimeModelTest.java --- |
(0125496) hgbot (developer) 2021-01-18 11:40 |
Merge request merged: https://gitlab.com/openbravo/product/openbravo/-/merge_requests/267 [^] |
Issue History | |||
Date Modified | Username | Field | Change |
2021-01-18 09:59 | shuehner | New Issue | |
2021-01-18 09:59 | shuehner | Assigned To | => shuehner |
2021-01-18 09:59 | shuehner | Modules | => Core |
2021-01-18 09:59 | shuehner | Triggers an Emergency Pack | => No |
2021-01-18 10:25 | hgbot | Note Added: 0125489 | |
2021-01-18 11:40 | hgbot | Resolution | open => fixed |
2021-01-18 11:40 | hgbot | Status | new => closed |
2021-01-18 11:40 | hgbot | Fixed in Version | => PR21Q2 |
2021-01-18 11:40 | hgbot | Note Added: 0125495 | |
2021-01-18 11:40 | hgbot | Note Added: 0125496 |
Copyright © 2000 - 2009 MantisBT Group |