Merge pull request #358 from laeubi/issue357

Fix #357
cucumber · Apr 15, 2019 · 40f6a4b · 40f6a4b
2 parents 0d144e1 + 229762d
commit 40f6a4b
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/....eclipse.editor/src/main/java/cucumber/eclipse/editor/builder/CucumberGherkinBuilder.java b/....eclipse.editor/src/main/java/cucumber/eclipse/editor/builder/CucumberGherkinBuilder.java
@@ -333,7 +333,7 @@ public MarkerFormatter(IDocument document, IResource gherkinFile, MarkerFactory
 
 		@Override
 		public void syntaxError(String state, String event, List<String> legalEvents, String uri, Integer line) {
-			this.markerFactory.syntaxErrorOnGherkin(this.gherkinFile, new ParseException(event, line));
+			this.markerFactory.syntaxErrorOnGherkin(this.gherkinFile, "Found "+event+" when expecting one of: "+legalEvents+". (Current state: "+state+").", line);
 		}
 
 		@Override

diff --git a/cucumber.eclipse.editor/src/main/java/cucumber/eclipse/editor/editors/GherkinModel.java b/cucumber.eclipse.editor/src/main/java/cucumber/eclipse/editor/editors/GherkinModel.java
@@ -61,7 +61,7 @@ public PositionedElement getStepElement(int offset) throws BadLocationException
 	public void updateFromDocument(final IDocument document) {
 		elements.clear();
 
-		Parser p = new Parser(new Formatter() {
+		Formatter formatter = new Formatter() {
 
 			private Stack<PositionedElement> stack = new Stack<PositionedElement>();
 
@@ -70,8 +70,7 @@ public void uri(String arg0) {
 			}
 
 			@Override
-			public void syntaxError(String arg0, String arg1,
-					List<String> arg2, String arg3, Integer arg4) {
+			public void syntaxError(String state, String event, List<String> legalEvents, String uri, Integer line) {
 			}
 
 			@Override
@@ -157,7 +156,8 @@ public void startOfScenarioLifeCycle(Scenario arg0) {
 				// TODO Auto-generated method stub
 
 			}
-		});
+		};
+		Parser p = new Parser(formatter, false);
 
 		try {
 			p.parse(document.get(), "", 0);