Skip to content
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

Standard library integration tests are not working properly in the distribution build #4625

Closed
ayeshLK opened this issue Jun 26, 2023 · 3 comments · Fixed by #5259
Closed
Assignees
Labels
good first issue Good for newcomers Reason/Regression The issue has introduced a regression. Type/Bug

Comments

@ayeshLK
Copy link
Member

ayeshLK commented Jun 26, 2023

Description:

$subject

When running the distribution build, standard library integration tests are not running properly. Tested with following integration test [1] which is written for OpenAPI definition generation for HTTP service, this test passed even though it should fail according to recent compiler plugin changes.

[1] - https://github.com/ballerina-platform/ballerina-distribution/blob/master/stdlib-integration-tests/http/tests/rest_introspection_test.bal

@keizer619 keizer619 added the good first issue Good for newcomers label Sep 15, 2023
dsplayerX added a commit to dsplayerX/ballerina-distribution that referenced this issue Mar 20, 2024
Copy link

This issue is NOT closed with a proper Reason/ label. Make sure to add proper reason label before closing. Please add or leave a comment with the proper reason label now.

      - Reason/EngineeringMistake - The issue occurred due to a mistake made in the past.
      - Reason/Regression - The issue has introduced a regression.
      - Reason/MultipleComponentInteraction - Issue occured due to interactions in multiple components.
      - Reason/Complex - Issue occurred due to complex scenario.
      - Reason/Invalid - Issue is invalid.
      - Reason/Other - None of the above cases.

dsplayerX added a commit to dsplayerX/ballerina-distribution that referenced this issue Mar 22, 2024
@keizer619 keizer619 reopened this Mar 26, 2024
@dsplayerX
Copy link
Contributor

This issue was fixed with #5259. Tests testRestApiDoc and testIntrospectionResourceAvailability were disabled in the same PR (commit:b1cd993).

Created new issue #5272 to keep track of fixing and re-enabling these disabled tests.

Copy link

This issue is NOT closed with a proper Reason/ label. Make sure to add proper reason label before closing. Please add or leave a comment with the proper reason label now.

      - Reason/EngineeringMistake - The issue occurred due to a mistake made in the past.
      - Reason/Regression - The issue has introduced a regression.
      - Reason/MultipleComponentInteraction - Issue occured due to interactions in multiple components.
      - Reason/Complex - Issue occurred due to complex scenario.
      - Reason/Invalid - Issue is invalid.
      - Reason/Other - None of the above cases.

@keizer619 keizer619 added the Reason/Regression The issue has introduced a regression. label Mar 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers Reason/Regression The issue has introduced a regression. Type/Bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants