Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0046140 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Openbravo ERP] A. Platform | major | have not tried | 2021-03-24 17:30 | 2021-03-25 06:55 | |||
Reporter | AugustoMauch | View Status | public | |||||
Assigned To | AugustoMauch | |||||||
Priority | immediate | 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 | |||||||
Merge Request Status | approved | |||||||
Review Assigned To | ||||||||
OBNetwork customer | No | |||||||
Web browser | ||||||||
Modules | Core | |||||||
Support ticket | ||||||||
Regression level | Coding ( Testing ) | |||||||
Regression date | 2021-03-23 | |||||||
Regression introduced in release | pi | |||||||
Regression introduced by commit | https://gitlab.com/openbravo/product/openbravo/-/commit/27a59980bfc52db096819cb3848e2dd8febb2a83 [^] | |||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0046140: Parameter passed to npm test are not being taken into account | |||||||
Description | This changeset [1] introduced a wrapper script around jest in order to use different parameters based on some conditions. The problem is that the parameters that were passed when running npm test are not being forwarded to jest, they are ignored. [1] https://gitlab.com/openbravo/product/openbravo/-/commit/27a59980bfc52db096819cb3848e2dd8febb2a83 [^] | |||||||
Steps To Reproduce | Execute: npm test ./modules/org.openbravo.client.application/web-test It should execute only the tests located following that folder, but instead it is executed all tests. | |||||||
Tags | No tags attached. | |||||||
Attached Files | ||||||||
![]() |
|
![]() |
|
(0126893) hgbot (developer) 2021-03-24 17:46 |
Merge Request created: https://gitlab.com/openbravo/product/openbravo/-/merge_requests/337 [^] |
(0126901) hgbot (developer) 2021-03-25 06:55 |
Merge request merged: https://gitlab.com/openbravo/product/openbravo/-/merge_requests/337 [^] |
(0126902) hgbot (developer) 2021-03-25 06:55 |
Directly closing issue as related merge request is already approved. Repository: https://gitlab.com/openbravo/product/openbravo [^] Changeset: 6c6dcde7faf431bb66a30496893e8638fbfe27c2 Author: Augusto Mauch <augusto.mauch@openbravo.com> Date: 2021-03-24T18:23:58+01:00 URL: https://gitlab.com/openbravo/product/openbravo/-/commit/6c6dcde7faf431bb66a30496893e8638fbfe27c2 [^] Fixes BUG-46140: parameters can be passed to npm test and npm coverage Now all parameters are forwarded from run-tests to jest. Both positional parameters and named parameters are supported. Remember that in order to pass named parameters to npm test they must be included after -- [1] Tests done: npm test (executes all tests) npm test modules/org.openbravo.client.application/web-test/ob-utilities-date.test.js (executes single test) npm coverage modules/org.openbravo.client.application/web-test/ob-utilities-date.test.js (executes coverage for single test) [1] https://jestjs.io/docs/cli/#using-with-npm-scripts [^] --- M modules/org.openbravo.client.kernel/test-scripts/run-tests M package.json --- |
![]() |
|||
Date Modified | Username | Field | Change |
2021-03-24 17:30 | AugustoMauch | New Issue | |
2021-03-24 17:30 | AugustoMauch | Assigned To | => AugustoMauch |
2021-03-24 17:30 | AugustoMauch | OBNetwork customer | => No |
2021-03-24 17:30 | AugustoMauch | Modules | => Core |
2021-03-24 17:30 | AugustoMauch | Regression level | => Coding ( Testing ) |
2021-03-24 17:30 | AugustoMauch | Regression date | => 2021-03-23 |
2021-03-24 17:30 | AugustoMauch | Regression introduced in release | => pi |
2021-03-24 17:30 | AugustoMauch | Regression introduced by commit | => https://gitlab.com/openbravo/product/openbravo/-/commit/27a59980bfc52db096819cb3848e2dd8febb2a83 [^] |
2021-03-24 17:30 | AugustoMauch | Triggers an Emergency Pack | => No |
2021-03-24 17:46 | hgbot | Merge Request Status | => open |
2021-03-24 17:46 | hgbot | Note Added: 0126893 | |
2021-03-25 06:55 | hgbot | Merge Request Status | open => approved |
2021-03-25 06:55 | hgbot | Resolution | open => fixed |
2021-03-25 06:55 | hgbot | Status | new => closed |
2021-03-25 06:55 | hgbot | Note Added: 0126901 | |
2021-03-25 06:55 | hgbot | Fixed in Version | => PR21Q2 |
2021-03-25 06:55 | hgbot | Note Added: 0126902 |
Copyright © 2000 - 2009 MantisBT Group |