← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/tests/test_versionedfile.py

  • Committer: Martin Pool
  • Date: 2008-06-04 06:15:34 UTC
  • mto: This revision was merged to the branch mainline in revision 3475.
  • Revision ID: mbp@sourcefrog.net-20080604061534-irtkvhd1cq9s788m
Test and fix #234748 problems in trailing newline diffs

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/tests/test_versionedfile.py
530
530
        self.assertRaises(errors.ReservedId, vf.get_lines, 'b:')
531
531
        self.assertRaises(errors.ReservedId, vf.get_text, 'b:')
532
532
 
 
533
    def test_add_unchanged_last_line_noeol_snapshot(self):
 
534
        """Add a text with an unchanged last line with no eol should work."""
 
535
        # Test adding this in a number of chain lengths; because the interface
 
536
        # for VersionedFile does not allow forcing a specific chain length, we
 
537
        # just use a small base to get the first snapshot, then a much longer
 
538
        # first line for the next add (which will make the third add snapshot)
 
539
        # and so on. 20 has been chosen as an aribtrary figure - knits use 200
 
540
        # as a capped delta length, but ideally we would have some way of
 
541
        # tuning the test to the store (e.g. keep going until a snapshot
 
542
        # happens).
 
543
        for length in range(20):
 
544
            version_lines = {}
 
545
            vf = self.get_file('case-%d' % length)
 
546
            prefix = 'step-%d'
 
547
            parents = []
 
548
            for step in range(length):
 
549
                version = prefix % step
 
550
                lines = (['prelude \n'] * step) + ['line']
 
551
                vf.add_lines(version, parents, lines)
 
552
                version_lines[version] = lines
 
553
                parents = [version]
 
554
            vf.add_lines('no-eol', parents, ['line'])
 
555
            vf.get_texts(version_lines.keys())
 
556
            self.assertEqualDiff('line', vf.get_text('no-eol'))
 
557
 
 
558
    def test_get_texts_eol_variation(self):
 
559
        # similar to the failure in <http://bugs.launchpad.net/234748>
 
560
        vf = self.get_file()
 
561
        sample_text_nl = ["line\n"]
 
562
        sample_text_no_nl = ["line"]
 
563
        versions = []
 
564
        version_lines = {}
 
565
        parents = []
 
566
        for i in range(4):
 
567
            version = 'v%d' % i
 
568
            if i % 2:
 
569
                lines = sample_text_nl
 
570
            else:
 
571
                lines = sample_text_no_nl
 
572
            # left_matching blocks is an internal api; it operates on the
 
573
            # *internal* representation for a knit, which is with *all* lines
 
574
            # being normalised to end with \n - even the final line in a no_nl
 
575
            # file. Using it here ensures that a broken internal implementation
 
576
            # (which is what this test tests) will generate a correct line
 
577
            # delta (which is to say, an empty delta).
 
578
            vf.add_lines(version, parents, lines,
 
579
                left_matching_blocks=[(0, 0, 1)])
 
580
            parents = [version]
 
581
            versions.append(version)
 
582
            version_lines[version] = lines
 
583
        vf.check()
 
584
        vf.get_texts(versions)
 
585
        vf.get_texts(reversed(versions))
 
586
 
533
587
    def test_make_mpdiffs(self):
534
588
        from bzrlib import multiparent
535
589
        vf = self.get_file('foo')