Skip to content

Allure cucumber jvm#35

Merged
baev merged 26 commits into
allure-framework:masterfrom
clicman:allure-cucumber-jvm
Mar 21, 2017
Merged

Allure cucumber jvm#35
baev merged 26 commits into
allure-framework:masterfrom
clicman:allure-cucumber-jvm

Conversation

@clicman
Copy link
Copy Markdown
Contributor

@clicman clicman commented Mar 20, 2017

Add allure-cucumber-jvm adaptor

import gherkin.I18n;
import gherkin.formatter.Formatter;
import gherkin.formatter.Reporter;
import gherkin.formatter.model.*;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do not use wildcard imports

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Damn Idea!

Comment thread settings.gradle Outdated
//include 'allure-junit5'
include 'allure-testng'

include 'allure-cucumber-jvm' No newline at end of file
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add a new line in end of file

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Got it!

@baev
Copy link
Copy Markdown
Member

baev commented Mar 20, 2017

you also need to fix all the pmd/checkstyle/findbugs issues

@clicman
Copy link
Copy Markdown
Contributor Author

clicman commented Mar 20, 2017

you also need to fix all the pmd/checkstyle/findbugs issues

Is these check done automatically? I can fix it in right way only if it will be hardly declared rules or automatically checked by right configured sonar of smth. similar.

@baev
Copy link
Copy Markdown
Member

baev commented Mar 20, 2017

simply run ./gradlew clean build and you will see the errors in console

@baev baev added the work:in progress Some work in progress at the moment label Mar 20, 2017
Copy link
Copy Markdown
Member

@baev baev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM in general

import io.qameta.allure.model.StepResult;
import io.qameta.allure.model.TestResult;

import java.util.*;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

imports


import cucumber.runtime.CucumberException;
import cucumber.runtime.StepDefinitionMatch;
import gherkin.formatter.model.*;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

imports

}
}

public static String getStackTraceAsString(final Throwable throwable) {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you can use ResultsUtils.getStatusDetails() instead

@baev baev merged commit a7d88f2 into allure-framework:master Mar 21, 2017
@baev baev removed work:in progress Some work in progress at the moment labels Mar 21, 2017
@baev
Copy link
Copy Markdown
Member

baev commented Mar 21, 2017

@clicman thanks!

@clicman clicman deleted the allure-cucumber-jvm branch March 21, 2017 10:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants