Skip to content

Unit test GLOBAL_TEST_CLASS in Y_CHECK_BASE fails #525

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
ConjuringCoffee opened this issue Nov 10, 2021 · 6 comments
Closed

Unit test GLOBAL_TEST_CLASS in Y_CHECK_BASE fails #525

ConjuringCoffee opened this issue Nov 10, 2021 · 6 comments

Comments

@ConjuringCoffee
Copy link
Contributor

ConjuringCoffee commented Nov 10, 2021

The unit test method GLOBAL_TEST_CLASS of class Y_CHECK_BASE fails. It expects a true boolean value for IS_TEST_CODE, but it is false.
I can confirm that Y_UNIT_TEST_BASE is available in the system, containing test code on the global class level.

I'm running the tests of commit c782c26 on an S/4 system with component SAP_BASIS release 755 SP-Level 0002. Can anyone confirm if the test runs successfully on older releases?

@ConjuringCoffee ConjuringCoffee added the bug Something isn't working correctly label Nov 10, 2021
@ConjuringCoffee
Copy link
Contributor Author

Please note that this isn't the only unit test that fails in my system: #526
The issues might be connected.

@lucasborin
Copy link
Member

Tested on SAP_BASIS 757/0000:
image

All fine here. Is the latest version of the class y_code_pal_ref_scan_double active on your system?

@ConjuringCoffee
Copy link
Contributor Author

Interesting. Yes, the latest version is active and identical to the code on GitHub.

@lucasborin
Copy link
Member

Do you have the SAP Note 2527903 implemented in your system?

@lucasborin lucasborin removed the bug Something isn't working correctly label Nov 11, 2021
@ConjuringCoffee
Copy link
Contributor Author

It looks like the task of implementing that note fell through on our side. I'll see to it being implemented and will report back.

@ConjuringCoffee
Copy link
Contributor Author

Implementing Note 2527903 fixed the issue, thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants