Fossil SCM
Verify revert output in revert tests.
Commit
3949c9a33e219cad057ff649898c141eb6b52b90
Parent
e66a12ec4bfa24b…
1 file changed
+35
-17
+35
-17
| --- test/revert.test | ||
| +++ test/revert.test | ||
| @@ -2,30 +2,25 @@ | ||
| 2 | 2 | # Tests for 'fossil revert' |
| 3 | 3 | # |
| 4 | 4 | # |
| 5 | 5 | |
| 6 | 6 | # Test 'fossil revert' against expected results from 'fossil changes' and |
| 7 | -# 'fossil addremove --test', as well as by verifying the existence of files | |
| 7 | +# 'fossil addremove -n', as well as by verifying the existence of files | |
| 8 | 8 | # on the file system. 'fossil undo' is called after each test |
| 9 | 9 | # |
| 10 | -proc revert-test {testid args} { | |
| 10 | +proc revert-test {testid revertArgs expectedRevertOutput args} { | |
| 11 | 11 | global RESULT |
| 12 | 12 | set passed 1 |
| 13 | 13 | |
| 14 | - if {[llength $args] % 2} { | |
| 15 | - set revertArgs [lindex $args 0] | |
| 16 | - set args [lrange $args 1 end] | |
| 17 | - } else { | |
| 18 | - set revertArgs {} | |
| 19 | - } | |
| 20 | 14 | set args [dict merge { |
| 21 | 15 | -changes {} -addremove {} -exists {} -notexists {} |
| 22 | 16 | } $args] |
| 23 | 17 | |
| 24 | - fossil revert {*}$revertArgs | |
| 18 | + set result [fossil revert {*}$revertArgs] | |
| 19 | + test_status_list revert-$testid $result $expectedRevertOutput | |
| 25 | 20 | |
| 26 | - set statusListTests [list -changes changes -addremove {addremove --test}] | |
| 21 | + set statusListTests [list -changes changes -addremove {addremove -n}] | |
| 27 | 22 | foreach {key fossilArgs} $statusListTests { |
| 28 | 23 | set expected [dict get $args $key] |
| 29 | 24 | set result [fossil {*}$fossilArgs] |
| 30 | 25 | test_status_list revert-$testid$key $result $expected |
| 31 | 26 | } |
| @@ -68,53 +63,76 @@ | ||
| 68 | 63 | file rename -force f3 f3n |
| 69 | 64 | fossil mv f3 f3n |
| 70 | 65 | |
| 71 | 66 | # Test 'fossil revert' with no arguments |
| 72 | 67 | # |
| 73 | -revert-test 1 -addremove { | |
| 68 | +revert-test 1-1 {} { | |
| 69 | + UNMANAGE: f0 | |
| 70 | + REVERTED: f1 | |
| 71 | + REVERTED: f2 | |
| 72 | + REVERTED: f3 | |
| 73 | + DELETE: f3n | |
| 74 | +} -addremove { | |
| 74 | 75 | ADDED f0 |
| 75 | 76 | } -exists {f0 f1 f2 f3} -notexists f3n |
| 76 | 77 | |
| 77 | 78 | # Test with a single filename argument |
| 78 | 79 | # |
| 79 | -revert-test 2 f0 -changes { | |
| 80 | +revert-test 1-2 f0 { | |
| 81 | + UNMANAGE: f0 | |
| 82 | +} -changes { | |
| 80 | 83 | DELETED f1 |
| 81 | 84 | EDITED f2 |
| 82 | 85 | RENAMED f3n |
| 83 | 86 | } -addremove { |
| 84 | 87 | ADDED f0 |
| 85 | 88 | } -exists {f0 f2 f3n} -notexists f3 |
| 86 | 89 | |
| 87 | -revert-test 3 f1 -changes { | |
| 90 | +revert-test 1-3 f1 { | |
| 91 | + REVERTED: f1 | |
| 92 | +} -changes { | |
| 88 | 93 | ADDED f0 |
| 89 | 94 | EDITED f2 |
| 90 | 95 | RENAMED f3n |
| 91 | 96 | } -exists {f0 f1 f2 f3n} -notexists f3 |
| 92 | 97 | |
| 93 | -revert-test 4 f2 -changes { | |
| 98 | +revert-test 1-4 f2 { | |
| 99 | + REVERTED: f2 | |
| 100 | +} -changes { | |
| 94 | 101 | ADDED f0 |
| 95 | 102 | DELETED f1 |
| 96 | 103 | RENAMED f3n |
| 97 | 104 | } -exists {f0 f2 f3n} -notexists {f1 f3} |
| 98 | 105 | |
| 99 | 106 | # Both files involved in a rename are reverted regardless of which filename |
| 100 | 107 | # is used as an argument to 'fossil revert' |
| 101 | 108 | # |
| 102 | -revert-test 5 f3 -changes { | |
| 109 | +revert-test 1-5 f3 { | |
| 110 | + REVERTED: f3 | |
| 111 | + DELETE: f3n | |
| 112 | +} -changes { | |
| 103 | 113 | ADDED f0 |
| 104 | 114 | DELETED f1 |
| 105 | 115 | EDITED f2 |
| 106 | 116 | } -exists {f0 f2 f3} -notexists {f1 f3n} |
| 107 | 117 | |
| 108 | -revert-test 6 f3n -changes { | |
| 118 | +revert-test 1-6 f3n { | |
| 119 | + REVERTED: f3 | |
| 120 | + DELETE: f3n | |
| 121 | +} -changes { | |
| 109 | 122 | ADDED f0 |
| 110 | 123 | DELETED f1 |
| 111 | 124 | EDITED f2 |
| 112 | 125 | } -exists {f0 f2 f3} -notexists {f1 f3n} |
| 113 | 126 | |
| 114 | 127 | # Test with multiple filename arguments |
| 115 | 128 | # |
| 116 | -revert-test 7 {f0 f2 f3n} -changes { | |
| 129 | +revert-test 1-7 {f0 f2 f3n} { | |
| 130 | + UNMANAGE: f0 | |
| 131 | + REVERTED: f2 | |
| 132 | + REVERTED: f3 | |
| 133 | + DELETE: f3n | |
| 134 | +} -changes { | |
| 117 | 135 | DELETED f1 |
| 118 | 136 | } -addremove { |
| 119 | 137 | ADDED f0 |
| 120 | 138 | } -exists {f0 f2 f3} -notexists {f1 f3n} |
| 121 | 139 |
| --- test/revert.test | |
| +++ test/revert.test | |
| @@ -2,30 +2,25 @@ | |
| 2 | # Tests for 'fossil revert' |
| 3 | # |
| 4 | # |
| 5 | |
| 6 | # Test 'fossil revert' against expected results from 'fossil changes' and |
| 7 | # 'fossil addremove --test', as well as by verifying the existence of files |
| 8 | # on the file system. 'fossil undo' is called after each test |
| 9 | # |
| 10 | proc revert-test {testid args} { |
| 11 | global RESULT |
| 12 | set passed 1 |
| 13 | |
| 14 | if {[llength $args] % 2} { |
| 15 | set revertArgs [lindex $args 0] |
| 16 | set args [lrange $args 1 end] |
| 17 | } else { |
| 18 | set revertArgs {} |
| 19 | } |
| 20 | set args [dict merge { |
| 21 | -changes {} -addremove {} -exists {} -notexists {} |
| 22 | } $args] |
| 23 | |
| 24 | fossil revert {*}$revertArgs |
| 25 | |
| 26 | set statusListTests [list -changes changes -addremove {addremove --test}] |
| 27 | foreach {key fossilArgs} $statusListTests { |
| 28 | set expected [dict get $args $key] |
| 29 | set result [fossil {*}$fossilArgs] |
| 30 | test_status_list revert-$testid$key $result $expected |
| 31 | } |
| @@ -68,53 +63,76 @@ | |
| 68 | file rename -force f3 f3n |
| 69 | fossil mv f3 f3n |
| 70 | |
| 71 | # Test 'fossil revert' with no arguments |
| 72 | # |
| 73 | revert-test 1 -addremove { |
| 74 | ADDED f0 |
| 75 | } -exists {f0 f1 f2 f3} -notexists f3n |
| 76 | |
| 77 | # Test with a single filename argument |
| 78 | # |
| 79 | revert-test 2 f0 -changes { |
| 80 | DELETED f1 |
| 81 | EDITED f2 |
| 82 | RENAMED f3n |
| 83 | } -addremove { |
| 84 | ADDED f0 |
| 85 | } -exists {f0 f2 f3n} -notexists f3 |
| 86 | |
| 87 | revert-test 3 f1 -changes { |
| 88 | ADDED f0 |
| 89 | EDITED f2 |
| 90 | RENAMED f3n |
| 91 | } -exists {f0 f1 f2 f3n} -notexists f3 |
| 92 | |
| 93 | revert-test 4 f2 -changes { |
| 94 | ADDED f0 |
| 95 | DELETED f1 |
| 96 | RENAMED f3n |
| 97 | } -exists {f0 f2 f3n} -notexists {f1 f3} |
| 98 | |
| 99 | # Both files involved in a rename are reverted regardless of which filename |
| 100 | # is used as an argument to 'fossil revert' |
| 101 | # |
| 102 | revert-test 5 f3 -changes { |
| 103 | ADDED f0 |
| 104 | DELETED f1 |
| 105 | EDITED f2 |
| 106 | } -exists {f0 f2 f3} -notexists {f1 f3n} |
| 107 | |
| 108 | revert-test 6 f3n -changes { |
| 109 | ADDED f0 |
| 110 | DELETED f1 |
| 111 | EDITED f2 |
| 112 | } -exists {f0 f2 f3} -notexists {f1 f3n} |
| 113 | |
| 114 | # Test with multiple filename arguments |
| 115 | # |
| 116 | revert-test 7 {f0 f2 f3n} -changes { |
| 117 | DELETED f1 |
| 118 | } -addremove { |
| 119 | ADDED f0 |
| 120 | } -exists {f0 f2 f3} -notexists {f1 f3n} |
| 121 |
| --- test/revert.test | |
| +++ test/revert.test | |
| @@ -2,30 +2,25 @@ | |
| 2 | # Tests for 'fossil revert' |
| 3 | # |
| 4 | # |
| 5 | |
| 6 | # Test 'fossil revert' against expected results from 'fossil changes' and |
| 7 | # 'fossil addremove -n', as well as by verifying the existence of files |
| 8 | # on the file system. 'fossil undo' is called after each test |
| 9 | # |
| 10 | proc revert-test {testid revertArgs expectedRevertOutput args} { |
| 11 | global RESULT |
| 12 | set passed 1 |
| 13 | |
| 14 | set args [dict merge { |
| 15 | -changes {} -addremove {} -exists {} -notexists {} |
| 16 | } $args] |
| 17 | |
| 18 | set result [fossil revert {*}$revertArgs] |
| 19 | test_status_list revert-$testid $result $expectedRevertOutput |
| 20 | |
| 21 | set statusListTests [list -changes changes -addremove {addremove -n}] |
| 22 | foreach {key fossilArgs} $statusListTests { |
| 23 | set expected [dict get $args $key] |
| 24 | set result [fossil {*}$fossilArgs] |
| 25 | test_status_list revert-$testid$key $result $expected |
| 26 | } |
| @@ -68,53 +63,76 @@ | |
| 63 | file rename -force f3 f3n |
| 64 | fossil mv f3 f3n |
| 65 | |
| 66 | # Test 'fossil revert' with no arguments |
| 67 | # |
| 68 | revert-test 1-1 {} { |
| 69 | UNMANAGE: f0 |
| 70 | REVERTED: f1 |
| 71 | REVERTED: f2 |
| 72 | REVERTED: f3 |
| 73 | DELETE: f3n |
| 74 | } -addremove { |
| 75 | ADDED f0 |
| 76 | } -exists {f0 f1 f2 f3} -notexists f3n |
| 77 | |
| 78 | # Test with a single filename argument |
| 79 | # |
| 80 | revert-test 1-2 f0 { |
| 81 | UNMANAGE: f0 |
| 82 | } -changes { |
| 83 | DELETED f1 |
| 84 | EDITED f2 |
| 85 | RENAMED f3n |
| 86 | } -addremove { |
| 87 | ADDED f0 |
| 88 | } -exists {f0 f2 f3n} -notexists f3 |
| 89 | |
| 90 | revert-test 1-3 f1 { |
| 91 | REVERTED: f1 |
| 92 | } -changes { |
| 93 | ADDED f0 |
| 94 | EDITED f2 |
| 95 | RENAMED f3n |
| 96 | } -exists {f0 f1 f2 f3n} -notexists f3 |
| 97 | |
| 98 | revert-test 1-4 f2 { |
| 99 | REVERTED: f2 |
| 100 | } -changes { |
| 101 | ADDED f0 |
| 102 | DELETED f1 |
| 103 | RENAMED f3n |
| 104 | } -exists {f0 f2 f3n} -notexists {f1 f3} |
| 105 | |
| 106 | # Both files involved in a rename are reverted regardless of which filename |
| 107 | # is used as an argument to 'fossil revert' |
| 108 | # |
| 109 | revert-test 1-5 f3 { |
| 110 | REVERTED: f3 |
| 111 | DELETE: f3n |
| 112 | } -changes { |
| 113 | ADDED f0 |
| 114 | DELETED f1 |
| 115 | EDITED f2 |
| 116 | } -exists {f0 f2 f3} -notexists {f1 f3n} |
| 117 | |
| 118 | revert-test 1-6 f3n { |
| 119 | REVERTED: f3 |
| 120 | DELETE: f3n |
| 121 | } -changes { |
| 122 | ADDED f0 |
| 123 | DELETED f1 |
| 124 | EDITED f2 |
| 125 | } -exists {f0 f2 f3} -notexists {f1 f3n} |
| 126 | |
| 127 | # Test with multiple filename arguments |
| 128 | # |
| 129 | revert-test 1-7 {f0 f2 f3n} { |
| 130 | UNMANAGE: f0 |
| 131 | REVERTED: f2 |
| 132 | REVERTED: f3 |
| 133 | DELETE: f3n |
| 134 | } -changes { |
| 135 | DELETED f1 |
| 136 | } -addremove { |
| 137 | ADDED f0 |
| 138 | } -exists {f0 f2 f3} -notexists {f1 f3n} |
| 139 |