opendev/gerrit
Code Issues Proposed changes

Revert "Fix issues with conversational message parsing"

Browse Source 
This reverts commit 26efaf4436.

Reason for revert: internal issue

Change-Id: Ia2bbf6d5262c5c6669b07ae4f96971cb2775cdd8
This commit is contained in:
Kasper Nilsson
2018-09-07 16:41:54 +00:00
parent 26efaf4436
commit 78834bc16f
2 changed files with 3 additions and 26 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
polygerrit-ui/app/elements/change/gr-message/gr-message.js
View File

@@ -17,7 +17,7 @@
(function() {
'use strict';
const PATCH_SET_PREFIX_PATTERN = /^Patch Set (\d+):[ ]?/;
const PATCH_SET_PREFIX_PATTERN = /^Patch Set (\d)+:[ ]?/;
const COMMENTS_COUNT_PATTERN = /^\((\d+)( inline)? comments?\)$/;
const LABEL_TITLE_SCORE_PATTERN = /^([A-Za-z0-9-]+)([+-]\d+)$/;
@@ -268,9 +268,6 @@
* @param {string} text
*/
_consumeMessage(text) {
// If this variable is defined, the parsing process has already executed.
if (this._successfulParse !== undefined) { return; }
this._parsedPatchNum = '';
this._parsedCommentCount = '';
this._parsedChangeMessage = '';

24
polygerrit-ui/app/elements/change/gr-message/gr-message_test.html
View File

@@ -242,7 +242,7 @@ limitations under the License.
test('just change message', () => {
const str = [
'Patch Set 20:',
'Patch Set 2:',
'',
'I think you should reconsider this approach.',
'It really makes no sense.',
@@ -251,7 +251,7 @@ limitations under the License.
element._consumeMessage(str);
assert.isTrue(element._successfulParse);
assert.equal(element._parsedPatchNum, '20');
assert.equal(element._parsedPatchNum, '2');
assert.equal(element._parsedCommentCount, '');
assert.deepEqual(element._parsedVotes, []);
assert.equal(element._parsedChangeMessage, [
@@ -340,26 +340,6 @@ limitations under the License.
'Nice work.',
].join('\n'));
});
test('vote parsing happened already', () => {
const str = [
'Patch Set 2: Code-Review-Label+2 Verified+1',
'',
'LGTM',
'',
'Nice work.',
].join('\n');
element._successfulParse = true;
element._consumeMessage(str);
assert.notOk(element._parsedPatchNum);
element._successfulParse = false;
element._consumeMessage(str);
assert.notOk(element._parsedPatchNum);
});
});
});
</script>