Uploaded image for project: 'Spark'
  1. Spark
  2. SPARK-31003

Fix incorrect use of assume() in tests

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 3.0.0
    • 2.4.6, 3.0.0
    • Tests
    • None

    Description

      Ahir Reddy and I found some cases where assume and assert were mixed up in test suites.

      If an assume call fails then it will cause the test to be marked as skipped instead of failed: this is often used to skip tests if certain prerequisites are missing (e.g. to disable certain tests on Windows). If assume is mistakenly used in place of assert then this can cause logically-failing tests to be skipped, masking bugs.

      This patch fixes several such cases, replacing certain assume calls with assert

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            joshrosen Josh Rosen
            joshrosen Josh Rosen
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment