~bzr/bzr/bzr.1.5

« back to all changes in this revision

Viewing changes to bzrlib/tests/test_remote.py

  • Committer: Canonical.com Patch Queue Manager
  • Date: 2008-05-07 08:48:35 UTC
  • mfrom: (3408.3.1 branch_conf)
  • Revision ID: pqm@pqm.ubuntu.com-20080507084835-066pb8ddwett0ne3
(mbp) Remove erroneous handling of branch.conf for RemoteBranch

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2006, 2007 Canonical Ltd
 
1
# Copyright (C) 2006, 2007, 2008 Canonical Ltd
2
2
#
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
603
603
 
604
604
 
605
605
class TestBranchControlGetBranchConf(tests.TestCaseWithMemoryTransport):
606
 
    """Test branch.control_files api munging...
607
 
 
608
 
    We special case RemoteBranch.control_files.get('branch.conf') to
609
 
    call a specific API so that RemoteBranch's can intercept configuration
610
 
    file reading, allowing them to signal to the client about things like
611
 
    'email is configured for commits'.
 
606
    """Getting the branch configuration should use an abstract method not vfs.
612
607
    """
613
608
 
614
609
    def test_get_branch_conf(self):
 
610
        raise tests.KnownFailure('branch.conf is not retrieved by get_config_file')
 
611
        # We should see that branch.get_config() does a single rpc to get the
 
612
        # remote configuration file, abstracting away where that is stored on
 
613
        # the server.  However at the moment it always falls back to using the
 
614
        # vfs, and this would need some changes in config.py.
 
615
 
615
616
        # in an empty branch we decode the response properly
616
 
        client = FakeClient([(('ok', ), 'config file body')], self.get_url())
 
617
        client = FakeClient([(('ok', ), '# config file body')], self.get_url())
617
618
        # we need to make a real branch because the remote_branch.control_files
618
619
        # will trigger _ensure_real.
619
620
        branch = self.make_branch('quack')
621
622
        # we do not want bzrdir to make any remote calls
622
623
        bzrdir = RemoteBzrDir(transport, _client=False)
623
624
        branch = RemoteBranch(bzrdir, None, _client=client)
624
 
        result = branch.control_files.get('branch.conf')
 
625
        config = branch.get_config()
625
626
        self.assertEqual(
626
627
            [('call_expecting_body', 'Branch.get_config_file', ('quack/',))],
627
628
            client._calls)
628
 
        self.assertEqual('config file body', result.read())
629
629
 
630
630
 
631
631
class TestBranchLockWrite(tests.TestCase):