Mailing-List: contact notifications-help@asterixdb.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@asterixdb.apache.org
Date: Sun, 6 Aug 2017 13:52:33 -0700
From: "Michael Blow (Code Review)" <do-not-reply@asterixdb.incubator.apache.org>
CC: Jenkins <jenkins@fulliautomatix.ics.uci.edu>,
	Till Westmann <tillw@apache.org>,
	Murtadha Hubail <mhubail@apache.org>
Reply-To: mblow@apache.org
Subject: Change in asterixdb[master]: [ASTERIXDB-2024][TEST] Test fwk comment patterns
In-Reply-To: <gerrit.1501960869485.I4c317ccee242bd5fa2f5ec5bed85a69f0a47cf11@asterix-gerrit.ics.uci.edu>
References: <gerrit.1501960869485.I4c317ccee242bd5fa2f5ec5bed85a69f0a47cf11@asterix-gerrit.ics.uci.edu>
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Content-Disposition: inline
User-Agent: Gerrit/2.12.7
Message-Id: <20170806205233.305FC2420AC@unhygienix.ics.uci.edu>
archived-at: Sun, 06 Aug 2017 20:52:37 -0000

Michael Blow has submitted this change and it was merged.

Change subject: [ASTERIXDB-2024][TEST] Test fwk comment patterns
......................................................................


[ASTERIXDB-2024][TEST] Test fwk comment patterns

Update the test framework to match comment patterns only at head of
line, to avoid truncating other instances of these characters i.e.
http://foobar.com -> http:

Change-Id: I4c317ccee242bd5fa2f5ec5bed85a69f0a47cf11
Reviewed-on: https://asterix-gerrit.ics.uci.edu/1919
Reviewed-by: Michael Blow <mblow@apache.org>
Integration-Tests: Michael Blow <mblow@apache.org>
Tested-by: Michael Blow <mblow@apache.org>
---
M asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Michael Blow: Looks good to me, approved; Verified; Verified


diff --git a/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java b/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java
index 1079832..1c29e86 100644
--- a/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java
+++ b/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java
@@ -105,8 +105,8 @@
     private static final long MAX_URL_LENGTH = 2000l;
     private static final Pattern JAVA_BLOCK_COMMENT_PATTERN = Pattern.compile("/\\*.*\\*/",
             Pattern.MULTILINE | Pattern.DOTALL);
-    private static final Pattern JAVA_LINE_COMMENT_PATTERN = Pattern.compile("//.*$", Pattern.MULTILINE);
-    private static final Pattern SHELL_LINE_COMMENT_PATTERN = Pattern.compile("#.*$", Pattern.MULTILINE);
+    private static final Pattern JAVA_LINE_COMMENT_PATTERN = Pattern.compile("^//.*$", Pattern.MULTILINE);
+    private static final Pattern SHELL_LINE_COMMENT_PATTERN = Pattern.compile("^#.*$", Pattern.MULTILINE);
     private static final Pattern REGEX_LINES_PATTERN = Pattern.compile("^(-)?/(.*)/([im]*)$");
     private static final Pattern POLL_TIMEOUT_PATTERN = Pattern.compile("polltimeoutsecs=(\\d+)(\\D|$)",
             Pattern.MULTILINE);

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1919
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I4c317ccee242bd5fa2f5ec5bed85a69f0a47cf11
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow <mblow@apache.org>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Michael Blow <mblow@apache.org>
Gerrit-Reviewer: Murtadha Hubail <mhubail@apache.org>
Gerrit-Reviewer: Till Westmann <tillw@apache.org>