

class OwnersClient(object):
  """Interact with OWNERS files in a repository.

  This class allows you to interact with OWNERS files in a repository both the
  Gerrit Code-Owners plugin REST API, and the owners database implemented by
  Depot Tools in owners.py:

   - List all the owners for a change.
   - Check if a change has been approved.
   - Check if the OWNERS configuration in a change is valid.

  All code should use this class to interact with OWNERS files instead of the
  owners database in owners.py
  """
  def __init__(self, host):
    self._host = host

  def ListOwnersForFile(self, project, branch, path):
    """List all owners for a file."""
    raise Exception('Not implemented')

  def IsChangeApproved(self, change_number):
    """Check if the latest patch set for a change has been approved."""
    raise Exception('Not implemented')

  def IsOwnerConfigurationValid(self, change_number, patch):
    """Check if the owners configuration in a change is valid."""
    raise Exception('Not implemented')


import owners


class OwnersClient(object):
  """Interact with OWNERS files in a repository.

  This class allows you to interact with OWNERS files in a repository both the
  Gerrit Code-Owners plugin REST API, and the owners database implemented by
  Depot Tools in owners.py:

   - List all the owners for a change.
   - Check if a change has been approved.
   - Check if the OWNERS configuration in a change is valid.

  All code should use this class to interact with OWNERS files instead of the
  owners database in owners.py
  """
  def __init__(self, host):
    self._host = host

  def ListOwnersForFile(self, project, branch, path):
    """List all owners for a file."""
    raise Exception('Not implemented')

  def IsChangeApproved(self, change_id):
    """Check if the latest patch set for a change has been approved."""
    raise Exception('Not implemented')

  def IsOwnerConfigurationValid(self, change_id, patch):
    """Check if the owners configuration in a change is valid."""
    raise Exception('Not implemented')


class DepotToolsClient(OwnersClient):
  """Implement OwnersClient using owners.py Database."""
  def __init__(self, host, root):
    super(DepotToolsClient, self).__init__(host)
    self._root = root
    self._db = owners.Database(root, open, os.path)

  def ListOwnersForFile(self, _project, _branch, path):
    return sorted(self._db.all_possible_owners([arg], None))

Implement ListOwnersForFile for Depot Tools
[owners] Implement ListOwnersForFile for Depot Tools

Add DepotToolsClient to implement the OwnersClient API
for Depot Tools, and implement the ListOwnersForFile method.

Change-Id: I933a262898439d879c919d695aa62d7702b4c9a4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2530509
Auto-Submit: Edward Lesmes <d05609f027dc667cc37492b1dc1bdea5d60fa82a@chromium.org>
Reviewed-by: Anthony Polito <05bb588aa9020353543fee7ac4e6e0a3d0d6f6ed@google.com>
Commit-Queue: Edward Lesmes <d05609f027dc667cc37492b1dc1bdea5d60fa82a@chromium.org>

+ 
+ import owners
  
  
  class OwnersClient(object):
    """Interact with OWNERS files in a repository.
  
    This class allows you to interact with OWNERS files in a repository both the
    Gerrit Code-Owners plugin REST API, and the owners database implemented by
    Depot Tools in owners.py:
  
     - List all the owners for a change.
     - Check if a change has been approved.
     - Check if the OWNERS configuration in a change is valid.
  
    All code should use this class to interact with OWNERS files instead of the
    owners database in owners.py
    """
    def __init__(self, host):
      self._host = host
  
    def ListOwnersForFile(self, project, branch, path):
      """List all owners for a file."""
      raise Exception('Not implemented')
  
-   def IsChangeApproved(self, change_number):
+   def IsChangeApproved(self, change_id):
      """Check if the latest patch set for a change has been approved."""
      raise Exception('Not implemented')
  
-   def IsOwnerConfigurationValid(self, change_number, patch):
+   def IsOwnerConfigurationValid(self, change_id, patch):
      """Check if the owners configuration in a change is valid."""
      raise Exception('Not implemented')
  
+ 
+ class DepotToolsClient(OwnersClient):
+   """Implement OwnersClient using owners.py Database."""
+   def __init__(self, host, root):
+     super(DepotToolsClient, self).__init__(host)
+     self._root = root
+     self._db = owners.Database(root, open, os.path)
+ 
+   def ListOwnersForFile(self, _project, _branch, path):
+     return sorted(self._db.all_possible_owners([arg], None))
+ 
Implement ListOwnersForFile for Depot Tools
## Code Before:


class OwnersClient(object):
  """Interact with OWNERS files in a repository.

  This class allows you to interact with OWNERS files in a repository both the
  Gerrit Code-Owners plugin REST API, and the owners database implemented by
  Depot Tools in owners.py:

   - List all the owners for a change.
   - Check if a change has been approved.
   - Check if the OWNERS configuration in a change is valid.

  All code should use this class to interact with OWNERS files instead of the
  owners database in owners.py
  """
  def __init__(self, host):
    self._host = host

  def ListOwnersForFile(self, project, branch, path):
    """List all owners for a file."""
    raise Exception('Not implemented')

  def IsChangeApproved(self, change_number):
    """Check if the latest patch set for a change has been approved."""
    raise Exception('Not implemented')

  def IsOwnerConfigurationValid(self, change_number, patch):
    """Check if the owners configuration in a change is valid."""
    raise Exception('Not implemented')

## Instruction:
Implement ListOwnersForFile for Depot Tools
## Code After:

import owners


class OwnersClient(object):
  """Interact with OWNERS files in a repository.

  This class allows you to interact with OWNERS files in a repository both the
  Gerrit Code-Owners plugin REST API, and the owners database implemented by
  Depot Tools in owners.py:

   - List all the owners for a change.
   - Check if a change has been approved.
   - Check if the OWNERS configuration in a change is valid.

  All code should use this class to interact with OWNERS files instead of the
  owners database in owners.py
  """
  def __init__(self, host):
    self._host = host

  def ListOwnersForFile(self, project, branch, path):
    """List all owners for a file."""
    raise Exception('Not implemented')

  def IsChangeApproved(self, change_id):
    """Check if the latest patch set for a change has been approved."""
    raise Exception('Not implemented')

  def IsOwnerConfigurationValid(self, change_id, patch):
    """Check if the owners configuration in a change is valid."""
    raise Exception('Not implemented')


class DepotToolsClient(OwnersClient):
  """Implement OwnersClient using owners.py Database."""
  def __init__(self, host, root):
    super(DepotToolsClient, self).__init__(host)
    self._root = root
    self._db = owners.Database(root, open, os.path)

  def ListOwnersForFile(self, _project, _branch, path):
    return sorted(self._db.all_possible_owners([arg], None))
