)]}'
{
  "commit": "5abfea2ae7e0486834c2b283a482abaebf61eda2",
  "tree": "f02b74772b874c46245bdd2d11146483562234f1",
  "parents": [
    "d607dd0130228035552982858edfe0fa22573ba5"
  ],
  "author": {
    "name": "Abhijeet Kaur",
    "email": "abkaur@google.com",
    "time": "Mon Oct 26 18:38:33 2020 +0000"
  },
  "committer": {
    "name": "Abhijeet Kaur",
    "email": "abkaur@google.com",
    "time": "Wed Dec 16 14:43:50 2020 +0000"
  },
  "message": "Run tests only on VISIBLE volumes that are accessible by apps\n\nThis is a quick fix to unblock partners. The MediaProvider fix\nfor this will go out with the mainline release to return only VISIBLE\nvolumes for MediaStore.getExternalVolumeNames().\n\nNon-VISIBLE volumes are not indexed by MediaProvider and are not mounted\nby FUSE; Hence, MediaStore APIs do not have access on such directories,\nas a result of which tests are failing. The reason this has not be\ncaught in Pixel devices is because all public volumes have ADOPTABLE\nflag set and hence VISIBLE flag set. (b/171736035).\n\nCopying javadocs recommended guidelines for VISIBLE volumes here for\ncontext from VolumeInfo.java :\n\n```\n{@link #MOUNT_FLAG_VISIBLE} means the volume is visible to third-party\napps for direct filesystem access. The system should send out relevant\nstorage broadcasts and index any media on visible volumes.\nVisible volumes are considered a more stable part of the device, which\nis why we take the time to index them. In particular, transient volumes\nlike USB OTG devices \u003cem\u003eshould not\u003c/em\u003e be marked as visible; their\ncontents should be surfaced to apps through the Storage Access\nFramework.\n```\n\nBug: 170403461\nTest: Manual; Non-adoptable public volume which is not-VISIBLE is not\nrun for CtsProviderTestCases. (Only able to repro this manually due to\nb/171736035)\n\nChange-Id: I443c776751119e8f38909c84a6f154837e646efd\nMerged-In: I443c776751119e8f38909c84a6f154837e646efd\n(cherry picked from commit 263293373a9dd5d671c4ebd8f55a343c8c7f3f55)\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "942d4f47de6f175264805d104b0d49c6406640f5",
      "old_mode": 33188,
      "old_path": "tests/tests/provider/src/android/provider/cts/ProviderTestUtils.java",
      "new_id": "5951ce9f3b070bde76e33b8a09d15719917578e6",
      "new_mode": 33188,
      "new_path": "tests/tests/provider/src/android/provider/cts/ProviderTestUtils.java"
    }
  ]
}