Once we have a list of required tests please shard out the implementations. 

Re-assigning to Joel - PTAL for M67.

Lowering priority to P2.

Would be great if we could get some data on our coverage dashboard, e.g.:

https://chromium-coverage.appspot.com/reports/559420/linux/chromium/src/cc/animation/report.html

Not suer how this report is generated :)

See the post on chromium-dev Friday-last [1], about the new coverage tool.  It's meant to cover ChromeOS but I don't see out tests in reports as yet.

[1] https://groups.google.com/a/chromium.org/forum/#!topic/chromium-dev/ewIXgtLncwI

Assigning to sergei, who is working on tests for DriveFS. We should ensure we have good FMP API test coverage so we can catch any regressions DriveFS introduces.

When we add new File Manager Private API ... need tests. And see also the tests in chrome/test/data/extensions/api_test/file_browser. These are browser tests, driven by external_filesystem_apitest.cc

  chrome/browser/chromeos/file_manager/external_filesystem_apitest.cc

and in addition to the unit tests: file_manager_private_apitest.cc. HTH.

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b4268a4de0dfd26787354ecab6e1298d1c04cf15

commit b4268a4de0dfd26787354ecab6e1298d1c04cf15
Author: Joel Hockey <joelhockey@chromium.org>
Date: Thu May 31 05:13:59 2018

Add a fileManagerPrivate.isCrostiniEnabled test

Additional test for fileManagerPrivate.mountCrostiniContainer
will be added to these files next.

Bug:  845075 , 822130
Change-Id: I8e4fbb840764e975b5d670556905f3b4b411f2da
Reviewed-on: https://chromium-review.googlesource.com/1075878
Commit-Queue: Joel Hockey <joelhockey@chromium.org>
Reviewed-by: Noel Gordon <noel@chromium.org>
Reviewed-by: Yoshiki Iguchi <yoshiki@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563153}
[modify] https://crrev.com/b4268a4de0dfd26787354ecab6e1298d1c04cf15/chrome/browser/chromeos/extensions/file_manager/file_manager_private_apitest.cc
[add] https://crrev.com/b4268a4de0dfd26787354ecab6e1298d1c04cf15/chrome/test/data/extensions/api_test/file_browser/crostini_test/manifest.json
[add] https://crrev.com/b4268a4de0dfd26787354ecab6e1298d1c04cf15/chrome/test/data/extensions/api_test/file_browser/crostini_test/test.js

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/817a0f93053534947536a8c98811aa541f5e9926

commit 817a0f93053534947536a8c98811aa541f5e9926
Author: Joel Hockey <joelhockey@chromium.org>
Date: Fri Jun 01 01:35:50 2018

fileManagerPrivate.mountCrostiniContainer test

* Added CrostiniManager::SkipRestartForTesting
  to bypass component loading and starting
  concierge to start VM and container.
* Added MockDiskMountManager::NotifyMountEvent
  to trigger OnMountEvent to observers.

Bug:  845075 
Bug: 822130
Change-Id: Ifd9464730c6cd50a683cc40f60a289c80a36158c
Reviewed-on: https://chromium-review.googlesource.com/1080487
Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Reviewed-by: Nicholas Verne <nverne@chromium.org>
Reviewed-by: Noel Gordon <noel@chromium.org>
Commit-Queue: Joel Hockey <joelhockey@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563476}
[modify] https://crrev.com/817a0f93053534947536a8c98811aa541f5e9926/chrome/browser/chromeos/crostini/crostini_manager.cc
[modify] https://crrev.com/817a0f93053534947536a8c98811aa541f5e9926/chrome/browser/chromeos/crostini/crostini_manager.h
[modify] https://crrev.com/817a0f93053534947536a8c98811aa541f5e9926/chrome/browser/chromeos/extensions/file_manager/file_manager_private_apitest.cc
[modify] https://crrev.com/817a0f93053534947536a8c98811aa541f5e9926/chrome/test/data/extensions/api_test/file_browser/crostini_test/manifest.json
[modify] https://crrev.com/817a0f93053534947536a8c98811aa541f5e9926/chrome/test/data/extensions/api_test/file_browser/crostini_test/test.js
[modify] https://crrev.com/817a0f93053534947536a8c98811aa541f5e9926/chromeos/disks/mock_disk_mount_manager.cc
[modify] https://crrev.com/817a0f93053534947536a8c98811aa541f5e9926/chromeos/disks/mock_disk_mount_manager.h

re c#7, the test needs to be added to https://cs.chromium.org/chromium/src/tools/code_coverage/test_suite.txt so that code coverage bots would run it